Closed
Bug 1270649
Opened 9 years ago
Closed 9 years ago
Rename mExceptionHasBeenRisen to mExceptionWasRaised
Categories
(Core :: DOM: Events, defect)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
mozilla49
| Tracking | Status | |
|---|---|---|
| firefox49 | --- | fixed |
People
(Reporter: dbaron, Assigned: dbaron)
References
Details
(Whiteboard: btpp-active)
Attachments
(1 file)
|
5.89 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
risen is the past participle of rise; raised is the past participle of
raise, which is what is done to exceptions.
See:
https://en.wiktionary.org/wiki/rise#Verb
https://en.wiktionary.org/wiki/raise#Verb (definition 2, though not sure
if the use for exceptions is from 2.3 or 2.5)
(I wonder if mExceptionRaised would be simpler, though.)
MozReview-Commit-ID: 8Ynup8aDcLT
| Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8749418 -
Flags: review?(bugs)
| Assignee | ||
Comment 2•9 years ago
|
||
(This was introduced in https://hg.mozilla.org/mozilla-central/rev/fd53318c79ce .)
Blocks: 813445
Updated•9 years ago
|
Whiteboard: btpp-active
Comment 3•9 years ago
|
||
Comment on attachment 8749418 [details] [diff] [review]
Rename mExceptionHasBeenRisen to mExceptionHasBeenRaised
Hmm, shouldn't it then be
mExceptionWasRaised.
But you're the native English speaker, rs+
Attachment #8749418 -
Flags: review?(bugs) → review+
| Assignee | ||
Comment 4•9 years ago
|
||
Yes, mExceptionWasRaised is probably better. (simple past rather than (present) perfect.) I'll do that.
| Assignee | ||
Updated•9 years ago
|
Summary: Rename mExceptionHasBeenRisen to mExceptionHasBeenRaised → Rename mExceptionHasBeenRisen to mExceptionWasRaised
| Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/730147bfebe3d2c7d3fe1d0848189a8cb0cc8927
Bug 1270649 - Rename mExceptionHasBeenRisen to mExceptionWasRaised. r=smaug
Comment 6•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•