[MPlayer-dev-eng] [PATCH] Dynamic list of EOSD sources

Diego Biurrun diego at biurrun.de
Tue Sep 14 11:05:44 CEST 2010


On Mon, Sep 13, 2010 at 09:51:36PM +0200, Nicolas George wrote:
> 
> Updated patch attached, except for the possible move of the list functions
> to static inline.
> 
> --- a/eosd.c
> +++ b/eosd.c
> @@ -26,54 +26,126 @@
> +    r = image_pool;
> +    image_pool = image_pool->next;

Here and in other places I think the readability of your code would
be enhanced by vertically aligning the =.

Diego


More information about the MPlayer-dev-eng mailing list