Skip to content

Blapping the last of the warnings - fixing #31#88

Merged
heliosfa merged 21 commits intodevelopmentfrom
warnings_tidy
Aug 16, 2019
Merged

Blapping the last of the warnings - fixing #31#88
heliosfa merged 21 commits intodevelopmentfrom
warnings_tidy

Conversation

@heliosfa
Copy link
Copy Markdown
Contributor

This PR addresses the remaining warnings left over from #31 and adds the -pedantic flag to the make file.

This means that a make -j X -s is now completely silent on POETS Boxes (GCC 7.3.0):

gmb@heaney:~/Orchestrator/Build/gcc$ make clean
if [ -d "../../application_staging" ]; then \
        find "../../application_staging" \
                ! -regex '../../application_staging/xml.*' \
                ! -wholename "../../application_staging" \
                -exec rm --force --recursive {} +; \
fi
rm --force --recursive ./Objects ./Dependency_lists ../../bin \
../../orchestrate.sh ../../test_bin
gmb@heaney:~/Orchestrator/Build/gcc$ make -j 12 -s
gmb@heaney:~/Orchestrator/Build/gcc$

@mvousden also tested on GCC 9.1.0 and I have tested on GCC 6.3.0

Testing carried out:

  • make tests and then ./run-tests.sh passes
gmb@heaney:~/Orchestrator/test_bin$ ./run-tests.sh
Running all tests...
All tests passed.
Running memory checks on all tests...
All memory checks passed.
  • A 40x40 heated plate builds, deploys, executes and completes correctly.
  • A 280x280 heated plate builds, deploys, executes and completes correctly.

@heliosfa heliosfa added the bug Report of a bug label Aug 16, 2019
@heliosfa heliosfa requested review from AlexRast and mvousden August 16, 2019 12:32
@heliosfa heliosfa self-assigned this Aug 16, 2019
@mvousden
Copy link
Copy Markdown
Contributor

Still builds silently on GCC 9.1.0 post-merging-in-development.

I guess you still plan to add -Werror - please do so.

@heliosfa
Copy link
Copy Markdown
Contributor Author

As per quick chat with @mvousden, we are leaving -Werror off for the time being - it will be selectively enabled for CI.

As a general rule, things with warnings won't be accepted for merge.

@heliosfa heliosfa merged commit 2765eed into development Aug 16, 2019
@heliosfa heliosfa mentioned this pull request Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Report of a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants