[FFmpeg-devel] [PATCH 2/2] lavd/sdl: reset context variables after destroy

Nicolas George george at nsup.org
Fri Feb 7 17:08:23 CET 2014


Le nonidi 19 pluviôse, an CCXXII, Lukasz M a écrit :
> Yes, I investigated.
> write_trailer is called for the first time in write_packet when quit flag
> is set.
> Second write_trailer comes from ffmpeg.c
> 
> I was thinking about removing that call from write_packet too, seems
> redundant.

I see, thanks for explaining. I do not know whether this is redundant (there
is unfortunately no common cleanup code for muxers, that would be called
even when muxing fails), but it is clearly wanted and not a bug, and this
patch is therefore necessary and correct. Maybe copy-paste this message to
the commit message.

Regards,

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


More information about the ffmpeg-devel mailing list