Sent from my iPad

> On Feb 18, 2014, at 23:25, slava77 <notifications@github.com> wrote:
> 
> In RecoLocalTracker/SiStripClusterizer/src/ThreeThresholdAlgorithm.cc:
> 
> > @@ -80,19 +90,21 @@
> >  candidateAccepted() const {
> >    return ( !candidateLacksSeed &&
> >  	   noiseSquared * ClusterThresholdSquared
> > -	   <=  std::pow( std::accumulate(ADCs.begin(),ADCs.end(),float(0)), 2));
> > +	   <=  std::pow( float(std::accumulate(ADCs.begin(),ADCs.end(), int(0))), 2.f));
> pow(x, 2.f) still hurts my eyes.
> From my last reading on this, these things are not optimized to x*x
> 
it is optimized to x*x by the compiler.
sorry is the original code, the author prefer as it.
v.
> —
> Reply to this email directly or view it on GitHub.