<AWT Dev> [13] Review Request: 8220495 Update GIFlib library to the latest 5.1.8

Philip Race philip.race at oracle.com
Wed Mar 20 22:45:15 UTC 2019

I am surprised that giflib does not build on windows, we can't
be the only people using it on windows ?

Except for the legal header, I think your "patches" should be
inside a platform ifdef to show exactly where they are needed
and also be bracketed by comments

So something like this :
/** Begin JDK modifications to support building on Solaris **/
#ifdef SOLARIS
#endif /* SOLARIS */
/** End JDK modifications to support building on Solaris **/


On 3/20/19, 3:28 PM, Sergey Bylokhov wrote:
> Hello.
> Please review the fix for JDK 13.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8220495
> Fix: http://cr.openjdk.java.net/~serb/8220495/webrev.02
> The giflib was updated from 5.1.4 to 5.1.8.
> The is mostly a replacement of old files(5.1.4) by the new 
> files(5.1.8) except two additional steps.
>  - The library uses "<unistd.h>" which breaks the build on windows, 
> and "<stdbool.h>" wihch does not work properly on old Solaris.
>    It was fixed by this patch(same as in 5.1.4 during previous upgrade):
>    http://cr.openjdk.java.net/~serb/8220495/giflib
>  - I have drop the usage of "tabs" and manually reformat the code to 
> make the diff smaller.
> Mach5 on all platforms is green.

More information about the awt-dev mailing list