X-Git-Url: https://pd.if.org/git/?a=blobdiff_plain;f=functions%2Fstdlib%2Fstrtol.c;h=05271e05054e8cc877b79eb94326c1a42a900987;hb=8fca29703edff11c81af1e6039bcf115a83b6bbc;hp=8fc95b0fce7c66d842bcb8d325e371f211f2cac9;hpb=0d54a75af25ca44411e7c4190cc2a93a390e61a2;p=pdclib.old

diff --git a/functions/stdlib/strtol.c b/functions/stdlib/strtol.c
index 8fc95b0..05271e0 100644
--- a/functions/stdlib/strtol.c
+++ b/functions/stdlib/strtol.c
@@ -37,11 +37,6 @@ long int strtol( const char * s, char ** endptr, int base )
 #ifdef TEST
 #include <_PDCLIB_test.h>
 
-#ifndef _PDCLIB_INT_H
-#define _PDCLIB_INT_H
-#include <_PDCLIB_int.h>
-#endif
-
 #include <errno.h>
 
 int main( void )
@@ -49,19 +44,27 @@ int main( void )
     char * endptr;
     /* this, to base 36, overflows even a 256 bit integer */
     char overflow[] = "-ZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZ_";
+    /* tricky border case */
+    char tricky[] = "+0xz";
     errno = 0;
     /* basic functionality */
     TESTCASE( strtol( "123", NULL, 10 ) == 123 );
     /* proper detecting of default base 10 */
-    TESTCASE( strtol( "123", NULL, 0 ) == 123 );
+    TESTCASE( strtol( "456", NULL, 0 ) == 456 );
     /* proper functioning to smaller base */
     TESTCASE( strtol( "14", NULL, 8 ) == 12 );
     /* proper autodetecting of octal */
-    TESTCASE( strtol( "014", NULL, 0 ) == 12 );
+    TESTCASE( strtol( "016", NULL, 0 ) == 14 );
     /* proper autodetecting of hexadecimal, lowercase 'x' */
     TESTCASE( strtol( "0xFF", NULL, 0 ) == 255 );
     /* proper autodetecting of hexadecimal, uppercase 'X' */
-    TESTCASE( strtol( "0XFF", NULL, 0 ) == 255 );
+    TESTCASE( strtol( "0Xa1", NULL, 0 ) == 161 );
+    /* proper handling of border case: 0x followed by non-hexdigit */
+    TESTCASE( strtol( tricky, &endptr, 0 ) == 0 );
+    TESTCASE( endptr == tricky + 2 );
+    /* proper handling of border case: 0 followed by non-octdigit */
+    TESTCASE( strtol( tricky, &endptr, 8 ) == 0 );
+    TESTCASE( endptr == tricky + 2 );
     /* errno should still be 0 */
     TESTCASE( errno == 0 );
     /* overflowing subject sequence must still return proper endptr */
@@ -74,7 +77,7 @@ int main( void )
     TESTCASE( errno == ERANGE );
     TESTCASE( ( endptr - overflow ) == 53 );
     /* testing skipping of leading whitespace */
-    TESTCASE( strtol( " \n\v\t\f123", NULL, 0 ) == 123 );
+    TESTCASE( strtol( " \n\v\t\f789", NULL, 0 ) == 789 );
     /* testing conversion failure */
     TESTCASE( strtol( overflow, &endptr, 10 ) == 0 );
     TESTCASE( endptr == overflow );
@@ -85,7 +88,7 @@ int main( void )
     /* one-complement and signed magnitude just as well. Anyone having a   */
     /* platform to test this on?                                           */
     errno = 0;
-#if _PDCLIB_LONG_BYTES == 4
+#if LONG_MAX == 0x7fffffffL
     /* testing "even" overflow, i.e. base is power of two */
     TESTCASE( strtol( "0x7FFFFFFF", NULL, 0 ) == 0x7fffffff );
     TESTCASE( errno == 0 );
@@ -99,7 +102,7 @@ int main( void )
     TESTCASE( strtol( "-0x80000001", NULL, 0 ) == LONG_MIN );
     TESTCASE( errno == ERANGE );
     /* TODO: test "odd" overflow, i.e. base is not power of two */
-#elif _PDCLIB_LONG_BYTES == 8
+#elif LONG_MAX == 0x7fffffffffffffffL
     /* testing "even" overflow, i.e. base is power of two */
     TESTCASE( strtol( "0x7FFFFFFFFFFFFFFF", NULL, 0 ) == 0x7fffffffffffffff );
     TESTCASE( errno == 0 );