[Ffmpeg-devel] [PATCH] flacenc - lpc and options
Sun Jul 2 05:33:22 CEST 2006
Michael Niedermayer wrote:
>>>>+ * sets audio frame size using frame duration in milliseconds
>>>>+ * - encoding: set by user.
>>>>+ * - decoding: unused.
>>>>+ int block_time_ms;
>>>iam a little unsure about this, maybe you could leave this one out for now
>>>and set it just based on compression_level
>>This was to give the user the option to set the block size. Do you think
>>the user should instead be able to set AVCodecContext.frame_size
>>explicitly before calling encode_init()? Currently it is set by the
>>encoder only, but i see no reason not to let the user set it. The
>>encoders currently don't check the value anyway and would just
>>over-write it with what it's supposed to be. Would there be anything
>>wrong with some encoders checking first for a user-preferred value? I
>>could change the comment to something like:
>> * samples per packet.
>> * - encoding: set by user, but may be changed by 'init'.
>> * - decoding: set by lavc.
>>or would it be better to let the user set it and return an error if it
>>is an invalid value for that codec, like is done with most of the other
> hmm, iam more in favor of check & error
I took out block_time_ms and switched to using frame_size which already
has the ability to be set using AVOption. If frame_size > 0, it is used
for the block size. Otherwise, a predefined block size is used based on
the compression level. An error is returned if frame_size is
out-of-range for a valid FLAC file.
I have made a few other very minor changes as well.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the ffmpeg-devel