From a674e1c88c632c237d7e56fbb6972b46c39a1fb0 Mon Sep 17 00:00:00 2001
From: Dodo <kasper199914@gmail.com>
Date: Mon, 2 Mar 2020 23:43:50 +0100
Subject: [PATCH] remove unused mut, restructure the test

---
 src/libcore/tests/mem.rs | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/libcore/tests/mem.rs b/src/libcore/tests/mem.rs
index 9ed2323e2873..8337ab103419 100644
--- a/src/libcore/tests/mem.rs
+++ b/src/libcore/tests/mem.rs
@@ -132,18 +132,18 @@ fn test_discriminant_send_sync() {
 
 #[test]
 fn test_const_forget() {
-    const fn test_const_forget<T>(x: T) {
-        forget(x);
-    }
+    const _: () = forget(0i32);
+    const _: () = forget(Vec::<Vec<Box<i32>>>::new());
 
     // Writing this function signature without const-forget
     // triggers compiler errors:
     // 1) That we use a non-const fn inside a const fn
     // 2) without the forget, it complains about the destructor of Box
-    const fn const_forget_box<T>(mut x: Box<T>) {
+    const fn const_forget_box<T>(x: Box<T>) {
         forget(x);
     }
 
-    const _: () = test_const_forget(0i32);
-    const _: () = test_const_forget(Vec::<Vec<Box<i32>>>::new());
+    // Call the forget_box at runtime,
+    // as we can't const-construct a box yet.
+    const_forget_box(Box::new(0i32));
 }