Re: [PATCH 2/2] kernel32/tests: Close all handles on process exit.

[Zebediah Figura]

On 11/16/2017 09:56 PM, Dmitry Timoshkov wrote:
> > - CloseHandle(loader_lock_event);
> > - CloseHandle(peb_lock_event);
> > - CloseHandle(heap_lock_event);
> > - CloseHandle(ack_event);
> > - CloseHandle(stop_event);
> > - CloseHandle(process);
> > - /* Don't close event, mutex and semaphore here; we want to test if they are
> > -
> > * abandoned on process termination. */
> > -
> > trace("call ExitProcess(195)\n");
> > ExitProcess(195);
> > }
>
> Please don't add these CloseHandle() calls, they clutter the code and don't do
> anything useful.

Thanks. I've resent the patch just to remove the existing CloseHandle() 
call.