Ahh, ok. What I meant that if we need to get rid off setters then both classes SiStripRecHit1D and SiStripRecHit2D will need to be modified and therefore the places I quote should be properly handle it.

On Dec 12, 2013, at ,Dec 12, 11:49 AM, Chris Jones <notifications@github.com> wrote:

> In DataFormats/TrackerRecHit2D/interface/SiStripRecHit1D.h:
> 
> >  
> >    virtual int dimension() const {return 1;}
> >    virtual void getKfComponents( KfComponentsHolder & holder ) const {getKfComponents1D(holder);}
> >  
> > - 
> > -  double sigmaPitch() const { return sigmaPitch_;}
> > -  void setSigmaPitch(double sigmap) const { sigmaPitch_=sigmap;}
> > +
> > +  double sigmaPitch() const;
> > +  void setSigmaPitch(double sigmap) const;
> The lines you quote, only 
> https://cmssdt.cern.ch/SDT/lxr/source/RecoLocalTracker/SiStripRecHitConverter/interface/SiStripRecHitMatcher.h#207
> is actually calling SiStripRecHit1D::setSigmaPitch. The others are calling SiStripRecHit2D::setSigmaPitch.
> 
> —
> Reply to this email directly or view it on GitHub.
>