From bfb1bee6feba9f06452a0e4b572ec4d15df325cf Mon Sep 17 00:00:00 2001
From: Jeff Epler <jepler@gmail.com>
Date: Sun, 26 Jan 2025 20:54:14 -0600
Subject: [PATCH] py/parsenumbase: Favor clarity of code over manual
 optimisation.

Follow up to 13b13d1fdd05549d504eeded0b5aa8871d5e5dcf, based on some
testing on godbolt, the manual code optimisation seems unnecessary for code
size, at least on gcc x86_64 and ARM, and it's definitely not good for
clarity.

Signed-off-by: Jeff Epler <jepler@gmail.com>
---
 py/parsenumbase.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/py/parsenumbase.c b/py/parsenumbase.c
index cc3275c45..fbf07a119 100644
--- a/py/parsenumbase.c
+++ b/py/parsenumbase.c
@@ -41,11 +41,11 @@ size_t mp_parse_num_base(const char *str, size_t len, int *base) {
     if (c == '0') {
         c = *(p++) | 32;
         int b = *base;
-        if (c == 'x' && !(b & ~16)) {
+        if (c == 'x' && (b == 0 || b == 16)) {
             *base = 16;
-        } else if (c == 'o' && !(b & ~8)) {
+        } else if (c == 'o' && (b == 0 || b == 8)) {
             *base = 8;
-        } else if (c == 'b' && !(b & ~2)) {
+        } else if (c == 'b' && (b == 0 || b == 2)) {
             *base = 2;
         } else {
             p -= 2;