Opened 10 years ago

Closed 10 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:
Has a Patch: no 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 Changed 10 years ago by umccullough

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 Changed 10 years ago by aljen

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

comment:3 Changed 10 years ago by umccullough

Cc: umccullough added

comment:4 Changed 10 years ago by richienyhus

Cc: richienyhus@… added

comment:5 Changed 10 years ago by axeld

Status: newassigned

comment:6 Changed 10 years ago by axeld

Resolution: fixed
Status: assignedclosed

Fixed in hrev31250.

Note: See TracTickets for help on using tickets.