fix uninitialised var timetype to avoid error from checker

[apfeiffer1]

... as reported by DavidA in https://hypernews.cern.ch/HyperNews/CMS/get/swDevelopment/3051.html

[apfeiffer1]

+1

[cmsbuild]

A new Pull Request was created by @apfeiffer1 (Andreas Pfeiffer) for CMSSW_7_2_X.

fix uninitialised var timetype to avoid error from checker

It involves the following packages:

CondCore/ESSources

@cmsbuild, @diguida, @rcastello, @cerminar, @nclopezo can you please review it and eventually sign? Thanks.
@ghellwig 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.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[apfeiffer1]

+1
as the robot did not see my earlier +1 .. :(

On Thu, Sep 18, 2014 at 12:12 PM, cmsbuild notifications@github.com wrote:

A new Pull Request was created by @apfeiffer1
https://github.com/apfeiffer1 (Andreas Pfeiffer) for CMSSW_7_2_X.

fix uninitialised var timetype to avoid error from checker

It involves the following packages:

CondCore/ESSources

@cmsbuild https://github.com/cmsbuild, @diguida
https://github.com/diguida, @rcastello https://github.com/rcastello,
@cerminar https://github.com/cerminar, @nclopezo
https://github.com/nclopezo can you please review it and eventually
sign? Thanks.
@ghellwig https://github.com/ghellwig 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.
@nclopezo https://github.com/nclopezo you are the release manager for
this.
You can merge this pull request by typing 'merge' in the first line of
your comment.

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

Thanks,
cheers, andreas

[cmsbuild]

+1
Tested at: deb3c82
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5405/999/summary.html

[diguida]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). @nclopezo can you please take care of it?

[cmsbuild]

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

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). @nclopezo can you please take care of it?

[davidlt]

There is extra ; at the end.

[diguida]

@davidlt
Thanks for noticing this: it seems that I need a review of my glasses! ;-)
Question: why do the tests run fine?