[MPlayer-dev-eng] [PATCH] initial DragonFly BSD support

Bernd Ernesti mplayer at lists.veego.de
Tue Aug 31 20:05:24 CEST 2004


On Tue, Aug 31, 2004 at 07:20:48PM +0200, Pawel Biernacki wrote:
[..]
> Ok, is this patch is good enough to be commited?

No.

[..]
> --- /dev/null   2004-08-31 20:19:47.000000000 +0200
> +++ osdep/lrintf.c      2004-08-31 21:01:39.000000000 +0200
> @@ -0,0 +1,7 @@
> +#include "../config.h"
> +
> +#ifndef lrint
> +#include <math.h>
> +#define        lrintf(x) rintf(x)
> +#endif
> +

Why are you checking for lrint?

Thats seems for me plain wrong.
Hint: HAVE_LRINTF

Did you even test that on a system which has lrintf?
I would say no.

Bernd




More information about the MPlayer-dev-eng mailing list