[MPlayer-dev-eng] Re: [PATCH] esd: support non-default tcp port

Alban Bedel albeu at free.fr
Sun Oct 9 21:06:58 CEST 2005


On Sun, 09 Oct 2005 20:36:17 +0200
Németh Márton <nm127 at freemail.hu> wrote:

> +#define AO_ESD_OVERWRITES_PARAMETER
> +
> +#ifdef AO_ESD_OVERWRITES_PARAMETER
> +#define AO_ESD_FREE_BUFFER(var_name)	if (var_name) free(var_name);
> +#else
> +#define AO_ESD_FREE_BUFFER(var_name)
> +#endif

All this pre-processor madness is not really usefull. I can't see
why one would want to disable only that part of the code.
imho it would be much simpler and cleaner w/o all the #ifdef.

	Albeu




More information about the MPlayer-dev-eng mailing list