[debuginfo-tests] r219618 - Address review comments from Justin Bogner.

Adrian Prantl aprantl at apple.com
Mon Oct 13 09:34:32 PDT 2014


Author: adrian
Date: Mon Oct 13 11:34:31 2014
New Revision: 219618

URL: http://llvm.org/viewvc/llvm-project?rev=219618&view=rev
Log:
Address review comments from Justin Bogner.
- raise without arguments is preserving the backtrace
- move the call to terminate lldb to the exit handler

Modified:
    debuginfo-tests/trunk/llgdb.py

Modified: debuginfo-tests/trunk/llgdb.py
URL: http://llvm.org/viewvc/llvm-project/debuginfo-tests/trunk/llgdb.py?rev=219618&r1=219617&r2=219618&view=diff
==============================================================================
--- debuginfo-tests/trunk/llgdb.py (original)
+++ debuginfo-tests/trunk/llgdb.py Mon Oct 13 11:34:31 2014
@@ -142,14 +142,14 @@ NOTE: There are several resons why this
             print frame.EvaluateExpression(' '.join(cmd[1:]))
 
         elif re.match('^q|(quit)$', cmd[0]):
-            lldb.SBDebugger_Terminate()
             sys.exit(0)
 
         else:
             print debugger.HandleCommand(' '.join(cmd))
 
-    except SystemExit, e:
-        raise e
+    except SystemExit:
+        lldb.SBDebugger_Terminate()
+        raise
     except:
         print 'Could not handle the command "%s"' % ' '.join(cmd)
 





More information about the llvm-commits mailing list