[Insight-developers] removing the dash in the module names

[Gaëtan Lehmann]

Hi Xiaxio,
Hi David,

Le 23 juin 11 à 17:35, David Cole a écrit :

> Why couldn't there be a name-mapping (reversible, 2-way) in just the  
> Wrapping libraries? Are the "real" module names referenced directly  
> in many places other than target_link_libraries calls?

They are reused in many places in wrapping.
Moving to the new modules defined in ITK is not that easy  
unfortunately, and dealing with non compatible module names make it  
even harder.

>
> How many languages can't handle the "-" in names? Is it just python,  
> or are dashes also bad in some other language?
>

At least python and java.
I haven't checked for Tcl.

>
> On Thu, Jun 23, 2011 at 11:15 AM, Xiaoxiao Liu <xiaoxiao.liu at kitware.com 
> > wrote:
> Yup, not only most of the cmake files need some string replacements,  
> all the header files (for doxygen) need to be touched as well.

Yes - nothing some regular expressions can't do.

> It might affect all other under-development external modules (e.g.  
> Lesion sizing toolkit, maybe a couple of other out there).
>

We are doing many changes which are affecting the development of other  
modules.
I think it shouldn't affect a design decision at that point.
In that case, using underscores instead of dashes doesn't make the  
names less good, but it does make the wrapping easier.

> If there is no easy work-around, I prefer Bra'd's  "underscore"  
> naming.

Ok!

Gaëtan



>
> 2011/6/23 Gaëtan Lehmann <gaetan.lehmann at jouy.inra.fr>
>
> Le 23 juin 11 à 15:19, Bill Lorensen a écrit :
>
>
> Virtually every .h file in the toolkit will need editing... Is there
> some other, perhaps wrapping-specific solution?
>
> That would force us to do a lot of conversions with cmake.
> In addition to make the code more complicated, I'm concerned that it  
> may make the already quite slow configuration even slower.
>
> Is it really a big problem to touch all the headers?
>
> Gaëtan
>
>
>
>
>
> On Thu, Jun 23, 2011 at 8:53 AM, Brad King <brad.king at kitware.com>  
> wrote:
> On 06/23/2011 04:07 AM, Gaëtan Lehmann wrote:
> most of the languages won't support a dash in the module name.
>
> Would it be ok to remove those dash in the module names?
>
>   ITK-Common
>   ITK-IO-Base
>
> would become
>
>   ITKCommon
>   ITKIOBase
>
> Perhaps underscores?
>
>  ITK_Common
>  ITK_IO_Base
>
> IMO having a separator breaks with our C++ coding guideline naming
> style enough to clearly distinguish the names as modules.  It also
> makes the name components clear, especially when the module names
> are all uppercase.  Consider
>
>  ITK-IO-PNG   =>  ITKIOPNG  v. ITK_IO_PNG
>  ITK-IO-TIFF  =>  ITKIOTIFF v. ITK_IO_TIFF
>
> -Brad
> _______________________________________________
> 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
>
>
> -- 
> Gaëtan Lehmann
> Biologie du Développement et de la Reproduction
> INRA de Jouy-en-Josas (France)
> tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
> http://mima2.jouy.inra.fr  http://www.itk.org
> http://www.bepo.fr
>
>
> _______________________________________________
> 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
>
>
>
>
> -- 
>
>
> ---------------------------------------------
> Xiaoxiao Liu, Ph.D.
> R & D Engineer
> Kitware Inc.
> Clifton Park, NY
> Phone: (518) 881-4924  or  (518) 371-3971 x124
>
>
>
> _______________________________________________
> 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
>
>

-- 
Gaëtan Lehmann
Biologie du Développement et de la Reproduction
INRA de Jouy-en-Josas (France)
tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
http://mima2.jouy.inra.fr  http://www.itk.org
http://www.bepo.fr

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 203 bytes
Desc: Ceci est une signature ?lectronique PGP
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110623/2925ab8d/attachment.pgp>