OK, I will send the new patch ASAP
Le 3 févr. 2016 18:39, "Mike Frysinger" <[email protected]> a écrit :

> On 03 Feb 2016 12:45, Daniel Thompson wrote:
> > On 03/02/16 12:29, Nicolas CARRIER wrote:
> > > Honestly, I hesitated to propose the inverse feature.
> > > I think the key point is : what we consider "normal" behavior ?
> > > I tend to consider it's "keep your arguments as they were passed to
> > > you", but I felt there was more people favoring the "do as you've
> > > always done".
> >
> > Defaults are enough to handle "normal" behavior; inverting the
> > definition of a feature isn't helpful.
> >
> > It is far better (and more intuitive) if: feature off -> smaller code
>
> naming wise, FEATURE_INIT_MODIFY_CMDLINE is fine.  the "default" value
> can be easily used to set the state regardless of naming and inverted.
>
> i have no strong opinion either way as to the default.  disabling the
> munging by default does yield smaller code.
> -mike
>
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to