[Insight-developers] Change (perhaps) needed for itk::NumericTraits to support WrapITK

[Kent Williams]

Gaetan Lehmann et al are working on a new wrapping Framework for ITK, 
and I noticed one problem in my efforts to get the TCL support fully 
working.  WrapITK optionally supports the itk::RGBPixel<unsigned short> 
data type, and their wrapping generates a reference to 
itk::NumericTraits<RGBPixel<unsigned short> >::One, which doesn't get 
defined.  To get WrapITK's version of Wish to link, I have to add that 
type (patch below).

My question: is there a reason not to check in this change? I'm a little 
confused about what the case is here, either:
A) ITK doesn't and won't support itk::RGBPixel<unsigned short>
or
B) Any scalar type works with itk::RGBPixel<>, and the NumericTraits is 
beside the point, but WrapITK (incorrectly?) demands 
itk::NumericTraits<RGBPixel<unsigned short> >

If WrapITK will be replacing the current wrapping code, I would 
certainly welcome someone besides me looking at how well it works with 
TCL.