Right. Fixed.
Thanks!
-F
On 03/19/2014 06:59 AM, Chris Jones wrote:
>
> In DataFormats/EcalDetId/src/EKDetId.cc:
>
> > +    0b011111111111111111111, // 9
> > +    0b001111111111111111111, // 10
> > +    0b001111111111111111111, // 11
> > +    0b000111111111111111111, // 12
> > +    0b000011111111111111111, // 13
> > +    0b000011111111111111111, // 14
> > +    0b000001111111111111111, // 15
> > +    0b000000111111111111111, // 16
> > +    0b000000011111111111111, // 17
> > +    0b000000000111111111111, // 18
> > +    0b000000000011111111111, // 19
> > +    0b000000000000111111111, // 20
> > +    0b000000000000000111111  // 21
> > +  };
> > +
> > +  const int* EK_CONFIG [] = {blackBox, noTaperEcalEta4, noTaperEcalEta3, taperEcalEta4, taperEcalEta3, 0};
>
> Should be
> |int const * const EK_CONFIG [] = ...|
> Else this will be flagged as a potential thread safety problem.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/cms-sw/cmssw/pull/2925/files#r10745755>.
>