On Wed, Jan 20, 2016 at 23:46:02 +0100, Clément Bœsch wrote:
> Subject: [PATCH] lavu: add ff_pthread_setname() and use it in various places
[...]
> +static inline void ff_thread_setname(const char *name)

The commit message is out of line with the adapted function name.

Shouldn't the introduction of the new functionality and its use in
various threads be split into two commits? Just wondering.

And shouldn't libavutil's version be minor-bumped, so that others
using it/wanting to use it could in theory check for availability?

Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to