[MPlayer-cvslog] r25079 - trunk/stream/tvi_dshow.c

[Vladimir Voroshilov]

2007/11/18, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> Hello,
> On Sat, Nov 17, 2007 at 08:08:03PM +0100, voroshil wrote:
> >  static const img_fmt img_fmt_list[] = {
> > -    {IMGFMT_YUY2, &MEDIASUBTYPE_YUY2, 0, IMGFMT_YUY2, 0},
> > -    {IMGFMT_YV12, &MEDIASUBTYPE_YV12, 0, IMGFMT_YV12, 0},
> > -    {IMGFMT_IYUV, &MEDIASUBTYPE_IYUV, 0, IMGFMT_IYUV, 0},
> > -    {IMGFMT_I420, &MEDIASUBTYPE_I420, 0, IMGFMT_I420, 0},
> > -    {IMGFMT_UYVY, &MEDIASUBTYPE_UYVY, 0, IMGFMT_UYVY, 0},
> > -    {IMGFMT_YVYU, &MEDIASUBTYPE_YVYU, 0, IMGFMT_YVYU, 0},
> > -    {IMGFMT_YVU9, &MEDIASUBTYPE_YVU9, 0, IMGFMT_YVU9, 0},
> > +    {IMGFMT_YUY2, &MEDIASUBTYPE_YUY2, 16, IMGFMT_YUY2, 0},
> > +    {IMGFMT_YV12, &MEDIASUBTYPE_YV12, 12, IMGFMT_YV12, 0},
> > +    {IMGFMT_IYUV, &MEDIASUBTYPE_IYUV, 12, IMGFMT_IYUV, 0},
> > +    {IMGFMT_I420, &MEDIASUBTYPE_I420, 12, IMGFMT_I420, 0},
> > +    {IMGFMT_UYVY, &MEDIASUBTYPE_UYVY, 16, IMGFMT_UYVY, 0},
> > +    {IMGFMT_YVYU, &MEDIASUBTYPE_YVYU, 16, IMGFMT_YVYU, 0},
> > +    {IMGFMT_YVU9, &MEDIASUBTYPE_YVU9, 9, IMGFMT_YVU9, 0},
> >      {IMGFMT_BGR32, &MEDIASUBTYPE_RGB32, 32, 0, 0},
> >      {IMGFMT_BGR24, &MEDIASUBTYPE_RGB24, 24, 0, 0},
> >      {IMGFMT_BGR16, &MEDIASUBTYPE_RGB565, 16, 3, 12},
>
> Wouldn't it be nicer to align all these fields as well? :-)

Obviously.
Done.

-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719