[FFmpeg-devel] [PATCH]Set sample_aspect_ratio in real demuxer

Paul B Mahol onemda at gmail.com
Thu Dec 20 12:27:31 CET 2012


On 12/20/12, Nicolas George <nicolas.george at normalesup.org> wrote:
> Le decadi 30 frimaire, an CCXXI, Carl Eugen Hoyos a ecrit :
>> Isn't this an independent bug?
>
> I do not think so. If there is a bug at work here, it may be to set
> codec->width and codec->height from these fields in the container. Do you
> know whether there is an authoritative documentation available?
>
>> What I want to fix is that ffmpeg -i file shows
>> "unknown DAR" while it is actually known.
>
> I know you want to fix that, but that precise fix is invalid and causes
> worse problems later, i.e. an invalid aspect ratio instead of an unknown
> one.
>
>> Do you think there is a situation where is does not work?
>
> I had just given an example.
>
>
> As a side note, IIRC, Git commit messages are supposed to be "context:
> short
> summary\n\nLonger summary if necessary." In this particular case, that
> would
> be something like "lavf/rmdec: set sample_aspect_ratio".

I'm for making this as rule/policy.


More information about the ffmpeg-devel mailing list