[FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

James Almer jamrial at gmail.com
Thu Jan 21 16:24:22 CET 2016


On 1/21/2016 6:52 AM, Moritz Barsnick wrote:
> 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

thread.h is an internal header and these are all inline functions, so
no need to bump anything.


More information about the ffmpeg-devel mailing list