Print

Print


Author: [log in to unmask]
Date: Tue May 17 18:00:07 2016
New Revision: 4361

Log:
Minor changes to class accessibly.

Modified:
    java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/EcalCrystal.java
    java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/GVector.java
    java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/Geant4Position.java

Modified: java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/EcalCrystal.java
 =============================================================================
--- java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/EcalCrystal.java	(original)
+++ java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/EcalCrystal.java	Tue May 17 18:00:07 2016
@@ -15,7 +15,7 @@
  * 
  * @author Jeremy McCormick
  */
-public class EcalCrystal extends DetectorElement {
+public final class EcalCrystal extends DetectorElement {
     
     private Hep3Vector positionFront;
 

Modified: java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/GVector.java
 =============================================================================
--- java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/GVector.java	(original)
+++ java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/GVector.java	Tue May 17 18:00:07 2016
@@ -8,7 +8,7 @@
  * 
  * @author Annie Simonyan
  */
-public class GVector {
+class GVector {
 
     private double x = 0, y = 0, z = 0;
     private double[] vec;

Modified: java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/Geant4Position.java
 =============================================================================
--- java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/Geant4Position.java	(original)
+++ java/branches/HPSJAVA-409/detector-model/src/main/java/org/hps/detector/ecal/Geant4Position.java	Tue May 17 18:00:07 2016
@@ -14,7 +14,8 @@
  * 
  * @author Annie Simonyan
  */
-public class Geant4Position {
+// FIXME: There is not anything specific to Geant4 about this class, so it should be renamed.
+public final class Geant4Position {
     
     private GVector centerPoint; // center coordinate
     private GVector crysvec; // crystal vector formed by front face center point and back face center point