[FFmpeg-devel] [PATCHv3] lavf: add av_guess_frame_sample_aspect_ratio function
Joakim Plate
elupus at ecce.se
Thu May 17 11:36:13 CEST 2012
>>
>> No further comments received, please apply the v3 patch.
>
> locally applied, will be in my next push
>
Damn.. :). Missed updates here.
You have declared function as:
AVRational av_guess_sample_aspect_ratio(AVFormatContext *format,
AVStream *stream, AVFrame *frame)
This makes it hard to use the function if there is no AVFrame. (ie
directly after av_find_stream_info). Secondly i may be blind, but i
can't see that you use the format pointer at all in the function. Was
that just for future safety?
/Joakim
More information about the ffmpeg-devel
mailing list