From 8769049e935a89daa51883755457a083a5589d4e Mon Sep 17 00:00:00 2001
From: Damien George <damien.p.george@gmail.com>
Date: Tue, 20 Feb 2018 19:19:02 +1100
Subject: [PATCH] py/objstr: Remove unnecessary check for positive splits
variable.
At this point in the code the variable "splits" is guaranteed to be
positive due to the check for "splits == 0" above it.
---
py/objstr.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/py/objstr.c b/py/objstr.c
index 13d957105..c42f38e75 100644
--- a/py/objstr.c
+++ b/py/objstr.c
@@ -663,9 +663,7 @@ STATIC mp_obj_t str_rsplit(size_t n_args, const mp_obj_t *args) {
}
res->items[idx--] = mp_obj_new_str_of_type(self_type, s + sep_len, last - s - sep_len);
last = s;
- if (splits > 0) {
- splits--;
- }
+ splits--;
}
if (idx != 0) {
// We split less parts than split limit, now go cleanup surplus
--
GitLab