Commit in GeomConverter/src/org/lcsim/geometry/segmentation on RefactorBranch
GridXYZ.java+1-11.15.2.1 -> 1.15.2.2
NonprojectiveCylinder.java+31.15.2.2 -> 1.15.2.3
ProjectiveZPlane.java-11.16.2.1 -> 1.16.2.2
+4-2
3 modified files
Just added some comments

GeomConverter/src/org/lcsim/geometry/segmentation
GridXYZ.java 1.15.2.1 -> 1.15.2.2
diff -u -r1.15.2.1 -r1.15.2.2
--- GridXYZ.java	28 Sep 2005 00:31:53 -0000	1.15.2.1
+++ GridXYZ.java	28 Sep 2005 00:34:55 -0000	1.15.2.2
@@ -71,7 +71,7 @@
         }
     }
     
-    /* FIXME: How to impl this?  It requires specific information about the subdetector topology. */
+    /* FIXME: Implementation requires very specific information about the subdetector topology. --JM */
     public long[] getNeighbourIDs(int a, int b, int c)
     {
         return new long[3];

GeomConverter/src/org/lcsim/geometry/segmentation
NonprojectiveCylinder.java 1.15.2.2 -> 1.15.2.3
diff -u -r1.15.2.2 -r1.15.2.3
--- NonprojectiveCylinder.java	28 Sep 2005 00:31:53 -0000	1.15.2.2
+++ NonprojectiveCylinder.java	28 Sep 2005 00:34:55 -0000	1.15.2.3
@@ -203,6 +203,9 @@
 
     // Not for public use, this is needed to calculate positions
     // and number of cells, etc.
+    /*
+     * FIXME: Are these next four functions really needed? --JM
+     */
     private double getZMin() {
 	return ((CylindricalSubdetector)detector).getZMin();
     }

GeomConverter/src/org/lcsim/geometry/segmentation
ProjectiveZPlane.java 1.16.2.1 -> 1.16.2.2
diff -u -r1.16.2.1 -r1.16.2.2
--- ProjectiveZPlane.java	28 Sep 2005 00:13:29 -0000	1.16.2.1
+++ ProjectiveZPlane.java	28 Sep 2005 00:34:55 -0000	1.16.2.2
@@ -61,7 +61,6 @@
     
     public double getZ()
     {
-        /* getRadius() is misnamed here.  Actually returning distance to layer center along Z. */
         return -Math.signum(getTheta()-PI/2) * getDistanceToSensitive(getLayer() );
     }
     
CVSspam 0.2.8