ref: c2f5b177c6ada717746917581b5d3d98e1ab91a0
parent: d7511b681b42b4367cb5818a5d949010c5910980
author: Quentin Rameau <[email protected]>
date: Fri Dec 28 19:23:45 EST 2018
[libc] Put strto* character to number convertion in _dtoi function
--- a/src/libc/libc.h
+++ b/src/libc/libc.h
@@ -34,6 +34,7 @@
extern int _daysyear(int year);
extern int _newyear(int year);
extern void *_getheap(void);
+extern int _dtoi(char);
#ifdef FILE
extern int _flsbuf(FILE *fp);
extern void _allocbuf(FILE *fp);
--- a/src/libc/stdlib/Makefile
+++ b/src/libc/stdlib/Makefile
@@ -3,7 +3,8 @@
include $(PROJECTDIR)/scripts/rules.mk
include ../rules.mk
-OBJS = abort.o\
+OBJS = _dtoi.o\
+ abort.o\
abs.o\
atexit.o\
atoi.o\
--- /dev/null
+++ b/src/libc/stdlib/_dtoi.c
@@ -1,0 +1,15 @@
+#include <ctype.h>
+#include <limits.h>
+#include <string.h>
+
+int
+_dtoi(char c)
+{
+ static const char digits[] = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ";
+ char *p;
+
+ if (p = strchr(digits, toupper(c)))
+ return p - digits;
+
+ return INT_MAX;
+}
--- a/src/libc/stdlib/strtol.c
+++ b/src/libc/stdlib/strtol.c
@@ -4,6 +4,8 @@
#include <stdlib.h>
#include <string.h>
+#include "../libc.h"
+
#undef strtol
long
@@ -11,8 +13,7 @@
{
int d, sign = -1;
long n;
- static const char digits[] = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ";
- const char *t, *p;
+ const char *t;
if (end)
*end = s;
@@ -38,9 +39,7 @@
n = 0;
/* Compute n as a negative number to avoid overflow on LONG_MIN */
- for (t = s; p = strchr(digits, toupper(*t)); ++t) {
- if ((d = p - digits) >= base)
- break;
+ for (t = s; (d = _dtoi(*t)) < base; ++t) {
if (n < LONG_MIN/base)
goto overflow;
n *= base;
--- a/src/libc/stdlib/strtoll.c
+++ b/src/libc/stdlib/strtoll.c
@@ -4,6 +4,8 @@
#include <stdlib.h>
#include <string.h>
+#include "../libc.h"
+
#undef strtoll
long long
@@ -11,8 +13,7 @@
{
int d, sign = -1;
long long n;
- static const char digits[] = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ";
- const char *t, *p;
+ const char *t;
if (end)
*end = s;
@@ -38,9 +39,7 @@
n = 0;
/* Compute n as a negative number to avoid overflow on LLONG_MIN */
- for (t = s; p = strchr(digits, toupper(*t)); ++t) {
- if ((d = p - digits) >= base)
- break;
+ for (t = s; (d = _dtoi(*t)) < base; ++t) {
if (n < LLONG_MIN/base)
goto overflow;
n *= base;
--- a/src/libc/stdlib/strtoul.c
+++ b/src/libc/stdlib/strtoul.c
@@ -4,6 +4,8 @@
#include <stdlib.h>
#include <string.h>
+#include "../libc.h"
+
#undef strtoul
unsigned long
@@ -11,8 +13,7 @@
{
int d, sign = 1;
unsigned long n;
- static const char digits[] = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ";
- const char *t, *p;
+ const char *t;
if (end)
*end = s;
@@ -37,9 +38,7 @@
s += 2;
n = 0;
- for (t = s; p = strchr(digits, toupper(*t)); ++t) {
- if ((d = p - digits) >= base)
- break;
+ for (t = s; (d = _dtoi(*t)) < base; ++t) {
if (n > ULONG_MAX/base)
goto overflow;
n *= base;
--- a/src/libc/stdlib/strtoull.c
+++ b/src/libc/stdlib/strtoull.c
@@ -4,6 +4,8 @@
#include <stdlib.h>
#include <string.h>
+#include "../libc.h"
+
#undef strtoull
unsigned long long
@@ -11,8 +13,7 @@
{
int d, sign = 1;
unsigned long long n;
- static const char digits[] = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ";
- const char *t, *p;
+ const char *t;
if (end)
*end = s;
@@ -37,9 +38,7 @@
s += 2;
n = 0;
- for (t = s; p = strchr(digits, toupper(*t)); ++t) {
- if ((d = p - digits) >= base)
- break;
+ for (t = s; (d = _dtoi(*t)) < base; ++t) {
if (n > ULLONG_MAX/base)
goto overflow;
n *= base;