Source-Changes-D archive

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

Re: CVS commit: src/sys/kern




On 09.01.2017 04:36, Christos Zoulas wrote:
> In article <20170109003130.33807FBAE%cvs.NetBSD.org@localhost>,
> Kamil Rytarowski <source-changes-d%NetBSD.org@localhost> wrote:
>> -=-=-=-=-=-
>>
>> Module Name:	src
>> Committed By:	kamil
>> Date:		Mon Jan  9 00:31:30 UTC 2017
>>
>> Modified Files:
>> 	src/sys/kern: kern_exec.c kern_exit.c kern_fork.c
>>
>> Log Message:
>> Cleanup dead code after revert of racy vfork(2) commit
>>
>> This removes dead code introduced with the following commit:
> 
> That's not dead code; it is commented out code that we would like
> to eventually get working. Rmind committed the change but it caused
> some regressions we did not understand. It did not hurt anything as
> it was commented out, and served as a reminder that something is
> wrong and we should one day track it down and fix it. Now there is
> no such reminder.
> 
> dead code = unreachable code but still compiled.
> 
> christos
> 

It's almost 5 years since commenting out - I assumed that it will rot to
the point that it might need to be rewritten. A reminder could be added
to roadmap and the code is still there in code history.

But if there is a request, I will revert it.

Attachment: signature.asc
Description: OpenPGP digital signature



Home | Main Index | Thread Index | Old Index