limit report line number

sonar-cxx-plugin/src/main/java/org/sonar/plugins/cxx/utils/CxxReportSensor.java

@@ -52,6 +52,7 @@ public abstract class CxxReportSensor implements Sensor {
   private ResourcePerspectives perspectives;
   private Set<String> notFoundFiles = new HashSet<String>();
   private Set<String> uniqueIssues = new HashSet<String>();
+  private HashMap<String, Integer> maxLineFiles = new HashMap<String, Integer>();
   private final Metric metric;
   private int violationsCount;
 
@@ -239,6 +240,21 @@ private boolean saveViolation(Project project, SensorContext context, String rul
           = org.sonar.api.resources.File.fromIOFile(new File(normalPath), project);
         if (context.getResource(file) != null) {
           lineNr = getLineAsInt(line);
+          int maxLine = lineNr;
+          if (maxLineFiles.containsKey(normalPath)){
+            maxLine = maxLineFiles.get(normalPath);
+          }
+          else {
+            maxLine = CxxUtils.countLines(normalPath);
+            if (maxLine > 0)
+              maxLineFiles.put(normalPath, maxLine);
+          }
+          if (lineNr > maxLine) {
+            if(CxxUtils.LOG.isWarnEnabled()) {
+            CxxUtils.LOG.warn("Correct line number ("+ lineNr +") from report for '"+ normalPath +"' to max. line (" + maxLine +")");
+            }
+            lineNr = maxLine;
+          }
           resource = file;
           add = true;
         } else {
@@ -303,3 +319,4 @@ protected String reportPathKey() {
     return "";
   }
 }
+

sonar-cxx-plugin/src/main/java/org/sonar/plugins/cxx/utils/CxxUtils.java

@@ -20,6 +20,10 @@
 package org.sonar.plugins.cxx.utils;
 
 import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.FileReader;
+import java.io.IOException;
+import java.io.LineNumberReader;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -49,7 +53,7 @@ public static String fileToAbsolutePath(File file) {
     }
     return file.getAbsolutePath();
   }
-
+  
   /**
    * Normalize the given path to pass it to sonar. Return null if normalization
    * has failed.
@@ -78,8 +82,32 @@ public static String normalizePathFull(String filename, String baseDir) {
     return filePath;
   }
 
+
+  /**
+   * @return returns true for Reactor Projects
+   */
   public static boolean isReactorProject(Project project) {
     return project.isRoot() && !project.getModules().isEmpty();
   }
+
+  /**
+   * @return returns number of Lines of a File
+   */
+  public static int countLines(String normalPath) {
+    LineNumberReader reader;
+    int cnt = 0;    
+    try {
+      reader = new LineNumberReader(new FileReader(normalPath));
+      String lineRead = "";
+      while ((lineRead = reader.readLine()) != null) {}
+      cnt = reader.getLineNumber() + 1; 
+      reader.close();
+    } catch (FileNotFoundException e) {
+      LOG.error("countLines of '{}' failed: '{}'", normalPath, e.toString());
+    } catch (IOException e1) {
+      LOG.error("countLines of '{}' failed: '{}'", normalPath, e1.toString());
+    }
+    return cnt;
+  }
 }