Question for Vodz about getopt_ulflags ([PATCH] inside)

[Rob Landley]

On Wednesday 03 May 2006 7:32 am, Tito wrote:
> On Wednesday 3 May 2006 12:42, you wrote:
> > If you have more than 8*sizeof(ulong) options, split them:
> >
> > opts1 = bb_getopt_ulflags(argc, argv, hdparm_options1);
> > opts2 = bb_getopt_ulflags(argc, argv, hdparm_options2);
>
> Thanks!

How does that work?  When I try to feed it an option it doesn't recognize, I 
expect:
./busybox ps -z
ps: invalid option -- z

Plus a dump of the help text...

> > BTW, see attached. I do want doc to be in English.
> > Please help me to push it through if you can :)
>
> Ok. There are a lot of whitespace changes, couldn't they be ignored?

I just applied it.

> Maybe just this few minor changes (see ++)...... and couldn't we add this
> too:
>
> ++If you have more than 8*sizeof(ulong) options, split them:
>
> ++opts1 = bb_getopt_ulflags(argc, argv, hdparm_options1);
> ++opts2 = bb_getopt_ulflags(argc, argv, hdparm_options2);

I'm open to a patch on top of the patch... :)

Rob
-- 
Never bet against the cheap plastic solution.