Rename to GetPythonThreadID

gpetrou · gpetrou · commit 6bcda676be66 · 2021-01-21T07:42:25.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,7 +11,7 @@ This document follows the conventions laid out in [Keep a CHANGELOG][].
 
 -   Ability to instantiate new .NET arrays using `Array[T](dim1, dim2, ...)` syntax
 -   Python operator method will call C# operator method for supported binary and unary operators ([#1324][p1324]).
--   Add GetNativeThreadID and Interrupt methods in PythonEngine
+-   Add GetPythonThreadID and Interrupt methods in PythonEngine
 
 ### Changed
 -   Drop support for Python 2, 3.4, and 3.5
diff --git a/src/embed_tests/TestInterrupt.cs b/src/embed_tests/TestInterrupt.cs
@@ -31,12 +31,12 @@ public void Dispose()
         public void InterruptTest()
         {
             int runSimpleStringReturnValue = int.MinValue;
-            ulong nativeThreadID = ulong.MinValue;
+            ulong pythonThreadID = ulong.MinValue;
             Task.Factory.StartNew(() =>
             {
                 using (Py.GIL())
                 {
-                    nativeThreadID = PythonEngine.GetNativeThreadID();
+                    pythonThreadID = PythonEngine.GetPythonThreadID();
                     runSimpleStringReturnValue = PythonEngine.RunSimpleString(@"
 import time
 
@@ -47,11 +47,11 @@ import time
 
             Thread.Sleep(200);
 
-            Assert.AreNotEqual(ulong.MinValue, nativeThreadID);
+            Assert.AreNotEqual(ulong.MinValue, pythonThreadID);
 
             using (Py.GIL())
             {
-                int interruptReturnValue = PythonEngine.Interrupt(nativeThreadID);
+                int interruptReturnValue = PythonEngine.Interrupt(pythonThreadID);
                 Assert.AreEqual(1, interruptReturnValue);
             }
 
diff --git a/src/runtime/pythonengine.cs b/src/runtime/pythonengine.cs
@@ -568,13 +568,13 @@ public static void Exec(string code, IntPtr? globals = null, IntPtr? locals = nu
         }
 
         /// <summary>
-        /// Gets the native thread ID.
+        /// Gets the Python thread ID.
         /// </summary>
-        /// <returns>The native thread ID.</returns>
-        public static ulong GetNativeThreadID()
+        /// <returns>The Python thread ID.</returns>
+        public static ulong GetPythonThreadID()
         {
             dynamic threading = Py.Import("threading");
-            return threading.get_ident();
+            return threading.InvokeMethod("get_ident");
         }
 
         /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -31,12 +31,12 @@ public void Dispose()`
`31`	`31`	`public void InterruptTest()`
`32`	`32`	`{`
`33`	`33`	`int runSimpleStringReturnValue = int.MinValue;`
`34`		`- ulong nativeThreadID = ulong.MinValue;`
	`34`	`+ ulong pythonThreadID = ulong.MinValue;`
`35`	`35`	`Task.Factory.StartNew(() =>`
`36`	`36`	`{`
`37`	`37`	`using (Py.GIL())`
`38`	`38`	`{`
`39`		`- nativeThreadID = PythonEngine.GetNativeThreadID();`
	`39`	`+ pythonThreadID = PythonEngine.GetPythonThreadID();`
`40`	`40`	`runSimpleStringReturnValue = PythonEngine.RunSimpleString(@"`
`41`	`41`	`import time`
`42`	`42`
`@@ -47,11 +47,11 @@ import time`
`47`	`47`
`48`	`48`	`Thread.Sleep(200);`
`49`	`49`
`50`		`- Assert.AreNotEqual(ulong.MinValue, nativeThreadID);`
	`50`	`+ Assert.AreNotEqual(ulong.MinValue, pythonThreadID);`
`51`	`51`
`52`	`52`	`using (Py.GIL())`
`53`	`53`	`{`
`54`		`- int interruptReturnValue = PythonEngine.Interrupt(nativeThreadID);`
	`54`	`+ int interruptReturnValue = PythonEngine.Interrupt(pythonThreadID);`
`55`	`55`	`Assert.AreEqual(1, interruptReturnValue);`
`56`	`56`	`}`
`57`	`57`
Original file line number	Diff line number	Diff line change
`@@ -568,13 +568,13 @@ public static void Exec(string code, IntPtr? globals = null, IntPtr? locals = nu`
`568`	`568`	`}`
`569`	`569`
`570`	`570`	`/// <summary>`
`571`		`- /// Gets the native thread ID.`
	`571`	`+ /// Gets the Python thread ID.`
`572`	`572`	`/// </summary>`
`573`		`- /// <returns>The native thread ID.</returns>`
`574`		`- public static ulong GetNativeThreadID()`
	`573`	`+ /// <returns>The Python thread ID.</returns>`
	`574`	`+ public static ulong GetPythonThreadID()`
`575`	`575`	`{`
`576`	`576`	`dynamic threading = Py.Import("threading");`
`577`		`- return threading.get_ident();`
	`577`	`+ return threading.InvokeMethod("get_ident");`
`578`	`578`	`}`
`579`	`579`
`580`	`580`	`/// <summary>`