fixup! Fix #8260 Improve check: Pointer calculation result not null

Author: francois-berder

lib/valueflow.cpp

@@ -1183,7 +1183,7 @@ static void valueFlowImpossibleValues(TokenList& tokenList, const Settings& sett
             const Token* op1 = tok->astOperand1();
             const Token* op2 = tok->astOperand2();
             if ((op1 && op1->hasKnownIntValue() && op1->getKnownIntValue() != 0)
-            || (op2 && op2->hasKnownIntValue() && op2->getKnownIntValue() != 0)) {
+                || (op2 && op2->hasKnownIntValue() && op2->getKnownIntValue() != 0)) {
                 ValueFlow::Value val(0);
                 val.setImpossible();
                 val.errorPath.emplace_back(tok, "Pointer arithmetic result cannot be null");

test/testcondition.cpp

@@ -6141,7 +6141,7 @@ class TestCondition : public TestFixture {
               "  if (q);\n"
               "}");
         ASSERT_EQUALS("[test.cpp:3:7]: (warning) Comparison is wrong. Result of 'q' can't be 0 unless there is pointer overflow, and pointer overflow is undefined behaviour. [pointerAdditionResultNotNull]\n", errout_str());
-      }
+    }
 
     void duplicateConditionalAssign() {
         setMultiline();