Fix #14487: FP duplicateBranch with empty macro (#8481)

My reasoning here is that a single semicolon most likely is a result of
an empty macro expansion. I can't think of a case where one would put a
single semicolon where it would make sense to warn.

Author: ludviggunne

lib/checkother.cpp

@@ -2692,7 +2692,7 @@ void CheckOther::checkDuplicateBranch()
                 const std::string branch2 = tokElse->stringifyList(scope.bodyEnd->linkAt(2));
 
                 // check for duplicates
-                if (branch1 == branch2) {
+                if (branch1 == branch2 && branch1 != ";") {
                     duplicateBranchError(scope.classDef, scope.bodyEnd->next(), ErrorPath{});
                     continue;
                 }

test/testother.cpp

@@ -7316,6 +7316,15 @@ class TestOther : public TestFixture {
               "    }\n"
               "}");
         ASSERT_EQUALS("", errout_str());
+
+        check("void f(int i) {\n"
+              "  if (1 == i) {\n"
+              "    ;\n"
+              "  } else {\n"
+              "    ;\n"
+              "  }\n"
+              "}\n");
+        ASSERT_EQUALS("", errout_str());
     }
 
     void duplicateBranch6() {