Refactored loading of GLSL capabilities

- Fixed bug where versions below 400 that are not in the hardcoded list were just ignored.
- Applied same approach with mapping as with the loadCapabilities for GL.
- Refactored out the commonalities between the two.
-

jme3-core/src/main/java/com/jme3/renderer/opengl/GLRenderer.java

@@ -172,16 +172,9 @@ private void loadCapabilitiesES() {
 
     private void loadCapabilitiesGL2() {
         int oglVer = extractVersion(gl.glGetString(GL.GL_VERSION));
+        Map<Integer, List<Caps>> versionCapMap = getGLVersionCapabilityMapping();
+        addCapabilitiesBasedOnVersion(versionCapMap, oglVer);
 
-        Map<Integer, List<Caps>> versionCapMap = getVersionCapabilityMapping();
-
-        for (Map.Entry<Integer, List<Caps>> entry : versionCapMap.entrySet()) {
-            if (oglVer >= entry.getKey()) {
-                for (Caps cap : entry.getValue()) {
-                    caps.add(cap);
-                }
-            }
-        }
 
         // Fix issue in TestRenderToMemory when GL.GL_FRONT is the main
         // buffer being used.
@@ -194,31 +187,22 @@ private void loadCapabilitiesGL2() {
      */
     private void loadCapabilitiesGLSL() {
         int glslVer = extractVersion(gl.glGetString(GL.GL_SHADING_LANGUAGE_VERSION));
+        Map<Integer, List<Caps>> versionCapMap = getGSLVersionCapabilityMapping();
+        addCapabilitiesBasedOnVersion(versionCapMap, glslVer);
+    }
 
-        switch (glslVer) {
-            default:
-                if (glslVer < 400) {
-                    break;
+    /**
+     * Add capabilities for all versions below the given version.
+     * @param versionCapMap mapping of existing versions and there capabilities
+     * @param version version to add capabilities for
+     */
+    private void addCapabilitiesBasedOnVersion(Map<Integer, List<Caps>> versionCapMap, int version) {
+        for (Map.Entry<Integer, List<Caps>> entry : versionCapMap.entrySet()) {
+            if (version >= entry.getKey()) {
+                for (Caps cap : entry.getValue()) {
+                    caps.add(cap);
                 }
-                // so that future OpenGL revisions wont break jme3
-                // fall through intentional
-            case 400:
-                caps.add(Caps.GLSL400);
-            case 330:
-                caps.add(Caps.GLSL330);
-            case 150:
-                caps.add(Caps.GLSL150);
-            case 140:
-                caps.add(Caps.GLSL140);
-            case 130:
-                caps.add(Caps.GLSL130);
-            case 120:
-                caps.add(Caps.GLSL120);
-            case 110:
-                caps.add(Caps.GLSL110);
-            case 100:
-                caps.add(Caps.GLSL100);
-                break;
+            }
         }
     }
 
@@ -2714,7 +2698,7 @@ public void setLinearizeSrgbImages(boolean linearize) {
         }
     }
 
-    public Map<Integer,List<Caps>> getVersionCapabilityMapping() {
+    public Map<Integer,List<Caps>> getGLVersionCapabilityMapping() {
 
         Map<Integer,List<Caps>> map = new TreeMap<>();
 
@@ -2728,4 +2712,19 @@ public Map<Integer,List<Caps>> getVersionCapabilityMapping() {
 
         return map;
     }
+
+    public Map<Integer,List<Caps>> getGSLVersionCapabilityMapping() {
+        Map<Integer,List<Caps>> map = new TreeMap<>();
+
+        map.put(100, Collections.singletonList(Caps.GLSL100));
+        map.put(110, Collections.singletonList(Caps.GLSL110));
+        map.put(120, Collections.singletonList(Caps.GLSL120));
+        map.put(130, Collections.singletonList(Caps.GLSL130));
+        map.put(140, Collections.singletonList(Caps.GLSL140));
+        map.put(150, Collections.singletonList(Caps.GLSL150));
+        map.put(330, Collections.singletonList(Caps.GLSL330));
+        map.put(400, Collections.singletonList(Caps.GLSL400));
+
+        return map;
+    }
 }