[FFmpeg-devel] [PATCH] ffmpeg: remove unused sync_ist attribute from OutputStream

Michael Niedermayer michaelni at gmx.at
Fri Mar 23 05:02:09 CET 2012


On Wed, Mar 21, 2012 at 02:23:44PM +0100, Matthieu Bouron wrote:
> Hi,
> 
> This patch remove the sync_ist attribute from OutputStream.
> sync_ist is not used anymore in ffmpeg.c. Usage was removed by commit:
> d97fa416d62fe00a69e5be947d7c51b71982dd12.

theres more dead code resulting from this, up to the parsing of the
sync streams. I think it would be better to fix the code unless it
has no use case

maybe wolfram could comment on the usecase part? he was the author of
this IIRC
also if we keep it we need a regression test so this doesnt happen
again

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120323/b5046cd1/attachment.asc>


More information about the ffmpeg-devel mailing list