[Lldb-commits] [lldb] c42f188 - [lldb] Fix up comment placement (NFC) (#200671)

[via lldb-commits]

Author: Dave Lee
Date: 2026-05-31T09:54:09-07:00
New Revision: c42f1884730fb51a5f802ad621c0cc7f68715424

URL: https://github.com/llvm/llvm-project/commit/c42f1884730fb51a5f802ad621c0cc7f68715424
DIFF: https://github.com/llvm/llvm-project/commit/c42f1884730fb51a5f802ad621c0cc7f68715424.diff

LOG: [lldb] Fix up comment placement (NFC) (#200671)

Follow up to https://github.com/llvm/llvm-project/pull/200084

Added: 
    

Modified: 
    lldb/source/Commands/CommandObjectFrame.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/source/Commands/CommandObjectFrame.cpp b/lldb/source/Commands/CommandObjectFrame.cpp
index 742f7026464f0..5c79d758bafd7 100644
--- a/lldb/source/Commands/CommandObjectFrame.cpp
+++ b/lldb/source/Commands/CommandObjectFrame.cpp
@@ -731,6 +731,10 @@ may even involve JITing and running code in the target program.)");
             valobj_sp = frame->GetValueForVariableExpressionPath(
                 entry.ref(), m_varobj_options.use_dynamic, expr_path_options,
                 var_sp, error);
+            // Check only the `error` argument, because doing
+            // `valobj_sp->GetError()` will update the value and potentially
+            // return a new error that happens during the update, even if
+            // `GetValueForVariableExpressionPath` reported no errors.
             if (valobj_sp && error.Success()) {
               result.GetValueObjectList().Append(valobj_sp);
 
@@ -770,10 +774,6 @@ may even involve JITing and running code in the target program.)");
                     break;
                   }
               if (!found_recognized) {
-                // Check only the `error` argument, because doing
-                // `valobj_sp->GetError()` will update the value and potentially
-                // return a new error that happens during the update, even if
-                // `GetValueForVariableExpressionPath` reported no errors.
                 if (error.Fail())
                   result.SetError(error.takeError());
                 else