Removed not so useful function BeamFitter::formatBTime()

[gartung]

which made "char ts[]" a static because otherwise the function returns the address of
a temporary. This function was only called by BeamFitter::updateBTime,
which was changed to produce the same results of formating two GMT times as strings
and putting the results in member variables.

[cmsbuild]

A new Pull Request was created by @gartung (Patrick Gartung) for CMSSW_7_1_X.

Removed not so useful function BeamFitter::formatBTime()

It involves the following packages:

RecoVertex/BeamSpotProducer

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

[cmsbuild]

Pull request #2135 was updated. @nclopezo, @cmsbuild, @anton-a, @diguida, @rcastello, @slava77, @Degano, @thspeer can you please check and sign again.

[nclopezo]

+1
The errors in the RelVals should be unrelated to this pr
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2135/15/summary.html

[rcastello]

+1

[slava77]

+1

based on 3d36851
code change OK
jenkins kind of ran (compiled OK, otherwise this part of the code is not run in regular tests)

[cmsbuild]

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

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2135/36/summary.html