Fixed location of return statement. Should not be within "finally" block.
When return statement is within "finally" statement any exception
that happens within the try block will not be reported. Exceptions are
supposed to be rethrown at the end of "finally" if there was no "except"
but, as the code was written, the end of "finally" is never reached.
TEST=Verified exceptions are reported correctly. Also, with asharif
verified with small test case that our understanding of "finally" was
correct.
PRESUBMIT=passed
R=asharif
APPROVED=asharif
DELTA=1 (0 added, 0 deleted, 1 changed)
OCL=43770739
P4 change: 43792110
diff --git a/v14/build_tc.py b/v14/build_tc.py
index 0a7e15e..fc937b4 100755
--- a/v14/build_tc.py
+++ b/v14/build_tc.py
@@ -315,7 +315,7 @@
rv = rv + tp.Build()
finally:
print "Exiting..."
- return rv
+ return rv
if __name__ == "__main__":
retval = Main(sys.argv)