[libc-commits] [PATCH] D112176: [libc] fix strtol returning the wrong length
Siva Chandra via Phabricator via libc-commits
libc-commits at lists.llvm.org
Wed Oct 20 13:53:08 PDT 2021
sivachandra added inline comments.
================
Comment at: libc/src/__support/str_conv_utils.h:65
unsigned long long result = 0;
+ bool seen_digit = false;
+ const char *original_src = src;
----------------
Nit: s/`seen_digit`/`is_number`?
================
Comment at: libc/src/__support/str_conv_utils.h:83
+ seen_digit = true;
+ }
+
----------------
Do we need this `if` block at all?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112176/new/
https://reviews.llvm.org/D112176
More information about the libc-commits
mailing list