[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: [OpenJDK 2D-Dev] Fix integer overflow warnings in icu layout code
From: Omair Majid <omajid () redhat ! com>
Date: 2016-02-17 15:45:44
Message-ID: 20160217154544.GB3900 () redhat ! com
[Download RAW message or body]
* Omair Majid <omajid@redhat.com> [2016-01-27 15:35]:
> Hi,
>
> * Phil Race <philip.race@oracle.com> [2016-01-21 17:55]:
> > On 01/21/2016 01:50 PM, Omair Majid wrote:
> > >That said, do you think some sort of add-and-check-for-overflow test
> > >would be suitable here? I can take a shot at implementing it, if you
> > >like.
> >
> > Sure ! (to the code change option).
>
> The following webrev implements that:
> http://cr.openjdk.java.net/~omajid/webrevs/le-signed-overflow-warnings/00/
>
> It uses comparisons with INT32_MAX and INT32_MIN instead of performing
> an addition or subtraction on the variable itself to avoid an overflow.
> With this, I no longer get any overflow warnings.
>
> On the other hand, if you think these checks are redundant, then I would
> be happy to remove these too.
Ping. Anyone have any comments?
Thanks,
Omair
--
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95 0056 F286 F14F 6648 4681
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic