[Lldb-commits] [PATCH] D127895: [lldb] Don't overwrite quit and exit builtins in the interactive python interpreter

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere marked 3 inline comments as done.
JDevlieghere added inline comments.


================
Comment at: lldb/test/Shell/ScriptInterpreter/Python/exit.test:1-5
+# RUN: %lldb -o 'script quit' | FileCheck %s --check-prefix SILENT
+# RUN: %lldb -o 'script quit()' | FileCheck %s --check-prefix SILENT
+
+# RUN: %lldb -o 'script exit' | FileCheck %s --check-prefix SILENT
+# RUN: %lldb -o 'script exit()' | FileCheck %s --check-prefix SILENT
----------------
kastiglione wrote:
> is there a reason to support both with and without `()`?
Yes, the latter is the function call, which works out of the box. This patch is mostly concerned with the former, which allows you to exit the script interpreter the same way you're allowed to exit the lldb prompt (i.e. with `quit`). 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127895/new/

https://reviews.llvm.org/D127895