[MPlayer-dev-eng] [PATCH] fix linker issues with Apple GCC 4

Ivan Kalvachev ivan at cacad.com
Sat Jan 29 13:41:01 CET 2005


On Sat, 22 Jan 2005 00:22:58 -0500
Alexander Strange <astrange at ithinksw.com> wrote:

> As it stands, mplayer compiles but doesn't link using the latest Apple 
> gcc 4 and cctools; there are two problems:
> • several sets of headers declare global variables in them, which 
> causes multiple definition errors (how did this ever work?)

committed
I also wonder how it worked.

> • gtf.c defines a round() after including math.h, but math.h already 
> has a round in it. glibc also has a round() call, so I simply removed 
> the static one in gtf.
> 

I had delayed that.
In short I am not sure that this is the right solution.
e.g. floor is defined in POSIX  while round in C99.
Probably an #ifndef would be better.

But i have small doubtfor the above fix, in case this code is workaround for buggy round,
or simply algorithm in gtf  need rounding to upper value (not nearest integer away from zero).
These are less probable (e.g. no one of the round usage seems to work with negative values)
but I would be happy to hear others oppinion.

Wish You Best
   Ivan Kalvachev
 iive




More information about the MPlayer-dev-eng mailing list