[jdk8u-dev] RFR: 8328999: Update GIFlib to 5.2.2 [v3]

[Severin Gehwolf]

On Thu, 10 Oct 2024 22:13:32 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:

>> This updates the in-tree giflib to 5.2.2. The actual code changes are a clean backport, but the license file changes needed adapting from the `giflib.md` file in 11u and later to the `THIRD_PARTY_README` file in 8u (now only one copy following [JDK-8338144](https://bugs.openjdk.org/browse/JDK-8338144))
>> 
>> 8u is also missing [JDK-8225487](https://bugs.openjdk.org/browse/JDK-8225487) which I've included as part of this patch. This updates the license to credit `openbsd-reallocarray.c` and is updated further by 8238999. I also included changes made by the 5.2.1 update which were missed because 8225487 was missing.
>> 
>> In `gif_lib.h`, I have restored the workaround for a missing `stdbool.h` on Windows only. This is needed to build on Visual Studio 2010.
>> 
>> The `giflib.md` changes include some odd changes, replacing the giflib VCS URL with just `tree/README`. I've raised [8339271](https://bugs.openjdk.org/browse/JDK-8339271) to correct this and will look at backporting that as a follow-up.
>
> Andrew John Hughes has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add missing #endif

Looks good to me.

-------------

Marked as reviewed by sgehwolf (Reviewer).

PR Review: https://git.openjdk.org/jdk8u-dev/pull/571#pullrequestreview-2393212433