View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002342Slicer4Core: Base Codepublic2012-07-24 02:302012-07-24 06:36
Reporterjcfr Assigned Tonicole  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version 
Target VersionSlicer 4.2.0Fixed in VersionSlicer 4.2.0 
Summary0002342: Check state of warning about "spacing" in vtkImageAccumulateDiscreteExecute.cxx
Description

/home/jchris/Projects/vvn/Libs/MRML/Core/vtkImageAccumulateDiscrete.cxx: In instantiation of ‘void vtkImageAccumulateDiscreteExecute(vtkImageAccumulateDiscrete, vtkImageData, T, vtkImageData, int*) [with T = char]’:
/home/jchris/Projects/vvn/Libs/MRML/Core/vtkImageAccumulateDiscrete.cxx:159:55: required from here
/home/jchris/Projects/vvn/Libs/MRML/Core/vtkImageAccumulateDiscrete.cxx:73:34: warning: variable ‘spacing’ set but not used [-Wunused-but-set-variable]

Does this warning illustrate an incomplete implementation or is it just a false alarm ?

The variable wasn't used in the original commit: http://viewvc.slicer.org/viewvc.cgi/Slicer4/trunk/Libs/MRML/vtkImageAccumulateDiscrete.cxx?view=markup&pathrev=6525

TagsNo tags attached.

Activities

nicole

nicole

2012-07-24 06:31

administrator   ~0005220

I think it's a false warning, svn 20680 removes the variable.
jcfr

jcfr

2012-07-24 06:36

administrator   ~0005221

Confirmed that the warning doesn't appear anymore.

Issue History

Date Modified Username Field Change
2012-07-24 02:30 jcfr New Issue
2012-07-24 02:33 jcfr Assigned To => nicole
2012-07-24 02:33 jcfr Status new => assigned
2012-07-24 02:33 jcfr Target Version => Slicer 4.2.0 - October 1st 2012
2012-07-24 02:33 jcfr Description Updated
2012-07-24 02:33 jcfr Description Updated
2012-07-24 06:31 nicole Note Added: 0005220
2012-07-24 06:32 nicole Status assigned => resolved
2012-07-24 06:32 nicole Resolution open => fixed
2012-07-24 06:36 jcfr Note Added: 0005221
2012-07-24 06:36 jcfr Status resolved => closed
2012-07-24 06:36 jcfr Fixed in Version => Slicer 4.2.0 - October 1st 2012