[MPlayer-users] (lack of) code style

Ivan Kowalenko ivan.kowalenko at gmail.com
Mon Aug 21 18:37:19 CEST 2006


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Aug 19, 2006, at 13.18, Denis Vlasenko wrote:

> Hello,
>
> I took a look into mplayer source.
> Well. I saw sources of quite a number of open-source
> projects, and mplayer is definitely won't make to the
> "cleanest code" award.
>
> Two random examples:

[snip]

> What is going on with the indentation here?
> Entire mplayer.c is littered with similar things.
>

[snip]

> Gross code duplication due to excessive inlining.
>
> This is my impression from just a 5-minute cursory
> examination, so maybe I had an extremely bad luck
> of hitting the worst places to look at.

Well, try talking to the guys on the -dev mailing list. We're use  
lusers here. I'm sure a good number of us (including myself) have  
never even looked at the source.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (Darwin)

iD4DBQFE6eFDh/R/wkI6oPQRArkvAJ9bvsjA31d0gjiWcX9qLDv+R0HUPgCY8+tR
lhdYBvvBJBxLjlgNW6lDTQ==
=Md32
-----END PGP SIGNATURE-----



More information about the MPlayer-users mailing list