Skip to content

Commit

Permalink
Remove unneeded code
Browse files Browse the repository at this point in the history
  • Loading branch information
jmao-denver committed Nov 27, 2024
1 parent d6be74b commit 159e78e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
5 changes: 0 additions & 5 deletions src/main/java/org/jpy/PyObject.java
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,6 @@ public class PyObject implements AutoCloseable {

private static final AtomicReference<Thread> CLEANUP_THREAD = new AtomicReference<>();

private static final boolean CLEANUP_ON_INIT = Boolean.parseBoolean(System.getProperty("PyObject.cleanup_on_init", "false"));

private static final boolean CLEANUP_ON_THREAD = Boolean.parseBoolean(System.getProperty("PyObject.cleanup_on_thread", "true"));

private static void startCleanupThread() {
Expand Down Expand Up @@ -71,9 +69,6 @@ public static int cleanup() {
PyObject(long pointer, boolean fromJNI) {
state = new PyObjectState(pointer);
if (fromJNI) {
if (CLEANUP_ON_INIT) {
REFERENCES.threadSafeCleanup(); // only performs *one* cleanup
}
if (CLEANUP_ON_THREAD) {
// ensures that we've only started after python has been started, and we know there is something to cleanup
startCleanupThread();
Expand Down
6 changes: 1 addition & 5 deletions src/main/java/org/jpy/PyObjectReferences.java
Original file line number Diff line number Diff line change
Expand Up @@ -76,11 +76,7 @@ private Reference<PyObject> asRef(PyObject pyObject) {
/**
* This should *only* be invoked through the proxy, or when we *know* we have the GIL.
*/
public int threadSafeCleanup() {
return threadSafeCleanup(buffer);
}

private synchronized int threadSafeCleanup(long[] buffer) {
public synchronized int threadSafeCleanup() {
return PyLib.ensureGil(() -> {
int index = 0;
while (index < buffer.length) {
Expand Down

0 comments on commit 159e78e

Please sign in to comment.