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.

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

Compile warnings on Ubuntu build #4911

Closed
scsides opened this issue Apr 16, 2022 · 2 comments
Closed

Compile warnings on Ubuntu build #4911

scsides opened this issue Apr 16, 2022 · 2 comments
Assignees
Labels
bug Something isn't working inactive Issue that has been inactive for at least 6 months

Comments

@scsides
Copy link
Contributor

scsides commented Apr 16, 2022

ISIS version(s) affected: dev 2022/04/16

Description

There are several compiler warnings on the Ubuntu build of dev

How to reproduce

Compile ISIS on the Ubuntu prog machine

Possible Solution

Additional context
[866/3612] Building CXX object objects/CMakeFiles/isis.dir/base/apps/isisimport/isisimport.cpp.o
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/src/base/apps/isisimport/isisimport.cpp: In lambda function:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/src/base/apps/isisimport/isisimport.cpp:136:12: warning: variable ‘foundfilter’ set but not used [-Wunused-but-set-variable]
bool foundfilter = false;
^~~~~~~~~~~
[3498/3612] Building CXX object tests/CMakeFiles/runISISTests.dir/FunctionalTestsCkwriter.cpp.o
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsCkwriter.cpp: In member function ‘virtual void DefaultCube_FunctionalTestCkwriterDefault_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsCkwriter.cpp:68:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < newKernelRotation->TimeBasedMatrix().size(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsCkwriter.cpp:73:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < newKernelRotation->AngularVelocity().size(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsCkwriter.cpp: In member function ‘virtual void DefaultCube_FunctionalTestCkwriterFromlist_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsCkwriter.cpp:137:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < newKernelRotation->TimeBasedMatrix().size(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsCkwriter.cpp:142:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < newKernelRotation->AngularVelocity().size(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[3518/3612] Building CXX object tests/CMakeFiles/runISISTests.dir/FunctionalTestsFootprintinit.cpp.o
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp: In member function ‘virtual void DefaultCube_FunctionalTestFootprintinitDefault_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp:37:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < coordArray.getSize(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp: In member function ‘virtual void DefaultCube_FunctionalTestFootprintinitLincSinc_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp:59:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < coordArray.getSize(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp: In member function ‘virtual void DefaultCube_FunctionalTestFootprintinitVertices_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp:81:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < coordArray.getSize(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp: In member function ‘virtual void DefaultCube_FunctionalTestFootprintinitCamera_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp:103:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < coordArray.getSize(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp: In member function ‘virtual void DefaultCube_FunctionalTestFootprintinitTestXY_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp:125:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < coordArray.getSize(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp: In member function ‘virtual void DefaultCube_FunctionalTestFootprintinitPrecision_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsFootprintinit.cpp:153:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < coordArray.getSize(); i++) {
~~^~~~~~~~~~~~~~~~~~~~~~
[3542/3612] Building CXX object tests/CMakeFiles/runISISTests.dir/FunctionalTestsJigsaw.cpp.o
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsJigsaw.cpp: In member function ‘virtual void ApolloNetwork_FunctionalTestJigsawApollo_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsJigsaw.cpp:174:15: warning: ‘outputPoint’ may be used uninitialized in this function [-Wmaybe-uninitialized]
measure = outputPoint->GetMeasure(csvLine[2]);
^
[3569/3612] Building CXX object tests/CMakeFiles/runISISTests.dir/FunctionalTestsSpkwriter.cpp.o
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsSpkwriter.cpp: In member function ‘virtual void DefaultCube_FunctionalTestSpkwriterDefault_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsSpkwriter.cpp:49:11: warning: variable ‘newCamera’ set but not used [-Wunused-but-set-variable]
Camera *newCamera = nullptr;
^~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsSpkwriter.cpp: In member function ‘virtual void DefaultCube_FunctionalTestSpkwriterFromlist_Test::TestBody()’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/FunctionalTestsSpkwriter.cpp:108:11: warning: variable ‘newCamera’ set but not used [-Wunused-but-set-variable]
Camera *newCamera = nullptr;
^~~~~~~~~
[3605/3612] Building CXX object tests/CMakeFiles/runISISTests.dir/TestUtilities.cpp.o
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/TestUtilities.cpp: In function ‘QString Isis::fileListToString(QVector)’:
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/TestUtilities.cpp:284:26: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (size_t i = 0; i < fileList.size(); i++) {
~~^~~~~~~~~~~~~~~~~
/usgs/cpkgs/isisTest/isisHRSCSRC/ISIS3/isis/tests/TestUtilities.cpp:288:13: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (i != fileList.size() - 1) {
~~^~~~~~~~~~~~~~~~~~~~~~

@scsides scsides added the bug Something isn't working label Apr 16, 2022
@github-actions
Copy link

Thank you for your contribution!

Unfortunately, this issue hasn't received much attention lately, so it is labeled as 'stale.'

If no additional action is taken, this issue will be automatically closed in 180 days.

@github-actions github-actions bot added the inactive Issue that has been inactive for at least 6 months label Oct 13, 2022
@AustinSanders
Copy link
Contributor

This issue should be resolved as of #5050

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working inactive Issue that has been inactive for at least 6 months
Projects
None yet
Development

No branches or pull requests

2 participants