Skip to content
New issue

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

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

Already on GitHub? # to your account

Warnings from new tests #613

Closed
jessemapel opened this issue Nov 27, 2018 · 0 comments
Closed

Warnings from new tests #613

jessemapel opened this issue Nov 27, 2018 · 0 comments
Assignees
Labels
unit test label to use for unit test gtest refactor

Comments

@jessemapel
Copy link
Contributor

I get the following warnings from our new tests when I build:

/usgs/pkgs/isis3nightly2018-11-27/isis/tests/PixelTests.cpp:468:3: warning: floating constant exceeds range of ‘double’ [-Woverflow]
   EXPECT_EQ(std::string("Invalid"), Pixel::ToString(-1.0e+1000));
   ^
/usgs/pkgs/isis3nightly2018-11-27/isis/tests/PixelTests.cpp:480:3: warning: floating constant exceeds range of ‘double’ [-Woverflow]
   EXPECT_EQ(std::string("Invalid"), Pixel(1,2,3,-1.0e+1000).ToString());
   ^
@jessemapel jessemapel added the unit test label to use for unit test gtest refactor label Nov 27, 2018
@jessemapel jessemapel self-assigned this Dec 17, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
unit test label to use for unit test gtest refactor
Projects
None yet
Development

No branches or pull requests

1 participant