+1 Gruß, Thomas
On Wed, Feb 19, 2020, 10:35 Baesken, Matthias <matthias.baes...@sap.com> wrote: > Hello Thomas / Lance / Martin, thanks for the reviews . > > I added a little helper function, new webrev : > > > > http://cr.openjdk.java.net/~mbaesken/webrevs/8239351.1/ > > > > Best regards, Matthias > > > > > > > > I like this too. +1 for factoring out throwing the error. > > > > ..Thomas > > > > On Tue, Feb 18, 2020 at 6:51 PM Martin Buchholz <marti...@google.com> > wrote: > > Thanks for doing this. Looks good to me. > I would probably create a tiny helper function to encapsulate the error > throw. > > On Tue, Feb 18, 2020 at 7:03 AM Baesken, Matthias < > matthias.baes...@sap.com> > wrote: > > > Hello, please review this change to Deflater.c . > > > > When running the jtreg test java/util/zip/DeInflate.java , we currently > > have errors on SLES 15.1 s390x when using the system zlib (1.2.11), while > > the bundled zlib (of OpenJDK) seems to be okay. > > What's worse, the error messages are not very meaningful / helpful . > > > > We just get > > java.lang.InternalError > > at java.base/java.util.zip.Deflater.deflateBytesBytes(Native Method) > > at java.base/java.util.zip.Deflater.deflate(Deflater.java:595) > > at java.base/java.util.zip.Deflater.deflate(Deflater.java:474) > > ... > > without more any meaningful error text. > > > > I would suggest to improve a bit the error messages in Deflater.c ; this > > would lead in this case to : > > > > java.lang.InternalError: unknown error in checkDeflateStatus, setParams > > case > > at > > java.base/java.util.zip.Deflater.deflateBytesBytes(Native Method) > > at > > java.base/java.util.zip.Deflater.deflate(Deflater.java:586) > > at > > java.base/java.util.zip.Deflater.deflate(Deflater.java:465) > > > > > > Bug/webrev : > > > > https://bugs.openjdk.java.net/browse/JDK-8239351 > > > > http://cr.openjdk.java.net/~mbaesken/webrevs/8239351.0/ > > > > > > Thanks, Matthias > > > >