Fix support for no_std (#3180)

Author: kennykerr

.github/workflows/clippy.yml

@@ -34,8 +34,6 @@ jobs:
         uses: ./.github/actions/fix-environment
       - name: Clippy cppwinrt
         run:  cargo clippy -p cppwinrt
-      - name: Clippy no_std
-        run:  cargo clippy -p no_std
       - name: Clippy riddle
         run:  cargo clippy -p riddle
       - name: Clippy sample_bits
@@ -204,6 +202,8 @@ jobs:
         run:  cargo clippy -p test_metadata
       - name: Clippy test_msrv
         run:  cargo clippy -p test_msrv
+      - name: Clippy test_no_std
+        run:  cargo clippy -p test_no_std
       - name: Clippy test_no_use
         run:  cargo clippy -p test_no_use
       - name: Clippy test_noexcept

.github/workflows/no_std.yml

@@ -0,0 +1,26 @@
+name: no_std
+
+on:
+  pull_request:
+  push:
+    paths-ignore:
+      - '.github/ISSUE_TEMPLATE/**'
+    branches:
+      - master
+
+env:
+  RUSTFLAGS: -Dwarnings
+
+jobs:
+  check:
+    strategy:
+      matrix:
+        rust: [stable, nightly]
+    runs-on: windows-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Prepare
+        run: rustup update --no-self-update ${{ matrix.rust }} && rustup default ${{ matrix.rust }}
+      - name: Check
+        run: cargo check -p test_no_std

.github/workflows/test.yml

@@ -58,8 +58,6 @@ jobs:
         run:  cargo clean
       - name: Test cppwinrt
         run:  cargo test -p cppwinrt --target ${{ matrix.target }} ${{ matrix.etc }}
-      - name: Test no_std
-        run:  cargo test -p no_std --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test riddle
         run:  cargo test -p riddle --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test sample_bits
@@ -156,10 +154,10 @@ jobs:
         run:  cargo test -p test_alternate_success_code --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_arch
         run:  cargo test -p test_arch --target ${{ matrix.target }} ${{ matrix.etc }}
-      - name: Clean
-        run:  cargo clean
       - name: Test test_arch_feature
         run:  cargo test -p test_arch_feature --target ${{ matrix.target }} ${{ matrix.etc }}
+      - name: Clean
+        run:  cargo clean
       - name: Test test_array
         run:  cargo test -p test_array --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_bcrypt
@@ -230,6 +228,8 @@ jobs:
         run:  cargo test -p test_metadata --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_msrv
         run:  cargo test -p test_msrv --target ${{ matrix.target }} ${{ matrix.etc }}
+      - name: Test test_no_std
+        run:  cargo test -p test_no_std --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_no_use
         run:  cargo test -p test_no_use --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_noexcept

crates/libs/bindgen/src/lib.rs

@@ -292,6 +292,6 @@ fn extension(path: &str) -> &str {
 }
 
 fn directory(path: &str) -> &str {
-    path.rsplit_once(&['/', '\\'])
+    path.rsplit_once(['/', '\\'])
         .map_or("", |(directory, _)| directory)
 }

crates/libs/bindgen/src/winmd/writer/mod.rs

@@ -45,9 +45,7 @@ impl Writer {
             MethodList: 0,
         });
 
-        let name = name
-            .rsplit_once(&['/', '\\'])
-            .map_or(name, |(_, name)| name);
+        let name = name.rsplit_once(['/', '\\']).map_or(name, |(_, name)| name);
 
         writer.tables.Module.push(Module {
             Name: writer.strings.insert(name),

crates/libs/core/Cargo.toml

@@ -24,15 +24,17 @@ path = "../targets"
 [dependencies.windows-result]
 version = "0.2.0"
 path = "../result"
+default-features = false
 
 [dependencies.windows-strings]
 version = "0.1.0"
 path = "../strings"
+default-features = false
 
 [dependencies]
 windows-implement = { path = "../implement",  version = "0.58.0" }
 windows-interface = { path = "../interface",  version = "0.58.0" }
 
 [features]
 default = ["std"]
-std = []
+std = ["windows-result/std", "windows-strings/std"]

crates/libs/registry/Cargo.toml

@@ -24,7 +24,9 @@ path = "../targets"
 [dependencies.windows-result]
 version = "0.2.0"
 path = "../result"
+default-features = false
 
 [dependencies.windows-strings]
 version = "0.1.0"
 path = "../strings"
+default-features = false

crates/libs/windows/Cargo.toml

@@ -26,9 +26,14 @@ default-target = "x86_64-pc-windows-msvc"
 targets = []
 rustdoc-args = ["--cfg", "docsrs"]
 
-[dependencies]
-windows-core = { path = "../core", version = "0.58.0" }
-windows-targets = { path = "../targets", version = "0.52.6" }
+[dependencies.windows-core]
+version = "0.58.0"
+path = "../core"
+default-features = false
+
+[dependencies.windows-targets]
+version = "0.52.6"
+path = "../targets"
 
 [features]
 default = ["std"]

crates/libs/windows/src/lib.rs

@@ -12,6 +12,7 @@ Learn more about Rust for Windows here: <https://github.com/microsoft/windows-rs
 #![cfg_attr(not(feature = "docs"), doc(hidden))]
 // TODO: workaround for https://github.com/rust-lang/rust/issues/126169
 #![allow(unused)]
+#![cfg_attr(all(not(feature = "std")), no_std)]
 
 #[allow(unused_extern_crates)]
 extern crate self as windows;

crates/tests/no_std/Cargo.toml

@@ -1,5 +1,5 @@
 [package]
-name = "no_std"
+name = "test_no_std"
 version = "0.1.0"
 edition = "2021"
 
@@ -11,11 +11,6 @@ doctest = false
 path = "../../libs/core"
 default-features = false
 
-[dependencies.windows]
-path = "../../libs/windows"
-default-features = false
-features = ["implement"]
-
 [dependencies.windows-registry]
 path = "../../libs/registry"
 default-features = false
@@ -24,5 +19,26 @@ default-features = false
 path = "../../libs/result"
 default-features = false
 
+[dependencies.windows-strings]
+path = "../../libs/strings"
+default-features = false
+
+[dependencies.windows-sys]
+path = "../../libs/sys"
+default-features = false
+
+[dependencies.windows-targets]
+path = "../../libs/targets"
+default-features = false
+
+[dependencies.windows-version]
+path = "../../libs/version"
+default-features = false
+
+[dependencies.windows]
+path = "../../libs/windows"
+default-features = false
+features = ["implement"]
+
 [lints]
 workspace = true