2008/7/15 Bernhard Fischer <[EMAIL PROTECTED]>:
> On Tue, Jul 15, 2008 at 09:24:00AM +0200, Loïc Grenié wrote:
>>2008/7/15 Denys Vlasenko <[EMAIL PROTECTED]>:
>>> On Tuesday 15 July 2008 08:58, Loïc Grenié wrote:
>>>> This is a patch that implements a config option for the
>>>> fractional sleep arguments.
>>>
>>> -#if ENABLE_FEATURE_FANCY_SLEEP
>>> +#if ENABLE_FEATURE_FANCY_SLEEP && !ENABLE_FEATURE_FLOAT_SLEEP
>>> static const struct suffix_mult sfx[] = {
>>>
>>> But fractional second sleep needs sfx[],
>>> because "sleep 0.9m" should work too (coreutils does that).
>>> You might also wat to mention that in help text.
>>
>> My bad.
>
> It should not be FANCY_SLEEP || FLOAT_SLEEP but independent options so
> one can have FANCY && FLOAT if desired.
>
> Can you send an updated patch, please?
FLOAT contains FANCY (I had missed it at the beginning, but FLOAT has
the functionality of FANCY). I thing making FLOAT depends on FANCY is
not a very good idea (someone looking for the FLOAT functionality will
probably not think about enabling FANCY).
Btw the name FLOAT is not cast in stone obviously.
Loïc
_______________________________________________
busybox mailing list
busybox@busybox.net
http://busybox.net/cgi-bin/mailman/listinfo/busybox
