Initial support for LongRunningCSharpRegion.cs

budgetdevv · budgetdevv · commit 2779e038b4e2 · 2024-03-21T02:06:24.000+08:00
diff --git a/PythonNETExtensions/Core/LongRunningCSharpRegion.cs b/PythonNETExtensions/Core/LongRunningCSharpRegion.cs
@@ -1,20 +1,21 @@
-﻿using System;
-using Python.Runtime;
+﻿using System.Runtime.CompilerServices;
 
 namespace PythonNETExtensions.Core
 {
-    public readonly struct LongRunningCSharpRegion: IDisposable
+    public readonly ref struct LongRunningCSharpRegion
     {
-        private readonly nint Handle;
+        private readonly ref PythonHandle Handle;
         
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public LongRunningCSharpRegion()
         {
-            Handle = PythonEngine.BeginAllowThreads();
+            Handle.Dispose();
         }
         
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public void Dispose()
         {
-            PythonEngine.EndAllowThreads(Handle);
+            Handle = new PythonHandle();
         }
     }
 }
diff --git a/PythonNETExtensions/Core/PythonHandle.cs b/PythonNETExtensions/Core/PythonHandle.cs
@@ -42,11 +42,13 @@ static PythonHandle()
                 .GetValue(null)!;
         }
         
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public PythonHandle()
         {
             GILState = PyGILState_Ensure();
         }
         
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public void Dispose()
         {
             PyGILState_Release(GILState);

Original file line number	Diff line number	Diff line change
`@@ -1,20 +1,21 @@`
`1`		`-using System;`
`2`		`-using Python.Runtime;`
	`1`	`+using System.Runtime.CompilerServices;`
`3`	`2`
`4`	`3`	`namespace PythonNETExtensions.Core`
`5`	`4`	`{`
`6`		`- public readonly struct LongRunningCSharpRegion: IDisposable`
	`5`	`+ public readonly ref struct LongRunningCSharpRegion`
`7`	`6`	`{`
`8`		`- private readonly nint Handle;`
	`7`	`+ private readonly ref PythonHandle Handle;`
`9`	`8`
	`9`	`+ [MethodImpl(MethodImplOptions.AggressiveInlining)]`
`10`	`10`	`public LongRunningCSharpRegion()`
`11`	`11`	`{`
`12`		`- Handle = PythonEngine.BeginAllowThreads();`
	`12`	`+ Handle.Dispose();`
`13`	`13`	`}`
`14`	`14`
	`15`	`+ [MethodImpl(MethodImplOptions.AggressiveInlining)]`
`15`	`16`	`public void Dispose()`
`16`	`17`	`{`
`17`		`- PythonEngine.EndAllowThreads(Handle);`
	`18`	`+ Handle = new PythonHandle();`
`18`	`19`	`}`
`19`	`20`	`}`
`20`	`21`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,11 +42,13 @@ static PythonHandle()`
`42`	`42`	`.GetValue(null)!;`
`43`	`43`	`}`
`44`	`44`
	`45`	`+ [MethodImpl(MethodImplOptions.AggressiveInlining)]`
`45`	`46`	`public PythonHandle()`
`46`	`47`	`{`
`47`	`48`	`GILState = PyGILState_Ensure();`
`48`	`49`	`}`
`49`	`50`
	`51`	`+ [MethodImpl(MethodImplOptions.AggressiveInlining)]`
`50`	`52`	`public void Dispose()`
`51`	`53`	`{`
`52`	`54`	`PyGILState_Release(GILState);`