Page MenuHomePhabricator

Should QmitkAdaptiveRegionGrowingToolGUI AccessPixel() method return TPixel instead of int as value type?
Open, LowPublic

Description

Current signature is

template <typename TPixel>
static void AccessPixel(mitk::PixelType /*ptype*/, mitk::Image* im, mitk::Point3D p, int& val)
template <typename TPixel>
static void AccessPixel(mitk::PixelType /*ptype*/, const mitk::Image* im, mitk::Point3D p, int& val)

Question posed: Shouldn't be val of type TPixel&?
If we touch this part we should also rethink if this access method is either needed at all or if it shouldn't be in a more general place.

Event Timeline

floca triaged this task as Low priority.May 4 2020, 2:17 PM
floca created this task.