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.

llvm-svn: 194206

clang/lib/Sema/ScopeInfo.cpp

@@ -185,7 +185,7 @@ void FunctionScopeInfo::markSafeWeakUse(const Expr *E) {
 }
 
 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];