[MPlayer-dev-eng] [PATCH] Support for video capture from National Instruments cards
Diego Biurrun
diego at biurrun.de
Fri Apr 24 16:56:25 CEST 2009
On Fri, Apr 24, 2009 at 11:27:36AM +0200, Sebastien Raveau wrote:
> On Thu, Apr 23, 2009 at 8:28 PM, Diego Biurrun <diego at biurrun.de> wrote:
> >> +#define _IMG_BASE 0x3FF60000
> >
> > Identifiers starting with an underscore and uppercase letters are
> > reserved for the system.
>
> Wanted to stay consistent with the National Instruments API, but I'll
> change that, no problem.
Everybody and their brother does this wrong..
Probably they look at libc headers and mindlessly copy from there..
> >> + switch(cmd)
> >> + {
> >
> > switch (cmd) {
>
> Same, this is code from tvi_dummy.c and/or tvi_bsdbt848.c files, but
> ok I'll change it too.
New files should be K&R style.
Diego
More information about the MPlayer-dev-eng
mailing list