Fix feature dependency

.github/workflows/build.yml

@@ -103,7 +103,7 @@ jobs:
       - uses: actions-rs/toolchain@v1
         with:
           profile: minimal
-          toolchain: 1.64.0 # clippy is too much of a moving target at the moment
+          toolchain: 1.75.0 # clippy is too much of a moving target at the moment
           override: true
           components: clippy
       - uses: actions-rs/clippy-check@v1

Cargo.toml

@@ -24,11 +24,22 @@ all = [
     "strlen",
     "strtol",
     "strtoul",
+    "strtoll",
+    "strtoull",
+    "strtoimax",
+    "strtoumax",
+    "strtoul",
     "strstr",
     "strchr",
     "atoi",
+    "utoa",
     "itoa",
     "snprintf",
+    "errno",
+    "isspace",
+    "isdigit",
+    "isalpha",
+    "isupper",
 ]
 abs = []
 strcmp = []
@@ -38,8 +49,18 @@ strncpy = []
 strlen = []
 strtol = []
 strtoul = []
+strtoll = []
+strtoull = []
+strtoimax = []
+strtoumax = []
 strstr = []
 strchr = []
 atoi = []
+utoa = []
 itoa = []
 snprintf = []
+errno = []
+isspace = []
+isdigit = []
+isalpha = []
+isupper = []

README.md

@@ -11,13 +11,21 @@ This crate basically came about so that the [nrfxlib](https://github.com/NordicP
 * abs
 * strol
 * atoi
+* isspace
+* isdigit
+* isalpha
+* isupper
 * strcmp
 * strncmp
 * strcpy
 * strncpy
 * strlen
 * strtol
+* strtoll
 * strtoul
+* strtoull
+* strtoimax
+* strtoumax
 * strstr
 * strchr
 * snprintf

build.rs

@@ -1,11 +1,30 @@
 fn main() {
 	if cfg!(feature = "snprintf") {
 		// Build our snprintf substitute (which has to be C as Rust doesn't do varargs)
-		cc::Build::new()
+		let mut build = cc::Build::new();
+
+		build
 			.warnings(true)
 			.extra_warnings(true)
 			.flag("-std=c99")
-			.file("./src/snprintf.c")
-			.compile("clocal");
+			.file("./src/snprintf.c");
+
+		#[cfg(not(feature = "itoa"))]
+		{
+			build.define("itoa", "tinyrlibc_itoa");
+		}
+		#[cfg(not(feature = "utoa"))]
+		{
+			build.define("utoa", "tinyrlibc_utoa");
+		}
+		#[cfg(not(feature = "strtoul"))]
+		{
+			build.define("strtoul", "tinyrlibc_strtoul");
+		}
+
+		build.compile("clocal");
 	}
+
+	println!("cargo:rerun-if-changed=build.rs");
+	println!("cargo:rerun-if-changed=src/snprintf.c");
 }

src/abs.rs

@@ -4,13 +4,8 @@
 
 use crate::CInt;
 
-/// Calculates the integer absolute value
-///
-/// ```
-/// use tinyrlibc::abs;
-/// assert_eq!(abs(-2), 2);
-/// ```
-#[no_mangle]
+/// Rust implementation of C library function `abs`
+#[cfg_attr(feature = "abs", no_mangle)]
 pub extern "C" fn abs(i: CInt) -> CInt {
 	i.abs()
 }

src/ctype.rs

@@ -0,0 +1,67 @@
+//! A tiny C library, written in Rust.
+//!
+//! See README.md for more details.
+//!
+//! This file is Copyright (c) Jonathan 'theJPster' Pallant 2019
+//! Licensed under the Blue Oak Model Licence 1.0.0
+//!
+//! See each module for its respective licence.
+
+/// `void`
+pub type CVoid = ::core::ffi::c_void;
+
+/// `long long int`
+pub type CLongLong = ::core::ffi::c_longlong;
+
+/// `unsigned long long int`
+pub type CULongLong = ::core::ffi::c_ulonglong;
+
+/// `intmax_t`
+pub type CUIntMax = CULongLong;
+
+/// `uintmax_t`
+pub type CIntMax = CLongLong;
+
+/// `long int`
+pub type CLong = ::core::ffi::c_long;
+
+/// `unsigned long int`
+pub type CULong = ::core::ffi::c_ulong;
+
+/// `int`
+pub type CInt = ::core::ffi::c_int;
+
+/// `unsigned int`
+pub type CUInt = ::core::ffi::c_uint;
+
+/// Represents an 8-bit `char`. Rust does not (and will never) support
+/// platforms where `char` is not 8-bits long.
+pub type CChar = u8;
+
+/// This allows you to iterate a null-terminated string in a relatively simple
+/// way.
+pub struct CStringIter {
+	ptr: *const CChar,
+	idx: isize,
+}
+
+impl CStringIter {
+	/// Create a new iterator from a pointer to a null-terminated string. The
+	/// behaviour is undefined if the string is not null-terminated.
+	pub fn new(s: *const CChar) -> CStringIter {
+		CStringIter { ptr: s, idx: 0 }
+	}
+}
+
+impl core::iter::Iterator for CStringIter {
+	type Item = CChar;
+	fn next(&mut self) -> Option<Self::Item> {
+		let c = unsafe { *self.ptr.offset(self.idx) };
+		if c == 0 {
+			None
+		} else {
+			self.idx += 1;
+			Some(c)
+		}
+	}
+}

src/itoa.rs

@@ -8,14 +8,15 @@
 
 use crate::CChar;
 
-#[no_mangle]
 /// Formats the given value `i`, with the given radix, into the given buffer (of the given length).
 ///
 /// No prefixes (like 0x or 0b) are generated. Only radix values in the range
 /// 2..=16 are supported.
 ///
 /// Returns the number of bytes written on success (not including the null),
 /// or -1 if the buffer wasn't large enough.
+#[cfg_attr(not(feature = "itoa"), export_name = "tinyrlibc_itoa")]
+#[cfg_attr(feature = "itoa", no_mangle)]
 pub unsafe extern "C" fn itoa(i: i64, s: *mut CChar, s_len: usize, radix: u8) -> i32 {
 	let (is_negative, pos_i) = if i < 0 {
 		(true, (-i) as u64)
@@ -31,14 +32,15 @@ pub unsafe extern "C" fn itoa(i: i64, s: *mut CChar, s_len: usize, radix: u8) ->
 	}
 }
 
-#[no_mangle]
 /// Formats the given value `u`, with the given radix, into the given buffer (of the given length).
 ///
 /// No prefixes (like 0x or 0b) are generated. Only radix values in the range
 /// 2..=16 are supported. Negative numbers are not supported.
 ///
 /// Returns the number of bytes written on success (not including the null),
 /// or -1 if the buffer wasn't large enough.
+#[cfg_attr(not(feature = "utoa"), export_name = "tinyrlibc_utoa")]
+#[cfg_attr(feature = "utoa", no_mangle)]
 pub unsafe extern "C" fn utoa(mut u: u64, s: *mut CChar, s_len: usize, radix: u8) -> i32 {
 	let buffer_slice = core::slice::from_raw_parts_mut(s, s_len);