You are right, we forgot to add the accessors other than the most crucial one. The others will be (secondarily) needed as well, so we gotta add the functions.


On Nov 29, 2013, at 1:22 PM, slava77 <notifications@github.com> wrote:

> In DataFormats/HeavyIonEvent/interface/VoronoiBackground.h:
> 
> > +public:
> > +      VoronoiBackground();
> > +      VoronoiBackground(double pt0, double pt1, double pt2, double mt0, double mt1, double mt2);
> > +      virtual ~VoronoiBackground();
> > +
> > +  double pt() const{
> > +     return pt_corrected;
> > +  }
> > +
> > +protected:
> > +
> > +  double pt_preeq;
> > +  double pt_posteq;
> > +  double pt_corrected;
> > +
> > +  double mt_preeq;
> There are no accessors to these data members.
> What's the point of storing if you can't read it?
> 
> —
> Reply to this email directly or view it on GitHub.
>