[FFmpeg-devel] [PATCH] astenc: Add an option to set the loop flag

jamal jamrial at gmail.com
Wed Dec 12 19:49:53 CET 2012

On 12/12/12 6:19 AM, Carl Eugen Hoyos wrote:
> Please do not re-indent these lines, it makes the review 
> unnecessarily more difficult.
> Sorry for not answering your last mail:
> I wondered if values like "-1" cannot be used to indicate 
> "no loop"?
> Carl Eugen

In which option you mean using -1? Either loopend or loopstart? That seems less intuitive than
an option that's essentially "Enable loop"...

Otherwise I'm really not seeing what you mean. The user should have the option to set only the flag,
or both the flag and the loop values.
I could make it so providing any of the loop values automatically enables the flag (instead of
ignoring the values altogether), but an option to enable the flag alone would still be needed.


More information about the ffmpeg-devel mailing list