Source-Changes-D archive

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

Re: CVS commit: src/bin/sleep



On Mon, Jan 28, 2019 at 07:52:22AM +0700, Robert Elz wrote:
> That PR needed to be fixed, even before it was filed, as stray
> characters after the numeric value are more likely to be something
> attempting linux "sleep 2m" (ie: sleep 120) raher than someone
> attempting to sleep for a currency value.   And that does need
> to be detected as an error, unlike "sleep 1,2" (or even sleep 0,2)
> which does no real harm, even if it was an accident (we're
> talking about a sleep of some extra fraction of a second...)

I'd argue that "300ms" or "120us" or "15ns" are all pretty unambiguous,
and that an exactly two char suffix would be rather unlikely to be line
noise.



Home | Main Index | Thread Index | Old Index