return pointer to streamCache type

[gartung]

Small typo found when implementing test classes for edm:global namespace. Returned pointer to Class type T instead of Cache type C. Compared to RunCacheHolder and LuminosityBlockCacheHolder to determine what return type should be.

[cmsbuild]

A new Pull Request was created by @gartung (Patrick Gartung) for CMSSW_7_0_X.

return pointer to streamCache type

It involves the following packages:

FWCore/Framework

@Dr15Jones, @ktf can you please review it and eventually sign? Thanks.
@wmtan this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@ktf you are the release manager for this.

[cmsbuild]

Pull request #848 was updated. @Dr15Jones, @ktf can you please check and sign again.

[Dr15Jones]

It should be 'C*' since the module is allowed to modify the object since that object is only accessible in a 'stream' call.

[gartung]

On 9/16/2013 10:59 AM, Chris Jones wrote:

It should be 'C*' since the module is allowed to modify the object
since that object is only accessible in a 'stream' call.

—
Reply to this email directly or view it on GitHub
#848 (comment).

OK I will back out the second commit.

[gartung]

runCache() and luminosityCache() both return C const*

[Dr15Jones]

runCache() and luminosityBlockCache() should return a 'C const*' since multiple threads could access them simultaneously. The StreamCache is only accessible to one thread.

[Dr15Jones]

+1

[nclopezo]

+1
https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc481/585/consoleFull
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/585

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @ktf can you please take care of it?