Proposals:Refactoring Statistics Framework 2007 Iterators: Difference between revisions
From KitwarePublic
Jump to navigationJump to search
| Line 10: | Line 10: | ||
*Decrement operator was not incremented correctly in itkListSample.h (Fixed) | *Decrement operator was not incremented correctly in itkListSample.h (Fixed) | ||
*Testing coverage of iterators is not complete, allowing problems like the one above to go unnoticed. | |||
*Functionality was not available to create (or assign) a const iterator from a non-const iterator in itkSubsample.h (Fixed). This problem was found in a number of classes (Not yet fixed). | *Functionality was not available to create (or assign) a const iterator from a non-const iterator in itkSubsample.h (Fixed). This problem was found in a number of classes (Not yet fixed). | ||
*In general a consistent set of iterator operations is not present. | *In general a consistent set of iterator operations is not present. | ||
Revision as of 13:53, 29 March 2007
Review of Iterators from the Statistics Framework
Summary Table
The ITK Statistics Framework has a collection of data containers many of which have Iterators associated with them.
This page presents a review of issues that have been identified with some iterators in the Statistics Framework and describes proposed actions intended to fix those issues.
List of Issues
- Decrement operator was not incremented correctly in itkListSample.h (Fixed)
- Testing coverage of iterators is not complete, allowing problems like the one above to go unnoticed.
- Functionality was not available to create (or assign) a const iterator from a non-const iterator in itkSubsample.h (Fixed). This problem was found in a number of classes (Not yet fixed).
- In general a consistent set of iterator operations is not present.
Proposed Solutions
- Propose a set of core iterator functionality for sample containers.
- Update containers to include this core functionality where necessary.
- Write tests that cover all iterator operations.
- Fix iterator implementation bugs as uncovered by tests.
