[Lldb-commits] [lldb] [lldb] Fix incorrect logical operator in 'if' condition check (NFC) (PR #94779)

Adrian Prantl via lldb-commits lldb-commits at lists.llvm.org
Mon Jun 17 10:05:37 PDT 2024


================
@@ -85,7 +85,7 @@ class ScriptedPythonInterface : virtual public ScriptedInterface {
     bool has_class_name = !class_name.empty();
     bool has_interpreter_dict =
         !(llvm::StringRef(m_interpreter.GetDictionaryName()).empty());
-    if (!has_class_name && !has_interpreter_dict && !script_obj) {
+    if (!has_class_name || !has_interpreter_dict || !script_obj) {
       if (!has_class_name)
----------------
adrian-prantl wrote:

it would be better to hoist this above the previous condition?

https://github.com/llvm/llvm-project/pull/94779


More information about the lldb-commits mailing list