Yes, it would be safe to set them both to nullptr, will do.

On Sep 18, 2013, at ,Sep 18, 10:22 AM, Chris Jones wrote:

> In CondFormats/CastorObjects/src/CastorElectronicsMap.cc:
> 
> > @@ -16,9 +16,7 @@
> >  
> >  CastorElectronicsMap::CastorElectronicsMap() : 
> >    mPItems(CastorElectronicsId::maxLinearIndex+1),
> > -  mTItems(CastorElectronicsId::maxLinearIndex+1),
> > -  sortedByPId(false),
> > -  sortedByTId(false)
> > +  mTItems(CastorElectronicsId::maxLinearIndex+1)
> For safety, shouldn't this set mPItemsById and mTItemsByTrigId to nullptr?
> 
> —
> Reply to this email directly or view it on GitHub.
>