Enforce variable conventions
  Default using_scin and using_prune selection to off
  Break different track selection methods into different class methods

section of THcHallCSpectrometer to be clearer.
Does not follow the fortran in detail, but should be doing the same
thing.

  There is some unclear code in the track selection that could
  do with improvement.

  Move hodo_slop retrieval to THcHodoscope
  Get fPosCenter array length from fNelem instead of fixing length at 16
  Remove now unneeded THcScintillatorPlane creation method

  Don't start variables that are not member variables with f
  Declare temporary variables as locally as possible
  Remove some unused member variables from THcHodoscope.h

negative side PMT signals from the HMS and SOS calorimeters.

    Changes in the Ycor method of the THcShower class, and in
    examples/PARAM/hcana.param input file.

Minimm value of normalized energy  fNormETot and minum value of number of
cherenkov photo electrons, fNCerNPE, is added to parameter files for the
calculation good scintillator event for tracking efficiency.
They are  optional parameters

fScinShould and fScinDid are set to 0 or 1
and only incrmented when Shower and Cerenkov detectors are setup.

This file was missing.

NCerNPE and fNormETot be optional parameters
When there is not the  !fChern || !fShower and instead just
not increment the counter.

If calorimeter and cherenkov is not added to SOS or HMS then it prints
a warning message and does not calcualte the efficiency for that
spectrometer.

Minimm value of normalized energy and minum value of number of
cherenkov photo electrons is added to parameter files for the
calculation of tracking efficiency.

  Variable found in a report template is first tested to see if it is a
  Hall C style parameter (in gHcParms).  If not, then tested to see if it
  is a global variable (in gHaVars).

  Move some stuff outside of # tracks > 0 test

  Clean up THcShower.h and THcShower.cxx.
  typedef vector<THcShowerHit*> THcShowerHitList by
  typedef set<THcShowerHit*> THcShowerHitList.
  Remove inheritance of class THcShowerCluster from THcShowerHitList.
  Define THcShowerCluster as THcShowerHitList, aka set<THcShowerHit*>.
  Remove inheritance of class THcShowerClusterList from
        THcShClusterList, a.k.a. vector<THcShowerCluster*> container.
  Define THcShowerClusterList as vector<THcShowerCluster*>.
  Rename THcShowerHitSet by THcShowerHitList.
  Rename HitList (related to the HMS calorimeter) to HitSet.
  Add operator< in THcShowerHit class, in order to have THcShowerHitSet
    objects to be properly sorted.

    THcHallCSpectrometer inherits from THaSpectrometer which has method
    GetPcentral and a member fPcentral.
     THcHallCSpectrometer had been using fPCentral but decided to
    use fPcentral.
    Delete fPCentral in THcHallCSpectrometer.h
    Replace all fPCentral  by fPcentral in ReadDatabase and FindVertices methods.

    Add variable fEtot= the total energy of the calorimeter which
      is calculated in Decode method
    Add variable fEtotnorm= the total energy of the calorimeter
      divided by central spectrometer which is calculated in Decode method
    Add both to DefineVariables list as etot and etotnorm

      Declare loop variables in the for statement
      Go back to UInt_t for some variables
      Don't prefix non-member variables with f

  Adds tests to THcHodoscope fiducial tracking efficiency
  Adds tests to THcCherenkov for Cerenkov efficiency

       State neighbours hits in a row separated by single block in
       THcShower::IsNeighbour method. This allows for inclusion of a
       stand alone hit of low energy deposition (presumably coming
       from noise in a channel) into a big cluster, hence reducing
       differences between engine and hcana.

  When it was UInt_t, variable length global variables didn't get the
  length correctly and there were run time errors.

  For the most part this is making sure that the loop variable in
  a for loop is of the same type as the variable that it is compared
  against (Often a UInt_t).

This reverts commit e9ce1040.

Conflicts:
	src/THcHallCSpectrometer.cxx

  For the most part this is making sure that the loop variable in
  a for loop is of the same type as the variable that it is compared
  against (Often a UInt_t).