[MPlayer-dev-eng] [PATCH] Remove ugly #include from tvi_bsdbt848.c

[Vladimir Voroshilov]

2007/4/9, Diego Biurrun <diego at biurrun.de>:
> On Mon, Apr 09, 2007 at 12:53:01AM +0700, Vladimir Voroshilov wrote:
> > Hm. forgot to attach file.
> >
> > --- stream/tvi_bsdbt848.c     (revision 22945)
> > +++ stream/tvi_bsdbt848.c     (working copy)
> > @@ -38,20 +38,18 @@
> >
> >  #include <sys/param.h>
> > -#ifdef __NetBSD__
> > -#include <dev/ic/bt8xx.h>
> > +#ifdef USE_SUN_AUDIO
> >  #include <sys/audioio.h>
> >  #endif
>
> Are you sure NetBSD does not need the first header?
It is included later in code:
+#ifdef IOCTL_BT848_H_NAME
+#include IOCTL_BT848_H_NAME
+#endif


> > --- configure (revision 22945)
> > +++ configure (working copy)
> > @@ -6674,20 +6674,80 @@
> >
> > -if bsd; then
>
> The test was under this check for a reason, why did you remove it?  IMO
> it should go around the header checks as well.

With this check _tv_bsdbt848 variable will (as did before) have value
"auto". IMO this is not good. In my patch this variable will get
either "yes"'  or "no" value (under another os checks will just fail,
_tv_bsdbt848 will get "no" value).  So i suggest to drop "if bsd" out.

>
> > @@ -6700,9 +6760,7 @@
> >    _noinputmodules="tv-bsdbt848 $_noinputmodules"
> >  fi
> >  echores "$_tv_bsdbt848"
> > -fi #if bsd
> >
> > -
> >  echocheck "Video 4 Linux TV interface"
>
> Don't remove that extra blank line, I like having tests separated by
> more than just one line.
Will be fixed.


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