[FFmpeg-devel] [PATCH] Make ffmpeg dump output format in case of missing output stream

[Stefano Sabatini]

On date Tuesday 2008-07-29 00:13:17 +0200, Stefano Sabatini encoded:
> Hi, as in subject.
> 
> BTW the patch makes more evident this bug:
> stefano at geppetto ~/s/ffmpeg> 
> ffmpeg -i ~/test.flv -f rtp -map 0.1:0.0 rtp://localhost:5008 -map 0.0:1.0 -f avi -map 0.1:1.1 -y foo.avi
> [...]
> Input #0, flv, from '/home/stefano/test.flv':
>   Duration: 00:00:30.01, start: 0.000000, bitrate: 96 kb/s
>     Stream #0.0: Video: vp6f, yuv420p, 320x240, 25.00 tb(r)
>     Stream #0.1: Audio: mp3, 44100 Hz, mono, 96 kb/s
> Output #1, avi, to 'foo.avi':
>     Stream #1.0: Video: mpeg4, yuv420p, 320x240, q=2-31, 200 kb/s, 25.00 tb(c)
>     Stream #1.1: Audio: mp2, 44100 Hz, mono, 64 kb/s
> Could not find sync stream #1.0
> 
> Patch for it attached into another thread.
> 
> Also I suggest to change the term "sync stream" with the term "output
> stream" in code and docs, I think it's clearer and more consistent
> with the output of dump_format.
> 
> Regards.
> -- 
> FFmpeg = Fast and Freak Maxi Patchable Enigmatic Guru

> Index: ffmpeg.c
> ===================================================================
> --- ffmpeg.c	(revision 14462)
> +++ ffmpeg.c	(working copy)
> @@ -1567,6 +1567,7 @@
>          si = stream_maps[i].sync_stream_index;
>          if (fi < 0 || fi > nb_input_files - 1 ||
>              si < 0 || si > file_table[fi].nb_streams - 1) {
> +            dump_format(output_files[fi], fi, output_files[fi]->filename, 1);
>              fprintf(stderr,"Could not find sync stream #%d.%d\n", fi, si);
>              av_exit(1);
>          }

Ugh this is *wrong*, if the stream doesn't exist I cannot dump its
description...

(Note to self, avoid to send patches at night).

Sorry for the noise.

Regards.
-- 
FFmpeg = Fostering and Fantastic Monstrous Portable Everlasting Gadget