Opened 15 years ago

Closed 15 years ago

#4039 closed bug (fixed)

PI in math.h

Reported by: kaliber Owned by: axeld
Priority: normal Milestone: R1
Component: System/libroot.so Version: R1/pre-alpha1
Keywords: Cc: umccullough, richienyhus@…
Blocked By: Blocking:
Platform: All

Description

Please remove following lines from math.h:

#define PI                              M_PI
#define PI2                             M_PI_2

It breaks build of some applications and it isn't POSIX http://opengroup.org/onlinepubs/007908775/xsh/math.h.html

Change History (6)

comment:1 by umccullough, 15 years ago

Ah, yes... this explains why I had problems compiling SETI@home. I worked around it differently in that case by changing their definition to PI_

I had meant to look into it myself, but got pre-occupied :)

comment:2 by aljen, 15 years ago

yeah, while porting llvm to haiku i needed to #undef PI too :/

comment:3 by umccullough, 15 years ago

Cc: umccullough added

comment:4 by richienyhus, 15 years ago

Cc: richienyhus@… added

comment:5 by axeld, 15 years ago

Status: newassigned

comment:6 by axeld, 15 years ago

Resolution: fixed
Status: assignedclosed

Fixed in hrev31250.

Note: See TracTickets for help on using tickets.