Skip to content

Commit 10742f9

Browse files
author
Kapil Borle
committed
Fix range parameter processing
1 parent 03cdb79 commit 10742f9

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

Engine/Commands/InvokeFormatterCommand.cs

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,18 @@ private void SetRange()
145145

146146
var objArr = Range as object[];
147147
int[] intArr;
148-
if (objArr == null)
148+
if (objArr != null)
149+
{
150+
if (!objArr.All(x => x is int))
151+
{
152+
throw new ArgumentException(
153+
"Array should contain integer elements.",
154+
nameof(Range));
155+
}
156+
intArr = new int[objArr.Length];
157+
objArr.CopyTo(intArr, 0);
158+
}
159+
else
149160
{
150161
// todo check passing int[] casted parameter
151162
intArr = Range as int[];
@@ -157,22 +168,13 @@ private void SetRange()
157168
}
158169
}
159170

160-
if (objArr.Length != 4)
171+
if (intArr.Length != 4)
161172
{
162173
throw new ArgumentException(
163174
"Array should be of length 4.",
164175
nameof(Range));
165176
}
166177

167-
if (!objArr.All(x => x is int))
168-
{
169-
throw new ArgumentException(
170-
"Array should contain integer elements.",
171-
nameof(Range));
172-
}
173-
174-
intArr = new int[objArr.Length];
175-
objArr.CopyTo(intArr, 0);
176178
this.range = new Range(intArr[0], intArr[1], intArr[2], intArr[3]);
177179
}
178180

0 commit comments

Comments
 (0)