Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make getObj in the *Generator classes thread safe #261

Merged
merged 1 commit into from Aug 19, 2013
Merged

Make getObj in the *Generator classes thread safe #261

merged 1 commit into from Aug 19, 2013

Conversation

gartung
Copy link
Member

@gartung gartung commented Aug 8, 2013

Make getObj in the *Generator classes thread safe by not returning pointer to member data obj_.

@ghost ghost assigned nclopezo Aug 9, 2013
@ktf
Copy link
Contributor

ktf commented Aug 9, 2013

@nclopezo please test.

@demattia please review and sign.

@nclopezo
Copy link
Contributor

nclopezo commented Aug 9, 2013

Hi

i tested theses changes in CMSSW_7_0_X_2013-08-09-0200, I ran the unit tests and RelVals.

The logs for building, unit tests and RelVals can be found here:

https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc472/79/console

@ghost ghost assigned demattia Aug 16, 2013
@ktf
Copy link
Contributor

ktf commented Aug 16, 2013

@demattia can you please review and sign?

@cmsbuild
Copy link
Contributor

The following categories have been signed by @demattia: Calibration and Alignment

@cms-git-alca

ktf added a commit that referenced this pull request Aug 19, 2013
…getObj-thread-safe

Make getObj in the *Generator classes thread safe
@ktf ktf merged commit 4211ada into cms-sw:CMSSW_7_0_X Aug 19, 2013
@gartung gartung deleted the CalibTracker-SiStripESProducers-getObj-thread-safe branch August 19, 2013 15:01
parbol pushed a commit to parbol/cmssw that referenced this pull request Mar 5, 2015
H2TauTau: uniformed ntuple production sequence across htt channels.
makortel pushed a commit to makortel/cmssw that referenced this pull request Apr 22, 2015
if full release has disabled debug subpackage then disable it for patch too
andjuo pushed a commit to andjuo/cmssw that referenced this pull request Sep 19, 2018
alignment_new.c now fully working. still some features to be added an…
mandrenguyen added a commit to mandrenguyen/cmssw that referenced this pull request Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants