swift-reflection-test: Don't leave test process hanging

bitjammer · bitjammer · commit ce3ed51485c8 · 2016-03-04T17:10:40.000-08:00
diff --git a/tools/swift-reflection-test/swift-reflection-test.cpp b/tools/swift-reflection-test/swift-reflection-test.cpp
@@ -417,15 +417,14 @@ static int doDumpHeapInstance(std::string BinaryFilename) {
 
       error = mach_port_deallocate(mach_task_self(), childPort);
       guardMachError(error, "mach_port_deallocate");
-      exit(EXIT_SUCCESS);
     }
   }
 
+  kill(pid, SIGTERM);
+
   error = task_set_bootstrap_port(mach_task_self(), bootstrap_port);
   guardMachError(error, "reset task_set_bootstrap_port");
 
-  kill(pid, SIGTERM);
-
   error = mach_port_deallocate(mach_task_self(), childPort);
   guardMachError(error, "mach_port_deallocate");
 

Original file line number	Diff line number	Diff line change
`@@ -417,15 +417,14 @@ static int doDumpHeapInstance(std::string BinaryFilename) {`
`417`	`417`
`418`	`418`	`error = mach_port_deallocate(mach_task_self(), childPort);`
`419`	`419`	`guardMachError(error, "mach_port_deallocate");`
`420`		`- exit(EXIT_SUCCESS);`
`421`	`420`	`}`
`422`	`421`	`}`
`423`	`422`
	`423`	`+ kill(pid, SIGTERM);`
	`424`	`+`
`424`	`425`	`error = task_set_bootstrap_port(mach_task_self(), bootstrap_port);`
`425`	`426`	`guardMachError(error, "reset task_set_bootstrap_port");`
`426`	`427`
`427`		`- kill(pid, SIGTERM);`
`428`		`-`
`429`	`428`	`error = mach_port_deallocate(mach_task_self(), childPort);`
`430`	`429`	`guardMachError(error, "mach_port_deallocate");`
`431`	`430`