Findbugs: Remove dead-code / unused [temp] storage and it's assignment

21 files changed, 133 insertions, 115 deletions

diff --git a/src/jogl/classes/com/jogamp/graph/curve/opengl/RegionRenderer.java b/src/jogl/classes/com/jogamp/graph/curve/opengl/RegionRenderer.java
index 9a175c030..289730ca0 100644
--- a/src/jogl/classes/com/jogamp/graph/curve/opengl/RegionRenderer.java
+++ b/src/jogl/classes/com/jogamp/graph/curve/opengl/RegionRenderer.java
@@ -478,7 +478,7 @@ public class RegionRenderer {
             posFp = rsFp.insertShaderSource(0, posFp, GLSL_USE_COLOR_CHANNEL);
         }
         if( Region.hasColorTexture( renderModes ) ) {
-            posVp = rsVp.insertShaderSource(0, posVp, GLSL_USE_COLOR_TEXTURE);
+                    rsVp.insertShaderSource(0, posVp, GLSL_USE_COLOR_TEXTURE);
             posFp = rsFp.insertShaderSource(0, posFp, GLSL_USE_COLOR_TEXTURE);
         }
         if( !pass1 ) {
diff --git a/src/jogl/classes/javax/media/opengl/GLContext.java b/src/jogl/classes/javax/media/opengl/GLContext.java
index e4e29ee2b..d5d8792d8 100644
--- a/src/jogl/classes/javax/media/opengl/GLContext.java
+++ b/src/jogl/classes/javax/media/opengl/GLContext.java
@@ -1878,7 +1878,7 @@ public abstract class GLContext {
           ok = GLContext.getAvailableGLVersion(device, 3, GLContext.CTX_PROFILE_CORE, major, minor, ctp);
       }
       if( !ok ) {
-          ok = GLContext.getAvailableGLVersion(device, 3, GLContext.CTX_PROFILE_COMPAT, major, minor, ctp);
+          GLContext.getAvailableGLVersion(device, 3, GLContext.CTX_PROFILE_COMPAT, major, minor, ctp);
       }
       return 0 != ( ctp[0] & CTX_IMPL_ES3_COMPAT );
   }
diff --git a/src/jogl/classes/jogamp/graph/font/typecast/ot/mac/ResourceMap.java b/src/jogl/classes/jogamp/graph/font/typecast/ot/mac/ResourceMap.java
index b22a06325..781e0adf3 100644
--- a/src/jogl/classes/jogamp/graph/font/typecast/ot/mac/ResourceMap.java
+++ b/src/jogl/classes/jogamp/graph/font/typecast/ot/mac/ResourceMap.java
@@ -31,19 +31,19 @@ import java.io.IOException;
 public class ResourceMap {
 
     private final byte[] headerCopy = new byte[16];
-    private final int nextResourceMap;
-    private final int fileReferenceNumber;
-    private final int attributes;
+    // private final int nextResourceMap;
+    // private final int fileReferenceNumber;
+    // private final int attributes;
     private final ResourceType[] types;
 
     /** Creates new ResourceMap */
     public ResourceMap(final DataInput di) throws IOException {
         di.readFully(headerCopy);
-        nextResourceMap = di.readInt();
-        fileReferenceNumber = di.readUnsignedShort();
-        attributes = di.readUnsignedShort();
-        final int typeOffset = di.readUnsignedShort();
-        final int nameOffset = di.readUnsignedShort();
+        /* nextResourceMap = */ di.readInt();
+        /* fileReferenceNumber = */ di.readUnsignedShort();
+        /* attributes = */ di.readUnsignedShort();
+        /* final int typeOffset = */ di.readUnsignedShort();
+        /* final int nameOffset = */ di.readUnsignedShort();
         final int typeCount = di.readUnsignedShort() + 1;
 
         // Read types
@@ -65,7 +65,6 @@ public class ResourceMap {
 
     public ResourceType getResourceType(final String typeName) {
         for (int i = 0; i < types.length; i++) {
-            final String s = types[i].getTypeAsString();
             if (types[i].getTypeAsString().equals(typeName)) {
                 return types[i];
             }
diff --git a/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GposTable.java b/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GposTable.java
index f10d4e299..ce6f308db 100644
--- a/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GposTable.java
+++ b/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GposTable.java
@@ -36,10 +36,10 @@ public class GposTable implements Table {
         _de = (DirectoryEntry) de.clone();
 
         // GPOS Header
-        final int version = di.readInt();
-        final int scriptList = di.readInt();
-        final int featureList = di.readInt();
-        final int lookupList = di.readInt();
+        /* final int version = */ di.readInt();
+        /* final int scriptList = */ di.readInt();
+        /* final int featureList = */ di.readInt();
+        /* final int lookupList = */ di.readInt();
     }
 
     /** Get the table type, as a table directory value.
diff --git a/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GsubTable.java b/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GsubTable.java
index f09973f99..3147fc8c5 100644
--- a/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GsubTable.java
+++ b/src/jogl/classes/jogamp/graph/font/typecast/ot/table/GsubTable.java
@@ -76,7 +76,7 @@ public class GsubTable implements Table, LookupSubtableFactory {
         final DataInputStream dis = new DataInputStream(new ByteArrayInputStream(buf));
 
         // GSUB Header
-        final int version = dis.readInt();
+        /* final int version = */ dis.readInt();
         final int scriptListOffset = dis.readUnsignedShort();
         final int featureListOffset = dis.readUnsignedShort();
         final int lookupListOffset = dis.readUnsignedShort();
diff --git a/src/jogl/classes/jogamp/graph/font/typecast/ot/table/KernSubtable.java b/src/jogl/classes/jogamp/graph/font/typecast/ot/table/KernSubtable.java
index df5d0394f..dcf02cb2b 100644
--- a/src/jogl/classes/jogamp/graph/font/typecast/ot/table/KernSubtable.java
+++ b/src/jogl/classes/jogamp/graph/font/typecast/ot/table/KernSubtable.java
@@ -28,8 +28,8 @@ public abstract class KernSubtable {
 
     public static KernSubtable read(final DataInput di) throws IOException {
         KernSubtable table = null;
-        final int version = di.readUnsignedShort();
-        final int length = di.readUnsignedShort();
+        /* final int version = */ di.readUnsignedShort();
+        /* final int length = */ di.readUnsignedShort();
         final int coverage = di.readUnsignedShort();
         final int format = coverage >> 8;
 
diff --git a/src/jogl/classes/jogamp/graph/font/typecast/t2/T2Interpreter.java b/src/jogl/classes/jogamp/graph/font/typecast/t2/T2Interpreter.java
index 7bfb8cbd5..80d76a7f8 100644
--- a/src/jogl/classes/jogamp/graph/font/typecast/t2/T2Interpreter.java
+++ b/src/jogl/classes/jogamp/graph/font/typecast/t2/T2Interpreter.java
@@ -307,10 +307,20 @@ public class T2Interpreter {
                 dxb[count - i - 1] = popArg().intValue();
                 dya[count - i - 1] = popArg().intValue();
             }
+            /**
+             * Not using the 'popped' arguments,
+             * hence simply pop them from stack!
+             *
             final int dy3 = popArg().intValue();
             final int dy2 = popArg().intValue();
             final int dx2 = popArg().intValue();
             final int dx1 = popArg().intValue();
+            */
+            popArg();
+            popArg();
+            popArg();
+            popArg();
+
             for (int i = 0; i < count; ++i) {
                 final Point lastPoint = getLastPoint();
                 final int xa = lastPoint.x;
diff --git a/src/jogl/classes/jogamp/graph/font/typecast/tt/engine/Interpreter.java b/src/jogl/classes/jogamp/graph/font/typecast/tt/engine/Interpreter.java
index b296886cc..a7408e381 100644
--- a/src/jogl/classes/jogamp/graph/font/typecast/tt/engine/Interpreter.java
+++ b/src/jogl/classes/jogamp/graph/font/typecast/tt/engine/Interpreter.java
@@ -452,8 +452,8 @@ public class Interpreter {
     /**
      * JuMP Relative
      */
-    private int _jmpr(int instructionIndex) {
-        return instructionIndex += pop() - 1;
+    private int _jmpr(final int instructionIndex) {
+        return instructionIndex + ( pop() - 1 );
     }
 
     /**
@@ -484,7 +484,7 @@ public class Interpreter {
      * LOOP and CALL function
      */
     private void _loopcall() {
-        final int index = pop();
+        /* final int index = */ pop();
         final int count = pop();
         for (int i = 0; i < count; i++) {
             execute(functionMap[i]);
@@ -946,13 +946,13 @@ public class Interpreter {
     private void _sfvtl(final short param) {
         pop();
         pop();
-        if (param == 1) {
-            gs.freedom_vector[0] = 0x0000;
-            gs.freedom_vector[1] = 0x0000;
-        } else {
+        // if (param == 1) {
             gs.freedom_vector[0] = 0x0000;
             gs.freedom_vector[1] = 0x0000;
-        }
+        // } else {
+        //    gs.freedom_vector[0] = 0x0000;
+        //    gs.freedom_vector[1] = 0x0000;
+        //}
     }
 
     /**
@@ -1039,19 +1039,24 @@ public class Interpreter {
      */
     private void _spvtl(final short param) {
 
+        // below block is dead code, reduce to pop() calls.
+        pop();
+        pop();
+        /**
         // We'll get a copy of the line and normalize it -
         // divide the x- and y-coords by the vector's dot product.
         final Point p1 = zone[gs.zp2][pop()];
         final Point p2 = zone[gs.zp1][pop()];
         final int x = p2.x - p1.x;
         final int y = p2.y - p1.y;
-        if(param == 1) {
+         */
+        // if(param == 1) {
             gs.projection_vector[0] = 0x0000;
             gs.projection_vector[1] = 0x0000;
-        } else {
-            gs.projection_vector[0] = 0x0000;
-            gs.projection_vector[1] = 0x0000;
-        }
+        // } else {
+        //    gs.projection_vector[0] = 0x0000;
+        //    gs.projection_vector[1] = 0x0000;
+        // }
     }
 
     private void _sround() {
diff --git a/src/jogl/classes/jogamp/opengl/egl/EGLGraphicsConfigurationFactory.java b/src/jogl/classes/jogamp/opengl/egl/EGLGraphicsConfigurationFactory.java
index 9962c0617..d4e5e7d62 100644
--- a/src/jogl/classes/jogamp/opengl/egl/EGLGraphicsConfigurationFactory.java
+++ b/src/jogl/classes/jogamp/opengl/egl/EGLGraphicsConfigurationFactory.java
@@ -333,7 +333,6 @@ public class EGLGraphicsConfigurationFactory extends GLGraphicsConfigurationFact
         final int winattrmask = GLGraphicsConfigurationUtil.getExclusiveWinAttributeBits(capsChosen);
         List<GLCapabilitiesImmutable> availableCaps = null;
         int recommendedIndex = -1;
-        long recommendedEGLConfig = -1;
         final IntBuffer numConfigs = Buffers.newDirectIntBuffer(1);
 
         if(!EGL.eglGetConfigs(eglDisplay, null, 0, numConfigs)) {
@@ -380,7 +379,7 @@ public class EGLGraphicsConfigurationFactory extends GLGraphicsConfigurationFact
         if( hasEGLChosenCaps ) {
             availableCaps = eglConfigs2GLCaps(device, glp, configs, numConfigs.get(0), winattrmask, forceTransparentFlag, skipCapsChooser /* onlyFirsValid */);
             if(availableCaps.size() > 0) {
-                recommendedEGLConfig =  configs.get(0);
+                final long recommendedEGLConfig =  configs.get(0);
                 recommendedIndex = 0;
                 if (DEBUG) {
                     System.err.println("EGLGraphicsConfiguration.eglChooseConfig: #1 eglChooseConfig: recommended fbcfg " + toHexString(recommendedEGLConfig) + ", idx " + recommendedIndex);
@@ -399,7 +398,6 @@ public class EGLGraphicsConfigurationFactory extends GLGraphicsConfigurationFact
         // 2nd choice: get all GLCapabilities available, no preferred recommendedIndex available
         if( null == availableCaps || 0 == availableCaps.size() ) {
             // reset ..
-            recommendedEGLConfig = -1;
             recommendedIndex = -1;
 
             if(!EGL.eglGetConfigs(eglDisplay, configs, configs.capacity(), numConfigs)) {
diff --git a/src/jogl/classes/jogamp/opengl/glu/gl2/nurbs/GLUgl2nurbsImpl.java b/src/jogl/classes/jogamp/opengl/glu/gl2/nurbs/GLUgl2nurbsImpl.java
index 338d7e6d6..a4507b767 100644
--- a/src/jogl/classes/jogamp/opengl/glu/gl2/nurbs/GLUgl2nurbsImpl.java
+++ b/src/jogl/classes/jogamp/opengl/glu/gl2/nurbs/GLUgl2nurbsImpl.java
@@ -37,7 +37,6 @@ import jogamp.opengl.glu.nurbs.*;
 
 import java.lang.reflect.Method;
 
-import javax.media.opengl.GL;
 import javax.media.opengl.GL2;
 import javax.media.opengl.glu.GLUnurbs;
 
@@ -71,13 +70,13 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
   /**
    * Using callback
+   * private final int callBackFlag;
    */
-  private final int callBackFlag;
 
   /**
    * Object for error call backs
+   * private final Object errorCallback;
    */
-  private final Object errorCallback;
 
   /**
    * List of map definitions
@@ -116,8 +115,8 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
   /**
    * Is curve modified
+   * private int isCurveModified;
    */
-  private int isCurveModified;
 
   /**
    * Object holding rendering settings
@@ -126,8 +125,8 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
   /**
    * Display list
+   * private DisplayList dl;
    */
-  private DisplayList dl;
 
   /**
    * Object for subdividing curves and surfaces
@@ -136,13 +135,13 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
   /**
    * Object responsible for rendering
+   * private Backend backend;
    */
-  private Backend backend;
 
   /**
    * Next picewise linear curve in linked list
+   * private O_pwlcurve nextPwlcurve;
    */
-  private O_pwlcurve nextPwlcurve;
 
   /**
    * Next trimming NURBS curve in linked list
@@ -156,13 +155,13 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
   /**
    * Are there any changes in trimming
+   * private boolean isTrimModified;
    */
-  private boolean isTrimModified;
 
   /**
    * Are there any changes in surface data
+   * private boolean isDataSurfaceModified;
    */
-  private boolean isDataSurfaceModified;
 
   /**
    * Nurber of trmims of processed surface
@@ -181,20 +180,20 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
   /**
    * Nextr surface in linked list
+   * private O_nurbssurface nextNurbssurface;
    */
-  private O_nurbssurface nextNurbssurface;
 
   /**
    * Are there any changes in surface
+   * private boolean isSurfaceModified;
    */
-  private boolean isSurfaceModified;
 
   /**
    * Initializes default GLUgl2nurbs object
    */
   public GLUgl2nurbsImpl() {
     // DONE
-    maplist = new Maplist(backend);
+    maplist = new Maplist();
     renderhints = new Renderhints();
     subdivider = new Subdivider();
     // original code
@@ -265,9 +264,9 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
     this.autoloadmode = true;
 
-    this.callBackFlag = 0;
+    // this.callBackFlag = 0;
 
-    this.errorCallback = null;
+    // this.errorCallback = null;
   }
 
   /**
@@ -311,14 +310,14 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
    */
   private void thread(final String name, final Object arg) {
     // DONE
-    final Class partype[] = new Class[1];
+    final Class<?> partype[] = new Class[1];
     partype[0] = arg.getClass();
     Method m;
     try {
       m = this.getClass().getMethod(name, partype);
-      if (dl != null) {
+      /* if (dl != null) {
         dl.append(this, m, arg);
-      } else {
+      } else */ {
         m.invoke(this, new Object[] { arg });
       }
     } catch (final Throwable e) {
@@ -337,9 +336,9 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
     // DONE
     try {
       final Method m = this.getClass().getMethod(name, (Class[]) null);
-      if (dl != null) {
+      /* if (dl != null) {
         dl.append(this, m, null);
-      } else {
+      } else */ {
         m.invoke(this, (Object[]) null);
       }
     } catch (final Throwable e) {
@@ -365,7 +364,7 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
     if (inTrim) {
       if (!nextCurve.equals(o_curve)) {
-        isCurveModified = 1;
+        // isCurveModified = 1;
         nextCurve = o_curve;
       }
     } else {
@@ -375,7 +374,7 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
     }
     nextCurve = o_curve.next;
     // kind of solution of union
-    nextPwlcurve = o_curve.o_pwlcurve;
+    // nextPwlcurve = o_curve.o_pwlcurve;
     nextNurbscurve = o_curve.o_nurbscurve;
   }
 
@@ -395,13 +394,13 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
     if (!playBack)
       bgnrender();
 
-    isTrimModified = false;
-    isDataSurfaceModified = false;
+    // isTrimModified = false;
+    // isDataSurfaceModified = false;
     isDataValid = 1;
     numTrims = 0;
     currentSurface = o_surface;
     nextTrim = o_surface.o_trim;
-    nextNurbssurface = o_surface.o_nurbssurface;
+    // nextNurbssurface = o_surface.o_nurbssurface;
   }
 
   /**
@@ -429,14 +428,14 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
     inSurface = 0;
 
-    nextNurbssurface = null;
+    // nextNurbssurface = null;
 
     if (isDataValid <= 0) {
       return;
     }
 
     if (nextTrim != null) {
-      isTrimModified = true;
+      // isTrimModified = true;
       nextTrim = null;
     }
 
@@ -638,7 +637,7 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
 
     // if(!o_nurbscurve.equals(nextNurbscurve)){
     if (!o_nurbscurve.equals(currentCurve.o_nurbscurve)) {
-      isCurveModified = 1;
+      // isCurveModified = 1;
       currentCurve.o_nurbscurve = o_nurbscurve;
       // nextNurbscurve=o_nurbscurve;
 
@@ -647,15 +646,19 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
     nextNurbscurve = o_nurbscurve.next;
 
     if (!currentCurve.equals(o_nurbscurve.owner)) {
-      isCurveModified = 1;
+      // isCurveModified = 1;
       o_nurbscurve.owner = currentCurve;
     }
 
-    if (o_nurbscurve.owner == null)
+
+    /**
+    if (o_nurbscurve.owner == null) {
       isCurveModified = 1;
+    } */
 
-    if (inCurve == 2)
+    if (inCurve == 2) {
       endcurve();
+    }
   }
 
   /**
@@ -677,18 +680,18 @@ public class GLUgl2nurbsImpl implements GLUnurbs {
     } else
       o_nurbssurface.used = true;
 
-    if (!o_nurbssurface.equals(nextNurbscurve)) {
-      isSurfaceModified = true;
+    // Always true, instances of diff classes are compared: if (!o_nurbssurface.equals(nextNurbscurve)) {
+      // isSurfaceModified = true;
       // nextNurbssurface=o_nurbssurface;
       currentSurface.o_nurbssurface = o_nurbssurface;
-    }
+    // }
 
     if (!currentSurface.equals(o_nurbssurface.owner)) {
-      isSurfaceModified = true;
+      // isSurfaceModified = true;
       o_nurbssurface.owner = currentSurface;
     }
 
-    nextNurbssurface = o_nurbssurface.next;
+    // nextNurbssurface = o_nurbssurface.next;
 
     if (inSurface == 2)
       endsurface();
diff --git a/src/jogl/classes/jogamp/opengl/glu/mipmap/HalveImage.java b/src/jogl/classes/jogamp/opengl/glu/mipmap/HalveImage.java
index 100c6ba1f..95ce86a5b 100644
--- a/src/jogl/classes/jogamp/opengl/glu/mipmap/HalveImage.java
+++ b/src/jogl/classes/jogamp/opengl/glu/mipmap/HalveImage.java
@@ -1140,10 +1140,10 @@ public class HalveImage {
           final int imageSizeInBytes, final boolean isSwap ) {
     int ii, jj;
     final int halfWidth = width / 2;
-    final int halfHeight = height / 2;
+    // final int halfHeight = height / 2;
     final int halfDepth = depth / 2;
     int src = 0;
-    final int padBytes = rowSizeInBytes - ( width * pixelSizeInBytes );
+    // final int padBytes = rowSizeInBytes - ( width * pixelSizeInBytes );
     int outIndex = 0;
 
     assert( (width == 1 || height == 1) && depth >= 2 );
diff --git a/src/jogl/classes/jogamp/opengl/glu/nurbs/Curve.java b/src/jogl/classes/jogamp/opengl/glu/nurbs/Curve.java
index 5fccbc598..ce0387bff 100644
--- a/src/jogl/classes/jogamp/opengl/glu/nurbs/Curve.java
+++ b/src/jogl/classes/jogamp/opengl/glu/nurbs/Curve.java
@@ -182,27 +182,31 @@ public class Curve {
     } else {
       assert (order <= MAXORDER);
 
-      final float tmp[][] = new float[MAXORDER][MAXCOORDS];
-
-      final int tstride = (MAXORDER);
-
-      final int val = 0;
-      // mapdesc.project(spts,stride,tmp,tstride,order);
-
-      //                System.out.println("TODO curve.getsptepsize mapdesc.project");
-
-      if (val == 0) {
-        setstepsize(mapdesc.maxrate);
-      } else {
-        final float t = mapdesc.getProperty(NurbsConsts.N_PIXEL_TOLERANCE);
-        if (mapdesc.isParametricDistanceSampling()) {
-          //                        System.out.println("TODO curve.getstepsize - parametric");
-        } else if (mapdesc.isPathLengthSampling()) {
-          //                        System.out.println("TODO curve.getstepsize - pathlength");
-        } else {
-          setstepsize(mapdesc.maxrate);
-        }
-      }
+      setstepsize(mapdesc.maxrate);
+
+      /***
+          final float tmp[][] = new float[MAXORDER][MAXCOORDS];
+
+          final int tstride = (MAXORDER);
+
+          final int val = 0;
+          // mapdesc.project(spts,stride,tmp,tstride,order);
+
+          //                System.out.println("TODO curve.getsptepsize mapdesc.project");
+
+          if (val == 0) {
+            setstepsize(mapdesc.maxrate);
+          } else {
+            final float t = mapdesc.getProperty(NurbsConsts.N_PIXEL_TOLERANCE);
+            if (mapdesc.isParametricDistanceSampling()) {
+              //                        System.out.println("TODO curve.getstepsize - parametric");
+            } else if (mapdesc.isPathLengthSampling()) {
+              //                        System.out.println("TODO curve.getstepsize - pathlength");
+            } else {
+              setstepsize(mapdesc.maxrate);
+            }
+          }
+      */
 
     }
 
diff --git a/src/jogl/classes/jogamp/opengl/glu/nurbs/Mapdesc.java b/src/jogl/classes/jogamp/opengl/glu/nurbs/Mapdesc.java
index 95ce60c25..b6e83d94c 100644
--- a/src/jogl/classes/jogamp/opengl/glu/nurbs/Mapdesc.java
+++ b/src/jogl/classes/jogamp/opengl/glu/nurbs/Mapdesc.java
@@ -166,9 +166,8 @@ public class Mapdesc {
    * @param type map type
    * @param rational is rational
    * @param ncoords number of control points coords
-   * @param backend backend object
    */
-  public Mapdesc(final int type, final int rational, final int ncoords, final Backend backend) {
+  public Mapdesc(final int type, final int rational, final int ncoords) {
     // DONE
     this.type = type;
     this.isrational = rational;
diff --git a/src/jogl/classes/jogamp/opengl/glu/nurbs/Maplist.java b/src/jogl/classes/jogamp/opengl/glu/nurbs/Maplist.java
index 7f9d7c5a0..2cb92aeef 100644
--- a/src/jogl/classes/jogamp/opengl/glu/nurbs/Maplist.java
+++ b/src/jogl/classes/jogamp/opengl/glu/nurbs/Maplist.java
@@ -47,15 +47,14 @@ public class Maplist {
 
   /**
    * Backend class
+   * private final Backend backend;
    */
-  private final Backend backend;
 
   /**
    * Makes new Maplist
-   * @param backend Backend class
    */
-  public Maplist(final Backend backend) {
-    this.backend = backend;
+  public Maplist(/* final Backend backend */) {
+    // this.backend = backend;
   }
 
   /**
@@ -88,7 +87,7 @@ public class Maplist {
    */
   private void add(final int type, final int rational, final int ncoords) {
     // DONE
-    final Mapdesc map = new Mapdesc(type, rational, ncoords, backend);
+    final Mapdesc map = new Mapdesc(type, rational, ncoords);
     if (maps == null) {
       maps = map;
     } else {
diff --git a/src/jogl/classes/jogamp/opengl/glu/nurbs/Subdivider.java b/src/jogl/classes/jogamp/opengl/glu/nurbs/Subdivider.java
index d7c1dbf3c..573c7df11 100644
--- a/src/jogl/classes/jogamp/opengl/glu/nurbs/Subdivider.java
+++ b/src/jogl/classes/jogamp/opengl/glu/nurbs/Subdivider.java
@@ -133,12 +133,12 @@ public class Subdivider {
   /**
    * Breakpoints
    */
-  private Flist smbrkpts;
+  private final Flist smbrkpts = new Flist();
 
   /**
    * Not used
+   * private float[] stepsizes;
    */
-  private float[] stepsizes;
 
   /**
    * Domain distance in V direction
@@ -651,8 +651,8 @@ public class Subdivider {
         arctesselator.bezier(newright, s, s, t1, t2);
         arctesselator.bezier(newleft, s, s, t2, t1);
       } else {
-        arctesselator.pwl_right(newright, s, t1, t2, stepsizes[0]);
-        arctesselator.pwl_left(newright, s, t2, t1, stepsizes[2]);
+        arctesselator.pwl_right(newright, s, t1, t2, 0 /* stepsizes[0] */);
+        arctesselator.pwl_left(newright, s, t2, t1, 0 /* stepsizes[2] */);
       }
       link(jarc1, jarc2, newright, newleft);
       left.addarc(newright);
@@ -732,7 +732,6 @@ public class Subdivider {
         } else if (hdiff == 0) {
           tailonright.addarc(jarc);
         } else {
-          final Arc jtemp;
           switch (arc_split(jarc, param, value, 0)) {
           case 2:
             tailonright.addarc(jarc);
diff --git a/src/jogl/classes/jogamp/opengl/windows/wgl/awt/WindowsAWTWGLGraphicsConfigurationFactory.java b/src/jogl/classes/jogamp/opengl/windows/wgl/awt/WindowsAWTWGLGraphicsConfigurationFactory.java
index 517ca72cc..9e784ad73 100644
--- a/src/jogl/classes/jogamp/opengl/windows/wgl/awt/WindowsAWTWGLGraphicsConfigurationFactory.java
+++ b/src/jogl/classes/jogamp/opengl/windows/wgl/awt/WindowsAWTWGLGraphicsConfigurationFactory.java
@@ -165,7 +165,7 @@ public class WindowsAWTWGLGraphicsConfigurationFactory extends GLGraphicsConfigu
                     System.err.println("WindowsAWTWGLGraphicsConfigurationFactory: PFD IDs: "+pfdIDs.length+", unique: "+pfdIDOSet.size());
                 }
                 winConfig.preselectGraphicsConfiguration(drawableFactory, pfdIDs);
-                final int gcIdx = pfdIDOSet.indexOf(new Integer(winConfig.getPixelFormatID()));
+                final int gcIdx = pfdIDOSet.indexOf(Integer.valueOf(winConfig.getPixelFormatID()));
                 if( 0 > gcIdx ) {
                     chosenGC = configs[gcIdx];
                     if(DEBUG) {
diff --git a/src/test/com/jogamp/opengl/test/bugs/Issue344Base.java b/src/test/com/jogamp/opengl/test/bugs/Issue344Base.java
index 7692a30b7..54786bbc3 100644
--- a/src/test/com/jogamp/opengl/test/bugs/Issue344Base.java
+++ b/src/test/com/jogamp/opengl/test/bugs/Issue344Base.java
@@ -76,7 +76,7 @@ public abstract class Issue344Base implements GLEventListener
 
         final Rectangle2D bounds = renderer.getBounds(getText());
         final float w = (float) bounds.getWidth();
-        final float h = (float) bounds.getHeight();
+        // final float h = (float) bounds.getHeight();
         textScaleFactor = 2.0f / (w * 1.1f);
         gl.setSwapInterval(0);
     }
diff --git a/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/TextureSequenceCubeES2.java b/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/TextureSequenceCubeES2.java
index b49fa182a..99b943c65 100644
--- a/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/TextureSequenceCubeES2.java
+++ b/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/TextureSequenceCubeES2.java
@@ -180,7 +180,7 @@ public class TextureSequenceCubeES2 implements GLEventListener {
 
         int rsFpPos = preludeGLSLVersion ? rsFp.addGLSLVersion(gl) : 0;
         rsFpPos = rsFp.insertShaderSource(0, rsFpPos, texSeq.getRequiredExtensionsShaderStub());
-        rsFpPos = rsFp.addDefaultShaderPrecision(gl, rsFpPos);
+        rsFp.addDefaultShaderPrecision(gl, rsFpPos);
 
         final String texLookupFuncName = texSeq.getTextureLookupFunctionName(myTextureLookupName);
         rsFp.replaceInShaderSource(myTextureLookupName, texLookupFuncName);
diff --git a/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSBSStereo.java b/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSBSStereo.java
index a2a39d631..0c7bdc81d 100644
--- a/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSBSStereo.java
+++ b/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSBSStereo.java
@@ -369,7 +369,7 @@ public class MovieSBSStereo implements StereoGLEventListener {
 
         int rsFpPos = preludeGLSLVersion ? rsFp.addGLSLVersion(gl) : 0;
         rsFpPos = rsFp.insertShaderSource(0, rsFpPos, mPlayer.getRequiredExtensionsShaderStub());
-        rsFpPos = rsFp.addDefaultShaderPrecision(gl, rsFpPos);
+        rsFp.addDefaultShaderPrecision(gl, rsFpPos);
 
         final String texLookupFuncName = mPlayer.getTextureLookupFunctionName(myTextureLookupName);
         rsFp.replaceInShaderSource(myTextureLookupName, texLookupFuncName);
diff --git a/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSimple.java b/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSimple.java
index 5918e4e6b..50d16db1b 100644
--- a/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSimple.java
+++ b/src/test/com/jogamp/opengl/test/junit/jogl/demos/es2/av/MovieSimple.java
@@ -439,7 +439,7 @@ public class MovieSimple implements GLEventListener {
 
         int rsFpPos = preludeGLSLVersion ? rsFp.addGLSLVersion(gl) : 0;
         rsFpPos = rsFp.insertShaderSource(0, rsFpPos, mPlayer.getRequiredExtensionsShaderStub());
-        rsFpPos = rsFp.addDefaultShaderPrecision(gl, rsFpPos);
+        rsFp.addDefaultShaderPrecision(gl, rsFpPos);
 
         final String texLookupFuncName = mPlayer.getTextureLookupFunctionName(myTextureLookupName);
         rsFp.replaceInShaderSource(myTextureLookupName, texLookupFuncName);
diff --git a/src/test/com/jogamp/opengl/test/junit/jogl/math/TestPMVMatrix01NEWT.java b/src/test/com/jogamp/opengl/test/junit/jogl/math/TestPMVMatrix01NEWT.java
index e111887f7..8c2216f7a 100644
--- a/src/test/com/jogamp/opengl/test/junit/jogl/math/TestPMVMatrix01NEWT.java
+++ b/src/test/com/jogamp/opengl/test/junit/jogl/math/TestPMVMatrix01NEWT.java
@@ -331,7 +331,8 @@ public class TestPMVMatrix01NEWT extends UITestCase {
     @SuppressWarnings("unused")
     @Test
     public void test03MvTranslate() {
-        final FloatBuffer pmvMv, pmvMvi, pmvMvit;
+        final FloatBuffer pmvMv;
+        // final FloatBuffer pmvMvi, pmvMvit;
         {
             final PMVMatrix pmv = new PMVMatrix();
             pmv.glMatrixMode(GLMatrixFunc.GL_PROJECTION);
@@ -341,8 +342,8 @@ public class TestPMVMatrix01NEWT extends UITestCase {
             pmv.glTranslatef(5f, 6f, 7f);
 
             pmvMv = pmv.glGetMvMatrixf();
-            pmvMvi = pmv.glGetMviMatrixf();
-            pmvMvit = pmv.glGetMvitMatrixf();
+            // pmvMvi = pmv.glGetMviMatrixf();
+            // pmvMvit = pmv.glGetMvitMatrixf();
         }
 
         final FloatBuffer glMv = FloatBuffer.allocate(16);
@@ -369,7 +370,8 @@ public class TestPMVMatrix01NEWT extends UITestCase {
     @SuppressWarnings("unused")
     @Test
     public void test04MvTranslateRotate() {
-        final FloatBuffer pmvMv, pmvMvi, pmvMvit;
+        final FloatBuffer pmvMv;
+        // final FloatBuffer pmvMvi, pmvMvit;
         {
             final PMVMatrix pmv = new PMVMatrix();
             pmv.glMatrixMode(GLMatrixFunc.GL_PROJECTION);
@@ -380,8 +382,8 @@ public class TestPMVMatrix01NEWT extends UITestCase {
             pmv.glRotatef(90f, 1f, 0f, 0f);
 
             pmvMv = pmv.glGetMvMatrixf();
-            pmvMvi = pmv.glGetMviMatrixf();
-            pmvMvit = pmv.glGetMvitMatrixf();
+            // pmvMvi = pmv.glGetMviMatrixf();
+            // pmvMvit = pmv.glGetMvitMatrixf();
         }
 
         final FloatBuffer glMv = FloatBuffer.allocate(16);