From d51c84dde22549c7413e52a682158e6da3ff1fdc Mon Sep 17 00:00:00 2001
From: bunnei <ericbunnie@gmail.com>
Date: Thu, 29 May 2014 20:00:44 -0400
Subject: event: added support for ClearEvent, fixed a bug with CreateEvent,
 fixed some comments

---
 src/core/hle/kernel/event.cpp | 18 +++++++++++-------
 src/core/hle/kernel/event.h   |  5 +++--
 2 files changed, 14 insertions(+), 9 deletions(-)

(limited to 'src/core/hle')

diff --git a/src/core/hle/kernel/event.cpp b/src/core/hle/kernel/event.cpp
index e85f39f0..7cb8c514 100644
--- a/src/core/hle/kernel/event.cpp
+++ b/src/core/hle/kernel/event.cpp
@@ -54,11 +54,16 @@ public:
  * Changes whether an event is locked or not
  * @param handle Handle to event to change
  * @param locked Boolean locked value to set event
+ * @return Result of operation, 0 on success, otherwise error code
  */
-void SetEventLocked(const Handle handle, const bool locked) {
+Result SetEventLocked(const Handle handle, const bool locked) {
     Event* evt = g_object_pool.GetFast<Event>(handle);
+    if (!evt) {
+        ERROR_LOG(KERNEL, "SetEventLocked called with unknown handle=0x%08X", handle);
+        return -1;
+    }
     evt->locked = locked;
-    return;
+    return 0;
 }
 
 /**
@@ -67,23 +72,22 @@ void SetEventLocked(const Handle handle, const bool locked) {
  * @return Result of operation, 0 on success, otherwise error code
  */
 Result ClearEvent(Handle handle) {
-    ERROR_LOG(KERNEL, "Unimplemented function ClearEvent");
-    return 0;
+    return SetEventLocked(handle, true);
 }
 
 /**
  * Creates an event
  * @param handle Reference to handle for the newly created mutex
  * @param reset_type ResetType describing how to create event
- * @return Handle to newly created object
+ * @return Newly created Event object
  */
 Event* CreateEvent(Handle& handle, const ResetType reset_type) {
     Event* evt = new Event;
 
     handle = Kernel::g_object_pool.Create(evt);
 
+    evt->locked = true;
     evt->reset_type = evt->intitial_reset_type = reset_type;
-    evt->locked = false;
 
     return evt;
 }
@@ -91,7 +95,7 @@ Event* CreateEvent(Handle& handle, const ResetType reset_type) {
 /**
  * Creates an event
  * @param reset_type ResetType describing how to create event
- * @return Handle to newly created object
+ * @return Handle to newly created Event object
  */
 Handle CreateEvent(const ResetType reset_type) {
     Handle handle;
diff --git a/src/core/hle/kernel/event.h b/src/core/hle/kernel/event.h
index c1842cb8..b1d2c6ad 100644
--- a/src/core/hle/kernel/event.h
+++ b/src/core/hle/kernel/event.h
@@ -15,8 +15,9 @@ namespace Kernel {
  * Changes whether an event is locked or not
  * @param handle Handle to event to change
  * @param locked Boolean locked value to set event
+ * @return Result of operation, 0 on success, otherwise error code
  */
-void SetEventLocked(const Handle handle, const bool locked);
+Result SetEventLocked(const Handle handle, const bool locked);
 
 /**
  * Clears an event
@@ -28,7 +29,7 @@ Result ClearEvent(Handle handle);
 /**
  * Creates an event
  * @param reset_type ResetType describing how to create event
- * @return Handle to newly created object
+ * @return Handle to newly created Event object
  */
 Handle CreateEvent(const ResetType reset_type);
 
-- 
cgit v1.2.3