On Fri, 20 Mar 2026 23:16:23 GMT, Alexander Matveev <[email protected]> 
wrote:

>> modules/javafx.media/src/main/native/gstreamer/gstreamer-lite/gstreamer/gst/gstvecdeque.c
>>  line 112:
>> 
>>> 110: 
>>> 111:   array = gst_vec_deque_new_for_struct (sizeof (gpointer), 
>>> initial_size);
>>> 112: #ifdef GSTREAMER_LITE
>> 
>> Question: are all the callers of this method prepared to handle `NULL` ?
>> (here and in all other PRs)
>
> In most cases yes. I did check as much as possible, but it is possible that I 
> miss something.

gst_vec_deque_new_for_struct() will never return NULL, since we passing `sizeof 
(gpointer)` which cannot be less then 0. No need to fix it.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2115#discussion_r3025160567

Reply via email to