[Insight-developers] itkConceptChecking.h

[Stephen Aylward]

Hi,

I believe there is a bug (or at least an obscurity) in the concept
checking code for BracketOperator.   See the line below marked HERE

template <typename T1, typename T2=T1, typename T3=T1>
struct BracketOperator
{
  struct Constraints
    {
    void constraints()
      {
      a = static_cast<T3>(b [ c ]);
      const_constraints(b, c);
      }
    void const_constraints(const T1& d, const T2& e)
      {
      a = static_cast<T3>(b [ c ]);
         <<<<< HERE
      }
    T3 a;
    T1 b;
    T2 c;
    };

I've put in lines to avoid an unused variable warning for the two args
to that function, but it seems as though the unused vars really should
be used.    Perhaps there are missing lines:
b = d
c = e
?

Experts?

s

PS> Discovered this while fixing warnings....<insert Luis' and Bill's
speeches about importance of warnings here> :)

-- 
Stephen R. Aylward, Ph.D.
Chief Medical Scientist
Kitware, Inc. - North Carolina Office
http://www.kitware.com
(518) 371-3971 x300