[FFmpeg-devel] [PATCH 2/2] fftools: add -report option.

Nicolas George nicolas.george at normalesup.org
Thu Dec 8 16:23:56 CET 2011


L'octidi 18 frimaire, an CCXX, Clément Bœsch a écrit :
> Nice idea.

Thanks.

>	     Maybe we could detect a crash just like mplayer and request the
> user to "-report" the bug.

That is a possibility I thought about, but in the end, I do not like the
idea of tinkering with signal handlers unnecessarily: it makes getting
backtraces harder.

> I think you should conditionally use localtime_r (see in vf_drawtext).

I think it would not be worth the additional clutter: unlike drawtext, this
piece of code is not meant to be called repeatedly from various threads but
only once at startup.

> Note: we might want to update http://ffmpeg.org/bugreports.html after that.

That was the next step.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111208/9f608669/attachment.asc>


More information about the ffmpeg-devel mailing list