Closed
Bug 1417799
Opened 8 years ago
Closed 8 years ago
Remove conditional catch consumers in mail/.
Categories
(Thunderbird :: General, enhancement)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 59.0
People
(Reporter: arai, Assigned: arai)
References
Details
Attachments
(1 file)
|
1.51 KB,
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
non-standard conditional catch syntax is going to be removed in bug 1228841.
there are 2 cases for replacement.
case 1: only conditional catch
from:
try {
A
} catch (e if cond) {
B
}
to
try {
A
} catch (e) {
if (!cond) {
throw e;
}
B
}
case 2: conditional catch and unconditional catch
from:
try {
A
} catch (e if cond) {
B
} catch {
C
}
to
try {
A
} catch (e) {
if (cond) {
B
} else {
C
}
}
| Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8928864 -
Flags: review?(acelists)
Comment on attachment 8928864 [details] [diff] [review]
Remove conditional catch consumers in mail/.
Review of attachment 8928864 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks!
Attachment #8928864 -
Flags: review?(acelists) → review+
Pushed by arai_a@mac.com:
https://hg.mozilla.org/comm-central/rev/92e2cda4467f
Remove conditional catch consumers in mail/. r=aceman
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 4•8 years ago
|
||
https://hg.mozilla.org/comm-central/rev/92e2cda4467f367eb6e3b74a30a0494a71baf24c
Bug 1417799 - Remove conditional catch consumers in mail/. r=aceman
OS: Unspecified → All
Hardware: Unspecified → All
Target Milestone: --- → Thunderbird 59.0
Version: unspecified → Trunk
You need to log in
before you can comment on or make changes to this bug.
Description
•