Add check for trailing spaces

Author: ahornace

dev/checkstyle/style.xml

@@ -13,6 +13,13 @@
     <module name="FileLength"/>
     <module name="FileTabCharacter"/>
 
+    <module name="RegexpSingleline">
+        <property name="format" value="\s+$"/>
+        <property name="minimum" value="0"/>
+        <property name="maximum" value="0"/>
+        <property name="message" value="Line has trailing spaces."/>
+    </module>
+
     <module name="LineLength">
         <property name="max" value="150"/>
     </module>

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/AnalyzerGuru.java

@@ -216,7 +216,7 @@ public class AnalyzerGuru {
      * List of all registered {@code FileAnalyzerFactory} instances.
      */
     private static final List<AnalyzerFactory> factories = new ArrayList<>();
-    
+
     /**
      * Names of all analysis packages.
      */
@@ -768,35 +768,35 @@ public static AnalyzerFactory findByFileTypeName(String fileTypeName) {
      * @throws InvocationTargetException if the underlying constructor throws an exception
      */
     public static AnalyzerFactory findFactory(String factoryClassName)
-            throws ClassNotFoundException, IllegalAccessException, InstantiationException, NoSuchMethodException, 
+            throws ClassNotFoundException, IllegalAccessException, InstantiationException, NoSuchMethodException,
             InvocationTargetException {
         Class<?> fcn;
         try {
             fcn = Class.forName(factoryClassName);
-            
+
         } catch (ClassNotFoundException e) {
             fcn = getFactoryClass(factoryClassName);
-            
+
             if (fcn == null) {
                 throw new ClassNotFoundException("Unable to locate class " + factoryClassName);
             }
         }
-        
+
         return findFactory(fcn);
     }
 
     /**
      * Get Analyzer factory class using class simple name.
-     * 
+     *
      * @param simpleName which may be either the factory class
      * simple name (eg. CAnalyzerFactory), the analyzer name
      * (eg. CAnalyzer), or the language name (eg. C)
-     * 
+     *
      * @return the analyzer factory class, or null when not found.
      */
     public static Class<?> getFactoryClass(String simpleName) {
         Class<?> factoryClass = null;
-        
+
         // Build analysis package name list first time only
         if (analysisPkgNames.isEmpty()) {
             Package[] p = Package.getPackages();
@@ -807,17 +807,17 @@ public static Class<?> getFactoryClass(String simpleName) {
                 }
             }
         }
-        
+
         // This allows user to enter the language or analyzer name
         // (eg. C or CAnalyzer vs. CAnalyzerFactory)
         // Note that this assumes a regular naming scheme of
-        // all language parsers: 
+        // all language parsers:
         //      <language>Analyzer, <language>AnalyzerFactory
-        
+
         if (!simpleName.contains("Analyzer")) {
             simpleName += "Analyzer";
         }
-        
+
         if (!simpleName.contains("Factory")) {
             simpleName += "Factory";
         }
@@ -831,10 +831,10 @@ public static Class<?> getFactoryClass(String simpleName) {
                 // Ignore
             }
         }
-        
+
         return factoryClass;
     }
-    
+
     /**
      * Find a {@code FileAnalyzerFactory} which is an instance of the specified
      * class. If one doesn't exist, create one and register it.

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/FileAnalyzer.java

@@ -185,7 +185,7 @@ protected TokenStreamComponents createComponents(String fieldName) {
                 try (HistoryAnalyzer historyAnalyzer = new HistoryAnalyzer()) {
                     return historyAnalyzer.createComponents(fieldName);
                 }
-            //below is set by PlainAnalyzer to workaround #1376 symbols search works like full text search 
+            //below is set by PlainAnalyzer to workaround #1376 symbols search works like full text search
             case QueryBuilder.REFS: {
                 return new TokenStreamComponents(symbolTokenizerFactory.get());
             }

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/HistoryAnalyzer.java

@@ -53,26 +53,26 @@ public HistoryAnalyzer() {
         super(Analyzer.PER_FIELD_REUSE_STRATEGY);
         stopWords = StopFilter.makeStopSet(ENGLISH_STOP_WORDS);
     }
-   
+
     /**
      * Creates components using a new {@link PlainFullTokenizer}, filtered using
      * a default set of English stop-words.
-     * @param fieldName name of field for which to create components     
+     * @param fieldName name of field for which to create components
      * @return components for this analyzer (NB safe to use even if this
      * analyzer were to be garbage-collected)
      */
     @Override
-    protected TokenStreamComponents createComponents(String fieldName) {        
+    protected TokenStreamComponents createComponents(String fieldName) {
         JFlexTokenizer plainfull = new JFlexTokenizer(new PlainFullTokenizer(
                 AbstractAnalyzer.DUMMY_READER));
         //we are counting position increments, this might affect the queries
         //later and need to be in sync, especially for highlighting of results
         return new TokenStreamComponents(plainfull, new StopFilter(plainfull,
             stopWords));
     }
-    
+
     @Override
-    protected TokenStream normalize(String fieldName, TokenStream in) {        
+    protected TokenStream normalize(String fieldName, TokenStream in) {
            return new LowerCaseFilter(in);
-        }        
+        }
 }

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/JFlexNonXref.java

@@ -465,7 +465,7 @@ protected boolean writeSymbol(String symbol, Set<String> keywords, int line,
         boolean caseSensitive) throws IOException {
         return writeSymbol(symbol, keywords, line, caseSensitive, false);
     }
-    
+
     /**
      * Write a symbol and generate links as appropriate.
      *

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/PathTokenizer.java

@@ -49,7 +49,7 @@
  */
 public class PathTokenizer extends Tokenizer {
 
-    // below should be '/' since we try to convert even windows file separators 
+    // below should be '/' since we try to convert even windows file separators
     // to unix ones
     public static final char DEFAULT_DELIMITER = '/';
     private final CharTermAttribute termAtt = addAttribute(CharTermAttribute.class);

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/ada/AdaAnalyzerFactory.java

@@ -32,16 +32,16 @@
  */
 public class AdaAnalyzerFactory extends FileAnalyzerFactory {
 
-    private static final String name = "Ada";
-    
+    private static final String NAME = "Ada";
+
     private static final String[] SUFFIXES = {
         "ADA",
         "ADB",
         "ADS"
     };
 
     public AdaAnalyzerFactory() {
-        super(null, null, SUFFIXES, null, null, "text/plain", AbstractAnalyzer.Genre.PLAIN, name);
+        super(null, null, SUFFIXES, null, null, "text/plain", AbstractAnalyzer.Genre.PLAIN, NAME);
     }
 
     @Override

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/ada/Consts.java

@@ -104,7 +104,7 @@ public class Consts {
         kwd.add("while");
         kwd.add("with");
         kwd.add("xor");
- 
+
         kwd.add("ascii"); // A.1 The Package Standard
         kwd.add("base"); // A.1 The Package Standard
         kwd.add("boolean"); // A.1 The Package Standard

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/archive/BZip2Analyzer.java

@@ -127,5 +127,5 @@ public InputStream getStream() throws IOException {
                 }
             }
         };
-    }    
+    }
 }

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/archive/BZip2AnalyzerFactory.java

@@ -26,9 +26,9 @@
 import org.opengrok.indexer.analysis.FileAnalyzerFactory;
 
 public class BZip2AnalyzerFactory extends FileAnalyzerFactory {
-    
-    private static final String name = "Bzip(2)";
-    
+
+    private static final String NAME = "Bzip(2)";
+
     private static final String[] SUFFIXES = {
         "BZ", "BZ2"
     };
@@ -38,7 +38,7 @@ public class BZip2AnalyzerFactory extends FileAnalyzerFactory {
     };
 
     public BZip2AnalyzerFactory() {
-        super(null, null, SUFFIXES, MAGICS, null, null, null, name);
+        super(null, null, SUFFIXES, MAGICS, null, null, null, NAME);
     }
 
     @Override