Remove try..catch(ex if ex) statements from toolkit/components/osfile/ so that ESLint can parse the files

RESOLVED FIXED in Firefox 57

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: dbugs, Assigned: dbugs)

Tracking

unspecified
mozilla57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

try...catch(ex if ex) isn't on a standards line and is specific to gecko.

As a result, ESLint doesn't support it, and therefore can't parse files containing it.

As part of the work towards bug 1379256, we should remove the catch(ex if ex) statements.
Comment on attachment 8892929 [details]
Bug 1386666 - Remove try..catch(ex if ex) statements from toolkit/components/osfile/ so that ESLint can parse the files.

https://reviewboard.mozilla.org/r/163952/#review169320
Attachment #8892929 - Flags: review?(dtownsend) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/856215476aa4
Remove try..catch(ex if ex) statements from toolkit/components/osfile/ so that ESLint can parse the files. r=mossop
Flags: needinfo?(dbugs)
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4558c7896403
Remove try..catch(ex if ex) statements from toolkit/components/osfile/ so that ESLint can parse the files. r=mossop
https://hg.mozilla.org/mozilla-central/rev/4558c7896403
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
https://hg.mozilla.org/projects/date/rev/856215476aa45721d2b333e65666b3f2b21f0184
Bug 1386666 - Remove try..catch(ex if ex) statements from toolkit/components/osfile/ so that ESLint can parse the files. r=mossop

https://hg.mozilla.org/projects/date/rev/4558c7896403e46dc3f6c945ac6239b8cdbfa93e
Bug 1386666 - Remove try..catch(ex if ex) statements from toolkit/components/osfile/ so that ESLint can parse the files. r=mossop
You need to log in before you can comment on or make changes to this bug.