[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