DragonFly BSD
DragonFly commits List (threaded) for 2005-01
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: cvs commit: src/sbin/mount mntopts.h src/sbin/newfs fscopy.c mkfs.c newfs.c


From: Chris Pressey <cpressey@xxxxxxxxxxxxxxx>
Date: Fri, 7 Jan 2005 11:38:14 -0800

On Fri, 7 Jan 2005 11:24:58 -0800 (PST)
Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx> wrote:

>     Good work Chris, but I have one minor nit and that is this 
>     empty_string[] business.  At least for fscopy.c what really
>     needs to be done there is that the code needs to be cleaned
>     up and NULL assigned in the 0-length case instead of "".
>     The one or two write()'s that write node->fs_Bytes should then
>     check to see if node->fs_Bytes != 0 before doing the write.
> 
>     The only reason the code works at all is that it nevers frees
>     node->fs_Data except during the allocation phase if something
>     errors out during allocation.
> 
>     In this case the empty_string is a bit too much of a hack to
>     get around the const/non-const issue.

OK, I'll give it another go-round.

-Chris



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]