[FFmpeg-devel] [PATCH] Handle variable pix_fmt

[Michael Niedermayer]

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()

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090626/086d8321/attachment.pgp>