From 8f6d7fdf0a9bf22727616fbf60f81345229fd300 Mon Sep 17 00:00:00 2001 From: Taiki Endo Date: Sun, 11 Aug 2019 01:19:07 +0900 Subject: [PATCH] Apply suggestions from code review --- pin-project-internal/src/pin_projectable/mod.rs | 1 - tests/repr_packed.rs | 6 +++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/pin-project-internal/src/pin_projectable/mod.rs b/pin-project-internal/src/pin_projectable/mod.rs index 74e4f0de..e70cb287 100644 --- a/pin-project-internal/src/pin_projectable/mod.rs +++ b/pin-project-internal/src/pin_projectable/mod.rs @@ -35,7 +35,6 @@ fn handle_type(args: TokenStream, item: Item, pinned_drop: Option) -> Re Item::Struct(item) => { let packed_check = ensure_not_packed(&item)?; let mut res = structs::parse(args, item, pinned_drop)?; - //println!("Packed check: {}", packed_check); res.extend(packed_check); Ok(res) } diff --git a/tests/repr_packed.rs b/tests/repr_packed.rs index 2596360d..0d9d64bc 100644 --- a/tests/repr_packed.rs +++ b/tests/repr_packed.rs @@ -17,7 +17,7 @@ use std::cell::Cell; fn weird_repr_packed() { // We keep track of the field address during // drop using a thread local, to avoid changing - // the layout of our #[repr(packed)] type + // the layout of our #[repr(packed)] type. thread_local! { static FIELD_ADDR: Cell = Cell::new(0); } @@ -39,8 +39,8 @@ fn weird_repr_packed() { // We let this field drop by going out of scope, // rather than explicitly calling drop(foo). // Calling drop(foo) causes 'foo' to be moved - // into the 'drop' function, resulinting a different - // address + // into the 'drop' function, resulting in a different + // address. let foo = Foo { field: 27 }; let field_addr = &foo.field as *const u8 as usize; field_addr