Commit in GeomConverter/src/org/lcsim/geometry/subdetector on MAIN
AbstractTestBeam.java+5-51.12 -> 1.13
Another fix to the fix. With the changed variable check the position setting should be done outside of the loop

GeomConverter/src/org/lcsim/geometry/subdetector
AbstractTestBeam.java 1.12 -> 1.13
diff -u -r1.12 -r1.13
--- AbstractTestBeam.java	11 Feb 2013 21:48:30 -0000	1.12
+++ AbstractTestBeam.java	13 Feb 2013 10:47:50 -0000	1.13
@@ -14,7 +14,7 @@
 
 /**
  * @author Jeremy McCormick <[log in to unmask]>
- * @version $Id: AbstractTestBeam.java,v 1.12 2013/02/11 21:48:30 jeremy Exp $
+ * @version $Id: AbstractTestBeam.java,v 1.13 2013/02/13 10:47:50 grefe Exp $
  */
 public abstract class AbstractTestBeam extends AbstractLayeredSubdetector
 {
@@ -56,10 +56,10 @@
 
         Element positionElem = node.getChild("position");
 
+        double posX = 0;
+        double posY = 0;
+        double posZ = 0;
         if (positionElem != null) {
-            double posX = 0;
-            double posY = 0;
-            double posZ = 0;
             Attribute posAttrib = positionElem.getAttribute("x");
             if (posAttrib != null) {
                 posX = posAttrib.getDoubleValue();
@@ -73,8 +73,8 @@
             if (posAttrib != null) {
                 posZ = posAttrib.getDoubleValue();
             }
-            this.position = new BasicHep3Vector(posX, posY, posZ);
         }
+        this.position = new BasicHep3Vector(posX, posY, posZ);
     }
 
     /**
CVSspam 0.2.12


Use REPLY-ALL to reply to list

To unsubscribe from the LCD-CVS list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=LCD-CVS&A=1