diff options
author | Harvey Harrison <[email protected]> | 2012-10-09 23:11:18 -0700 |
---|---|---|
committer | Harvey Harrison <[email protected]> | 2012-10-09 23:11:18 -0700 |
commit | 4fabfa782ccbf76281ebdd9b68cbbc7e2d1cf972 (patch) | |
tree | 19952989eb649f7d3a93f58ac1ad30c16c7b0b96 /src/classes/share/javax | |
parent | 65cd260734a2441ca7ec63ea1c785f8babd73d01 (diff) |
j3dcore: factor out another repetative block with a helper
Signed-off-by: Harvey Harrison <[email protected]>
Diffstat (limited to 'src/classes/share/javax')
-rw-r--r-- | src/classes/share/javax/media/j3d/Shape3DCompileRetained.java | 163 |
1 files changed, 49 insertions, 114 deletions
diff --git a/src/classes/share/javax/media/j3d/Shape3DCompileRetained.java b/src/classes/share/javax/media/j3d/Shape3DCompileRetained.java index ab2d237..32cc619 100644 --- a/src/classes/share/javax/media/j3d/Shape3DCompileRetained.java +++ b/src/classes/share/javax/media/j3d/Shape3DCompileRetained.java @@ -132,148 +132,76 @@ ArrayList<ArrayList<Geometry>> geometryInfo = null; // Now, merged the mergelist and separate list based on geoType, // this enables dlist optmization for (i = 1; i <= GeometryRetained.GEO_TYPE_GEOMETRYARRAY; i++) { - GeometryArrayRetained cgeo = null; - ArrayList<GeometryArrayRetained> curList; switch (i) { case GeometryArrayRetained.GEO_TYPE_QUAD_SET: - if (mergedList[i] != null) { - cgeo = new QuadArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new QuadArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_TRI_SET: - if (mergedList[i] != null) { - cgeo = new TriangleArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new TriangleArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_POINT_SET: - if (mergedList[i] != null) { - cgeo = new PointArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new PointArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_LINE_SET: - if (mergedList[i] != null) { - cgeo = new LineArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new LineArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_TRI_STRIP_SET: - if (mergedList[i] != null) { - cgeo = new TriangleStripArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new TriangleStripArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_TRI_FAN_SET: - if (mergedList[i] != null) { - cgeo = new TriangleFanArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new TriangleFanArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_LINE_STRIP_SET: - if (mergedList[i] != null) { - cgeo = new LineStripArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new LineStripArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_INDEXED_QUAD_SET: - if (mergedList[i] != null) { - cgeo = new IndexedQuadArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new IndexedQuadArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_INDEXED_TRI_SET: - if (mergedList[i] != null) { - cgeo = new IndexedTriangleArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new IndexedTriangleArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_INDEXED_POINT_SET: - if (mergedList[i] != null) { - cgeo = new IndexedPointArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new IndexedPointArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_INDEXED_LINE_SET: - if (mergedList[i] != null) { - cgeo = new IndexedLineArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new IndexedLineArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_INDEXED_TRI_STRIP_SET: - if (mergedList[i] != null) { - cgeo = new IndexedTriangleStripArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new IndexedTriangleStripArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_INDEXED_TRI_FAN_SET: - if (mergedList[i] != null) { - cgeo = new IndexedTriangleFanArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new IndexedTriangleFanArrayRetained()); + addSeparateList(separateList[i]); break; case GeometryArrayRetained.GEO_TYPE_INDEXED_LINE_STRIP_SET: - if (mergedList[i] != null) { - cgeo = new IndexedLineStripArrayRetained(); - curList = mergedList[i]; - cgeo.setCompiled(curList); - geometryList.add(cgeo); - cgeo.setSource(((SceneGraphObjectRetained)curList.get(0)).source); - } - addSeparateList(separateList[i]); + if (mergedList[i] != null) + addMergedList(mergedList[i], new IndexedLineStripArrayRetained()); + addSeparateList(separateList[i]); break; } } @@ -281,6 +209,13 @@ ArrayList<ArrayList<Geometry>> geometryInfo = null; } +private void addMergedList(ArrayList<GeometryArrayRetained> glist, + GeometryArrayRetained cgeo) { + cgeo.setCompiled(glist); + geometryList.add(cgeo); + cgeo.setSource(((SceneGraphObjectRetained) glist.get(0)).source); +} + private void addSeparateList(ArrayList<GeometryArrayRetained> glist) { if (glist == null) return; |