Fixed NPE

src/main/java/org/sonar/plugins/findbugs/FindbugsConfiguration.java

@@ -207,7 +207,7 @@ File saveIncludeConfigXml() throws IOException {
  * Scan the given folder for classes. It will catch classes from Java, JSP and more.
  *
  * @param folder Folder to scan
- * @return
+ * @return List<File> of class files
  * @throws IOException
  */
  public static List<File> scanForAdditionalClasses(File folder) throws IOException {

src/main/java/org/sonar/plugins/findbugs/FindbugsSensor.java

@@ -197,9 +197,11 @@ public void execute(SensorContext context) {
  }
 
  }
- finally {
- classMappingWriter.flush();
- classMappingWriter.close();
+ finally {
+ if(classMappingWriter != null) {
+ classMappingWriter.flush();
+ classMappingWriter.close();
+ }
  }
  }
 

src/main/java/org/sonar/plugins/findbugs/resource/ByteCodeResourceLocator.java

@@ -54,7 +54,7 @@ public class ByteCodeResourceLocator {
  * findSourceFileKeyByClassName() is broken in SonarQube 6.3.1.. This method is fixing it.
  * @param className
  * @param javaResourceLocator
- * @return
+ * @return String filepath
  */
  public String findSourceFileKeyByClassName(String className, JavaResourceLocator javaResourceLocator) {
  InputFile input = javaResourceLocator.findResourceByClassName(className);

src/main/java/org/sonar/plugins/findbugs/resource/JasperUtils.java

@@ -43,7 +43,7 @@ public static final String mangleChar(char ch)
  /**
  * Detect if the character is printable
  * @param c Character to test
- * @return
+ * @return boolean true if its a printable char otherwise false
  */
  public static boolean isPrintableChar( char c ) {
  Character.UnicodeBlock block = Character.UnicodeBlock.of( c );