Add time windows for pos and neg ADC selection.

Added fGoodPosAdcMult and fGoodNegAdcMult variables

Remove cuts for  calculation of fXAtAero and fYAtAero

  So that the counts can be reported in report files.

   First hit above this cut is taken as the reference time.
   Cuts are per detector and read as an optional parameter in
   each detector class.  TDCs and Flash ADC times have separate cuts.

Implement a second method of selecting best reference time.
  If InitHitList is passed a possitive reference time cut,
    use the first reference time above that cut.  If none of the
    hits pass the cut, then no reference time is set
  If InitHitList is passed a negative cut, the cut is taken to be
    the absolute value of the cut, but a reference time is garuanteed
    to be set as long as there is at least one reference time hit.
    If none of the reference time hits pass the cut, then the last
    hit is taken to be the reference time.  (Should be the largest.)

Allow reference time cuts for all the spectrometer detector classes
  Hodoscopes, drift chambers, Aerogels, Gas Cherenkovs, Calorimeters

  Add End method to each detector that calls MissReport which
  prints the number of events with missing TDC and ADC reference times.

Add variable for the difference between the Hodoscope start time
and the ADC pulse time.

Select good adc hit based on window on this difference

  Each detector object will check the global variable X.present each
  event.  If X.present is false, it indicates that the parent
  spectrometer object determineted that this spectrometer is not in
  this event, and therefor reference times are not expected.  If
  X.present is false, then the reference time warning messages will
  be suppressed.

Modified so that the fGoodPosAdcPulseTime, fGoodNegAdcPulseTime
  or fGoodAdcPulseTime is set to kBig

Modified THcShowerRawHit::SetReference to takeADC signals

Modified THcTrigRawHit::SetReference to take ADC signals

Modified THcShowerArray::ProcessHits so that fAdcTdcOffset add to frAdcPulseTime

Modified THcShowerArray::ReadDatabase so that defaults for fAdcThreshold etc
  are before gHcParms->LoadParmValues

Fix typo in previous commit's THcCherenkov.cxx

Include methods for FADC conversions and apply to non-raw data objects in THcRawAdcHit

Add raw pulse time data object to Cherenkov detector classes

Add raw pulse time data object to shower detector classes

Add raw pulse time data object to hodoscope detector class

   Gets rid of vars.push_back(RVarDef ... stuff

Also do some whitespace cleanup

  Then the # tubes of the Aerogel is known.
  Remove MaxNumXXXAeroPmt constants from THcAerogel.h

The line fNpeSum += fNegNpeSum
was basically double counting the fNegNpeSum
Same for fNpeSum += fPosNpeSum
since they were in a loop over Neg and Pos hits.

Just set
fNpeSum = fNegNpeSum + fPosNpeSum
after the two loops over Neg and Pos hits

New parameters are added.  hallc_replay of 3/31/2017 or later is needed.

As of now, the size of the TClonesArray objects are hard-coded in the
respective header files. This is not okay in the long run. Ideally,
the size of the TClonesArrays should be defined by the parameter which
dictates how many readout detectors exist for the respective
detector. I tried doing this in the Init method however, podd
complained regarding the size of the TClonesArrays in the DEF-file
histograms. I was unsuccessful in my first attempt. This should be
addressed in the future even though things work for now.

	In THcAerogel.cxx, mean pedestals are calculated if
	fMinPeds>0. Set fMinPeds to default zero in
	THcAerogel::InitializePedestals. Reinitialize it with
	paero_min_peds parameter in paero.param file.

 - Added new leafs to the hodoscope tree.
 - Added new leafs to the aerogel tree.
 - Added new leafs to cerenkov tree.
 - Added new leafs to the shower tree.

Left all the old leafs in there. Will need a careful cleanup...

This makes the code nicer and with less duplication in logic.

Along with that I had to correct all the call sites to the RawHodoHit in all the
classes that use it.

  Otherwise InitHistlist doesn't have the detector map
  and can't do setup of reference time stuff or sanity checks on the
  detector mapping

Some detector classes lacked the ability to get the appropriate
Apparatus character and were only working for HMS.

  Add a new parameter Xaero_tdc_offset to make sure valid tdc
  hits from a reference time corrected tdc are positive.
  Accept first first positve tdc hit after applying offset.
  Move array filling in CoarseProcess into Decode

  Modify THcFormula and THcParmList to match changes to THaFormula
     (needed for reports)
  Small changes in THcInterface and detector classes for compatibility
     with new OO decoder and podd 1.6

  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).

  Use InitArrays method in THcCherenkov and THcAerogel
  Null out all array pointers in the InitArrays method and
  call that method from all constructors so that it is always
  safe to delete these arrays.

These were related to a) scope of some variable declarations, and b) use
of pre-increment vs. post-increment of non-primitive types (this has a
slight performance hit due to the way that pre- vs. post-increment
operators are overloaded in C++.

Also cleaned up a few places where initialized variables were not being
used.

1) Need to add fNelem to DBRequest to read parameter arrays.
2) Added optional variable and set to true. Flag in the
DBRequest to tell code that parameter is optional
and does not have to be found.
3) Move setting of fNelem to before DBRequest for aero_num_pairs
 so that it can be overwritten by parameter file.