[FFmpeg-devel] [PATCH] libavformat/mpegts.c minor crash and warning avoidance

Glenn A. Serre gaserre
Wed May 23 02:15:22 CEST 2007

Good afternoon,

Benoit Fouet wrote:
> Hi,
> Glenn A. Serre wrote:
>> Good evening,
>> This patch would be to avoid a compiler warning and avoid a crash that 
>> happened when I experimented with having mpegts add streams from all the 
>> PMTs in the transport stream, rather than just from the first one found.
> it is better to split patches
> the two ones you're proposing are not correlated, so they should be
> splitted.

OK.  What is protocol in these cases?  Should I reply with two 
attachments, make two replies, or ???

>> Please note that I can't actually check this in (or at least I shouldn't 
>>   be able to).  Also, please note that the revision number below is 
>> against our repository, not the official ffmpeg one.
> but it is not against svn head, so your patch cannot be applied as is

The diff was against a file imported from yesterdsy's head, and it does 
apply to the current svn head
(using patch -i ~/tmp/patch.txt libavformat/mpegts.c), but I'll resend 
svn diffs in the actual ffmpeg svn tree.

--Glenn S.

More information about the ffmpeg-devel mailing list