[Insight-users] Does itkSmartPointer.h need to do #include "itkMacro.h"?

Niels Dekker niels-xtk at xs4all.nl
Sat May 17 06:53:49 EDT 2008


Luis Ibanez wrote:
> It seems that we don't even need the ITK_EXPORT here,
> given that the SmartPointer is a template and it is
> always fully instantiated from the .h file. (e.g.
> there is no implementation that goes into a .cxx file
> and that need to be exported if we build a shared
> library).
>
> That is, we can remove both the ITK_EXPORT and the
> #include "itkMacro.h". There may not be a need for
> #include "itkWin32Header.h" either.
>
> I'm running an experimental build with this change,
> and if it goes well, I'll commit the change.


Cool!  I just saw your revision, and it looks fine to me:
www.itk.org/cgi-bin/viewcvs.cgi/Code/Common/itkSmartPointer.h?revision=1.39&root=Insight&view=markup

My first entrance into the long CVS history log of ITK.  :-)  Thank you, 
Luis!

Kind regards, Niels



More information about the Insight-users mailing list