move unique error check above output format check

Author: Nettozx

cli/cppcheckexecutor.cpp

@@ -561,20 +561,20 @@ void StdLogger::reportErr(const ErrorMessage &msg)
                                      mGuidelineMapping, msgCopy.severity);
     msgCopy.classification = getClassification(msgCopy.guideline, mSettings.reportType);
 
+    // TODO: there should be no need for verbose and default messages here
+    const std::string msgStr =
+        msgCopy.toString(mSettings.verbose, mSettings.templateFormat, mSettings.templateLocation);
+
+    // Alert only about unique errors
+    if (!mSettings.emitDuplicates && !mShownErrors.insert(msgStr).second)
+        return;
+
     if (mSettings.outputFormat == Settings::OutputFormat::sarif) {
         mSarifReport.addFinding(std::move(msgCopy));
+    } else if (mSettings.outputFormat == Settings::OutputFormat::xml) {
+        reportErr(msgCopy.toXML());
     } else {
-        // TODO: there should be no need for verbose and default messages here
-        const std::string msgStr = msgCopy.toString(mSettings.verbose, mSettings.templateFormat, mSettings.templateLocation);
-
-        // Alert only about unique errors
-        if (!mSettings.emitDuplicates && !mShownErrors.insert(msgStr).second)
-            return;
-
-        if (mSettings.outputFormat == Settings::OutputFormat::xml)
-            reportErr(msgCopy.toXML());
-        else
-            reportErr(msgStr);
+        reportErr(msgStr);
     }
 }