Hi Slava,
as far as I know a static method cannot be declared const  - it does not
have access to the object, so it cannot modify it anyway.

Ciao,
.Andrea


On 24 April 2014 01:05, Slava Krutelyov <notifications@github.com> wrote:

> In RecoParticleFlow/PFProducer/plugins/importers/TrackFromParentImporter.h:
>
> > @@ -0,0 +1,85 @@
> > +#ifndef __TrackFromParentImporter_H__
> > +#define __TrackFromParentImporter_H__
> > +
> > +#include "RecoParticleFlow/PFProducer/interface/BlockElementImporterBase.h"
> > +#include "DataFormats/ParticleFlowReco/interface/PFBlockElementTrack.h"
> > +#include "DataFormats/ParticleFlowReco/interface/PFRecTrackFwd.h"
> > +#include "DataFormats/ParticleFlowReco/interface/PFRecTrack.h"
> > +
> > +// this adaptor class gets redefined later to match the
> > +// needs of the collection and importing cuts we are using
> > +template<class Collection>
> > +class ParentCollectionAdaptor {
> > +public:
> > +  static bool check_importable(const typename Collection::value_type&) {
>
> does this method need to be non-const?
> All static class methods should be const (else it's implying possibility
> of static non-const values inside, which is not allowed).
> The same applies to a lot of methods here in this PR
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/cms-sw/cmssw/pull/3385/files#r11928391>
> .
>