GSI Forum: Bugs, Fixes, Releases

Home » PANDA » PandaRoot » Bugs, Fixes, Releases » Compiler Warnings

Show: Today's Messages :: Polls :: Message Navigator

Compiler Warnings [message #20112]

Fri, 13 January 2017 15:12

Ralf Kliemt
Messages: 507
Registered: May 2007
Location: GSI, Darmstadt

first-grade participant

From: *dyn.telefonica.de

Dear Developers,

There are a large amount of compiler warnings in our code. Even when disabling "-Weffc++" we were at over 6000 warnings. Several occurring multiple times due to the dependency structure.
I took the liberty to remove some of them. One major haul was submitted yesterday, removing unused variable declarations. Although I paid a lot of attention there may be one or two variables that are not used but their declaration is calling a function that does actual calculations.

Every package responsible should revisit those changes!

Cheers!
Ralf

Report message to a moderator

Re: Compiler Warnings [message #20138 is a reply to message #20112]

Wed, 18 January 2017 09:52

Ralf Kliemt
Messages: 507
Registered: May 2007
Location: GSI, Darmstadt

first-grade participant

From: *gsi.de

Hi all.

There is another commit, containing fixes for -Wextra, mainly copy constructor fixes and ignored "const" expressions.

Please note there are not initialized variables in the following files:

PndRichHitProducer.cxx
PndTrkClaenup.cxx
PndGlobalIdealTrackMerger.cxx
PndTrkTracking2.cxx

Cheers!
Ralf

Report message to a moderator

Re: Compiler Warnings [message #20194 is a reply to message #20112]

Fri, 27 January 2017 11:10

Ralf Kliemt
Messages: 507
Registered: May 2007
Location: GSI, Darmstadt

first-grade participant

From: *gsi.de

Dear Developers,

There is a new commit dealing mostly with the order of variable initialization. (2/3 of this part is done).

I want to steer the attention to the following files:

PndSttGeometryMap.cxx: l.227

PndEmc.cxx: l.468
PndEmcStructure.cxx: l.561
---> Here are logical operations not written explicitly with braces. Please revisit this part thoroughly!

Cheers!
Ralf

[Updated on: Fri, 27 January 2017 11:10]

Report message to a moderator

Re: Compiler Warnings [message #20197 is a reply to message #20194]

Fri, 27 January 2017 12:51

Dmitry Morozov
Messages: 34
Registered: May 2007
Location: Protvino, Russia

continuous participant

From: 192.160.233*

Dear Ralf,

do I understand you correclty that "Here are logical operations not written explicitly with braces" is dealing with readability and clearity of reading the code for other developers?

Thank you,
Dmitry.

Report message to a moderator

Re: Compiler Warnings [message #20199 is a reply to message #20197]

Fri, 27 January 2017 14:53

Ralf Kliemt
Messages: 507
Registered: May 2007
Location: GSI, Darmstadt

first-grade participant

From: *gsi.de

Hi Dmitry,

This is one aspect. However, I would like to have the developers (is that you?) revisit this special point, because it creates a compiler warning with the "-Wall" flag. I also fear that the logic contains a real bug in this place.

Cheers!
Ralf

[Updated on: Fri, 27 January 2017 14:53]

Report message to a moderator

Re: Compiler Warnings [message #20204 is a reply to message #20199]

Mon, 30 January 2017 12:51

Dmitry Morozov
Messages: 34
Registered: May 2007
Location: Protvino, Russia

continuous participant

From: 192.160.233*

Hi Ralf,

I touched only stuff concerning FSC within these files. I'm not the author of that lines with warnings. This part handles gemetry of Farward Endcap EMC.

Best regards,
Dmitry

Report message to a moderator

Re: Compiler Warnings [message #20661 is a reply to message #20112]

Wed, 22 March 2017 08:45

Ralf Kliemt
Messages: 507
Registered: May 2007
Location: GSI, Darmstadt

first-grade participant

From: *gsi.de

Hi Folks,

We're down to 13 Warnings, excluding the thousands from -Weffc++.
I think we don't need to fix the warnings concerning effc++ as it is about having for each class a copy constructor, an assignment operator and to initialize always before the scope of the constructors starts.

From now on we will see the warnings during compilation on our systems, so take care please.

Cheers!
Ralf

Report message to a moderator

Re: Compiler Warnings [message #20663 is a reply to message #20661]

Wed, 22 March 2017 09:02

Tobias Stockmanns
Messages: 489
Registered: May 2007

first-grade participant

From: *ikp.kfa-juelich.de

Great work Ralf!

Report message to a moderator

Previous Topic:	EvtGen - time information lost
Next Topic:	New PandaRoot Release feb17p1

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Wed Nov 27 21:38:10 CET 2024

Total time taken to generate the page: 0.00828 seconds