Hi Thomas, to me this fix looks good.
Best regards Christoph From: 2d-dev [mailto:[email protected]] On Behalf Of Thomas Stüfe Sent: Mittwoch, 21. März 2018 18:20 To: 2d-dev <[email protected]> Cc: Andrew Leonard <[email protected]>; Adam Farley8 <[email protected]> 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
