[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Check in src/intervals.c, offset_intervals
From: |
Paul Eggert |
Subject: |
Re: Check in src/intervals.c, offset_intervals |
Date: |
Tue, 14 Aug 2012 01:33:14 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:14.0) Gecko/20120714 Thunderbird/14.0 |
On 08/08/2012 04:56 AM, Andreas Schwab wrote:
> Unless length is TYPE_MINIMUM (ptrdiff_t) (but the length is never
> supposed to be negative).
In that particular context, 'length' can be negative.
I don't think 'length' can be PTRDIFF_MIN there. The IF_LINT test
pacifies "gcc -O2 -Werror=strict-overflow". This warning flag is
problematic because it sometimes has false alarms (as
here) but it has found many bugs for me. Perhaps
I should give up on that warning flag, in which case the
IF_LINT test can be removed.