Le sextidi 6 frimaire, an CCXXIV, wm4 a écrit : > Better do the looping internal in flvdec.c (if there's no huge number > of other demuxers which need this), instead of adding a new error code > that is also part of the public API.
There are a few, but not many. This was my first intent, but looping in the framework is way more elegant and much cleaner design. Nothing would prevent us from defining the error code in libavformat/internal.h if we make sure it does not leaks to the application, but as Michael mentioned earlier and I did in the comments for patch 2/3 in this series, having a public code allows to give the application fine-grained control over the looping, for example to maintain basic user interaction while a demuxer tries to find a sync word in a big corrupted file on a slow local medium. Therefore, I maintain the proposed patch as is. Regards, -- Nicolas George
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
