From 6a382e1fd6fbf78c10b72f221cbefca9dee599e4 Mon Sep 17 00:00:00 2001 From: "simon.cropp" Date: Mon, 2 May 2011 19:55:27 +1000 Subject: [PATCH] change a variable scope to be close to usage use debug assert instead of ArgNullExceptions because this is an internal constructor so no need to guard to that level --- Dapper/SqlMapper.cs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/Dapper/SqlMapper.cs b/Dapper/SqlMapper.cs index 2dc85cc..47fc9b6 100644 --- a/Dapper/SqlMapper.cs +++ b/Dapper/SqlMapper.cs @@ -8,6 +8,7 @@ using System.Collections.Concurrent; using System.Collections.Generic; using System.Data; +using System.Diagnostics; using System.Dynamic; using System.Linq; using System.Reflection; @@ -262,7 +263,6 @@ class DontMap {} { if (info.Deserializer == null) { - var split = 0; int current = 0; Func nextSplit = () => @@ -282,7 +282,7 @@ class DontMap {} var otherDeserializer = new List(); - split = nextSplit(); + int split = nextSplit(); info.Deserializer = GetDeserializer(identity, reader, 0, split); if (typeof(U) != typeof(DontMap)) @@ -767,10 +767,10 @@ public class GridReader : IDisposable private readonly string sql; internal GridReader(IDbCommand command, IDataReader reader, IDbConnection connection, string sql) { - if (reader == null) throw new ArgumentNullException("reader"); - if (connection == null) throw new ArgumentNullException("connection"); - if (sql == null) throw new ArgumentNullException("sql"); - if (command == null) throw new ArgumentNullException("command"); + Debug.Assert(reader != null); + Debug.Assert(connection != null); + Debug.Assert(sql != null); + Debug.Assert(command != null); this.sql = sql; this.command = command; -- GitLab