Apply patch https://git.eclipse.org/c/56954/7 to org.eclipse.core.jobs

GoogleCloudPlatform · Aug 22, 2017 · 1dc13c9 · 1dc13c9
1 parent e0c8002
commit 1dc13c9
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 37 deletions.
diff --git a/...arty/patches/mars/org.eclipse.core.jobs/src/org/eclipse/core/internal/jobs/ThreadJob.java b/...arty/patches/mars/org.eclipse.core.jobs/src/org/eclipse/core/internal/jobs/ThreadJob.java
@@ -288,9 +288,11 @@ else if (state != Job.NONE)
 		} finally {
 			if (interrupted)
 				Thread.currentThread().interrupt();
-			//only update the lock state if we ended up using the thread job that was given to us
-			waitEnd(threadJob, threadJob == result, monitor);
-			if (threadJob == result) {
+			ISchedulingRule resultRule = result.getRule();
+			ISchedulingRule threadJobRule = threadJob.getRule();
+			boolean containsRule = resultRule != null && threadJobRule != null && resultRule.contains(threadJobRule);
+			waitEnd(threadJob, containsRule, monitor);
+			if (containsRule) {
 				if (waiting)
 					manager.implicitJobs.removeWaiting(threadJob);
 			}

diff --git a/...arty/patches/neon/org.eclipse.core.jobs/src/org/eclipse/core/internal/jobs/ThreadJob.java b/...arty/patches/neon/org.eclipse.core.jobs/src/org/eclipse/core/internal/jobs/ThreadJob.java
@@ -288,9 +288,11 @@ else if (state != Job.NONE)
 		} finally {
 			if (interrupted)
 				Thread.currentThread().interrupt();
-			//only update the lock state if we ended up using the thread job that was given to us
-			waitEnd(threadJob, threadJob == result, monitor);
-			if (threadJob == result) {
+			ISchedulingRule resultRule = result.getRule();
+			ISchedulingRule threadJobRule = threadJob.getRule();
+			boolean containsRule = resultRule != null && threadJobRule != null && resultRule.contains(threadJobRule);
+			waitEnd(threadJob, containsRule, monitor);
+			if (containsRule) {
 				if (waiting)
 					manager.implicitJobs.removeWaiting(threadJob);
 			}

diff --git a/third_party/patches/oxygen/org.eclipse.core.jobs/pom.xml b/third_party/patches/oxygen/org.eclipse.core.jobs/pom.xml
diff --git a/...ty/patches/oxygen/org.eclipse.core.jobs/src/org/eclipse/core/internal/jobs/ThreadJob.java b/...ty/patches/oxygen/org.eclipse.core.jobs/src/org/eclipse/core/internal/jobs/ThreadJob.java
@@ -303,9 +303,11 @@ else if (state != Job.NONE)
 				manager.getLockManager().removeLockWaitThread(currentThread, threadJob.getRule());
 			}
 		} finally {
-			//only update the lock state if we ended up using the thread job that was given to us
-			waitEnd(threadJob, threadJob == result, monitor);
-			if (threadJob == result) {
+			ISchedulingRule resultRule = result.getRule();
+			ISchedulingRule threadJobRule = threadJob.getRule();
+			boolean containsRule = resultRule != null && threadJobRule != null && resultRule.contains(threadJobRule);
+			waitEnd(threadJob, containsRule, monitor);
+			if (containsRule) {
 				if (waiting)
 					manager.implicitJobs.removeWaiting(threadJob);
 			}