diff options
author | Sven Gothel <[email protected]> | 2014-07-08 22:03:51 +0200 |
---|---|---|
committer | Sven Gothel <[email protected]> | 2014-07-08 22:03:51 +0200 |
commit | f37592d54f3633c0f195a10770020cbb5a732809 (patch) | |
tree | ce668d86ed7af7caecc30761762a78bef0f43fa7 /src/jogl/classes/jogamp/opengl/glu/nurbs | |
parent | 815776c5760e42c8b3c858a9bab982b203c59c24 (diff) |
Findbugs: Remove dead-code / unused [temp] storage and it's assignment
Diffstat (limited to 'src/jogl/classes/jogamp/opengl/glu/nurbs')
4 files changed, 34 insertions, 33 deletions
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); |