Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit12501f8

Browse files
committed
make methods of PyObject inherited from its base C# classes GIL-safe
fixes#1642
1 parent7637884 commit12501f8

File tree

4 files changed

+25
-6
lines changed

4 files changed

+25
-6
lines changed

‎CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ details about the cause of the failure
4747
- floating point values passed from Python are no longer silently truncated
4848
when .NET expects an integer[#1342][i1342]
4949
- More specific error messages for method argument mismatch
50+
- members of`PyObject` inherited from`System.Object and`DynamicObject` now autoacquire GIL
5051
- BREAKING: when inheriting from .NET types in Python if you override`__init__` you
5152
must explicitly call base constructor using`super().__init__(.....)`. Not doing so will lead
5253
to undefined behavior.

‎src/embed_tests/TestPyObject.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,13 @@ public void GetAttrDefault_IgnoresAttributeErrorOnly()
9494
);
9595
Assert.AreEqual(Exceptions.TypeError,typeErrResult.Type);
9696
}
97+
98+
// regression test from https://github.com/pythonnet/pythonnet/issues/1642
99+
[Test]
100+
publicvoidInheritedMethodsAutoacquireGIL()
101+
{
102+
PythonEngine.Exec("from System import String\nString.Format('{0},{1}', 1, 2)");
103+
}
97104
}
98105

99106
publicclassPyObjectTestMethods

‎src/runtime/InternString.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ public static void Initialize()
3939
{
4040
NewReferencepyStr=Runtime.PyUnicode_InternFromString(name);
4141
varop=newPyString(pyStr.StealOrThrow());
42-
Debug.Assert(name==op.ToString());
42+
Debug.Assert(name==op.As<string>());
4343
SetIntern(name,op);
4444
varfield=type.GetField("f"+name,PyIdentifierFieldFlags)!;
4545
field.SetValue(null,op.rawPtr);

‎src/runtime/PythonTypes/PyObject.cs

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1056,6 +1056,7 @@ public PyList Dir()
10561056
/// </remarks>
10571057
publicoverridestring?ToString()
10581058
{
1059+
usingvar_=Py.GIL();
10591060
usingvarstrval=Runtime.PyObject_Str(obj);
10601061
returnRuntime.GetManagedString(strval.BorrowOrThrow());
10611062
}
@@ -1072,7 +1073,11 @@ public PyList Dir()
10721073
/// Return true if this object is equal to the given object. This
10731074
/// method is based on Python equality semantics.
10741075
/// </remarks>
1075-
publicoverrideboolEquals(objecto)=>Equals(oasPyObject);
1076+
publicoverrideboolEquals(objecto)
1077+
{
1078+
usingvar_=Py.GIL();
1079+
returnEquals(oasPyObject);
1080+
}
10761081

10771082
publicvirtualboolEquals(PyObject?other)
10781083
{
@@ -1101,6 +1106,7 @@ public virtual bool Equals(PyObject? other)
11011106
/// </remarks>
11021107
publicoverrideintGetHashCode()
11031108
{
1109+
usingvar_=Py.GIL();
11041110
nintpyHash=Runtime.PyObject_Hash(obj);
11051111
if(pyHash==-1&&Exceptions.ErrorOccurred())
11061112
{
@@ -1135,12 +1141,14 @@ public long Refcount
11351141

11361142
publicoverrideboolTryGetMember(GetMemberBinderbinder,outobject?result)
11371143
{
1144+
usingvar_=Py.GIL();
11381145
result=CheckNone(this.GetAttr(binder.Name));
11391146
returntrue;
11401147
}
11411148

11421149
publicoverrideboolTrySetMember(SetMemberBinderbinder,object?value)
11431150
{
1151+
usingvar_=Py.GIL();
11441152
usingvarnewVal=Converter.ToPythonDetectType(value);
11451153
intr=Runtime.PyObject_SetAttrString(obj,binder.Name,newVal.Borrow());
11461154
if(r<0)
@@ -1234,6 +1242,7 @@ private static NewReference GetPythonObject(object? target)
12341242

12351243
publicoverrideboolTryInvokeMember(InvokeMemberBinderbinder,object?[]args,outobject?result)
12361244
{
1245+
usingvar_=Py.GIL();
12371246
if(this.HasAttr(binder.Name)&&this.GetAttr(binder.Name).IsCallable())
12381247
{
12391248
PyTuple?pyargs=null;
@@ -1258,6 +1267,7 @@ public override bool TryInvokeMember(InvokeMemberBinder binder, object?[] args,
12581267

12591268
publicoverrideboolTryInvoke(InvokeBinderbinder,object?[]args,outobject?result)
12601269
{
1270+
usingvar_=Py.GIL();
12611271
if(this.IsCallable())
12621272
{
12631273
PyTuple?pyargs=null;
@@ -1282,6 +1292,7 @@ public override bool TryInvoke(InvokeBinder binder, object?[] args, out object?
12821292

12831293
publicoverrideboolTryConvert(ConvertBinderbinder,outobject?result)
12841294
{
1295+
usingvar_=Py.GIL();
12851296
// always try implicit conversion first
12861297
if(Converter.ToManaged(this.obj,binder.Type,outresult,false))
12871298
{
@@ -1307,6 +1318,7 @@ public override bool TryConvert(ConvertBinder binder, out object? result)
13071318

13081319
publicoverrideboolTryBinaryOperation(BinaryOperationBinderbinder,objectarg,outobject?result)
13091320
{
1321+
usingvar_=Py.GIL();
13101322
NewReferenceres;
13111323
if(!(argisPyObject))
13121324
{
@@ -1419,6 +1431,7 @@ public override bool TryBinaryOperation(BinaryOperationBinder binder, object arg
14191431

14201432
publicoverrideboolTryUnaryOperation(UnaryOperationBinderbinder,outobject?result)
14211433
{
1434+
usingvar_=Py.GIL();
14221435
intr;
14231436
NewReferenceres;
14241437
switch(binder.Operation)
@@ -1463,10 +1476,8 @@ public override bool TryUnaryOperation(UnaryOperationBinder binder, out object?
14631476
/// <returns>A sequence that contains dynamic member names.</returns>
14641477
publicoverrideIEnumerable<string>GetDynamicMemberNames()
14651478
{
1466-
foreach(PyObjectpyObjinDir())
1467-
{
1468-
yieldreturnpyObj.ToString()!;
1469-
}
1479+
usingvar_=Py.GIL();
1480+
returnDir().Select(pyObj=>pyObj.ToString()!).ToArray();
14701481
}
14711482

14721483
voidISerializable.GetObjectData(SerializationInfoinfo,StreamingContextcontext)

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp