Apply suggestions from code review

dmpots · JDevlieghere · web-flow · commit 1d910e18de92 · 2025-11-12T13:35:06.000-08:00
Improve comments and update file header to match coding standards.

Co-authored-by: Jonas Devlieghere &lt;jonas@devlieghere.com&gt;
diff --git a/lldb/unittests/Expression/ValueMatcher.cpp b/lldb/unittests/Expression/ValueMatcher.cpp
@@ -1,4 +1,4 @@
-//===-- ValueMatcher.cpp --------------------------------------------------===//
+//===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
@@ -97,7 +97,7 @@ bool ValueMatcher::MatchAndExplainImpl(const Value &val,
       }
     }
   } else {
-    // For Scalar, FileAddress, and LoadAddress - compare m_value
+    // For Scalar, FileAddress, and LoadAddress compare m_value.
     const Scalar &actual_scalar = val.GetScalar();
     if (actual_scalar != m_expected_scalar) {
       os << "scalar value mismatch: expected " << m_expected_scalar << ", got "
diff --git a/lldb/unittests/Expression/ValueMatcher.h b/lldb/unittests/Expression/ValueMatcher.h
@@ -1,4 +1,4 @@
-//===-- ValueMatcher.h ----------------------------------------------------===//
+//===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-//===-- ValueMatcher.cpp --------------------------------------------------===//`
	`1`	`+//===----------------------------------------------------------------------===//`
`2`	`2`	`//`
`3`	`3`	`// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.`
`4`	`4`	`// See https://llvm.org/LICENSE.txt for license information.`
`@@ -97,7 +97,7 @@ bool ValueMatcher::MatchAndExplainImpl(const Value &val,`
`97`	`97`	`}`
`98`	`98`	`}`
`99`	`99`	`} else {`
`100`		`- // For Scalar, FileAddress, and LoadAddress - compare m_value`
	`100`	`+ // For Scalar, FileAddress, and LoadAddress compare m_value.`
`101`	`101`	`const Scalar &actual_scalar = val.GetScalar();`
`102`	`102`	`if (actual_scalar != m_expected_scalar) {`
`103`	`103`	`os << "scalar value mismatch: expected " << m_expected_scalar << ", got "`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-//===-- ValueMatcher.h ----------------------------------------------------===//`
	`1`	`+//===----------------------------------------------------------------------===//`
`2`	`2`	`//`
`3`	`3`	`// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.`
`4`	`4`	`// See https://llvm.org/LICENSE.txt for license information.`