Changes between Version 6 and Version 7 of StandardizingNames


Ignore:
Timestamp:
04/20/2009 06:24:52 PM (10 years ago)
Author:
rowen
Comment:

Comment added.

Legend:

Unmodified
Added
Removed
Modified
  • StandardizingNames

    v6 v7  
    9292 
    9393 
     94==== Comment by rowen on Mon 20 Apr 2009 01:24:52 PM CDT ==== 
     95Mostly this sounds fine to me. A few specific points, some relevant to the TCT board's findings: 
     96 * Regarding "i" as an iterator name (C++ 3-23): in my opinion it is a very poor choice for an STL-ish iterator that points to something. For that I suggest we adopt names that name the source, e.g. "inImageIter". I believe most existing code already uses a name that is more obvious than "i" and so violates rule 3-23. 
     97 * Dick Shaw's suggestion to use foo_i instead of iFoo is reasonable, but it can be tricky to decide when the underscore is wanted so I am somewhat against it. In a similar vein, I would like a rule for X and Y -- is it a prefix or a suffix? (And if a suffix, do we want X or _x)? 
     98 * Need we use decl instead of dec if it is part of a longer name? I propose not, though it makes one more thing to remember. 
     99 
     100 
    94101[[AddComment]] 
    95102