Overview
Examples
Screenshots
Comparisons
Applications
Download
Documentation
Tutorials
Bazaar
Status & Roadmap
FAQ
Authors & License
Forums
Funding Ultimate++
Search on this site
Search in forums












SourceForge.net Logo
Home » U++ Library support » ScatterDraw & ScatterCtrl » Compilation of ScatterDraw cause warnings (Should be fixed before 2019.1)
Compilation of ScatterDraw cause warnings [message #51493] Wed, 03 April 2019 20:44 Go to next message
Klugier is currently offline  Klugier
Messages: 614
Registered: September 2012
Location: Poland, Kraków
Contributor
Hello,

Our latest changes in build methods (We added -Wall flags that enables a lot of additional warnings) there are a lot of errors related to Scatter. I tested with ScatterCtrl_Demo. Here is the list:
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (6): In file included from /home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h:6,
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.h (4): from /home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.h:4,
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (1): from /home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp:1:
 (): bool areas;
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (385): warning: unused variable 'posx' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (386): warning: unused variable 'posy' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (387): warning: unused variable 'dx' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (388): warning: unused variable 'dy' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'NONE' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'LEFT_UP' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'LEFT_DOUBLE' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'LEFT_MOVE' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'MIDDLE_UP' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'MIDDLE_MOVE' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'RIGHT_UP' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterCtrl/ScatterCtrl.cpp (759): warning: enumeration value 'RIGHT_MOVE' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_NULL' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_JUSTIFY' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1408): warning: unused variable 'r' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1448): warning: unused variable 'x_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1449): warning: unused variable 'y_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_NULL' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_JUSTIFY' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1408): warning: unused variable 'r' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1448): warning: unused variable 'x_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1449): warning: unused variable 'y_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_NULL' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_JUSTIFY' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1408): warning: unused variable 'r' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1448): warning: unused variable 'x_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1449): warning: unused variable 'y_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (717): warning: 'Upp::TableData::areas' will be initialized after [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (712): warning:   'int Upp::TableData::lendata' [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.h (671): warning:   when initialized here [-Wreorder]
/home/klugier/upp/uppsrc/ScatterDraw/DataSource.cpp (848): warning: this 'if' clause does not guard... [-Wmisleading-indentation]
/home/klugier/upp/uppsrc/ScatterDraw/Legend.cpp (142): warning: enumeration value 'TOP' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_NULL' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_JUSTIFY' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1408): warning: unused variable 'r' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1448): warning: unused variable 'x_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1449): warning: unused variable 'y_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_NULL' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1661): warning: enumeration value 'ALIGN_JUSTIFY' not handled in switch [-Wswitch]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1408): warning: unused variable 'r' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1448): warning: unused variable 'x_c' [-Wunused-variable]
/home/klugier/upp/uppsrc/ScatterDraw/ScatterDraw.h (1449): warning: unused variable 'y_c' [-Wunused-variable]


Koldo, it would be nice if we manage to fix that issues before release.

Sincerely,
Klugier


Ultimate++ - one framework to rule them all.
Re: Compilation of ScatterDraw cause warnings [message #51499 is a reply to message #51493] Fri, 05 April 2019 09:25 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 2944
Registered: August 2008
Veteran
Hi Klugier

I have fixed it with MinGW (I do not know if the version you use generates more warnings).

Big surprise as I use "pedantic" with no warning, and U++ unfortunately generates tons of warnings.

It seems "all" is stronger than "pedantic".


Best regards
Iñaki
Re: Compilation of ScatterDraw cause warnings [message #51506 is a reply to message #51499] Sat, 06 April 2019 15:17 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 2944
Registered: August 2008
Veteran
After compiling in Linux, I have found and fixed last warnings.

Klugier: Please solve Core warnings. They are not much but, but being in Core, they appear repeatedly.
I have been compiling with "pedantic" for years and having continuously those warnings has been unpleasant.


Best regards
Iñaki
Re: Compilation of ScatterDraw cause warnings [message #51507 is a reply to message #51506] Sun, 07 April 2019 11:16 Go to previous message
Klugier is currently offline  Klugier
Messages: 614
Registered: September 2012
Location: Poland, Kraków
Contributor
Hello Koldo,

I can confirm that warnings gone away. Thanks for fixing this!

Backing to pedantic flag - I can not commit to Core directory. So, I am probably not good person to solve that issue. You should ask Mirek. However, I think that pedantic mode is too strong to officially introduce it to U++ default build methods. When I compiled TheIDE it mainly warns that there is additional ";", which field of destruction is very small.

Sincerely,
Klugier


Ultimate++ - one framework to rule them all.
Previous Topic: Added Rotate()
Goto Forum:
  


Current Time: Fri Apr 19 00:20:32 CEST 2019

Total time taken to generate the page: 0.00600 seconds