[MPlayer-dev-eng] Re: [patch] improve dxr3 tv norm switching

Thomas Jarosch tomj at simonv.com
Thu Jan 2 21:18:33 CET 2003


> > Well, they check for em8300 features (don't know what's in 0.12.0),
> > but I just copied them above/below other code I've seen in the module.
> > I'd like to see most of the #ifdefs removed if you agree.
>
> I have removed the unnecessary ones in cvs.

Looks good.

We could furthermore remove

#ifdef MVCOMMAND_SYNC
#ifdef EM8300_IOCTL_FLUSH

as they are already in em8300 0.12.0 (just checked)

> > Because there is way more output without a "." than with a ".".
> > Though we could fix it the other way around,
> > I just wanted to have the same output style.
>
> Hmm, ok... most if not all of those lines were written by Pontscho
> and not me.

So, what is the right course of action? I'd say capitalized letters and ".".
And maybe replace perror() with printf()...

> > > Why did you remove the initialization values to some of the declared
> > > variables? I personally don't like having uninitialized variables
> > > floating around.
> >
> > Jens Axboe did this one, but I don't know why he shouldn't as these
> > are just variables for temporary storage.
>
> Because the change does not do anything useful so I don't see the point in
> it.

Maybe smaller code size? :o) Just kidding...

> > btw: Is "prev_pts" really needed?
> >      I just saw it used in one line.
>
> It's used in a newer version of the code I have lying around.

ok

Thomas




More information about the MPlayer-dev-eng mailing list