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

Clément Bœsch u at pkh.me
Wed Jan 20 21:54:57 CET 2016


On Wed, Jan 20, 2016 at 09:50:40PM +0100, Nicolas George wrote:
> Le primidi 1er pluviôse, an CCXXIV, Clement Boesch a écrit :
> > ---
> >  libavcodec/frame_thread_encoder.c | 2 ++
> >  libavcodec/pthread_slice.c        | 2 ++
> >  libavfilter/pthread.c             | 2 ++
> >  libavformat/async.c               | 2 ++
> >  libavformat/udp.c                 | 3 +++
> >  libavutil/thread.h                | 9 +++++++++
> >  6 files changed, 20 insertions(+)
> > 
> > diff --git a/libavcodec/frame_thread_encoder.c b/libavcodec/frame_thread_encoder.c
> > index f4d35f9..6c3b048 100644
> > --- a/libavcodec/frame_thread_encoder.c
> > +++ b/libavcodec/frame_thread_encoder.c
> > @@ -62,6 +62,8 @@ static void * attribute_align_arg worker(void *v){
> >      ThreadContext *c = avctx->internal->frame_thread_encoder;
> >      AVPacket *pkt = NULL;
> >  
> 
> > +    ff_pthread_setname("ffmpeg frame worker");
> > +
> 
> The Linux man page says "length is restricted to 16 characters".
> 

dammit, that's why it didn't show up all after adding "ffmpeg". Thanks...

> Also, I suggest to make it double-click-friendly, i.e. use underscores
> instead of spaces.

Mmh, OK, but many apps seems not to care about the spaces.

-- 
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/20160120/37b2eba4/attachment.sig>


More information about the ffmpeg-devel mailing list