Move assignment for Thread, make destructor optional, loop cv waits

1 files changed, 2 insertions, 4 deletions

diff --git a/test/core/gpr/sync_test.cc b/test/core/gpr/sync_test.cc
index 487f394b14..24b4562819 100644
--- a/test/core/gpr/sync_test.cc
+++ b/test/core/gpr/sync_test.cc
@@ -22,7 +22,6 @@
 
 #include <stdio.h>
 #include <stdlib.h>
-#include <new>
 
 #include <grpc/support/alloc.h>
 #include <grpc/support/log.h>
@@ -196,7 +195,7 @@ static void test_destroy(struct test* m) {
 static void test_create_threads(struct test* m, void (*body)(void* arg)) {
   int i;
   for (i = 0; i != m->nthreads; i++) {
-    new (&m->threads[i]) grpc_core::Thread("grpc_create_threads", body, m);
+    m->threads[i] = grpc_core::Thread("grpc_create_threads", body, m);
     m->threads[i].Start();
   }
 }
@@ -210,7 +209,6 @@ static void test_wait(struct test* m) {
   gpr_mu_unlock(&m->mu);
   for (int i = 0; i != m->nthreads; i++) {
     m->threads[i].Join();
-    m->threads[i].~Thread();
   }
 }
 
@@ -258,7 +256,7 @@ static void test(const char* name, void (*body)(void* m),
     m = test_new(10, iterations, incr_step);
     grpc_core::Thread extra_thd;
     if (extra != nullptr) {
-      new (&extra_thd) grpc_core::Thread(name, extra, m);
+      extra_thd = grpc_core::Thread(name, extra, m);
       extra_thd.Start();
       m->done++; /* one more thread to wait for */
     }