diff options
author | Benno Lossin <benno.lossin@proton.me> | 2023-08-14 10:47:03 +0200 |
---|---|---|
committer | Miguel Ojeda <ojeda@kernel.org> | 2023-08-21 14:31:48 +0200 |
commit | 92fd540d62701115b22b1f531c8c86454809931b (patch) | |
tree | 6d8456b4c69876e184ac66f189bb4a809d5b93a4 /drivers/ps3/vuart.h | |
parent | rust: init: wrap type checking struct initializers in a closure (diff) | |
download | linux-92fd540d62701115b22b1f531c8c86454809931b.tar.xz linux-92fd540d62701115b22b1f531c8c86454809931b.zip |
rust: init: make initializer values inaccessible after initializing
Previously the init macros would create a local variable with the name
and hygiene of the field that is being initialized to store the value of
the field. This would override any user defined variables. For example:
```
struct Foo {
a: usize,
b: usize,
}
let a = 10;
let foo = init!(Foo{
a: a + 1, // This creates a local variable named `a`.
b: a, // This refers to that variable!
});
let foo = Box::init!(foo)?;
assert_eq!(foo.a, 11);
assert_eq!(foo.b, 11);
```
This patch changes this behavior, so the above code would panic at the
last assertion, since `b` would have value 10.
Reviewed-by: Martin Rodriguez Reboredo <yakoyoku@gmail.com>
Reviewed-by: Alice Ryhl <aliceryhl@google.com>
Reviewed-by: Gary Guo <gary@garyguo.net>
Signed-off-by: Benno Lossin <benno.lossin@proton.me>
Link: https://lore.kernel.org/r/20230814084602.25699-7-benno.lossin@proton.me
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
Diffstat (limited to 'drivers/ps3/vuart.h')
0 files changed, 0 insertions, 0 deletions