[MPlayer-dev-eng] [PATCH]VO_VDPAU, round 4

Diego Biurrun diego at biurrun.de
Thu Jan 22 14:45:42 CET 2009


On Thu, Jan 22, 2009 at 08:47:20AM +0000, Carl Eugen Hoyos wrote:
> Diego Biurrun <diego <at> biurrun.de> writes:
> 
> >     if (condition) {
> >         statement1;
> >         statement2;
> >     }
> 
> I found one of those and fixed it locally.
> 
> If you you are talking about the switch case:
> It is copied 1:1 from vo_xv.c and I will not change it before it is changed
> there: I don't believe there is agreement of where to put them there (and I
> think it makes sense to leave the identical lines as they are, it was already
> wrong to change them on some places of the file).

That begs the question why the code is duplicated in the first place.

Diego



More information about the MPlayer-dev-eng mailing list