Closed Bug 1417800 Opened 6 years ago Closed 6 years ago

Remove conditional catch consumers in suite/.

Categories

(SeaMonkey :: General, enhancement)

enhancement
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

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
    }
  }
Comment on attachment 8928866 [details] [diff] [review]
Remove conditional catch consumers in suite/.

Great. Thanks
Attachment #8928866 - Flags: review?(frgrahl) → review+
Pushed by arai_a@mac.com:
https://hg.mozilla.org/comm-central/rev/e2ea6bf5f527
Remove conditional catch consumers in suite/. r=frg
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
https://hg.mozilla.org/comm-central/rev/e2ea6bf5f52766ee6b4c4db05c10b0b999165f00
Bug 1417800 - Remove conditional catch consumers in suite/. r=frg
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: