[FFmpeg-cvslog] r15043 - trunk/libavformat/utils.c
Baptiste Coudurier
baptiste.coudurier
Sat Aug 30 00:31:14 CEST 2008
Hi,
Ronald S. Bultje wrote:
> Hi,
>
> On Fri, Aug 29, 2008 at 6:13 PM, bcoudurier <subversion at mplayerhq.hu> wrote:
>> make has_codec_parameters not returning true when sample_fmt is not set
> [..]
>> @@ -1842,7 +1842,7 @@ static int has_codec_parameters(AVCodecC
>> int val;
>> switch(enc->codec_type) {
>> case CODEC_TYPE_AUDIO:
>> - val = enc->sample_rate && enc->channels;
>> + val = enc->sample_rate && enc->channels && enc->sample_fmt != SAMPLE_FMT_NONE;
>> if(!enc->frame_size &&
>> (enc->codec_id == CODEC_ID_VORBIS ||
>> enc->codec_id == CODEC_ID_AAC))
>
> Doesn't this mean, since s_fmt is codec-dependent (and can thus only
> be set by the codec) that we always have to open the codec and that
> demuxer header parsing is never sufficient to pass find_stream_info()?
> If so, I would guess that this is bad.
>
Video already does that ... see pix_fmt.
--
Baptiste COUDURIER GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc. http://www.smartjog.com
Key fingerprint 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
More information about the ffmpeg-cvslog
mailing list