RecoLocalCalo : fix gcc700 warning: class has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor] #18944

gartung · 2017-05-25T15:22:06Z

Fixes warnings like these by adding virtual destructor with default constructor.

/build/cmsbld/jenkins-workarea/workspace/build-any-ib/w/tmp/BUILDROOT/8fa3f79dab694f86b12f10b4e4cceb97/opt/cmssw/slc6_amd64_gcc700/cms/cmssw/CMSSW_9_2_X_2017-05-21-2300/src/RecoLocalCalo/EcalRecAlgos/interface/EcalUncalibRecHitRecAbsAlgo.h:17:25: warning: 'class EcalUncalibRecHitRecAbsAlgo' has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor]

cmsbuild · 2017-05-25T15:23:28Z

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

It involves the following packages:

RecoJets/JetAssociationAlgorithms
RecoLocalCalo/EcalRecAlgos

@perrotta, @cmsbuild, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@TaiSakuma, @imarches, @ahinzmann, @acaudron, @rappoccio, @argiro, @jdolen, @nhanvtran, @JyothsnaKomaragiri, @gkasieczka, @schoef, @ferencek, @mverzett, @mariadalfonso, @pvmulder, @seemasharmafnal this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

gartung · 2017-05-25T15:39:15Z

@Dr15Jones

slava77 · 2017-05-25T15:41:34Z

@cmsbuild please test

cmsbuild · 2017-05-25T15:41:50Z

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/20103/console Started: 2017/05/25 17:46

cmsbuild · 2017-05-25T15:51:02Z

-1

Tested at: 0219e0a

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18944/20103/summary.html

I found follow errors while testing this PR

Failed tests: Build ClangBuild

Build:

I found an error when building:

>> Compiling edm plugin /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoTBCalo/EcalTBRecProducers/src/EcalTBWeightUncalibRecHitProducer.cc 
>> Compiling edm plugin /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoTBCalo/EcalTBRecProducers/src/SealModule.cc 
>> Compiling  /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetTracksAssociationXtrpCalo.cc 
>> Compiling  /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetSignalVertexCompatibilityAlgo.cc 
>> Compiling  /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetTracksAssociationExplicit.cc 
/build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetTracksAssociationXtrpCalo.cc:26:1: error: definition of explicitly-defaulted 'JetTracksAssociationXtrpCalo::~JetTracksAssociationXtrpCalo()'
 JetTracksAssociationXtrpCalo::~JetTracksAssociationXtrpCalo() 
 ^
In file included from /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetTracksAssociationXtrpCalo.cc:4:0:
/build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/interface/JetTracksAssociationXtrpCalo.h:36:11: error: 'virtual JetTracksAssociationXtrpCalo::~JetTracksAssociationXtrpCalo()' explicitly defaulted here
   virtual ~JetTracksAssociationXtrpCalo() = default;

Clang:

I found a compilation error while trying to compile with clang:
I used this command:
scram b vclean && scram build -k -j 16 USER_CXXFLAGS='-fsyntax-only' COMPILER='llvm compile'

>> Compiling  /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetTracksAssociationDR.cc 
>> Compiling  /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetTracksAssociationDRVertex.cc 
>> Compiling edm plugin /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoLocalCalo/EcalRecAlgos/plugins/EcalSeverityLevelESProducer.cc 
>> Compiling  /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoLocalCalo/EcalRecAlgos/test/testRunner.cpp 
>> Compiling  /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoLocalCalo/EcalRecAlgos/test/testEcalSeverityLevelAlgo.cppunit.cc 
/build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-25-1100/src/RecoJets/JetAssociationAlgorithms/src/JetTracksAssociationXtrpCalo.cc:26:31: error: definition of explicitly defaulted destructor
JetTracksAssociationXtrpCalo::~JetTracksAssociationXtrpCalo() 
                              ^
1 error generated.
gmake: *** [tmp/slc6_amd64_gcc530/src/RecoJets/JetAssociationAlgorithms/src/RecoJetsJetAssociationAlgorithms/JetTracksAssociationXtrpCalo.o] Error 1
>> Compile sequence completed for CMSSW CMSSW_9_2_X_2017-05-25-1100

cmsbuild · 2017-05-25T15:51:04Z

Comparison not run due to Build errors (RelVals and Igprof tests were also skipped)

slava77 · 2017-05-25T16:06:23Z

-1

…rtual destructor [-Wnon-virtual-dtor]

cmsbuild · 2017-05-25T16:12:38Z

Pull request #18944 was updated. @perrotta, @cmsbuild, @slava77, @davidlange6 can you please check and sign again.

Dr15Jones · 2017-05-25T16:22:41Z

please test

cmsbuild · 2017-05-25T16:23:24Z

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/20113/console Started: 2017/05/25 18:29

cmsbuild · 2017-05-25T17:38:07Z

+1
Tested at: b27b04a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18944/20113/summary.html

cmsbuild · 2017-05-25T17:38:10Z

Comparison job queued.

cmsbuild · 2017-05-25T20:12:45Z

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

Comparison Summary:

No significant changes to the logs found
Reco comparison results: 1731 differences found in the comparisons
DQMHistoTests: Total files compared: 24
DQMHistoTests: Total histograms compared: 1833592
DQMHistoTests: Total failures: 24293
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 1809119
DQMHistoTests: Total skipped: 180
DQMHistoTests: Total Missing objects: 0
Checked 98 log files, 14 edm output root files, 24 DQM output files

perrotta · 2017-05-26T13:30:48Z

+1

technical fix,
jenkins tests pass
if c++ gurus endorse it, it is also fine for reco

cmsbuild · 2017-05-26T13:31:27Z

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 requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

davidlange6 · 2017-05-27T07:41:37Z

+1

cmsbuild added this to the CMSSW_9_2_X milestone May 25, 2017

cmsbuild added comparison-pending orp-pending pending-signatures reconstruction-pending tests-pending labels May 25, 2017

cmsbuild added tests-started and removed tests-pending labels May 25, 2017

cmsbuild added comparison-notrun tests-rejected and removed comparison-pending tests-started labels May 25, 2017

cmsbuild added reconstruction-rejected and removed reconstruction-pending labels May 25, 2017

fix gcc700 warning: class has virtual functions and accessible non-vi…

b27b04a

…rtual destructor [-Wnon-virtual-dtor]

cmsbuild added comparison-pending reconstruction-pending tests-pending and removed comparison-notrun reconstruction-rejected tests-rejected labels May 25, 2017

cmsbuild added tests-started and removed tests-pending labels May 25, 2017

cmsbuild added tests-approved and removed tests-started labels May 25, 2017

cmsbuild added comparison-available and removed comparison-pending labels May 25, 2017

cmsbuild added fully-signed reconstruction-approved and removed pending-signatures reconstruction-pending labels May 26, 2017

cmsbuild added orp-approved and removed orp-pending labels May 27, 2017

cmsbuild merged commit 6410dcb into cms-sw:master May 27, 2017

gartung deleted the RecoLocalCalo-RecoJets-gcc700-dtor-warning-fix branch May 30, 2017 15:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

RecoLocalCalo : fix gcc700 warning: class has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor] #18944

RecoLocalCalo : fix gcc700 warning: class has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor] #18944

gartung commented May 25, 2017

cmsbuild commented May 25, 2017

gartung commented May 25, 2017

slava77 commented May 25, 2017

cmsbuild commented May 25, 2017 •

edited

cmsbuild commented May 25, 2017

cmsbuild commented May 25, 2017

slava77 commented May 25, 2017

cmsbuild commented May 25, 2017

Dr15Jones commented May 25, 2017

cmsbuild commented May 25, 2017 •

edited

cmsbuild commented May 25, 2017

cmsbuild commented May 25, 2017

cmsbuild commented May 25, 2017

perrotta commented May 26, 2017

cmsbuild commented May 26, 2017

davidlange6 commented May 27, 2017

RecoLocalCalo : fix gcc700 warning: class has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor] #18944

RecoLocalCalo : fix gcc700 warning: class has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor] #18944

Conversation

gartung commented May 25, 2017

cmsbuild commented May 25, 2017

gartung commented May 25, 2017

slava77 commented May 25, 2017

cmsbuild commented May 25, 2017 • edited

cmsbuild commented May 25, 2017

cmsbuild commented May 25, 2017

slava77 commented May 25, 2017

cmsbuild commented May 25, 2017

Dr15Jones commented May 25, 2017

cmsbuild commented May 25, 2017 • edited

cmsbuild commented May 25, 2017

cmsbuild commented May 25, 2017

cmsbuild commented May 25, 2017

perrotta commented May 26, 2017

cmsbuild commented May 26, 2017

davidlange6 commented May 27, 2017

cmsbuild commented May 25, 2017 •

edited

cmsbuild commented May 25, 2017 •

edited