From f76d712086ca548d8f220d9268280f0de0a32e04 Mon Sep 17 00:00:00 2001 From: Iskander Sharipov Date: Thu, 13 Sep 2018 21:43:37 +0300 Subject: [PATCH] use multi-value case clause instead of fallthrough Found using https://go-critic.github.io/overview#emptyFallthrough-ref --- iter_float.go | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/iter_float.go b/iter_float.go index 4f883c09..3ccd757c 100644 --- a/iter_float.go +++ b/iter_float.go @@ -145,9 +145,7 @@ non_decimal_loop: } // too many decimal places return iter.readFloat32SlowPath() - case invalidCharForNumber: - fallthrough - case dotInNumber: + case invalidCharForNumber, dotInNumber: return iter.readFloat32SlowPath() } decimalPlaces++ @@ -286,9 +284,7 @@ non_decimal_loop: } // too many decimal places return iter.readFloat64SlowPath() - case invalidCharForNumber: - fallthrough - case dotInNumber: + case invalidCharForNumber, dotInNumber: return iter.readFloat64SlowPath() } decimalPlaces++