# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 1435123201 25200
# Node ID 20de1ace07a9f0f1315514b0356dc49d5b97c509
# Parent  328739ea70c3aeab1b865f00db7d78660e99666b
check-code: detect legacy exception syntax

We just rewrote all files to use modern exception syntax. Ban the old
form.

This will detect the "except type, instance" and
"except (type1, type2), instance" forms.

diff -r 328739ea70c3 -r 20de1ace07a9 contrib/check-code.py
--- a/contrib/check-code.py	Tue Jun 23 22:20:08 2015 -0700
+++ b/contrib/check-code.py	Tue Jun 23 22:20:01 2015 -0700
@@ -281,6 +281,8 @@
     (r'\.debug\(\_', "don't mark debug messages for translation"),
     (r'\.strip\(\)\.split\(\)', "no need to strip before splitting"),
     (r'^\s*except\s*:', "naked except clause", r'#.*re-raises'),
+    (r'^\s*except\s([^\(,]+|\([^\)]+\))\s*,',
+     'legacy exception syntax; use "as" instead of ","'),
     (r':\n(    )*( ){1,3}[^ ]', "must indent 4 spaces"),
     (r'ui\.(status|progress|write|note|warn)\([\'\"]x',
      "missing _() in ui message (use () to hide false-positives)"),