[Insight-developers] IOCommon.h errors

[Miller, James V (Research)]

I might have jumped the gun but I tried to use the kwsys 
utilities yesterday but had compiler errors for a "missing" include file.

I tried to do a 

#include <itkkwsys/SystemTools.hxx>

inside a file in Testing/Code/Common but the compiler kept saying that file
could not be found.  I looked at the CMakeLists.txt files and it looks as
though the directory for kwsys gets created in the build tree as itkkwsys.
Digging through the CMakeCache, however, it looked like there were reference
to kwsys and not itkkwsys for libraries, etc.  So I figured you weren't
quite done with the configuration.


> -----Original Message-----
> From: Bill Hoffman [mailto:bill.hoffman@kitware.com]
> Sent: Thursday, April 10, 2003 8:56 AM
> To: norman-k-williams@uiowa.edu
> Cc: insight Developers List
> Subject: [Insight-developers] IOCommon.h errors
> 
> 
> As per the last phone conference we (Brad and I) created 
> itkkwsys, and this
> should be used instead of IOCommon.h.   I suppose we may want 
> to remove
> IOCommon.h at this point.  Or, perhaps we should create a sub-class
> of the itkkwsys/SystemTools class in Code/Common.
> 
> The functions added to IOCommon.h were both in SystemTools.hxx: 
> 
> 
> #include <itkkwsys/SystemTools.hxx>
> 
>   /**
>    * Make a new directory if it is not there.  This function
>    * can make a full path even if none of the directories existed
>    * prior to calling this function.
>    */
>   static bool MakeDirectory(const char* path); 
>   ///! return true if the file is a directory.
>   static bool FileIsDirectory(const char* name);
> 
> 
> Although the new functions in IOCommon.h were copied from 
> cmake, many of the
> supporting ifdef's from the .cxx file were not copied. 
> Also, it is not a good idea to put system specific
> function calls in .h files.    These should be implementation details
> that are implemented in the .cxx file.   This avoids system 
> header files
> being included in the files that use the class.   
> 
> 
> -Bill
> 
> 
> 
> _______________________________________________
> Insight-developers mailing list
> Insight-developers@public.kitware.com
> http://public.kitware.com/mailman/listinfo/insight-developers
>