I think itkCheck might be better.

On Wed, Nov 18, 2015 at 4:26 PM, Matt McCormick
<[email protected]> wrote:
> Hi Seun,
>
> Thanks for discussing the issue.
>
> I think a good name would be "Check".  Could you please submit a patch? [1]
>
> Thanks,
> Matt
>
> [1] http://www.itk.org/Wiki/ITK/Git/Develop
>
>
> On Wed, Nov 18, 2015 at 4:08 PM, Seun Odutola <[email protected]> wrote:
>> Hi Everyone,
>>
>>     Just wanted to bring to your attention a problem I faced working on a 
>> project using ITK 4.9 on Mac OS X 10.11, Apple declares a macro called 
>> 'check’  that takes just a parameter and apparently there is a name clash 
>> with a typedef of ITK’s (see itkPromoteType.h).
>>
>> // snippet
>> template <typename TA, typename TB> struct PromoteType
>> {
>>   static TA a;
>>   static TB b;
>>
>>   // Aimed at supporting overloads
>>   template <typename T> static Details::Identity<1>::Type& check(typename 
>> Details::SizeToType<1,  TA, TB>::Type, T);
>>   template <typename T> static Details::Identity<2>::Type& check(typename 
>> Details::SizeToType<2,  TA, TB>::Type, T);
>>
>>   // Common numeric types
>>   static Details::Identity<3 >::Type& itkcheck(typename 
>> Details::SizeToType<3,  TA, TB>::Type, int);
>>
>> // end of snippet
>>
>>  I set about trying to fix this issue and on renaming the ‘check’ in my case 
>> I chose ‘itkcheck’, rebuilt ITK and ran my project everything worked fine.
>>
>> My proposal to the team if indeed I’m right is the above typedef need 
>> renaming as we can’t change Apple’s implementation so it’s most likely ITK 
>> will need to fix this. Currently renaming the check typedef works fine.
>> Thanks
>>
>> Regards Seun
>>
>> p.s: I could submit a patch if that’s fine but I would like to know what 
>> name the ITK team would prefer to be substituted for the ‘check' typedef 
>> above.
>> _______________________________________________
>> 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.php
>>
>> 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://public.kitware.com/mailman/listinfo/insight-developers
> _______________________________________________
> 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.php
>
> 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://public.kitware.com/mailman/listinfo/insight-developers



-- 
Unpaid intern in BillsBasement at noware dot com
_______________________________________________
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.php

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://public.kitware.com/mailman/listinfo/insight-developers

Reply via email to