svn commit: r230354 - head/usr.sbin/makefs
brde at optusnet.com.au
Sat Jan 21 05:52:57 UTC 2012
On Fri, 20 Jan 2012, Eitan Adler wrote:
> On Fri, Jan 20, 2012 at 7:13 PM, Bruce Evans <brde at optusnet.com.au> wrote:
>> I agree. Â Never fix vendor code. Â Especially style bugs in it. Â Not all
>> vendor code is in contrib.
> I'm sure this is a silly question, but why isn't this utility in a
> contrib/ folder?
Probably historical, but src/contrib/ still has little more than `file'
and lukemftpd from NetBSD. I think those are there mainly because they
got portabilized and complicated by their use of autoconfig, so they
don't fit naturally in the BSD build framework any more.
> [snip other fixes to the api]
>> I don't understand this code well enough to fix it. Â Fixing it requires
>> understanding whether the unused variable was unused because of another
>> bug. Â Another bug seems likely here, since there is so much dead code.
>> Or maybe I'm just confused, and the dead code is actually undead. Â It
>> is complicated enough for this to be unclear.
> I did not just rely on the compiler - I went through the code manually
> to ensure it was correct.
It can take too long to see correctness for an apparently-simple change.
More information about the svn-src-head