[Insight-developers] Style Header for ITKv4

Hans Johnson hans-johnson at uiowa.edu
Tue Aug 10 13:26:36 EDT 2010


Brad,

I can give you a script that shows exactly what uncrustify will do.  There
are options in uncrustify to define how to handle this case.

I'll make some test cases this afternoon.


Hans



On 8/10/10 9:09 AM, "Brad Davis" <brad.davis at kitware.com> wrote:

> Here is a progress report:
> 
> The old indentation check contained some heuristics that made it fail
> in certain places.  The new check does not use these heuristics (but
> it is new and has some bugs to work out).  I have also successfully
> run it in minutes on large files that took hours to run using the old
> checker.  I am actually running it on ITK code now to evaluate it (but
> I'm not making changes to the ITK code, yet).  I am down to a few
> corner cases like the following one that require some sort of group
> decision.  I will elaborate on the others as they come up.
> 
> It is ITK custom to align typedefs.  In places we have:
> 
> typedef   A    B;
> typedef   C    D;
> typedef   EEEEEEEEE
>                       F;
> 
> If this is a mess in your email then the idea is that "F" is on a new
> line but it is aligned with "B" and "D" rather than by two spaces.
> 
> The new indentation checker flags this as an error.  Is this quirky
> alignment really what we want or should it be a normal indent for a
> continued line like this?
> 
> typedef   A    B
> typedef   C    D
> typedef   EEEEEEEEE
>    F
> 
> We could say both alignments are acceptable.
> 
> Hans,
> 
> What does uncrustify do in this case?
> 
> Best,
> Brad
> 
> On Thu, Aug 5, 2010 at 10:32 AM, Hans Johnson <hans-johnson at uiowa.edu> wrote:
>> Brad,
>> 
>> I am in contact with Brad Davis, and he is updating the KWStyle tool to
>> support a more consistent style (and less ambiguous) style.
>> 
>> We are hoping that the necessary changes can be completed by next week.  The
>> uncrustify solution is not possible until KWStyle can properly report on
>> indentation in a consistent manner.  Once this is done,  an uncrustify
>> procedure will be available to automatically produce KWStyle compliant code
>> formatting with out user intervention.
>> 
>> Regards,
>> 
>> Hans
>> 
>> 
>> 
>> On 8/5/10 9:21 AM, "Bradley Lowekamp" <blowekamp at mail.nih.gov> wrote:
>> 
>> It looks like KWStyle is failing for all files now due a related header
>> issue:
>> 
>>  Processing
>> /nfs/mead/Users/blowek1/src/ITK/Code/Review/itkLabelContourImageFilter.txx
>> 1: Error #13 (0) Header mismatch: itkLabelContourImageFilter.txx
>> (ITKHeader.h) :
>> /nfs/mead/Users/blowek1/src/ITK/Utilities/KWStyle/ITKHeader.h
>> 
>> I am not familiar enough with KWStyle to know if there is a quick fix for
>> this with the current... (actually I don't know if I am running the current
>> as I can't seem to get the version for the executable )  version.
>> 
>> Just trying to prevent unstyled code from getting in...
>> 
>> I suppose with Hans' plan to just uncrustify the code it should not be a
>> significant issue if it does.
>> 
>> Brad
>> 
>> On Jul 29, 2010, at 1:28 PM, Karthik Krishnan wrote:
>> 
>> Yup.  VTK has done away with those keyword substitutions. Its header looks
>> like this...
>> 
>> /*=========================================================================
>> 
>>   Program:   Visualization Toolkit
>>   Module:    vtkObject.h
>> 
>>   Copyright (c) Ken Martin, Will Schroeder, Bill Lorensen
>>   All rights reserved.
>>   See Copyright.txt or http://www.kitware.com/Copyright.htm for details.
>> 
>>      This software is distributed WITHOUT ANY WARRANTY; without even
>>      the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR
>>      PURPOSE.  See the above copyright notice for more information.
>> 
>> =========================================================================*/
>> 
>> I am guessing ITK will follow suit..
>> 
>> --
>> karthik
>> 
>> 
>> On Thu, Jul 29, 2010 at 8:55 PM, Hans Johnson <hans-johnson at uiowa.edu>
>> wrote:
>> 
>> I would vote to remove those.  They seem redundant with the information that
>> is available from git, and they make using external diff tools tedious to
>> work with.
>> 
>> Hans
>> 
>> On 7/29/10 10:11 AM, "Mathieu Malaterre" <mathieu.malaterre at gmail.com>
>> wrote:
>> 
>>> Hi,
>>> 
>>>   Quick question: are we keeping the same header after moving to git ?
>>> 
>>>   Module:    $RCSfile: itkImageIOBase.h,v $
>>>   Date:      $Date: 2009-11-29 15:51:11 $
>>>   Version:   $Revision: 1.56 $
>>> 
>>> Does those keywords still apply to git world ?
>>> 
>>> Thanks,
>> 
>> --
>> Hans J. Johnson, Ph.D.
>> Assistant Professor
>> 200 Hawkins Drive
>> T205 BT, The University of Iowa
>> Iowa City, IA 52242
>> 
>> hans-johnson at uiowa.edu
>> PHONE: 319 353 8587
>> 
>> _______________________________________________
>> Powered by www.kitware.com <http://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
>> 
>> <ATT00001..txt>
>> 
>> ========================================================
>> 
>> Bradley Lowekamp
>> 
>> Lockheed Martin Contractor for
>> 
>> Office of High Performance Computing and Communications
>> 
>> National Library of Medicine
>> 
>> blowekamp at mail.nih.gov
>> 
>> 
>> 
>> ________________________________
>> 
>> _______________________________________________
>> 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
>> 
>> _______________________________________________
>> 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
>> 
>> 
> 
> 

-- 
Hans J. Johnson, Ph.D.
Assistant Professor
200 Hawkins Drive
T205 BT, The University of Iowa
Iowa City, IA 52242

hans-johnson at uiowa.edu
PHONE: 319 353 8587



More information about the Insight-developers mailing list