- Added better logging
- Removed throws
	modified:   THcRawAdcHit.cxx

- added hit logging to THcRawAdcHit
- removed throws when too many pulses where set

 Conflicts:
	src/THcCherenkov.cxx

Do HMS pathlength correction using xptar and delta instead
of the focal plane quantities.

Fixed sign typo in fROC1_ePosCoinTime and fROC2_ePosCoinTime

Fix sign error in DeltaSHMSpathLength and DeltaHMSpathLength

Add tree variables
DeltaSHMSPathLength
DeltaHMSPathLength
had_coinCorr_Positron
elec_coinCorr

 Conflicts:
	src/THcHelicity.cxx

	modified:   src/THcHelicity.cxx
	modified:   src/THcHelicity.h

	modified:   ../src/THcCherenkov.h
	modified:   ../src/THcCherenkovHit.h
	modified:   ../src/THcHitList.cxx
	modified:   ../src/THcHitList.h
	modified:   ../src/THcRawAdcHit.h
	modified:   ../src/include/hcana/HallC_Data.h

	modified:   THcCherenkov.cxx
	modified:   THcCherenkov.h
	modified:   THcHodoscope.cxx
	modified:   THcHodoscope.h
	modified:   include/HallC_LinkDef.h
	new file:   include/hcana/HallC_Data.h

	modified:   src/THcDC.h
	modified:   src/THcDriftChamberPlane.cxx
	modified:   src/THcDriftChamberPlane.h
	modified:   src/include/HallC_LinkDef.h

	modified:   THcDC.cxx
	modified:   THcDC.h
	modified:   THcHodoscope.cxx
	modified:   include/HallC_LinkDef.h

- Each detector should override ManualInitTree(TTree*)
- This method should be used to add branches to the tree
- These branches should be data members of the detector.

Logger is used almost everywhere.

	modified:   ../CMakeLists.txt

	modified:   Dockerfile
	modified:   ../../src/CMakeLists.txt

- Lots of additional logging after moving spdlog out of project
- The loggers are mostly defined in the header `analyzer/include/podd2/Logger.h`
- Next step  is to add file logging and to set good defaults
- The spdlog severity level (in order of importance) are:
  - critical
  - error
  - warn
  - info
  - debug
  - trace

	modified:   THcHallCSpectrometer.cxx
	modified:   THcHallCSpectrometer.h
	modified:   THcPrimaryKine.cxx
	modified:   THcSecondaryKine.cxx
	modified:   THcTrigDet.cxx

- removed spdlog ( now in analyzer)
- added logging to THaAnalzyer (podd)

In THcScalerEvtHandler::AnalyzeBuffer there is a loop over
the scalers to keep track of scalers with a cut on the current.
There was a mistake in using scal_prev_read[nscal-1] instead of
dvars_prev_read[ivar] in the IF statement.

fGoodFlags holds event-by-event data, but was never cleared,
eventually growing to several hundred megabytes in a long analysis.
The effciency calculations in THcHodoEff that rely on the contents
of fGoodFlags have been most certainly garbage up to now.

I've replaced the event-by-event ClearEvent() method with the
standard virtual function Clear() and have added fGoodFlags.clear()
there.

Also, I've added assertions to the hodoscope code to check
the validity of some non-obvious array indices. Nothing triggers;
the index logic appears sound (despite its appearance ...)

This includes a particularly nasty leak in the shower code that caused
several hundred bytes to be lost _per event_.