Skip to content

Fix memory leak add pyobject finalizer #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jaredbroad merged 20 commits into from
Mar 28, 2019
Merged

Fix memory leak add pyobject finalizer #27

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
26e88d7
Finalizer for PyObject
amos402 Jun 23, 2018
fccd1d8
Avoid test interdependency
amos402 Jun 24, 2018
9281157
Add source to .csproj
amos402 Jun 24, 2018
481cb4f
Make sure recover the environment
amos402 Jun 24, 2018
1959a6a
Add StackTrace of C# exception
amos402 Jun 24, 2018
e7b8635
Clean up the test and interface
amos402 Jun 24, 2018
c51cebd
Update CHANGELOG.md
amos402 Jun 24, 2018
833d367
Add links to issues and PRs in CHANGELOG
civilx64 Jul 1, 2018
9aa9b1a
Mono doesn't have GC.WaitForFullGCComplete
amos402 Jun 24, 2018
338dcc1
Fixed PythonException leak
amos402 Jun 25, 2018
ef832a3
Fixed nPython.exe crash on Shutdown
amos402 Jul 10, 2018
3a9be30
Add error handler
amos402 Aug 4, 2018
eb569f2
Make collect callback without JIT
amos402 Sep 6, 2018
994449e
Add pending marker
amos402 Oct 23, 2018
fc53d94
Remove PYTHONMALLOC setting
amos402 Oct 25, 2018
8eb3591
Fix ref count error
amos402 Nov 22, 2018
3610220
Add ref count check for helping discover the bugs of decref too much
amos402 Nov 22, 2018
e4fd72e
Fix ref count error
amos402 Nov 25, 2018
0ed8bee
typo error
amos402 Nov 26, 2018
2c94c90
Version bump 1.0.5.18
Martin-Molinero Mar 27, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
typo error
  • Loading branch information
@amos402 @Martin-Molinero
amos402 authored and Martin-Molinero committed Mar 27, 2019
commit 0ed8bee39f3b0715e8fe7fb96bf161ce9ec00f4d
4 changes: 2 additions & 2 deletions src/embed_tests/TestFinalizer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@ public void ValidateRefCount()
Runtime.Runtime.XIncref(e.Handle);
return false;
};
Finalizer.Instance.IncorrectRefCntResovler += handler;
Finalizer.Instance.IncorrectRefCntResolver += handler;
try
{
ptr = CreateStringGarbage();
Expand All @@ -223,7 +223,7 @@ public void ValidateRefCount()
}
finally
{
Finalizer.Instance.IncorrectRefCntResovler -= handler;
Finalizer.Instance.IncorrectRefCntResolver -= handler;
}
}

Expand Down
6 changes: 3 additions & 3 deletions src/runtime/finalizer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ public IncorrectRefCountException(IntPtr ptr)
}

public delegate bool IncorrectRefCntHandler(object sender, IncorrectFinalizeArgs e);
public event IncorrectRefCntHandler IncorrectRefCntResovler;
public event IncorrectRefCntHandler IncorrectRefCntResolver;
public bool ThrowIfUnhandleIncorrectRefCount { get; set; } = true;

#endregion
Expand Down Expand Up @@ -304,9 +304,9 @@ private void ValidateRefCount()
ImpactedObjects = indexer[handle]
};
bool handled = false;
if (IncorrectRefCntResovler != null)
if (IncorrectRefCntResolver != null)
{
var funcList = IncorrectRefCntResovler.GetInvocationList();
var funcList = IncorrectRefCntResolver.GetInvocationList();
foreach (IncorrectRefCntHandler func in funcList)
{
if (func(this, args))
Expand Down