[MPlayer-dev-eng] [PATCH] fix for v4l1 driver

Vladimir Voroshilov voroshil at gmail.com
Mon May 21 19:15:44 CEST 2007


2007/4/29, Trent Piepho <xyzzy at speakeasy.org>:
> On Sun, 29 Apr 2007, Vladimir Voroshilov wrote:
> > 2007/4/28, Trent Piepho <xyzzy at speakeasy.org>:
> >
> > >@@ -798,6 +770,53 @@
> > >     if (ioctl(priv->video_fd, VIDIOCSPICT, &priv->picture) == -1)
> > >     {
> > >     mp_msg(MSGT_TV, MSGL_ERR, "ioctl set picture failed: %s\n", strerror(errno));
> > >+    mp_msg(MSGT_TV, MSGL_ERR, "The 'outfmt' of '%s' is likely not
> > supported by your card\n",
> > >+           vo_format_name(priv->format));
> > >+    return 0;
> >
> > Why don't you change existing message?
> > Other parts looks ok for for me at quick-look.
>
> It seemed like the existing message was still relevant, since it says what
> ioctl failed and the error code.
>
> >
> > In another cosmetics patch you also change comment in disabled block of
> > code.
> > Can i know the reason for it ?
>
> I thought it was a call to VIDIOSCAPTURE instead of VIDIOCCAPTURE, but it's
> not so I've taken that change out.
>
> I see you pulled the typo fix out of my patch and put in already, so I won't
> include that again in my patch.

since no objections received, patch applied.
Thanks for your work.


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



More information about the MPlayer-dev-eng mailing list