[RECO] Remove ROOT::Math dict which are unused

[smuzaffar]

PR description:

PR #26883 replaced the usage of ROOT::Math::* with explicit dictionaries for

  <class pattern="std::vector<ROOT::Math::SMatrix<*>>" />
  <class pattern="std::vector<ROOT::Math::MatRepStd<*>>" />
  <class pattern="std::vector<ROOT::Math::RowOffsets<*>>" />
  <class pattern="std::vector<ROOT::Math::SVector<*>>" />

rootcling complains that 3 of these are not used.

>> Building LCG reflex dict from header file src/DataFormats/Math/src/classes.h
  Warning: Unused class rule: std::vector<ROOT::Math::SVector<*>>
   Warning: Unused class rule: std::vector<ROOT::Math::RowOffsets<*>>
   Warning: Unused class rule: std::vector<ROOT::Math::MatRepStd<*>>

This PR proposes to drop these to avoid the warnings.

PR validation:

Local compilation does not show these warnings any more.

[smuzaffar]

please test

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26931/9973

• This PR adds an extra 12KB to repository

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/495/console Started: 2019/05/24 15:34
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/498/console Started: 2019/05/24 16:56
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/499/console Started: 2019/05/24 17:20
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/500/console Started: 2019/05/24 18:10

[cmsbuild]

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master.

It involves the following packages:

DataFormats/Math

@perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@makortel, @felicepantaleo, @rovere this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+1
Tested at: 384b996
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a8b8f8/500/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
e8ee23e
4e0cefc
472998f
7087b99
319d932
6beb290
acef289
cc0c012
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-26931/500/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-26931/500/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 33
• DQMHistoTests: Total histograms compared: 3206856
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3206521
• DQMHistoTests: Total skipped: 334
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 32 files compared)
• Checked 137 log files, 14 edm output root files, 33 DQM output files

[smuzaffar]

@slava77 , @perrotta and @fabiocos , we need to get in to avoid the build errors caused by #26883 . We did not see the errors in pr tests as the DataFormatsMath library was available from release area. In full build this resulted in build errors.

[slava77]

+1

for #26931 384b996

• technical
• jenkins tests pass
  • the build now does not show the compilation warnings

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[fabiocos]

+1