[FFmpeg-devel] [PATCH] Handle variable pix_fmt

Jason Garrett-Glaser darkshikari
Fri Jun 26 02:44:15 CEST 2009


On Thu, Jun 25, 2009 at 5:41 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
> On Thu, Jun 25, 2009 at 03:27:12PM -0700, Jason Garrett-Glaser wrote:
>> On Thu, Jun 25, 2009 at 3:24 PM, Jason
>> Garrett-Glaser<darkshikari at gmail.com> wrote:
>> > Works nicely with the framework added by variable frame-size support.
>> >
>> > Fixes, at a minimum, issue1165.
>> >
>> > Dark Shikari
>> >
>>
>> Seems even that was unnecessarily complicated. ?Simplified patch attached.
>
> ok

I just noticed that the printf is now not sufficiently descriptive.

Currently, it is:

fprintf(stderr,"Input Stream #%d.%d frame size changed to %dx%d\n",
ist->file_index, ist->index, ist->st->codec->width,
ist->st->codec->height);

Would this be fine?

fprintf(stderr,"Input Stream #%d.%d frame size changed to %dx%d,
%s\n", ist->file_index, ist->index, ist->st->codec->width,
ist->st->codec->height,sws_format_name(ist->st->codec->pix_fmt));

The problem with this is that sws_format_name isn't exported from
libswscale... what should I do?

Dark Shikari



More information about the ffmpeg-devel mailing list