Tidy up to enable C4800 on Windows

client/crashpad_client_win.cc

 // Copyright 2015 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 211 matching lines @@
 #else
   record.ExceptionAddress = reinterpret_cast<void*>(context.Eip);
 #endif  // ARCH_CPU_64_BITS
 
   exception_pointers.ExceptionRecord = &record;
 
   g_non_crash_exception_information.thread_id = GetCurrentThreadId();
   g_non_crash_exception_information.exception_pointers =
       reinterpret_cast<crashpad::WinVMAddress>(&exception_pointers);
 
-  bool set_event_result = SetEvent(g_signal_non_crash_dump);
+  bool set_event_result = !!SetEvent(g_signal_non_crash_dump);

[Mark Mentovai, 2015/10/22 21:22:40]

Ugly, but well, at least it’s not !!::.

   PLOG_IF(ERROR, !set_event_result) << "SetEvent";
 
   DWORD wfso_result = WaitForSingleObject(g_non_crash_dump_done, INFINITE);
   PLOG_IF(ERROR, wfso_result != WAIT_OBJECT_0) << "WaitForSingleObject";
 }
 
 }  // namespace crashpad