On Sun, Apr 22, 2018 at 09:33:04AM +0200, Clément Bœsch wrote: > On Sun, Apr 22, 2018 at 02:51:16AM +0100, Rostislav Pehlivanov wrote: > [...] > > I think av_thread_message_queue_elems would be a better name, had to think > > for a good period of time what "nelem" meant. > > I'm afraid of "queue_elems" implying "queuing elements" so I went for > the more explicit av_thread_message_queue_nb_elems() instead. >
> Also fixed the problem raised by Michael. confirmed [...] > diff --git a/libavutil/threadmessage.h b/libavutil/threadmessage.h > index 8480a0a3db..7ebf03a8ac 100644 > --- a/libavutil/threadmessage.h > +++ b/libavutil/threadmessage.h > @@ -95,6 +95,11 @@ void > av_thread_message_queue_set_err_recv(AVThreadMessageQueue *mq, > void av_thread_message_queue_set_free_func(AVThreadMessageQueue *mq, > void (*free_func)(void *msg)); > > +/** > + * Return the current number of messages in the queue. > + */ doesnt document the AVERROR case should be ok otherwise, no more comments from me thx also welcome back ! [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB "Nothing to hide" only works if the folks in power share the values of you and everyone you know entirely and always will -- Tom Scott
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel