Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Compiling with standard options on Ubuntu 20.04LTS gives 200+ warnings #644

Closed
rodyo opened this issue Aug 24, 2020 · 3 comments · May be fixed by #645
Closed

Compiling with standard options on Ubuntu 20.04LTS gives 200+ warnings #644

rodyo opened this issue Aug 24, 2020 · 3 comments · May be fixed by #645

Comments

@rodyo
Copy link

rodyo commented Aug 24, 2020

As per title.

Most are related to a new warning in GCC8.0+ (-Wcatch-value; performance optimization). Others are -Wunused-but-set-variable (minor quality thing) and -Wimplicit-fallthrough (genuine mistakes, fortunately without ill effects).

I fixed these in the PR.
@rodyo rodyo mentioned this issue Aug 24, 2020
Open
@SamFayolle
Copy link
Collaborator

Hi Rody,

First of all, many thanks for spotting and fixing this! As you might have noticed the Tudat support/development is a bit slowed down during the summer period. But just to let you know, the main developer will be back in a few weeks so he will be able to come back to you on this and also on your other posts and fixes. Thank you again very much for spotting all those issues and offering possible fixes for them :)

Best regards,

Marie

@rodyo
Copy link
Author

rodyo commented Aug 24, 2020

My pleasure :)

@DominicDirkx
Copy link
Member

NOTE: This Tudat version is no longer supported. See https://docs.tudat.space/en/stable/ and https://github.com/tudat-team/tudat-bundle for the new version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed all compilation warnings rodyo/tudat
3 participants
@rodyo @DominicDirkx @SamFayolle