Merge pull request #380 from benaadams/sanitise-loops

Sanitise loops
This commit is contained in:
David Fowler 2015-10-02 15:43:57 -07:00
commit fb79c6c8da
4 changed files with 6 additions and 5 deletions

View File

@ -27,7 +27,7 @@ namespace Microsoft.AspNet.Hosting.Startup
var serviceProvider = builder.ApplicationServices;
var parameterInfos = MethodInfo.GetParameters();
var parameters = new object[parameterInfos.Length];
for (var index = 0; index != parameterInfos.Length; ++index)
for (var index = 0; index < parameterInfos.Length; index++)
{
var parameterInfo = parameterInfos[index];
if (parameterInfo.ParameterType == typeof(IApplicationBuilder))
@ -54,4 +54,4 @@ namespace Microsoft.AspNet.Hosting.Startup
MethodInfo.Invoke(instance, parameters);
}
}
}
}

View File

@ -509,9 +509,10 @@ namespace Microsoft.AspNet.Hosting.Startup
private static IEnumerable<Exception> FlattenAndReverseExceptionTree(Exception ex)
{
var list = new List<Exception>();
for (; ex != null; ex = ex.InnerException)
while (ex != null)
{
list.Add(ex);
ex = ex.InnerException;
}
list.Reverse();
return list;

View File

@ -25,7 +25,7 @@ namespace Microsoft.AspNet.Server.Testing
CancellationToken cancellationToken = default(CancellationToken),
int retryCount = 60)
{
for (int retry = 0; retry < retryCount; retry++)
for (var retry = 0; retry < retryCount; retry++)
{
if (cancellationToken.IsCancellationRequested)
{

View File

@ -235,7 +235,7 @@ namespace Microsoft.AspNet.TestHost
// Assert
var buffer = new byte[1];
for (int i = 0; i < hello.Length; i++)
for (var i = 0; i < hello.Length; i++)
{
bool last = i == (hello.Length - 1);
var result = await clientSocket.ReceiveAsync(new System.ArraySegment<byte>(buffer), CancellationToken.None);