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

Build breaks on i386 after vtk-9.2.2 upgrade #458

Closed
yurivict opened this issue Oct 25, 2022 · 2 comments · Fixed by #459
Closed

Build breaks on i386 after vtk-9.2.2 upgrade #458

yurivict opened this issue Oct 25, 2022 · 2 comments · Fixed by #459
Labels
type:bug Something isn't working

Comments

@yurivict
Copy link
Contributor

/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:30: error: non-constant-expression cannot be narrowed from type 'unsigned int' to 'vtkIdType' (aka 'int') in initializer list [-Wc++11-narrowing]
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                             ^~~~~~~~~~~~~~~~
/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:30: note: insert an explicit cast to silence this issue
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                             ^~~~~~~~~~~~~~~~
                             static_cast<vtkIdType>( )
/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:48: error: non-constant-expression cannot be narrowed from type 'unsigned int' to 'vtkIdType' (aka 'int') in initializer list [-Wc++11-narrowing]
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                                               ^~~~~~~~~~~~~~~~
/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:48: note: insert an explicit cast to silence this issue
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                                               ^~~~~~~~~~~~~~~~
                                               static_cast<vtkIdType>( )
2 errors generated.

clang-14
FreeBSD 13.1

@yurivict yurivict added the type:bug Something isn't working label Oct 25, 2022
@Meakk
Copy link
Contributor

Meakk commented Oct 25, 2022

Thank you for reporting the problem. It will be fixed by #459
You can also disable F3D_STRICT_BUILD CMake variable (this variable sets -Werror compilation flag which converts warnings into errors)

@yurivict
Copy link
Contributor Author

yurivict commented Nov 1, 2022

There are still failures with F3D_STRICT_BUILD:

/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:30: error: non-constant-expression cannot be narrowed from type 'unsigned int' to 'vtkIdType' (aka 'int') in initializer list [-Wc++11-narrowing]
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                             ^~~~~~~~~~~~~~~~
/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:30: note: insert an explicit cast to silence this issue
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                             ^~~~~~~~~~~~~~~~
                             static_cast<vtkIdType>( )
/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:48: error: non-constant-expression cannot be narrowed from type 'unsigned int' to 'vtkIdType' (aka 'int') in initializer list [-Wc++11-narrowing]
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                                               ^~~~~~~~~~~~~~~~
/wrkdirs/usr/ports/graphics/f3d/work/f3d-1.3.1/library/VTKExtensions/Readers/vtkF3DAssimpImporter.cxx:450:48: note: insert an explicit cast to silence this issue
        vtkIdType fId[2] = { face.mIndices[0], face.mIndices[1] };
                                               ^~~~~~~~~~~~~~~~
                                               static_cast<vtkIdType>( )
2 errors generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants