[FFmpeg-devel] [PATCH] Handle variable pix_fmt

[Jason Garrett-Glaser]

On Thu, Jun 25, 2009 at 5:50 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
> On Thu, Jun 25, 2009 at 05:44:15PM -0700, Jason Garrett-Glaser wrote:
>> 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?
>
> use avcodec_get_pix_fmt_name()
>

Applied with that instead.

Dark Shikari