View Issue Details

IDProjectCategoryView StatusLast Update
0003812Slicer4Module VolumeRenderingpublic2018-10-16 11:54
Reporterjcfr Assigned Tofinetjul  
PrioritynormalSeverityminorReproducibilityhave not tried
Status resolvedResolutionfixed 
Product Version 
Target VersionFixed in VersionSlicer 4.8.1 
Summary0003812: Fix -Waggressive-loop-optimizations - undefined behavior warning in vtkSlicerGPURayCastMultiVolumeMapper
Description

Slicer r23591

Release build / Ubuntu 13.10 / g++ (Ubuntu/Linaro 4.8.1-10ubuntu9) 4.8.1

/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastVolumeMapper.cxx: In member function ‘void vtkSlicerGPURayCastVolumeMapper::SetupRayCastParameters(vtkRenderer, vtkVolume)’:
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastVolumeMapper.cxx:433:48: warning: iteration 4u invokes undefined behavior [-Waggressive-loop-optimizations]
volMat[i] = (GLfloat)((matrix->Element))[i];
^
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastVolumeMapper.cxx:432:3: note: containing loop
for (int i = 0; i < 16; i++)
^
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastVolumeMapper.cxx:433:48: warning: iteration 4u invokes undefined behavior [-Waggressive-loop-optimizations]
volMat[i] = (GLfloat)(
(matrix->Element))[i];
^
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastVolumeMapper.cxx:432:3: note: containing loop
for (int i = 0; i < 16; i++)
^

/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastMultiVolumeMapper.cxx: In member function ‘void vtkSlicerGPURayCastMultiVolumeMapper::SetupRayCastParameters(vtkRenderer, vtkVolume)’:
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastMultiVolumeMapper.cxx:403:48: warning: iteration 4u invokes undefined behavior [-Waggressive-loop-optimizations]
volMat[i] = (GLfloat)((matrix->Element))[i];
^
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastMultiVolumeMapper.cxx:402:3: note: containing loop
for (int i = 0; i < 16; i++)
^
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastMultiVolumeMapper.cxx:403:48: warning: iteration 4u invokes undefined behavior [-Waggressive-loop-optimizations]
volMat[i] = (GLfloat)(
(matrix->Element))[i];
^
/path/to/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerGPURayCastMultiVolumeMapper.cxx:402:3: note: containing loop
for (int i = 0; i < 16; i++)
^

TagsNo tags attached.

Activities

jcfr

jcfr

2018-10-16 11:54

administrator   ~0016157

Fixed in r25972: ENH: Remove unused VolumeRenderingReplacements fixed point raycaster override
See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&amp;revision=25972

Issue History

Date Modified Username Field Change
2014-08-20 11:55 jcfr New Issue
2014-08-20 11:55 jcfr Status new => assigned
2014-08-20 11:55 jcfr Assigned To => finetjul
2014-08-20 11:59 jcfr Description Updated
2018-10-16 11:54 jcfr Status assigned => resolved
2018-10-16 11:54 jcfr Resolution open => fixed
2018-10-16 11:54 jcfr Fixed in Version => Slicer 4.8.1
2018-10-16 11:54 jcfr Note Added: 0016157