CElement::GetElement used before definition

RESOLVED FIXED

Status

()

P3
normal
RESOLVED FIXED
19 years ago
19 years ago

People

(Reporter: tor, Assigned: rickg)

Tracking

Trunk
Sun
Solaris
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
CElement::GetElement() (in COtherElements.h) is being used before the
definition as "inline", which causes the SUNWspro4.2 compiler to choke.
I'm planning on checking in the following patch (reviewed by bruce)
when the tree opens.  This just moves GetElement up before any of the
inline functions.

If this offends your coding aesthetics, let me know.  Otherwise I'll
close this bug after checking in this fix.

Index: COtherElements.h
===================================================================
RCS file: /cvsroot/mozilla/htmlparser/src/COtherElements.h,v
retrieving revision 3.17
diff -u -r3.17 COtherElements.h
--- COtherElements.h    2000/05/15 07:58:53     3.17
+++ COtherElements.h    2000/05/15 16:58:51
@@ -1917,6 +1917,12 @@
 static eHTMLTags kObjectKids[]={eHTMLTag_param,eHTMLTag_unknown};
 static eHTMLTags kTBodyKids[]={eHTMLTag_tr,eHTMLTag_unknown};
 
+
+inline CElement* CElement::GetElement(eHTMLTags aTag) {
+  return gElementTable->mElements[aTag];
+}
+
+
 /******************************************************************************
*****
   This method is pretty interesting, because it's where the elements all get 
   initialized for this elementtable.
@@ -2550,11 +2556,6 @@
   }
   return 0;
 }
-
-inline CElement* CElement::GetElement(eHTMLTags aTag) {
-  return gElementTable->mElements[aTag];
-}
-
 
 inline CElement* CElement::GetDefaultContainerFor(CElement* anElement) {
   CElement* result=0;
(Reporter)

Comment 1

19 years ago
Checked in.
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Updated

19 years ago
QA Contact: janc → hong

Comment 2

19 years ago
changing qa contact for solaris bugs to hong
You need to log in before you can comment on or make changes to this bug.