Closed Bug 500870 Opened 15 years ago Closed 15 years ago

Add "override" method annotation and enforce that it matches a base class's virtual method

Categories

(Developer Infrastructure :: Source Code Analysis, enhancement)

Product:
Developer Infrastructure
Component:
Source Code Analysis
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(status1.9.2 .4-fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
status1.9.2 --- .4-fixed

People

(Reporter: jruderman, Assigned: benjamin)

References

Details

(Whiteboard: [fixed-lorentz])

Attachments

(1 file)

Add NS_OVERRIDE annotation and enforcement, rev. 1
7.34 KB, patch
taras.mozilla
: review+
Details | Diff | Splinter Review
Would be nice to be able to annotate methods as intending to overriding a virtual method. Then, if a base class changes unexpectedly, the derived class will fail to compile instead of mysteriously failing to work properly.
Keywords: student-project
Whiteboard: [good first bug]
Assignee: nobody → benjamin
Status: NEW → ASSIGNED
Attachment #401262 - Flags: review?
Keywords: student-project
Whiteboard: [good first bug]
Attachment #401262 - Flags: review? → review?(tglek)
Depends on: 512726
Comment on attachment 401262 [details] [diff] [review] Add NS_OVERRIDE annotation and enforcement, rev. 1 With following fixed: signaturesMatch should check shortname instead of doing the check externally. I did in bug 512726, so lets land this one after that.
Attachment #401262 - Flags: review?(tglek) → review+
Blocks: 517524
http://hg.mozilla.org/mozilla-central/rev/8150d919e280
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Blanket approval for Lorentz merge to mozilla-1.9.2 a=beltzner for 1.9.2.4 - please make sure to mark status1.9.2:.4-fixed
Product: Core → Firefox Build System
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: