Remove subTasksCompleted

gselzer · gselzer · commit b3fcb66c28b8 · 2024-05-06T16:26:13.000-05:00
It is no longer used
diff --git a/scijava-progress/src/main/java/org/scijava/progress/Task.java b/scijava-progress/src/main/java/org/scijava/progress/Task.java
@@ -68,9 +68,6 @@ public class Task {
 	/** Maximum of current stage */
 	private final AtomicLong max = new AtomicLong(1);
 
-	/** Number of subtasks completed */
-	private final AtomicLong subTasksCompleted = new AtomicLong(0);
-
 	/**
 	 * True iff a call to {@link Task#define(long, long)} has been made
 	 */
@@ -116,7 +113,6 @@ public void complete() {
 		if (hasElements) {
 			this.current.set(this.max.get());
 		}
-		if (parent != null) parent.recordSubtaskCompletion(this);
 	}
 
 	/**
@@ -172,6 +168,7 @@ public void define(final long elements, final long subTasks) {
 		if (elements == 0) {
 			this.hasElements = false;
 			if (this.tasks == 0) {
+				this.tasks = 1;
 				this.current.set(1);
 				this.max.set(1);
 				return;
@@ -210,18 +207,6 @@ public double progress() {
 		return totalCompletion / tasks;
 	}
 
-	/**
-	 * Iff {@code task} was a task generated by this {@link Task}, then we update
-	 * progress based on its completion.
-	 *
-	 * @param task a {@link Task} generated by this one
-	 */
-	private void recordSubtaskCompletion(final Task task) {
-		if (!subTasks.contains(task)) throw new IllegalArgumentException(
-			"Task " + task + " is not a subtask of Task " + this);
-		if (tasksDefined) subTasksCompleted.getAndIncrement();
-	}
-
 	/**
 	 * Used by the progressible {@link Object} to set the status of its
 	 * computation. This method should <b>not</b> used by any code not part of the
