Hi Yana

You are right - I just pushed this change in the same branch. Could you 
check if this is in or should I make a new push and pull request?

Sunanda

On Fri, 23 May 2014, ianna wrote:

> 
> In Geometry/CaloEventSetup/plugins/HGCalTopologyBuilder.cc:
> 
> > @@ -90,7 +90,7 @@ HGCalTopologyBuilder::produce(const IdealGeometryRecord&
>  iRecord ) {
> > 
> >    edm::ESHandle<HGCalDDDConstants>  pHGDC;
> >    iRecord.get(name_, pHGDC) ;
> > -  const HGCalDDDConstants* hgdc = &(*pHGDC);
> > +  const HGCalDDDConstants hgdc = (*pHGDC);
> 
> @bsunanda - I think, you meant const reference here:
> 
> const HGCalDDDConstants&
> 
> —
> Reply to this email directly or view it onGitHub.[5033146__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcxNjQ2Nj
> QxNywiZGF0YSI6eyJpZCI6MzI5NDgwNDR9fQ==--604acc6108f8ecf420414f99e02bf26c246
> 4dc9e.gif]
> 
> 
>