Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/reports/bin/parseNewReports.py

Issue 29350318: Issue 4382 - Use new syntax to catch exceptions in sitescripts (Closed)
Patch Set: Created Aug. 30, 2016, 1:28 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sitescripts/extensions/utils.py ('k') | sitescripts/reports/web/submitReport.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/reports/bin/parseNewReports.py
===================================================================
--- a/sitescripts/reports/bin/parseNewReports.py
+++ b/sitescripts/reports/bin/parseNewReports.py
@@ -88,7 +88,7 @@
parser.CharacterDataHandler = processText
try:
parser.ParseFile(source)
- except ExpatError, error:
+ except ExpatError as error:
reportData['warnings']['!parsing'] = 'Parsing error in the report: %s at line %i column %i' % (ErrorString(error.code), error.lineno, error.offset)
source.seek(0)
« no previous file with comments | « sitescripts/extensions/utils.py ('k') | sitescripts/reports/web/submitReport.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld