Hi Chris,

Uhm, reading the notes above:

    /// some classes to make this fit into a template footprint
    /// for RecoPFClusterRefCandidate so we can make jets and MET
    /// out of PFClusters.

So, this is trying to fit the signature of
DataFormats/Candidate/interface/LeafRefCandidateT.h

@rappoccio
Sal should point better to the right place.
Maybe a reference is not necessary.

		--slava


On 11/6/13, 4:31 PM, Chris Jones wrote:
> In DataFormats/ParticleFlowReco/interface/PFCluster.h:
> 
>> @@ -130,8 +128,7 @@
>>      
>>      /// dummy vertex access
>>      math::XYZPoint const & vertex() const { 
> 
> I had thought it might be a virtual function override from the base
> class so the signature couldn't change. But that is not the case. If you
> want, I can make the change you suggested. Of course, one can wonder why
> those methods exist at all.
> 
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cms-sw/cmssw/pull/1345/files#r7466580>.
> 

-- 
***********************************************************************
 Vyacheslav (Slava) Krutelyov
 TAMU: Physics Dept Texas A&M MS4242, College Station, TX 77843-4242
 CERN: 42-R-027
 AIM/Skype: siava16              googleTalk: slava77@gmail.com
 (630) 291-5128  Cell (US)       +41 76 275 7116 Cell (CERN)

-----------------------------------------------------------------------