[Insight-developers] Why is m_NumberOfThreads signed?

[Luis Ibanez]

Hi David,

After a quick "git grep"
I don't see any reason why m_NumberOfThreads has to be "int".

I agree with you in that "unsigned int" makes more sense.

Would you like to give it a try at producing a patch ?


   Thanks


       Luis


---------------------
On Thu, Apr 14, 2011 at 4:34 PM, David Doria <daviddoria at gmail.com> wrote:
> In itkProcessObject, m_NumberOfThreads is declared as an int. Is there a
> reason it is not unsigned int?
> Simple operations like this:
> for ( unsigned int t = 0; t < this->GetNumberOfThreads(); ++t )
> currently require casting to prevent signed/unsigned comparison warnings.
> Thoughts?
>
> David
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/protraining.html
>
> Please keep messages on-topic and check the ITK FAQ at:
> http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers
>
>