Updated the implementation as reviewer suggested.

Author: bloodlee

cxx-sensors/src/main/java/org/sonar/cxx/sensors/pclint/CxxPCLintSensor.java

@@ -158,6 +158,12 @@ private void addSecondaryLocationsToCurrentIssue(@Nonnull CxxReportIssue current
                                                        String file,
                                                        String line,
                                                        String msg) {
+        if (currentIssue.getLocations().isEmpty()) {
+          LOG.error("The issue of {} must have the primary location. Skip adding more locations",
+                  currentIssue.toString());
+          return;
+        }
+
         if (file != null && file.isEmpty() && msg != null) {
           Matcher matcher = SUPPLEMENTAL_MSG_PATTERN.matcher(msg);
 
@@ -171,12 +177,6 @@ private void addSecondaryLocationsToCurrentIssue(@Nonnull CxxReportIssue current
           return;
         }
 
-        if (currentIssue.getLocations().isEmpty()) {
-          LOG.error("The issue of {} must have the primary location. Skip adding more locations",
-                  currentIssue.toString());
-          return;
-        }
-
         // Due to SONAR-9929, even the API supports the extra/flow in different file,
         // the UI is not ready. For this case, use the parent issue's file and line for now.
         CxxReportLocation primaryLocation = currentIssue.getLocations().get(0);