_darwin.cfg doesn't define the DARWIN macro, but pratom.h tests it

NEW
Assigned to

Status

NSPR
NSPR
10 years ago
10 years ago

People

(Reporter: Wan-Teh Chang, Assigned: Wan-Teh Chang)

Tracking

4.7.1
All
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

10 years ago
NSPR's _xxxos.cfg usually defines the XXXOS macro,
for example, _linux.cfg defines LINUX, and _solaris.cfg
defines SOLARIS.

For some reason, _darwin.cfg doesn't define DARWIN.
This means NSPR's public headers should not test DARWIN
because we want our users to be able to use our public
headers without defining any macro.

But pratom.h is testing DARWIN.  We should either change
pratom.h to test the predefined macro __APPLE__, or change
_darwin.cfg to define DARWIN.
You need to log in before you can comment on or make changes to this bug.