[clang-tidy] NO.46,47 enable core.NonNullParamChecker,unhandled-self-assignment (#61494)

enkilee · web-flow · commit 4efa2b4bc7cc · 2024-02-22T11:49:06.000+08:00
* fix 47 46

* fix

* fix
diff --git a/paddle/pir/src/core/op_operand.cc b/paddle/pir/src/core/op_operand.cc
@@ -27,6 +27,9 @@
 
 namespace pir {
 OpOperand &OpOperand::operator=(const OpOperand &rhs) {  // NOLINT
+  if (this == &rhs) {
+    return *this;
+  }
   impl_ = rhs.impl_;
   return *this;
 }
diff --git a/paddle/testing/paddle_gtest_main.cc b/paddle/testing/paddle_gtest_main.cc
@@ -60,7 +60,7 @@ int main(int argc, char** argv) {  // NOLINT
   }
 
 #if defined(PADDLE_WITH_CUDA) || defined(PADDLE_WITH_HIP)
-  if (strstr(env_str, "enable_gpu_memory_usage_log")) {
+  if (strstr(env_str, "enable_gpu_memory_usage_log")) {  // NOLINT
     VLOG(1) << "Set FLAGS_enable_gpu_memory_usage_log to true";
     FLAGS_enable_gpu_memory_usage_log = true;
   }

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,9 @@`
`27`	`27`
`28`	`28`	`namespace pir {`
`29`	`29`	`OpOperand &OpOperand::operator=(const OpOperand &rhs) { // NOLINT`
	`30`	`+ if (this == &rhs) {`
	`31`	`+ return *this;`
	`32`	`+ }`
`30`	`33`	`impl_ = rhs.impl_;`
`31`	`34`	`return *this;`
`32`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ int main(int argc, char** argv) { // NOLINT`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`#if defined(PADDLE_WITH_CUDA) \|\| defined(PADDLE_WITH_HIP)`
`63`		`- if (strstr(env_str, "enable_gpu_memory_usage_log")) {`
	`63`	`+ if (strstr(env_str, "enable_gpu_memory_usage_log")) { // NOLINT`
`64`	`64`	`VLOG(1) << "Set FLAGS_enable_gpu_memory_usage_log to true";`
`65`	`65`	`FLAGS_enable_gpu_memory_usage_log = true;`
`66`	`66`	`}`