[MPlayer-dev-eng] [PATCH] new libvo driver: vo_ivtv [ask for review]

Diego Biurrun diego at biurrun.de
Wed Jul 12 13:36:13 CEST 2006


On Tue, Jul 11, 2006 at 09:24:36PM +0200, Benjamin Zores wrote:
> On Tue, 11 Jul 2006 10:16:13 +0200
> Reimar Doeffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> 
> > Ok, end of very quick surfacial review.
> 
> Attached a revised patch with all your comments taken into account.
> It is much simpler now. If it's ok for everyone, i'm gonna commit it on tomorrow.
> 
> --- libvo/vo_ivtv.c	(r??vision 0)
> +++ libvo/vo_ivtv.c	(r??vision 0)
> +
> +static uint32_t
> +ivtv_reset (int blank_screen)
> +{
> + 
> +  while (ioctl (ivtv_fd, IVTV_IOC_S_STOP_DECODE, &sd) < 0)
> +  {
> +    if (errno != EBUSY)
> +    {
> +      mp_msg (MSGT_VO, MSGL_ERR,
> +              "IVTV_IOC_STOP_DECODE: %s\n", strerror (errno));
> +      return 1;
> +    }
> +  }

Well, it's your code of course, but somehow I prefer putting function
type and name on one line as well as keeping the opening { on the same
line as if and while constructs.  It makes the code more compact.  It's
your decision.  I just looked over it superficially, all I found was
some trailing whitespace, IOW just some very minor nits..

> --- configure	(r??vision 19017)
> +++ configure	(copie de travail)
> @@ -4708,6 +4712,33 @@
>  
> +  if test "$_mp1e" = auto ; then
> +    # we don't need mp1e
> +    _mp1e=no

Heh, mp1e..  Does this thing still work nowadays?  I thought it was
looong obsolete..

Diego



More information about the MPlayer-dev-eng mailing list