rewrite macros using syn

.github/workflows/check.yml

@@ -39,9 +39,7 @@ jobs:
           toolchain: ${{ matrix.toolchain }}
           components: clippy
       - name: cargo clippy
-        uses: actions-rs/clippy-check@v1
-        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
+        run: cargo clippy -- -D warnings
   doc:
     runs-on: ubuntu-latest
     name: nightly / doc

.github/workflows/nostd.yml

@@ -22,3 +22,5 @@ jobs:
         run: rustup target add ${{ matrix.target }}
       - name: cargo check
         run: cargo check --target ${{ matrix.target }} --no-default-features
+        env:
+          RUSTFLAGS: "-Dwarnings"

.github/workflows/safety.yml

@@ -28,15 +28,24 @@ jobs:
         run: cargo test --lib --tests --all-features --target x86_64-unknown-linux-gnu
         env:
           ASAN_OPTIONS: "detect_odr_violation=0:detect_leaks=0"
-          RUSTFLAGS: "--cfg NO_UI_TESTS --cfg NO_ALLOC_FAIL_TESTS -Z sanitizer=address"
+          RUSTFLAGS: "--cfg NO_UI_TESTS --cfg NO_ALLOC_FAIL_TESTS -Z sanitizer=address -Dwarnings"
       - name: cargo test -Zsanitizer=leak
         if: always()
         run: cargo test --all-features --target x86_64-unknown-linux-gnu
         env:
           LSAN_OPTIONS: "suppressions=lsan-suppressions.txt"
-          RUSTFLAGS: "--cfg NO_UI_TESTS --cfg NO_ALLOC_FAIL_TESTS -Z sanitizer=leak"
+          RUSTFLAGS: "--cfg NO_UI_TESTS --cfg NO_ALLOC_FAIL_TESTS -Z sanitizer=leak -Dwarnings"
   miri:
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        flags: [
+          "",
+          "-Zmiri-tree-borrows",
+          "-Zmiri-strict-provenance",
+          "-Zmiri-tree-borrows -Zmiri-strict-provenance"
+        ]
     steps:
       - uses: actions/checkout@v4
         with:
@@ -48,10 +57,11 @@ jobs:
         with:
           toolchain: ${{ env.NIGHTLY }}
           components: miri
-      - name: cargo miri test
+      - name: ${{ matrix.flags }} cargo miri test
         run: cargo miri test
         env:
-          MIRIFLAGS: ""
+          RUSTFLAGS: "-Dwarnings"
+          MIRIFLAGS: ${{ matrix.flags }}
 # loom:
 #   runs-on: ubuntu-latest
 #   steps:

.github/workflows/scheduled.yml

@@ -27,6 +27,8 @@ jobs:
         run: cargo generate-lockfile
       - name: cargo test --locked
         run: cargo test --locked --all-features --all-targets
+        env:
+          RUSTFLAGS: "-Dwarnings"
   # https://twitter.com/alcuadrado/status/1571291687837732873
   update:
     runs-on: ubuntu-latest
@@ -46,4 +48,4 @@ jobs:
       - name: cargo test
         run: cargo test --locked --all-features --all-targets
         env:
-          RUSTFLAGS: -D deprecated
+          RUSTFLAGS: -Ddeprecated -Dwarnings

.github/workflows/test.yml

@@ -29,9 +29,13 @@ jobs:
       # https://twitter.com/jonhoo/status/1571290371124260865
       - name: cargo test --locked
         run: cargo test --locked --all-features --all-targets
+        env:
+          RUSTFLAGS: "-Dwarnings"
       # https://github.com/rust-lang/cargo/issues/6669
       - name: cargo test --doc
         run: cargo test --locked --all-features --doc
+        env:
+          RUSTFLAGS: "-Dwarnings"
   os-check:
     runs-on: ${{ matrix.os }}
     name: ${{ matrix.os }} / nightly
@@ -53,4 +57,6 @@ jobs:
         if: hashFiles('Cargo.lock') == ''
         run: cargo generate-lockfile
       - name: cargo test
-        run: cargo test --locked --all-features --all-targets
+        run: cargo test --locked --all-features --all-targets 
+        env:
+          RUSTFLAGS: "-Dwarnings"

README.md

@@ -63,13 +63,13 @@ and thus is a prime candidate for using this library.
 ### Using the [`pin_init!`] macro
 
 If you want to use [`PinInit`], then you will have to annotate your `struct` with
-`#[`[`pin_data`]`]`. It is a macro that uses `#[pin]` as a marker for
+[`#[pin_data]`](pin_data). It is a macro that uses `#[pin]` as a marker for
 [structurally pinned fields]. After doing this, you can then create an in-place constructor via
 [`pin_init!`]. The syntax is almost the same as normal `struct` initializers. The difference is
 that you need to write `<-` instead of `:` for fields that you want to initialize in-place.
 
 ```rust
-use pinned_init::*;
+use pinned_init::{pin_data, pin_init};
 #[pin_data]
 struct Foo {
     #[pin]
@@ -87,6 +87,8 @@ let foo = pin_init!(Foo {
 (or just the stack) to actually initialize a `Foo`:
 
 ```rust
+use pinned_init::InPlaceInit;
+
 let foo: Result<Pin<Box<Foo>>, _> = Box::pin_init(foo);
 ```
 
@@ -98,12 +100,16 @@ Many types that use this library supply a function/macro that returns an initial
 the above method only works for types where you can access the fields.
 
 ```rust
+use pinned_init::InPlaceInit;
+
 let mtx: Result<Pin<Arc<CMutex<usize>>>, _> = Arc::pin_init(CMutex::new(42));
 ```
 
 To declare an init macro/function you just return an [`impl PinInit<T, E>`]:
 
 ```rust
+use pinned_init::{pin_data, try_pin_init, PinInit, InPlaceInit};
+
 #[pin_data]
 struct DriverData {
     #[pin]
@@ -136,7 +142,7 @@ actually does the initialization in the correct way. Here are the things to look
   `slot` gets called.
 
 ```rust
-use pinned_init::*;
+use pinned_init::{pin_data, pinned_drop, pin_init_from_closure, PinInit};
 use core::{ptr::addr_of_mut, marker::PhantomPinned, cell::UnsafeCell, pin::Pin};
 mod bindings {
     extern "C" {

examples/big_struct_in_place.rs

@@ -1,7 +1,5 @@
 #![feature(allocator_api)]
 
-use std::convert::Infallible;
-
 use pinned_init::*;
 
 // Struct with size over 1GiB

examples/error.rs

@@ -1,3 +1,4 @@
+#![allow(unused_attributes)]
 #![feature(allocator_api)]
 
 use core::convert::Infallible;
@@ -18,4 +19,5 @@ impl From<AllocError> for Error {
     }
 }
 
+#[allow(dead_code)]
 fn main() {}

examples/mutex.rs

@@ -1,4 +1,6 @@
+#![allow(unused_attributes)]
 #![feature(allocator_api)]
+
 use core::{
     cell::{Cell, UnsafeCell},
     marker::PhantomPinned,
@@ -16,6 +18,7 @@ use pinned_init::*;
 #[allow(unused_attributes)]
 #[path = "./linked_list.rs"]
 pub mod linked_list;
+#[allow(unused_imports)]
 pub use linked_list::Error;
 use linked_list::*;
 
@@ -100,6 +103,7 @@ impl<T> CMutex<T> {
         }
     }
 
+    #[allow(dead_code)]
     pub fn get_data_mut(self: Pin<&mut Self>) -> &mut T {
         // SAFETY: we have an exclusive reference and thus nobody has access to data.
         unsafe { &mut *self.data.get() }

examples/pthread_mutex.rs

@@ -1,5 +1,7 @@
 // inspired by https://github.com/nbdd0121/pin-init/blob/trunk/examples/pthread_mutex.rs
+#![allow(dead_code)]
 #![feature(allocator_api)]
+
 #[cfg(not(windows))]
 mod pthread_mtx {
     use core::{

pinned-init-macro/Cargo.toml

@@ -15,3 +15,4 @@ proc-macro = true
 [dependencies]
 quote = "1.0"
 proc-macro2 = "1.0"
+syn = { version = "2.0", features = ["full", "visit-mut", "printing"] }