Skip to content

Commit

Permalink
Remove additional bounds check
Browse files Browse the repository at this point in the history
  • Loading branch information
xtqqczze authored Mar 3, 2023
1 parent ce18ccf commit 36d4416
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ private static bool TryParseNumber(ReadOnlySpan<byte> source, ref Number.NumberB

case Utf8Constants.Plus:
srcIndex++;
if (srcIndex == source.Length)
if (srcIndex >= source.Length)

This comment has been minimized.

Copy link
@stephentoub

stephentoub Mar 3, 2023

Member

"Remove additional bounds check"... does it actually remove the bounds check on L52? I'd be surprised.

{
bytesConsumed = 0;
return false;
Expand Down

0 comments on commit 36d4416

Please sign in to comment.