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. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
