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

[jamal]

On 07/12/12 5:00 AM, Carl Eugen Hoyos wrote:
> James Almer <jamrial <at> gmail.com> writes:
> 
>> +  { "astflags",  "AST Muxer flags", OFFSET(flags), AV_OPT_TYPE_FLAGS, 
>> {.i64 = 0}, INT_MIN, INT_MAX, AV_OPT_FLAG_ENCODING_PARAM, "astflags" },
>> +  { "loop",      "Enable looping",  0,             AV_OPT_TYPE_CONST, 
>> {.i64 = 1}, INT_MIN, INT_MAX, AV_OPT_FLAG_ENCODING_PARAM, "astflags" },
>>    { "loopstart", "Loopstart position in milliseconds.", OFFSET(loopstart), 
>> AV_OPT_TYPE_INT64, { .i64 = 0 }, 0, INT_MAX, AV_OPT_FLAG_ENCODING_PARAM },
>>    { "loopend",   "Loopend position in milliseconds.",   OFFSET(loopend), 
>> AV_OPT_TYPE_INT64, { .i64 = 0 }, 0, INT_MAX, AV_OPT_FLAG_ENCODING_PARAM },
> 
> Shouldn't instead a different value be written depending 
> on what the user set for loopstart and loopend?

The user might want to set no loop information (Meaning loopstart = 0 and loopend = total samples) but still enable the loop flag so the player loops through the whole file.

Regards.