Assign True instead of None to __clear_reentry_guard__ · pythonnet/pythonnet@8e0333d · GitHub

pythonnet / pythonnet Public

• Notifications You must be signed in to change notification settings
• Fork 780
• Star 5.5k

• Code
• Issues 153
• Pull requests 12
• Discussions
• Actions
• Projects
• Wiki
• Security and quality
• Insights

Additional navigation options

Commit 8e0333d

Browse filesBrowse the repository at this point in the historyBrowse files

filmor

committed

Assign True instead of None to __clear_reentry_guard__

Not at all sure why this helps, but when assigning `None` instead, the object is gone at the time of garbage collection.

1 parent e244503 commit 8e0333d
Copy full SHA for 8e0333d

1 file changed

+6-1Lines changed: 6 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• src/runtime/Types
  • ClassBase.cs

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎src/runtime/Types/ClassBase.cs‎

Copy file name to clipboardExpand all lines: src/runtime/Types/ClassBase.cs

+6-1Lines changed: 6 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -392,7 +392,12 @@ internal static unsafe int BaseUnmanagedClear(BorrowedReference ob)
392	392	using var dict = Runtime.PyObject_GenericGetDict(ob);
393	393	if (Runtime.PyMapping_HasKey(dict.Borrow(), PyIdentifier.__clear_reentry_guard__) != 0)
394	394	return 0;
395		- int res = Runtime.PyDict_SetItem(dict.Borrow(), PyIdentifier.__clear_reentry_guard__, Runtime.None);
	395	+
	396	+ int res = Runtime.PyDict_SetItem(
	397	+ dict.Borrow(),
	398	+ PyIdentifier.__clear_reentry_guard__,
	399	+ Runtime.PyTrue
	400	+ );
396	401	if (res != 0) return res;
397	402
398	403	res = clear(ob);

0 commit comments

Comments

0 (0)