Closed Bug 5028 Opened 26 years ago Closed 26 years ago

build error in RDF on HP-UX 10.20

Categories

(Core Graveyard :: RDF, defect, P3)

HP
HP-UX
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bruce, Assigned: waterson)

Details

Attachments

(1 file)

The HP build is currently dead because of this error in mozilla/rdf/base/public/rdf.h: Error 104: "../../../dist/./include/rdf.h", line 47 # Token concatenation did not create a single, valid new token. static const char* kURI##prefix##_##name = ##namespace #name ;\ ^^^^^^^^^^^^^ I've been watching it via tinderbox and the ports, so any prospective fix for it will need to be tested there (I haven't got an HP machine).
Summary: build error in RDF on HP-UX 10.20
Btw, this happens from mozilla/rdf/base/src/nsContainerCursor.cpp (line 68): DEFINE_RDF_VOCAB(RDF_NAMESPACE_URI, RDF, nextVal); // ad hoc way to make containers fast and RDF_NAMESPACE_URI is defined to be "http://www.w3.org/TR/WD-rdf-syntax#"
OK, I think DEFINE_RDF_VOCAB is just busted. I'm a little surprised that it works on GCC, even. Patch attached that should fix things, so waterson et alii can review and make sure that I'm not smoking crack. (It relies on the fact that |namespace| will always be a string literal, which I believe is the case.) If nobody objects, I'll check it in when the tree reopens.
Shaver, I verified on Mac and Win32: works fine. While you're at it, change |namespace| to |ns| -- turns out that |namespace| is a C++ keyword, which is surely just an accident waiting to happen.
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
shaver fixed this one. (thanks) but the change from namespace to ns didn't occur.
Okay, I'll do that next time I check in.
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: