[FFmpeg-devel] [PATCH] streamid does not work with newaudio, newvideo, newsubtitle (issue 2465)

Michael Niedermayer michaelni
Sun Jan 16 03:34:08 CET 2011


On Sat, Jan 15, 2011 at 12:11:55PM -0500, Mike Scheutzow wrote:
> Attached is a patch to fix roundup bug 2465.
>
> The problem is that the ffmpeg (the app) -streamid option did not work  
> with -newaudio/-newvideo/-newsubtitle.
>
> The cause was a conflict between the feature where streamid values were  
> reset to default for each output filename, and the implementation of  
> -new*, which requires that the -new* option be specified after the  
> target filename.
>
> My patch changes the ffmpeg behavior so that user-specified streamid  
> values apply to all the following output files on the command line  
> (rather than just the next output filename.)
>
> Mike Scheutzow
>
>

>  doc/ffmpeg.texi |    6 ++++--
>  ffmpeg.c        |    1 -
>  2 files changed, 4 insertions(+), 3 deletions(-)
> 3bb5be9c244fbe21abaff8eb6916aa1ba743f967  streamid_fix_v1.patch

seems like the simplest fix so ok if it work and passes tests

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110116/22ee3046/attachment.pgp>



More information about the ffmpeg-devel mailing list