[Insight-developers] ThreadedCalculateUpdateValue

[Brady McCary]

insight-developers,

The current signature of this function in
ParallelSparseFieldLevelSetImageFilter is:

  inline virtual ValueType ThreadedCalculateUpdateValue(const unsigned
int itkNotUsed(ThreadId),
                                                        const
IndexType itkNotUsed(index),
                                                        const TimeStepType &dt,
                                                        const ValueType &value,
                                                        const ValueType &change)

It probably should be:

  inline virtual ValueType ThreadedCalculateUpdateValue(const unsigned
int itkNotUsed(ThreadId),
                                                        const
IndexType &itkNotUsed(index),
                                                        const TimeStepType &dt,
                                                        const ValueType &value,
                                                        const ValueType &change)

I.e, the second argument should be a reference. I think this is an
omission, considering that there is no reason for it not to be a
reference, and the corresponding signature in
SparseFieldLevelSetImageFilter has index as a reference.

Brady