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

Fix basic suite narrowing conversions #1156

Open
ellnor01 opened this issue Feb 12, 2021 · 1 comment
Open

Fix basic suite narrowing conversions #1156

ellnor01 opened this issue Feb 12, 2021 · 1 comment

Comments

@ellnor01
Copy link
Contributor

Enabling narrowing errors by removing
if(CMAKE_COMPILER_IS_GNUCC OR "${CMAKE_CXX_COMPILER_ID}" MATCHES "(Apple)?Clang") add_cxx_flag_if_supported(-Wno-narrowing) endif()
in test_conformance/basic/CMakeLists.txt produces many errors. A common example is:
test_conformance/basic/test_readimage_int16.cpp:87:49: error: narrowing conversion of 'img_width' from 'int' to 'size_t {aka long unsigned int}' inside { } [-Werror=narrowing] size_t region[3] = {img_width, img_height, 1};

@ellnor01
Copy link
Contributor Author

This is a follow up for #787 and so requires #1144 to be merged first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant