Closed Bug 72168 Opened 24 years ago Closed 24 years ago

compile error: eCSSProperty_box_orient not defined

Categories

(Core :: CSS Parsing and Computation, defect)

x86
Linux
defect
Not set
major

Tracking

()

VERIFIED INVALID

People

(Reporter: matt, Assigned: pierre)

Details

When trying to compile the latest CVS source on Linux, I get the following compile error for file mozilla/content/html/style/src/nsCSSDeclaration.cpp: nsCSSDeclaration.cpp: In method `void nsCSSXUL::List(FILE * = stdout, int = 0) const': nsCSSDeclaration.cpp:1008: `eCSSProperty_box_orient' undeclared (first use this function) nsCSSDeclaration.cpp:1008: (Each undeclared identifier is reported only once nsCSSDeclaration.cpp:1008: for each function it appears in.) According to LXR, this isn't defined anywhere, but it used in several places, including files other than nsCSSDeclaration.cpp. In nsCSSDeclaration.cpp, it seems that eCSSProperty_box_orient is only used in sections of code in an "#ifdef INCLUDE_XUL", so I added --disable-xul to the configure command line. With that, nsCSSDeclaration.cpp compiles, but nsCSSParser.cpp fails to compile, with the error: nsCSSParser.cpp: In method `PRBool CSSParserImpl::ParseSelectorGroup(PRInt32 &, SelectorList *&)': nsCSSParser.cpp:1448: `IsOutlinerPseudoElement' undeclared (first use this function) and IsOutlinerPseudoElement() is defined in an INCLUDE_XUL area, so --disable-xul doesn't work. I've configured my build like this: ./configure --with-java-supplement=yes --with-xprint=yes --with-xlib=yes --enabl e-trace-malloc=yes --with-extensions=all --disable-jar-packaging --enable-perf-m etrics=yes --enable-mathml=yes --enable-svg=yes --enable-logrefcnt=yes --enable- jprof=yes --enable-xpctools=yes --enable-cpp-rtti=yes --enable-leaky=yes
The problem went away when I deleted mozilla/content and re-pulled that entire directory tree; and the reason that I wasn't seeing the symbol in LXR was that the enum was being generated with a macro. My bad; I'm marking this bug INVALID.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
rsverified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.