Re: [Freedos-kernel] Re: Re: [Freedos-cvs] kernel/kernel inthndlr.c,1.87.2.12,1.87.2.13
Hi! 5-Янв-2005 22:59 [EMAIL PROTECTED] (tom ehlert) wrote to Eric Auer : te> and while I agree completely with what he wrote, IMO he slightly te> missed the point (and you -eric- miss it completely) >> The whole point is that if you remove the break; completely, then >> somebody who later reads or changes the code might be mislead into >> thinking that the code SHOULD fall through ... Eric correctly repeat my point, as I try to express it. (Of course, there may be raised other issues about "shared programming effort", but I begun thread with only specific reason of readability, which was affected by patch.) --- The SF.Net email is sponsored by: Beat the post-holiday blues Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek. It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt ___ Freedos-kernel mailing list Freedos-kernel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freedos-kernel
Kernel patches: Was: Re: [Freedos-kernel] Re: Re: [Freedos-cvs] kernel/kernel
Peter Fedorow wrote: .. Are embedded developers welcome to send in clean patches to selectively remove those items? yes, however it depends largely on the patch as to if it will be applied or not I would prefer preventing lots of defines that effect a little piece here or there, but any cleanup or patches that can selectively remove functionality (e.g. the #ifdef FAT32 or #ifdef LFNAPI stuff) without obscuring the source unreasonably is fine Please note, if possible, any patches other than bugfixes should be against the development kernel, as that will allow it to be integrated sooner if accepted. Just to be clear, yes please send any patches you feel may be beneficial, Jeremy --- The SF.Net email is sponsored by: Beat the post-holiday blues Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek. It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt ___ Freedos-kernel mailing list Freedos-kernel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freedos-kernel