On Mon, Dec 04, 2017 at 02:53:36 +0000, Colin NG wrote: > fftools/ffprobe: Fix for ticket 6796 (ffprobe show_frames ts dvbsubs infinite > loop)
You are supposed to primarily write *what* you changed, not why. The reference to the ticket belongs in thew "body" of the commit message. Like: -- snip -- fftools/ffprobe: skip blabla for subtitle frames (I'm making this up) Avoids infinite looping for "ffprobe -show_frames" on mpegts files with dvbsubs. Fixes #6796. -- snip -- > +++ b/fftools/ffprobe.c > @@ -2280,6 +2280,7 @@ static av_always_inline int process_frame(WriterContext > *w, > break; > default: > *packet_new = 0; > + break; > } This is an irrelevant style change, right? > int is_sub = (par->codec_type == AVMEDIA_TYPE_SUBTITLE); > nb_streams_frames[pkt->stream_index]++; > + got_frame = (par->codec_type == AVMEDIA_TYPE_SUBTITLE) ? 0: > got_frame; Two lines above, "is_sub" was introduced as a handy shortcut. Why not just use it? Moritz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel