[cfe-commits] r56096 - in /cfe/trunk: lib/Sema/SemaStmt.cpp test/SemaCXX/condition.cpp

Mike Stump mrs at apple.com
Thu Sep 11 14:05:10 PDT 2008

On Sep 11, 2008, at 12:09 PM, Argiris Kirtzidis wrote:
> CheckCXXBooleanCondition follows the convention of other check  
> functions
> (CheckInitializerTypes, CheckSingleInitializer, CheckCastTypes, etc)

CheckFoo implies that the routine just Check for the condition Foo (to  
me).  Maybe we should pick a new naming scheme for these types of  
routines.  ConvertFoo at least implies an action.  ConvertAndCheck,  
though, that is a little wordy.

More information about the cfe-commits mailing list