View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0008624

ITK

public

2009-02-26 10:18

2010-10-26 15:05

Reporter

Andinet

Assigned To

kentwilliams

Priority

normal

Severity

minor

Reproducibility

always

Status

closed

Resolution

open

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0008624: Warning when itkHessianRecursiveGaussianImageFilter is instantiated with 2D image

Description

Instantiating itkHessianRecursiveGaussianImageFilter using a 2D image
will produce the following warning

/.../Insight/Code/BasicFilters/itkHessianRecursiveGaussianImageFilter.txx:38: warning: comparison of unsigned expression < 0 is always false

This is due to

for( unsigned int i = 0; i<NumberOfSmoothingFilters; i++ )
    {
    GaussianFilterPointer filter = GaussianFilterType::New();
    filter->SetOrder( GaussianFilterType::ZeroOrder );
    filter->SetNormalizeAcrossScale( m_NormalizeAcrossScale );
    filter->ReleaseDataFlagOn();
    m_SmoothingFilters.push_back( filter );
    }

NumberOfSmoothingFilters = 2 - 2 = 0

Additional Information

I added code to instantiate the 2D version of the filter, and then ran a compile with

-Wall -W -Wshadow -Wunused-variable -Wunused-parameter -Wunused-function -Wunused -Wno-system-headers -Wwrite-strings -Wno-deprecated -Woverloaded-virtual

and I didn't see any warnings for this filter.

Tags

No tags attached.

Resolution Date

Sprint

Sprint Status

backlog

Attached Files

Relationships

Notes
(0022647) Hans Johnson (developer) 2010-10-21 14:23	We believe it is fixed, but it deserves verification.

Issue History
Date Modified	Username	Field	Change
2009-02-26 10:18	Andinet	New Issue
2010-06-16 19:10	Ivan Macia	Status	new => assigned
2010-06-16 19:10	Ivan Macia	Assigned To	=> Ivan Macia
2010-10-21 14:23	Hans Johnson	Assigned To	Ivan Macia => kentwilliams
2010-10-21 14:23	Hans Johnson	Note Added: 0022647
2010-10-26 15:05	kentwilliams	Sprint Status	=> backlog
2010-10-26 15:05	kentwilliams	Additional Information Updated
2010-10-26 15:05	kentwilliams	Status	assigned => closed