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
