Hi Volker,

Yes, I really want to stop the cmsRun execution for a NULL pointer.
In this particular situation, NULL pointer should never happen and
it means there is a bug somewhere. If the NULL pointer is detected
at this location, it will happen during the processing of the first event,
so not much processing time will be lost.

Regards,
Igor

On 05/12/2014 05:21 AM, Volker Adler wrote:
>
> In JetMETCorrections/FFTJetObjects/interface/L2RecoScaleCalculator.h:
>
> > +    inline explicit L2RecoScaleCalculator(const edm::ParameterSet& ps)
> > +        : m_radiusFactor(ps.getParameter<double>("radiusFactor")) {}
> > +
> > +    inline virtual ~L2RecoScaleCalculator() {}
> > +
> > +    inline virtual void mapFFTJet(const reco::Jet& /* jet */,
> > +                                  const reco::FFTJet<float>& fftJet,
> > +                                  const math::XYZTLorentzVector& /* current */,
> > +                                  double* buf, const unsigned dim) const
> > +    {
> > +        if (dim != 1)
> > +            throw cms::Exception("FFTJetBadConfig")
> > +                << "In L2RecoScaleCalculator::mapFFTJet: "
> > +                << "invalid table dimensionality: "
> > +                << dim << std::endl;
> > +        assert(buf);
>
> Do you really want to stop the |cmsRun| execution for a NULL pointer instead of simply exiting the function?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/cms-sw/cmssw/pull/3810/files#r12521561>.
>