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

Implement isArray() and arrayLength() #1692

Merged
merged 1 commit into from Dec 5, 2013

Conversation

wmtan
Copy link
Contributor

@wmtan wmtan commented Dec 5, 2013

Add the implementation of two functions needed to replace the use of Reflex in the Conditions code.
Implementation provided by Paul Russo.

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 5, 2013

A new Pull Request was created by @wmtan for CMSSW_7_0_ROOT6_X.

Implement isArray() and arrayLength()

It involves the following packages:

FWCore/Utilities

@cmsbuild, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks.
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.

@apfeiffer1
Copy link
Contributor

+1
even if I'm not asked to review !! :)

On Thu, Dec 5, 2013 at 9:58 PM, cmsbuild notifications@github.com wrote:

A new Pull Request was created by @wmtan https://github.com/wmtan for
CMSSW_7_0_ROOT6_X.

Implement isArray() and arrayLength()

It involves the following packages:

FWCore/Utilities

@cmsbuild https://github.com/cmsbuild, @Dr15Joneshttps://github.com/Dr15Jones,
@ktf https://github.com/ktf, @nclopezo https://github.com/nclopezocan you please review it and eventually sign? Thanks.
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.


Reply to this email directly or view it on GitHubhttps://github.com//pull/1692#issuecomment-29937207
.

Thanks,
cheers, andreas

@Dr15Jones
Copy link
Contributor

+1

davidlt added a commit that referenced this pull request Dec 5, 2013
Implement isArray() and arrayLength()
@davidlt davidlt merged commit e1eed05 into cms-sw:CMSSW_7_0_ROOT6_X Dec 5, 2013
@wmtan wmtan deleted the AddArrayFunctions branch December 9, 2013 22:17
ggovi pushed a commit to ggovi/cmssw that referenced this pull request Jan 11, 2017
Advance data for RecoJets/JetProducers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants