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

Clément Bœsch u at pkh.me
Sun Jan 24 14:03:40 CET 2016


On Thu, Jan 21, 2016 at 10:52:46AM +0100, 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.
> 

fixed locally, thanks

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

it could, but it sounds a bit overkill in this case

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160124/e56f0246/attachment.sig>


More information about the ffmpeg-devel mailing list