inherited virtual function is hidden

VERIFIED FIXED

Status

()

P2
normal
VERIFIED FIXED
20 years ago
19 years ago

People

(Reporter: Brade, Assigned: vidur)

Tracking

Trunk
PowerPC
Mac System 8.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

20 years ago
[please reassign as appropriate]

From the Macintosh compiler:

Warning : 'nsIXMLContent::GetNameSpacePrefix(nsIAtom *&)const ' hides inherited
virtual function 'nsIContent::GetNameSpacePrefix(int, nsIAtom *&)'
nsIXMLContent.h line 44   };
(Reporter)

Updated

20 years ago
Assignee: guha → vidur
(Reporter)

Comment 1

20 years ago
Reassign to vidur (please further reassign as necessary)

Updated

20 years ago
QA Contact: 3849

Comment 2

20 years ago
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 3

20 years ago
folks, I'm going to need developer assistance on verifying this bug. Vidur (or
whoever fixes this) please note when you checked in the fix and please note how
you self-tested the fix, we will mark as verified per your self test.
(Assignee)

Comment 4

20 years ago
I'll change the name of one of the methods, but I'm not sure why this is a
problem. Since the methods have different signatures, it seems to me that there
shouldn't be a problem - C++ allows method overloading. Again, this kind of
overloading isn't very COM-friendly, so I'm happy to change it and close &
verify the bug when I do. Kathy, any idea why this is a problem for the MW
compiler?

Comment 5

20 years ago
Vidur - that would be great if you could mark it as such when you are complete.
(Assignee)

Updated

20 years ago
Status: NEW → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED
(Assignee)

Updated

20 years ago
Status: RESOLVED → VERIFIED
(Assignee)

Comment 6

20 years ago
Made the change on 3/19. The name of the nsIContent method is now
GetNameSpacePrefixFromId.

Updated

19 years ago
QA Contact: beppe → chrisd
You need to log in before you can comment on or make changes to this bug.