================
@@ -2438,8 +2443,7 @@ MallocChecker::FreeMemAux(CheckerContext &C, const Expr 
*ArgExpr,
       // Check if the memory location being freed is the actual location
       // allocated, or an offset.
       RegionOffset Offset = R->getAsOffset();
-      if (Offset.isValid() &&
-          !Offset.hasSymbolicOffset() &&
+      if (Offset.isValid() && !Offset.hasSymbolicOffset() &&
----------------
benedekaibas wrote:

Sorry, I did not check yesterday the files after opening the PR. It looks like 
my IDE setup for running `clang-format` when I save a file was doing it for the 
whole file instead of just the changed lines. I did force push the change in 
[2b7e695](https://github.com/llvm/llvm-project/commit/2b7e69544ab1e4a73cd06fd478e09c22f39455f1).
 

I know force push should be avoided, but since this is the only review yet on 
this PR and it highlights the exact problem I was confident doing the force 
push. 

https://github.com/llvm/llvm-project/pull/212883
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to