fix array overflow issues found by David A.

Issues reported at
https://hypernews.cern.ch/HyperNews/CMS/get/swReleases/4108/1/1/1/1/1/1/1/1.html
and follow ups are mostly fixed here. 
so then there is no need for the TMath::Abs(eta) at all (I was just reducing it from being called 4 times to 1). will fix that

On Sep 14, 2014, at 6:52 PM, StoyanStoynev <notifications@github.com> wrote:

> In RecoEcal/EgammaCoreTools/plugins/EcalClusterEnergyUncertaintyObjectSpecific.cc:
> 
> > @@ -204,10 +204,12 @@ float EcalClusterEnergyUncertaintyObjectSpecific::getValue( const reco::SuperClu
> >  	par2[5][5]=-7.27426;
> >  	par3[5][5]=0.201164;
> >  	
> > -	
> > +
> > +	double absEta=TMath::Abs(eta);
> 
> I understand it is an "old" code optimized but:
> eta is defined as "float eta = fabs(superCluster.eta())" which it shouldn't (I think) but 
> as is TMath::Abs is unnecessary and moreover in any of the places std:abs is a better choice; absEta is better to be float here.
> 
> —
> Reply to this email directly or view it on GitHub.
>