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.

Sign up for GitHub

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

Jump to bottom

Ref vs Ptr performance benefits #17

Closed
slava77 opened this issue Jan 16, 2014 · 1 comment
Closed

Ref vs Ptr performance benefits #17

slava77 opened this issue Jan 16, 2014 · 1 comment
Labels
question

Comments

@slava77
Copy link
Owner

slava77 commented Jan 16, 2014

No description provided.

@slava77
Copy link
Owner Author

slava77 commented Jan 22, 2014

Ref should perform better than Ptr.

Ptr is more useful when polymorphism is needed

@slava77 slava77 closed this as completed Jan 22, 2014
slava77 pushed a commit that referenced this issue May 13, 2014
@venturia
Merge pull request #17 from cms-sw/CMSSW_7_1_X
5824050 
Merge cms-sw CMSSW_7_1_X into my CMSSW_7_1_X
slava77 pushed a commit that referenced this issue May 29, 2014
@jshlee
Merge pull request #17 from unclok/CMSSW_6_2_X_SLHC
6cf2d4a 
Adding GEM Digi validation to CMSSW Validation
slava77 pushed a commit that referenced this issue May 29, 2014
Merge pull request #17 from dildick/GEM-ME0-Full-Sim_RecHits
47f2626 
Gem me0 full sim rec hits
slava77 pushed a commit that referenced this issue Sep 12, 2014
Merge pull request #17 from mguilbau/mydevel-branch
ad3b683 
New version of the q2 Trigger Algo
slava77 pushed a commit that referenced this issue May 26, 2015
@lgray
Merge pull request #17 from Sam-Harper/vid_heepV51_ecalDrivenFix
9c36eb5 
adding EcalDriven cut to VID for HEEP
slava77 pushed a commit that referenced this issue Aug 19, 2015
@pietverwilligen
Merge pull request #17 from cms-sw/CMSSW_7_6_X
a13d553 
PR_ToBringUpToDateMyBranch_ME0Segment_PR_v4
slava77 pushed a commit that referenced this issue Nov 30, 2015
@aknayak
Merge pull request #17 from aknayak/my76xReMiniAOD
3fe601d 
another small fix to take care of the comments from PR
slava77 pushed a commit that referenced this issue Jan 19, 2017
@Martin-Grunewald
Merge pull request #17 from cms-AlCaDB/80XHLTAfterMD5Train
e23f17d 
[80X] update the pA simulation GT to take latest L1T menu and HLT JECs
slava77 pushed a commit that referenced this issue Jul 17, 2018
@abrinke1
Merge pull request #17 from cms-sw/CMSSW_10_2_X
f6aef03 
Update branch CMSSW_10_2_X to cms-sw
slava77 pushed a commit that referenced this issue Jul 20, 2020
Merge pull request #17 from PFCal-dev/hgcal_eol_pulse_update_112X_six
b43af76 
Minor edits to tester
slava77 pushed a commit that referenced this issue Oct 9, 2021
@cerati
Merge pull request #17 from cerati/mk-build-improve
c5f8c4c 
Mk build improve2
slava77 pushed a commit that referenced this issue Jun 7, 2022
@kskovpen
Merge pull request #17 from cms-sw/master
0c7b484 
update master
slava77 pushed a commit that referenced this issue Mar 15, 2024
@ariostas
Merge pull request #17 from SegmentLinking/CMSSW_13_3_0_pre3_LST_X_Lo…
72edb80 
…adTwoSDLlibsFix

GPU->CUDA renaming of LST library
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@slava77