feat: always have no_std and import std when std feature is activ…

…ated
GrayJack · Jan 19, 2025 · 5ec9191 · 5ec9191
1 parent 0ee2e90
commit 5ec9191
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 4 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -1,8 +1,13 @@
 {
+    "rust-analyzer.cargo.extraEnv": {
+        "CFLAGS": "",
+        "CXXFLAGS": ""
+    },
     "rust-analyzer.checkOnSave.features": ["amalgation", "nightly"],
     "rust-analyzer.cargo.features": [
         "amalgation", "nightly"
     ],
+    "rust-analyzer.imports.preferNoStd": true,
     "rust-analyzer.completion.snippets.custom": {
         "JanetRS Test": {
             "prefix": ["janetrstest", "test"],

diff --git a/src/lib.rs b/src/lib.rs
@@ -54,7 +54,7 @@
 //! ### TODO: Lib level
 //!  - Better docs.
 //!  - Marshalling mechanism
-#![cfg_attr(not(feature = "std"), no_std)]
+#![no_std]
 #![cfg_attr(feature = "nightly", feature(allocator_api))]
 #![cfg_attr(docsrs, feature(doc_cfg))]
 
@@ -67,6 +67,9 @@ compile_error!(r#"You can only use either "amalgation" or "system" feature, not
 // Janet requires allocation
 extern crate alloc;
 
+#[cfg(any(test, feature = "std"))]
+extern crate std;
+
 /// This module has a expose the entire Janet C-API structures, constants and functions.
 ///
 /// This module exists in the case of a functionality of the Janet C-API can't be, or is

diff --git a/src/types.rs b/src/types.rs
@@ -12,7 +12,6 @@ use core::{
     num::ParseIntError,
 };
 
-#[cfg(not(feature = "std"))]
 use alloc::{
     string::{String, ToString},
     vec,

diff --git a/src/types/buffer.rs b/src/types/buffer.rs
@@ -11,7 +11,6 @@ use core::{
 
 use alloc::borrow::Cow;
 
-#[cfg(not(feature = "std"))]
 use alloc::{string::String, vec::Vec};
 
 #[cfg(feature = "std")]

diff --git a/src/types/string.rs b/src/types/string.rs
@@ -9,7 +9,6 @@ use core::{
 
 use alloc::borrow::Cow;
 
-#[cfg(not(feature = "std"))]
 use alloc::string::String;
 
 #[cfg(feature = "std")]