[OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c
thomas.stuefe at gmail.com
Wed Mar 21 18:57:04 UTC 2018
thanks for pointing out the history, I was not aware of that.
I looked at that huffman coding and tried to determine whether the
underflow may happen in real life scenarios. I could at least not exclude
that possibility. I looked thru the mailing list threads - did someone
analyse and conclude for sure this was just a pointless compiler warning?
I would prefer the pragmatic solution (and IMHO also safer one) of fixing
this underflow in the proposed fashion. I had opened a bug report earlier
today. However, if someone already spent brain cycles on it and a patch -
in whatever form - is forthcoming, I do not want to butt in. In that case I
will close this bug again.
I would just like to see this fixed this because it affects us at SAP too.
Kind Regards, Thomas
On Wed, Mar 21, 2018 at 6:56 PM, Phil Race <philip.race at oracle.com> wrote:
> I prefer the makefile fix, since we don't by policy, make changes to the
> imported libraries.
> On Jan 23rd  I expressed such a tool-chain specific makefile fix would
> be fine by me.
> Toolchain specific means ideally it would look like what Magnus wrote 
> Although you said GC 5.4.0 would need to be included in the logic.
> If it can be shown to affect current / future versions of gcc then it
> could be unqualified.
> I think we've just been waiting for a webrev since then ..
>  http://mail.openjdk.java.net/pipermail/2d-dev/2018-January/008855.html
>  http://mail.openjdk.java.net/pipermail/build-dev/2018-
> On 03/21/2018 09:53 AM, Adam Farley8 wrote:
> > Hi Adam,
> > no problem. I'll open a bug and if necessary find a second reviewer.
> Thanks for fixing, maybe I can stop building with warnings disabled on our
> s390 machines now.
> > ..Thomas
> > > On Wed, Mar 21, 2018 at 5:10 PM, Andrew Leonard
> <andrew_m_leonard at uk.ibm.com> <andrew_m_leonard at uk.ibm.com> wrote:
> > > Hi Thomas,
> > > I'm a "contributor", but not a "committer", so not on that list,
> didn't even know that
> > > list existed! I was sort of assuming since it was a trivial change,
> and the request was
> > > for a review, i'd chip in...!
> > > Thanks
> > > Andrew
> > > Andrew Leonard
> > > Java Runtimes Development
> > > IBM Hursley
> > > IBM United Kingdom Ltd
> > > Phone internal: 245913, external: 01962 815913
> > > internet email: andrew_m_leonard at uk.ibm.com
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the 2d-dev