r194206 - Remove an unnecessary condition that I added hastily: Unsigned numbers are obviously >= 0 ; )

Faisal Vali faisalv at yahoo.com
Thu Nov 7 08:57:57 PST 2013


Author: faisalv
Date: Thu Nov  7 10:57:56 2013
New Revision: 194206

URL: http://llvm.org/viewvc/llvm-project?rev=194206&view=rev
Log:
Remove an unnecessary condition that I added hastily: Unsigned numbers are obviously >= 0 ;)

Also - others have complained about some white space issues - sorry about that - continues to be a pain point for me - will try and see what I can do with clang-format this evening after work - as a short term fix, if anyone can email me the files that they have already identified with issues, it would help me speed up a focused fix. sorry.



Modified:
    cfe/trunk/lib/Sema/ScopeInfo.cpp

Modified: cfe/trunk/lib/Sema/ScopeInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/ScopeInfo.cpp?rev=194206&r1=194205&r2=194206&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/ScopeInfo.cpp (original)
+++ cfe/trunk/lib/Sema/ScopeInfo.cpp Thu Nov  7 10:57:56 2013
@@ -185,7 +185,7 @@ void FunctionScopeInfo::markSafeWeakUse(
 }
 
 void LambdaScopeInfo::getPotentialVariableCapture(unsigned Idx, VarDecl *&VD, Expr *&E) {
-  assert((Idx >= 0 && Idx < getNumPotentialVariableCaptures()) &&
+  assert(Idx < getNumPotentialVariableCaptures() &&
     "Index of potential capture must be within 0 to less than the "
     "number of captures!");
   E = PotentiallyCapturingExprs[Idx];





More information about the cfe-commits mailing list