[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: new snapshot available: coreutils-8.20.119-54cdb0
From: |
Assaf Gordon |
Subject: |
Re: new snapshot available: coreutils-8.20.119-54cdb0 |
Date: |
Mon, 11 Feb 2013 23:45:45 -0500 |
On Feb 11, 2013, at 20:32, Pádraig Brady <address@hidden> wrote:
> On 02/11/2013 08:47 PM, Assaf Gordon wrote:
>> - if (errno != 0)
>> + /* EINVAL can happen if 'base' is invalid (hardcoded as 10, so can't
>> happen),
>> + or if no conversion was performed (on some platforms). Ignore &
>> continue
>> + if no conversion was performed */
>> + if (errno != 0 && (errno != EINVAL))
>
> It might be better to do:
>
> if (errno == ERANGE)
> error ();
>
> In any case thanks for the fix.
> Pádraig.
I was thinking about that, my only concern was since we're already dealing with
non-standard code, should we worry about an even weirder implementation that
might return something that's not ERANGE and not EINVAL?
Re: new snapshot available: coreutils-8.20.113-1f1f4, Pádraig Brady, 2013/02/11
Re: new snapshot available: coreutils-8.20.119-54cdb0, Pádraig Brady, 2013/02/11
- Re: new snapshot available: coreutils-8.20.119-54cdb0, Assaf Gordon, 2013/02/11
- Re: new snapshot available: coreutils-8.20.119-54cdb0, Assaf Gordon, 2013/02/11
- Re: new snapshot available: coreutils-8.20.119-54cdb0, Assaf Gordon, 2013/02/11
- Re: new snapshot available: coreutils-8.20.119-54cdb0, Pádraig Brady, 2013/02/11
- Re: new snapshot available: coreutils-8.20.119-54cdb0,
Assaf Gordon <=
- Re: new snapshot available: coreutils-8.20.119-54cdb0, Pádraig Brady, 2013/02/12
- Re: new snapshot available: coreutils-8.20.119-54cdb0, Assaf Gordon, 2013/02/12
Re: new snapshot available: coreutils-8.20.119-54cdb0, Pádraig Brady, 2013/02/13