On Wed, 23 Apr 2025 23:26:23 GMT, Alexander Matveev <almat...@openjdk.org>
wrote:
> - Fixed gcc 14 compiler errors.
>
>
> ../../../gstreamer-lite/gstreamer/gst/gstclock.c: In function
> 'gst_clock_entry_new':
> ../../../gstreamer-lite/gstreamer/gst/gstclock.c:178:48: error: passing
> argument 1 of 'g_weak_ref_init' from incompatible pointer type
> [-Wincompatible-pointer-types]
> 178 | #define GST_CLOCK_ENTRY_CLOCK_WEAK_REF(entry) (&((GstClockEntryImpl
> *)(entry))->clock)
> | ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> | |
> | GWeakRef **
> ../../../gstreamer-lite/gstreamer/gst/gstclock.c:274:20: note: in expansion
> of macro 'GST_CLOCK_ENTRY_CLOCK_WEAK_REF'
> 274 | g_weak_ref_init (GST_CLOCK_ENTRY_CLOCK_WEAK_REF (entry), clock);
>
>
>
> ../../../plugins/av/videodecoder.c: In function 'videodecoder_dispose':
> ../../../plugins/av/videodecoder.c:202:31: error: passing argument 1 of
> 'basedecoder_close_decoder' from incompatible pointer type
> [-Wincompatible-pointer-types]
> 202 | basedecoder_close_decoder(decoder);
>
>
>
> ../../../plugins/av/videodecoder.c: In function 'videodecoder_convert_frame':
> ../../../plugins/av/videodecoder.c:548:50: error: passing argument 2 of
> 'decoder->sws_scale_func' from incompatible pointer type
> [-Wincompatible-pointer-types]
> 548 | base->frame->data,
> | ~~~~~~~~~~~^~~~~~
> | |
> | uint8_t ** {aka unsigned char **}
> ../../../plugins/av/videodecoder.c:548:50: note: expected 'const uint8_t *
> const*' {aka 'const unsigned char * const*'} but argument is of type 'uint8_t
> **' {aka 'unsigned char **'}
Changes look good. I can now compile using gcc14.2
-------------
Marked as reviewed by kcr (Lead).
PR Review: https://git.openjdk.org/jfx/pull/1794#pullrequestreview-2792633904
