<AWT Dev> [9] Review request for 8077982: GIFLIB upgrade
Alexander Zvegintsev
alexander.zvegintsev at oracle.com
Fri Apr 17 18:10:36 UTC 2015
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8077982/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8077982
This fix is GIFLIB upgrade to the latest version 5.1.1 [1]
webrev against GIFLIB 5.1.1 available here [2]
Please see some explanations below:
GIFLIB 5.0.0+ supports interlaced images properly,
so we don't need to handle it in splashscreen_gif.c anymore (pass = 4;
npass = 5).
We compile JDK with Microsoft's compilers on Windows, there is no
unistd.h[3] (but Cygwin/MinGW has).
stdbool.h also isn't supported [4]. However I got a lot of strange build
errors on Solaris with included stdbool.h and -std=gnu99.
So it was replaced with own definition of bool.
[1]
http://sourceforge.net/projects/giflib/files/giflib-5.1.1.tar.gz/download
[2] http://cr.openjdk.java.net/~azvegint/jdk/9/8077982/giflib/00/
[3]
http://stackoverflow.com/questions/341817/is-there-a-replacement-for-unistd-h-for-windows-visual-c/1759731#1759731
[4]
http://stackoverflow.com/questions/8548521/trying-to-use-include-stdbool-h-in-vs-2010/8549206#8549206
--
Thanks,
Alexander.
More information about the awt-dev
mailing list