Re: [FFmpeg-devel] threadmessage improvements v2

2015-12-02 Thread Nicolas George
Le duodi 12 frimaire, an CCXXIV, Clement Boesch a écrit :
> - the flushing function can be set through a dedicated function instead of a
>   new constructor prototype

If I read the patch correctly, you seem to have fumbled your rebasing. The
deprecation is still there and the setter function is not.

Regards,

-- 
  Nicolas George


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] threadmessage improvements v2

2015-12-02 Thread Clément Bœsch
On Wed, Dec 02, 2015 at 04:01:40PM +0100, Nicolas George wrote:
> Le duodi 12 frimaire, an CCXXIV, Clement Boesch a écrit :
> > - the flushing function can be set through a dedicated function instead of a
> >   new constructor prototype
> 
> If I read the patch correctly, you seem to have fumbled your rebasing. The
> deprecation is still there and the setter function is not.

Yeah I sent the wrong patchset, see v2.1. Sorry about that. It seems git
also sent 2 empty compose message for whatever reason.

Sorry about the noise.

-- 
Clément B.


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] threadmessage improvements v2

2015-12-02 Thread Clément Bœsch
Improvements & changes regarding last iteration:

[PATCH 1/3] avutil/threadmessage: add av_thread_message_flush()

- the flushing function can be set through a dedicated function instead of a
  new constructor prototype
- the flush function is now fixed regarding messages larger than a pointer...

[PATCH 2/3] fate/api: test threadmessage

- the test program is greatly improved by allowing multiple readers in addition
  to multiple senders.
- the flush now occurs randomly in readers and senders
- the messages are now larger than a pointer (see bug fix in 1/3) with a magic
  check

[PATCH 3/3] avutil/threadmessage: fix condition broadcasting

- instead of an inefficient wide signal broadcasting, it's now using 2 distinct
  conditions for reading and sending
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel