RFR: 8262366: Update glib to version 2.66.7

Kevin Rushforth kcr at openjdk.java.net
Thu Apr 1 12:22:14 UTC 2021


On Thu, 1 Apr 2021 11:04:35 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> It compiles for me (on gcc-9, 10.1 and 10.2) but I can imagine it doesn't compile on all configurations. `memset` is declared in `string.h` and that may or may not be included. 
>> Running gcc -H shows that on my system, `string.h` is included by `glib-2.0/glib/gtestutils.h` (which is included by `glib-2.0/glib.h`) but on other systems, that might not be the case.
>> Since this PR introduces `memset` in the `audio-buffer.c` file, I think it is safest to include `string.h` in this file as well, as this removes the dependency on that file to be included indirectly.
>
> Alexander tried it on our CI build machine yesterday and it builds fine there, too. I should add that I was building on a very old Ubuntu 16.04 system.
> 
>> Since this PR introduces memset in the audio-buffer.c file, I think it is safest to include string.h in this file as well, as this removes the dependency on that file to be included indirectly.
> 
> I was thinking the same thing. There is one more file that should include `<string.h>`: `qtdemux_tags.c` uses `strlen`and fails to compile on my system. With the change to include `string.h` in those two files, the build now passes on my system.

All my testing looks good on all three platforms. I'll take a look at the diffs next.

One thing I did spot is that you need to update `gstreamer.md` and `glib.md` to bump the version numbers.

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

PR: https://git.openjdk.java.net/jfx/pull/447


More information about the openjfx-dev mailing list