[Insight-developers] Proposal to change file naming conventions

[Bradley Lowekamp]

On Apr 20, 2011, at 5:08 PM, Brad King wrote:

> On 04/20/2011 04:35 PM, Johnson, Hans J wrote:
>> I would contend that ".txx" is so not anything but ITK.
> 
> http://www.ehow.com/facts_5656990_file-extension-txx_.html
> 
> ;)
> 
> On 04/20/2011 04:53 PM, Johnson, Hans J wrote:
>> 2) move to .hpp (because some do not like having the -impl.h)
> 
> 2b) move to .hxx, one letter change and consistent with use of .cxx
>    for non-template implementation files.  It is known to emacs,
>    vim, Xcode, VS, etc.
> 
> If we are going to do this then the massive rename commit should do
> the renames and modify the files mentioning .txx in their content
> at the same time.  In order to keep history connected easily we need
> to avoid renaming and modifying a file at the same time.  Therefore
> no .txx file should mention itself or other .txx files at the time
> of the rename.  This command shows the lines that we will need to
> fix ahead of time to avoid mentioning .txx if possible:
> 
> $ git grep '\.txx' -- '*.txx' |grep -v ThirdParty
> 
> As of commit fc6041b6 there are only 19 of them.

Additionally add txx file have the preprocessor guard:

__itkFileName_txx

So every txx file will need to be modified as well.

> 
> -Brad K
> _______________________________________________
> 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

========================================================
Bradley Lowekamp  
Lockheed Martin Contractor for
Office of High Performance Computing and Communications
National Library of Medicine 
blowekamp at mail.nih.gov


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110420/bbb06776/attachment.htm>