Merge pull request #201 from JeffersonLab/fix_compwarnings_rtj

* suppress compiler warnings about unused variables [rtj]
JeffersonLab · Jan 1, 2022 · 936a11f · 936a11f
2 parents bfafade + 9805752
commit 936a11f
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 11 deletions.
diff --git a/src/CobremsGeneration.cc b/src/CobremsGeneration.cc
@@ -72,8 +72,8 @@ CobremsGeneration::CobremsGeneration(double Emax_GeV, double Epeak_GeV)
    fBeamEmittance = 2.5e-9; // m r
    fCollimatorSpotrms = 0.0005; // m
    fCollimatorDistance = 76.0; // m
-   fCollimatorDiameter = 0.0034; // m
-   fTargetThickness = 20e-6; // m
+   fCollimatorDiameter = 0.005; // m
+   fTargetThickness = 50e-6; // m
    fTargetThetay = 0.050; // radians
    fTargetThetaz = 0; // radians
    setTargetCrystal("diamond");
@@ -656,11 +656,13 @@ double CobremsGeneration::Rate_dNcdxdp(double x, double phi)
 
 #if COBREMS_GENERATOR_VERBOSITY > 1
    if (qzmin < 99) {
+#else
+   if (false) {
+#endif
       std::cout << hmin << "," << kmin << "," << lmin
                 << " is the best plane at x=" << x
                 << std::endl;
    }
-#endif
 
    return sum;
 }

diff --git a/src/GlueXPhotonBeamGenerator.cc b/src/GlueXPhotonBeamGenerator.cc
@@ -100,12 +100,6 @@ GlueXPhotonBeamGenerator::GlueXPhotonBeamGenerator(CobremsGeneration *gen)
       }
    }
 
-   // These cutoffs should be set empirically, as low as possible
-   // for good efficiency, but not too low so as to avoid excessive
-   // warnings about Pcut violations.
-
-   double raddz = fCobrems->getTargetThickness() * m;
-
    prepareImportanceSamplingPDFs();
 
    // Create interface for interactive commands

diff --git a/src/GlueXSensitiveDetectorCDC.cc b/src/GlueXSensitiveDetectorCDC.cc
@@ -630,8 +630,6 @@ void GlueXSensitiveDetectorCDC::add_cluster(hit_vector_t &hits,
 
    // drift radius 
    double dradius_cm = hit.d_cm;
-   double d2 = dradius_cm * dradius_cm;
-   double d3 = dradius_cm * d2;  
 
    // Find the drift time for this cluster. Drift time depends on B:
    // (dependence derived from Garfield calculations)