Changed PluginFactoryBase::*findPMaker to return void*

The functions *findPMaker used to return an iterator to an internal structure of PluginFactoryBase. This exposed the implementation too much (since it will be changed next). However, the calling routine didn’t really care about the iterator and always just pull one particular value from the iterator. The functions now returns that one value.
Member data m_plugins was moved from protected to private
Use mutex to protect Plugin system

dlopen is not thread safe and therefore must be protected from simultaneous access. Given that this system is used outside of cmsRun using a mutex was considered the best option for serialization. Given the fact that the plugin system shouldn’t be called in a time critical manner, it was easiest to protect the member data also using a mutex. If this causes a performance problem, we can review it and try to use a tbb thread-safe container instead.
Add the valuemap PfCanditate-Photon
Add the valuemap PfCanditate-Photon
Add the valueMap to store the Photon Ref for each PF egamma Candidate
Fix bug, the singleLegConversions were not being really put in the event
Add the valueMap to store the Photon Ref for each PF egamma Candidate
Add the valueMap to store the Photon Ref for each PF egamma Candidate
Fix a mistake
change the simplePhotonAnalyzer to test value map
Shorten the name of the valu map
adding ValueMap
adding test module
Remove the requirement of PF candidate being a gamma
remove spurious cout
RecoEgamma Consumes migration
RecoEgamma Consumes migration
a few things to propagate from the consumes migration
fix some faulty config files, remove bug from GsfElectrons