[Tizen.Core] Fix managing native handle of Event (#6337)
authorhjhun <36876573+hjhun@users.noreply.github.com>
Fri, 20 Sep 2024 04:43:02 +0000 (13:43 +0900)
committerGitHub <noreply@github.com>
Fri, 20 Sep 2024 04:43:02 +0000 (13:43 +0900)
The native handle should not be null before calling Dispose() method.

Signed-off-by: Hwankyu Jhun <h.jhun@samsung.com>
src/Tizen.Core/Tizen.Core/Event.cs
src/Tizen.Core/Tizen.Core/Task.cs

index b0a473dd5e6964d7f4938a8217968911168a6fc4..89c3d84af0311ee53ffcd6b6a72ad8bd6b543cbc 100644 (file)
@@ -135,7 +135,7 @@ namespace Tizen.Core
             eventObject.Handle = IntPtr.Zero;
         }
 
-        internal IntPtr Handle { get { return _handle; } set { _handle = value; } }
+        internal IntPtr Handle { get { return _handle; } }
         internal IntPtr Source { get; set; }
         internal int Id { get; set; }
 
index 0be1f06fd5e4653e29a48d039e9eae8ff55579f2..5b05fbb1e5841cce75ce07343a43296b5ac97bb2 100644 (file)
@@ -402,7 +402,7 @@ namespace Tizen.Core
                 throw new ArgumentNullException(nameof(coreEvent));
             }
 
-            if (coreEvent.Handle == IntPtr.Zero)
+            if (coreEvent.Source != IntPtr.Zero)
             {
                 throw new ArgumentException("The event is already added");
             }
@@ -428,7 +428,6 @@ namespace Tizen.Core
                 }
 
                 coreEvent.Source = handle;
-                coreEvent.Handle = IntPtr.Zero;
             }
         }