Changeset 11374 in josm for trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSParser.jj

Timestamp:

2016-12-09T23:31:13+01:00 (3 years ago)

Author:

Don-vip

Message:

sonar - squid:S00112 - Generic exceptions should never be thrown: define JosmRuntimeException

File:

• trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSParser.jj (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSParser.jj

@@ -40 +40 @@
 import org.openstreetmap.josm.gui.mappaint.mapcss.Subpart;
 import org.openstreetmap.josm.tools.ColorHelper;
+import org.openstreetmap.josm.tools.JosmRuntimeException;
 import org.openstreetmap.josm.tools.Pair;
 import org.openstreetmap.josm.tools.Utils;
@@ -66 +67 @@
      */
     public static enum LexicalState {
-        PREPROCESSOR(0), /* the preprocessor */
-        DEFAULT(2);      /* the main parser */
-        
+        /** the preprocessor */
+        PREPROCESSOR(0),
+        /** the main parser */
+        DEFAULT(2);
+
         int idx; // the integer, which javacc assigns to this state
-        
+
         LexicalState(int idx) {
             if (!this.name().equals(MapCSSParserTokenManager.lexStateNames[idx])) {
-                throw new RuntimeException();
+                throw new JosmRuntimeException("Wrong name for index " + idx);
             }
             this.idx = idx;
         }
-    };
+    }
     
     /**
@@ -95 +98 @@
             scs = new SimpleCharStream(in, encoding, 1, 1);
         } catch (java.io.UnsupportedEncodingException e) {
-            throw new RuntimeException(e);
+            throw new JosmRuntimeException(e);
         }
         return new MapCSSParserTokenManager(scs, initState.idx);