drop-with-owned-fields 0.0.0-wip

Safe and sound *owned* access to a `struct`'s fields in `Drop`: no more `unsafe` usage of `ManuallyDrop`!
Documentation
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
# `::drop-with-owned-fields`

Safe and sound _owned_ access to a `struct`'s fields in `Drop`: no more `unsafe` usage of `ManuallyDrop`!

[![Repository](https://img.shields.io/badge/repository-GitHub-brightgreen.svg)](
https://github.com/danielhenrymantilla/drop-with-owned-fields.rs)
[![Latest version](https://img.shields.io/crates/v/drop-with-owned-fields.svg)](
https://crates.io/crates/drop-with-owned-fields)
[![Documentation](https://docs.rs/drop-with-owned-fields/badge.svg)](
https://docs.rs/drop-with-owned-fields)
[![MSRV](https://img.shields.io/badge/MSRV-1.79.0-white)](
https://gist.github.com/danielhenrymantilla/9b59de4db8e5f2467ed008b3c450527b)
[![unsafe forbidden](https://img.shields.io/badge/unsafe-forbidden-success.svg)](
https://github.com/rust-secure-code/safety-dance/)
[![License](https://img.shields.io/crates/l/drop-with-owned-fields.svg)](
https://github.com/danielhenrymantilla/drop-with-owned-fields.rs/blob/master/LICENSE-ZLIB)
[![CI](https://github.com/danielhenrymantilla/drop-with-owned-fields.rs/workflows/CI/badge.svg)](
https://github.com/danielhenrymantilla/drop-with-owned-fields.rs/actions)
[![no_std compatible](https://img.shields.io/badge/no__std-compatible-success.svg)](
https://github.com/rust-secure-code/safety-dance/)

<!-- Templated by `cargo-generate` using https://github.com/danielhenrymantilla/proc-macro-template -->

The [`#[drop_with_owned_fields]`][`drop_with_owned_fields`] attribute of this crate automates and
encapsulates the process of wrapping the fields of a struct in [`ManuallyDrop`], which is typically
needed when having the intention to [`drop()`-in-place][`ManuallyDrop::drop()`] a certain field
before others are, or when actually needing to [`take`][`ManuallyDrop::take()`] full, owned, access
to that field. These two operations are `unsafe`, despite the notorious soundness of the whole
pattern, which is quite unfortunate. The objective of this crate is to properly identify and
automate this "notoriously sound" pattern, so as to expose a _safe and sound_ API for users to take
advantage of, with all the power of the type system supporting them and gently nudging them away
from bugs.

## Examples

### Example: `Defer<impl FnOnce()>`

Take, for instance, the following, rather typical, example:

```rust ,compile_fail
struct Defer<F: FnOnce()> {
    f: F,
};

impl<F: FnOnce()> Drop for Defer<F> {
    fn drop(&mut self) {
        (self.f)() // Error, cannot move out of `self.f` which is behind a mutable reference
    }
}
```

Alas, our usage of the correct `FnOnce()` bound (since we only need to call it once) has made this
snippet fail!

  - Full error message:

    <details class="custom"><summary><span class="summary-box"><span>Click to show</span></span></summary>

    <span class="code_with_line_wrap">

    ```rust ,ignore
    # /*
    error[E0507]: cannot move out of `self.f` which is behind a mutable reference
     --> src/_lib.rs:37:9
      |
    7 |         (self.f)()
      |         ^^^^^^^^--
      |         |
      |         `self.f` moved due to this call
      |         move occurs because `self.f` has type `F`, which does not implement the `Copy` trait
      |
    note: this value implements `FnOnce`, which causes it to be moved when called
     --> src/_lib.rs:37:9
      |
    7 |         (self.f)()
      |         ^^^^^^^^
    # */
    ```

    </span>

    </details>

Enter [`#[drop_with_owned_fields]`][`drop_with_owned_fields`]:

```rust
# fn main() {}
#
use ::drop_with_owned_fields::drop_with_owned_fields;

#[drop_with_owned_fields]
struct Defer<F: FnOnce()> {
    f: F,
}

#[drop_with_owned_fields]
impl<F: FnOnce()> Drop for Defer<F> {
    fn drop(Self { f }: _) {
        f(); // ✅
    }
}
```

Note that the second usage of [`#[drop_with_owned_fields]`][`drop_with_owned_fields`] on that
`impl Drop` block is only supported by enabling the `"drop-sugar"` feature of the crate, which
shall, in turn, enable the `"full"` features of `::syn` (resulting in a slightly higher
from-scratch compile-time, _should no other crate in the dependency tree have enabled it
already_).

Without it, the `Drop` block and logic would have had to be spelled out a bit more explicitly, like so:

<details class="custom"><summary><span class="summary-box"><span>Click to show</span></span></summary>

```rust
# fn main() {}
#
use ::drop_with_owned_fields::prelude::*;

#[drop_with_owned_fields]
struct Defer<F: FnOnce()> { f: F }

impl<F: FnOnce()> DropWithOwnedFields for Defer<F> {
    fn drop(DestructuredFieldsOf::<Self> { f }: DestructuredFieldsOf<Self>) {
        f(); // ✅
    }
}
```

or, if the `DestructuredFieldsOf<Self>` stutter is then deemed unæsthetic:

```rust
# fn main() {}
#
use ::drop_with_owned_fields::prelude::*;

#[drop_with_owned_fields]
struct Defer<F: FnOnce()> { f: F }

impl<F: FnOnce()> DropWithOwnedFields for Defer<F> {
    fn drop(this: DestructuredFieldsOf<Self>) {
        if true {
            // One approach…
            (this.f)(); // ✅
        } else {
            // …or another.
            let DestructuredFieldsOf::<Self> { f } = this;
            f(); // ✅
        }
    }
}
```

  - (the advantage of destructuring is that you can be sure not to be forgetting to properly handle
    some field; with that being said, the "forgotten" fields are still owned in that `fn` body,
    just anonymously (or by `this`), so they get dropped, normally, at the end of the `fn`.)

---

</details>

If you forget to `impl DropWithOwnedFields` (with or without sugar), like so:

```rust ,compile_fail
# use ::drop_with_owned_fields::drop_with_owned_fields;
#
#[drop_with_owned_fields]
struct Example {
    // …
}
#
# fn main() {}
```

you will then get the following compiler error message:

<details class="custom"><summary><span class="summary-box"><span>Click to show</span></span></summary>

<span class="code_with_line_wrap">

```rust ,ignore
# /*
error[E0277]: the trait bound `Example: DropWithOwnedFields` is not satisfied
 --> src/_lib.rs:130:1
  |
6 | #[drop_with_owned_fields]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `DropWithOwnedFields` is not implemented for `Example`
  |
  = note: The `#[drop_with_owned_fields]` annotation expects 🫵 you to provide a companion `impl` of `DropWithOwnedFields` (the whole point!).

          If you have enabled the `"drop-sugar"` Cargo feature, you can even write a direct `impl` of `Drop` instead, but with a `#[drop_with_owned_fields]` annotation on top of it.

  = note: this error originates in the attribute macro `drop_with_owned_fields` (...)
# */
```

</span>

</details>

### Example: `.transaction.commit()` in `Drop`

<details open class="custom"><summary><span class="summary-box"><span>Click to hide</span></span></summary>

Another case where one may need owned access to a field in drop is when the field is doing its own linear/affine-ish types thing, and having different "destructors" requiring and consuming an owned `self`.

Typically, `transaction` handles do this for their `.commit()` and `.roll_back()` functions:

```rust
use ::drop_with_owned_fields::drop_with_owned_fields;

use example_lib::Transaction;
// where:
mod example_lib {
    pub struct Transaction {
        // …
    }

    impl Transaction {
        /// Owned access required for stronger type-safety 👌
        pub fn commit(self) {
            // …
        }
    }
}

#[drop_with_owned_fields]
struct CommitOnDrop {
    txn: Transaction,
}

#[drop_with_owned_fields]
impl Drop for CommitOnDrop {
    fn drop(Self { txn }: _) {
        txn.commit(); // ✅
    }
}
#
# fn main() {}
```

</details>

## Unsugaring

Taking the `Defer<F>` example, for instance, but renamed as Foo:

```rust
#[::drop_with_owned_fields::drop_with_owned_fields]
impl<F: FnOnce()> Drop for Foo<F> {
    fn drop(Self { f }: _) {
        f();
    }
}

#[::drop_with_owned_fields::drop_with_owned_fields]
struct Foo<F: FnOnce()> {
    f: F,
}
#
# fn main() {}
```

unsugars to code along the following lines (papering over robust namespacing and privacy):

```rust
# use ::core::{mem::ManuallyDrop, ops::{Deref, DerefMut}};
# use ::drop_with_owned_fields::prelude::*;
#
/// Unsugaring of the `Drop` impl.
impl<F: FnOnce()> DropWithOwnedFields for Foo<F> {
    fn drop(FooFields { f }: FooFields<F>) {
        f();
    }
}

// == Unsugaring of the `struct Foo` definition: ==

struct FooFields<F: FnOnce()> {
    f: F,
}

/// This is what makes `DestructuredFieldsOf<Foo<F>> == FooFields<F>`.
impl<F: FnOnce()> ::drop_with_owned_fields::DestructureFields for Foo<F> {
    type DestructuredFields = FooFields<F>;
    # fn destructure_fields_disabling_extra_drop(self) -> Self::DestructuredFields { todo!() }
}
# impl<F: FnOnce()> ::drop_with_owned_fields::ඞ::drop_with_owned_fields_annotation for Foo<F> {}

// The `ManuallyDrop` unsafe-but-sound pattern!
struct Foo<F: FnOnce()> {
    // real fields no longer in the `struct`, but moved to the companion `SelfFields` data type!
    // 👇
    manually_drop_fields: ManuallyDrop<FooFields<F>>,
}
impl<F: FnOnce()> Drop for Foo<F>
where
# /*
    // This is what makes the real `impl Drop` use and require your `DropWithOwned` logic
    //     👇
    Self : DropWithOwnedFields,
# */
{
    fn drop(&mut self) {
        let fields = unsafe {
            ManuallyDrop::take(&mut self.manually_drop_fields)
        };
        <Self as DropWithOwnedFields>::drop(fields);
    }
}

// -- Niceties --

// -- 1. `.field_name` access sugar:
impl<F: FnOnce()> Deref for Foo<F> {
    type Target = FooFields<F>;
    // …
    # fn deref(&self) -> &FooFields<F> { &self.manually_drop_fields }
}
// Ditto for `DerefMut`

# #[cfg(any())]
// -- 2. Constructor builder/helper
impl<F: FnOnce()> Into<Foo<F>> for FooFields<F> {
    // ...
}
#
# fn main() {}
```

Mainly, notice the very important addition of a "companion `struct`": `FooFields<F>`:

## The companion `struct FooFields<…>`

This is the `struct` containing all of the fields laid out as they initially were for the original
`Foo` definition. The trick having been about splitting the original `Foo` definition (as input to
the macro) into two `struct` definitions:

  - the `FooFields<…>` "copy", which has the fields **but has no extra/customized `Drop` glue nor
    `impl` whatsoever**.

  - the `Foo<…>` original type, which does have the desired extra/customized `Drop impl`, but in
    exchange of that it had to forsake carrying the fields directly, using a
    `ManuallyDrop<FooFields<…>>` layer instead.

More generally, for every `StructName<Generics…>`, there shall be a companion
`StructNameFields<Generics…>` definition.

  - There is a way to make the attribute name this data structure as _you_ desire: its default name
    can thus be overridden;

  - otherwise, the default name is _currently left unspecified_ (and will probably involve some ``
    sigil somewhere in the name to communicate this fact and deter from direct usage).

    Instead, the [`DestructureFields`] `trait` can be used, _especially its
    [`DestructuredFields`] associated type_.

    Or the convenience [`DestructuredFieldsOf<_>`][`DestructuredFieldsOf`] `type` alias:

    ```rust ,ignore
    DestructuredFieldsOf<Foo<F>> = <Foo<F> as DestructureFields>::DestructuredFields
                                 = FooFields<F>
    ```

    This yields a properly specified and usable type name.

It is guaranteed to have the same fields as the original `Foo` definition, in terms of:
  - deconstructing and/or accessing these fields in `Drop` of `Deref{,Mut}`;
  - constructing this `FooFields { … }` instance, which, as we are about to see, shall be not only
    handy but _paramount_ for the instantiation of a `Foo { … }` value.

## Braced literal construction of `Foo { … }`

It is not longer available once the [`#[drop_with_owned_fields]`][`drop_with_owned_fields`] pass has
happened onto `Foo`'s definition.

Indeed, instead, we have something along the lines of:

```rust ,ignore
struct Foo<F> {
    manually_drop_fiels: ManuallyDrop<FooFields<F>>,
}
```

  - (with `.manually_drop_fields` being a field name left _private_);

This, obviously, prevents the "typical" braced-`struct`-literal construction of a `Foo { … }`.

```rust ,compile_fail
# use ::core::mem::ManuallyDrop;
#
struct FooFields<F: FnOnce()> { f: F }

struct Foo<F: FnOnce()> {
    manually_drop_fiels: ManuallyDrop<FooFields<F>>,
}

let _foo = Foo {
    f: || (),
};
```

<span class="code_with_line_wrap">

```rust ,ignore
# /*
error[E0560]: struct `Foo<_>` has no field named `f`
  --> src/_lib.rs:392:5
   |
12 |     f: || (),
   |     ^ `Foo<_>` does not have this field
   |
# */
```

</span>

Instead, the workaround is to involve the perfectly-available braced-`struct`-literal construction
of the `FooFields { … } struct` and _its eponymous fields_, and then simply call `.into()` to
convert it "back" into a `Foo { … }`:

```rust
use ::drop_with_owned_fields::prelude::*;

#[drop_with_owned_fields]
pub struct Defer<F: FnOnce()> {
    f: F,
}

#[drop_with_owned_fields]
impl<F: FnOnce()> Drop for Defer<F> {
    fn drop(Self { f }: _) {
        f();
    }
}

impl<F: FnOnce()> Defer<F> {
    pub fn new(f: F) -> Self {
        DestructuredFieldsOf::<Self> { f }.into() // 👈
    }
}

fn main() {
    let _defer = Defer::new(|| println!("General Kenobi."));
    println!("Hello, there!");
}
```

## Renaming the companion `struct`

Since having to type `DestructuredFieldsOf<Foo<F>>` all the time can be deemed cumbersome, and
noisy, the [`#[drop_with_owned_fields]`][`drop_with_owned_fields`] attribute accepts an optional
attribute arg to:
- both override the  name of that companion struct,
- and adjust its visibility so that it be allowed to be fully `pub`lic should the author with so:

```rust
use ::drop_with_owned_fields::prelude::*;

#[drop_with_owned_fields(
    pub type FooFields = DestructuredFieldsOf<Self>,
)]
pub struct Foo<F: FnOnce()> { f: F }

impl<F: FnOnce()> DropWithOwnedFields for Foo<F> {
    fn drop(FooFields { f }: FooFields<F>) {
        f(); // ✅
    }
}
#
# fn main() {}
```

<!-- Note: the following links are just for Github's `README.md`, since docs.rs has these shadowed by the proper intra-doc links. -->

[`ManuallyDrop`]: https://doc.rust-lang.org/stable/std/mem/struct.ManuallyDrop.html

[`ManuallyDrop::drop()`]: https://doc.rust-lang.org/stable/std/mem/struct.ManuallyDrop.html#method.drop

[`ManuallyDrop::take()`]: https://doc.rust-lang.org/stable/std/mem/struct.ManuallyDrop.html#method.take

[`drop_with_owned_fields`]: https://docs.rs/drop-with-owned-fields/*/drop_with_owned_fields/attr.drop_with_owned_fields.html

[`DestructureFields`]: https://docs.rs/drop-with-owned-fields/*/drop_with_owned_fields/trait.DestructureFields.html

[`DestructuredFields`]: https://docs.rs/drop-with-owned-fields/*/drop_with_owned_fields/trait.DestructureFields.html#associatedtype.DestructuredFields

[`DestructuredFieldsOf`]: https://docs.rs/drop-with-owned-fields/*/drop_with_owned_fields/type.DestructuredFieldsOf.html