[ITK] [ITK-dev] Header clashes with Apple

[Bill Lorensen]

I think itkCheck might be better.

On Wed, Nov 18, 2015 at 4:26 PM, Matt McCormick
<matt.mccormick at kitware.com> 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 <seun at rogue-research.com> 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