[MPlayer-dev-eng] libvo changes

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sat Apr 5 19:57:29 CEST 2008


On Sat, Apr 05, 2008 at 07:55:14PM +0300, Uoti Urpala wrote:
> Here are the latest libvo changes. They add the basics of a new VO
> driver API and convert vo_xv to use it.

Another two comments:
I really think that "struct vo" is not really a proper name, vo_context
or some such seems much more appropriate to me.
I also do not really like that you use "struct ..." _everywhere_ IMO
that "struct" clutters the code without any information, and (also IMO)
at least the internal vo code would be much better of using a less
cluttery typedef.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list