Drop unused function.

[ktf]

This is actually appearing as unused now.

[cmsbuild]

A new Pull Request was created by @ktf (Giulio Eulisse) for CMSSW_7_4_X.

Drop unused function.

It involves the following packages:

RecoTracker/TkDetLayers

@cmsbuild, @cvuosalo, @nclopezo, @slava77 can you please review it and eventually sign? Thanks.
@ghellwig, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @appeltel, @gpetruc, @cerati, @dgulhan, @venturia 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.

[ktf]

please test

[davidlange6]

the phase 1 tracker code is still a work in progress- maybe we should revisit this one in a few months when that work is fully ported to 7xy..

[venturia]

David,

there is no problem to remove that function: with the code factorized as it is now there is no need for it anymore. I forgot to remove it myself.

                                     Andrea

On Jan 7, 2015, at 20:40 , David Lange <notifications@github.commailto:notifications@github.com>
wrote:

the phase 1 tracker code is still a work in progress- maybe we should revisit this one in a few months when that work is fully ported to 7xy..

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/7064#issuecomment-69078859.

[slava77]

please test

I'm curious why cms-bot didn't listen to Giulio's "please test"

[cmsbuild]

The tests are being triggered in jenkins.

[slava77]

+1

for #7064 8176af5
git grep shows no use other than the definition
letting cms-bot to handle the rest

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes or unless it breaks tests.

[cmsbuild]

+1
Tested at: 8176af5
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7064/1762/summary.html

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7064/1762/summary.html