[FFmpeg-devel] [PATCH 1/2] lavu: add thread message API.

[Nicolas George]

Le nonidi 29 germinal, an CCXXII, Michael Niedermayer a écrit :
> this will fail if there are no pthreads

You are right. I remembered wrong how the compat system works. Updated
version attached.

Clément:

> (just for the context, we are talking about av_thread_message_queue_send()
> returning ENOSYS in case HAVE_THREADS is false)
> 
> I was under the impression the function calling
> av_thread_message_queue_send() was called even without threading,
> typically assuming that lavu/threadmessage.c would be a working API even
> without threading if possible. I guess I was wrong.

Even if av_thread_message_queue_send() were to be called, it would fail
since av_thread_message_queue_alloc() would have failed too and returned a
NULL structure. I tested the last version with --disable-threads, and it
worked, so I guess the #ifdef are correct.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavu-add-thread-message-API.patch
Type: text/x-diff
Size: 10566 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140420/0d9fa9ab/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140420/0d9fa9ab/attachment.asc>