Thank you Christoph! On Tue, Mar 27, 2018 at 2:30 PM, Langer, Christoph <christoph.lan...@sap.com > wrote:
> Hi Thomas, > > > > to me this fix looks good. > > > > Best regards > > Christoph > > > > *From:* 2d-dev [mailto:2d-dev-boun...@openjdk.java.net] *On Behalf Of *Thomas > Stüfe > *Sent:* Mittwoch, 21. März 2018 18:20 > *To:* 2d-dev <2d-dev@openjdk.java.net> > *Cc:* Andrew Leonard <andrew_m_leon...@uk.ibm.com>; Adam Farley8 < > adam.far...@uk.ibm.com> > *Subject:* [OpenJDK 2D-Dev] RFR(xxxs): 8200052: libjavajpeg: Fix compile > warning in jchuff.c > > > > Hi all, > > > > may I please have another review for this really trivial change. It fixes > a gcc warning on s390 and ppc. Also, it is probably a good idea to fix this. > > > > bug: https://bugs.openjdk.java.net/browse/JDK-8200052 > > webrev: http://cr.openjdk.java.net/~stuefe/webrevs/ > 8200052-fix-warning-in-jchuff.c/webrev.00/webrev/ > > > > This was contributed by Adam Farley at IBM. > > > > I already reviewed this. I also test-built on zlinux and it works. > > > > Thanks, Thomas >
