Fix many build warnings #25
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Hey guys, I'm building jedi-bundle develop for the first time in a while, and am seeing lots of compiler warnings with icx / ifort. These compilers are being generous by passing much of this with warnings instead of raising errors, and I'd recommend tightening up the CI checks to catch these.
C++:
int field_vid[vars.size()]invokes undefined behavior. Preferstd::vector<int> field_vid(vars.size()).-Walland-warn all, there are many overloaded virtual function warnings. Essentially, base class functions are being masked (not overridden) with different parameters in derived classes, and that's messy.Fortran:
Impact
At the very least, this fixes failures in
test_util_signal_trap_fpe_div_by_zeroandtest_l95_getkf_cppthat I observed. Conversely, it exposes transient bugs intest_assimilation_etkfandtest_qg_incrementthat should be diagnosed.Checklist