Closed Bug 707352 Opened 12 years ago Closed 12 years ago

Remove NS_FINAL_CLASS

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla11

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(1 file)

Patch
4.61 KB, patch
cjones
: review+
Details | Diff | Splinter Review
Attached patch Patch — — Splinter Review 
It's had its day, and C++11 magic now supersedes it.  Let's kill it.  I think this patch will do the trick, although I base that belief mostly on MXR-searching and the like.

The Clang developers are of the opinion that it should be an error for an abstract class to be final, so nsDerivedSafe will not be marked with MOZ_FINAL.
Attachment #578759 - Flags: review?(jones.chris.g)
Attachment #578759 - Flags: review?(jones.chris.g) → review+
https://hg.mozilla.org/mozilla-central/rev/68b6b0c6045f
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.