From: David ‘Bombe’ Roden <bombe@pterodactylus.net>
Date: Thu, 27 Mar 2014 16:09:41 +0000 (+0100)
Subject: Move waiting for next reaction to start into its own method.
X-Git-Tag: v2~242
X-Git-Url: https://git.pterodactylus.net/?a=commitdiff_plain;h=078ecb834e21239ec0dc17b57a29a4efa9eede66;p=rhynodge.git

Move waiting for next reaction to start into its own method.
---

diff --git a/src/main/java/net/pterodactylus/rhynodge/engine/Engine.java b/src/main/java/net/pterodactylus/rhynodge/engine/Engine.java
index 2ff7666..bb42289 100644
--- a/src/main/java/net/pterodactylus/rhynodge/engine/Engine.java
+++ b/src/main/java/net/pterodactylus/rhynodge/engine/Engine.java
@@ -126,14 +126,7 @@ public class Engine extends AbstractExecutionThreadService {
 			long waitTime = nextReaction.get().getNextTime() - System.currentTimeMillis();
 			logger.debug(format("Time to wait for next Reaction: %d millseconds.", waitTime));
 			if (waitTime > 0) {
-				synchronized (reactions) {
-					try {
-						logger.info(format("Waiting until %tc.", nextReaction.get().getNextTime()));
-						reactions.wait(waitTime);
-					} catch (InterruptedException ie1) {
-						/* we’re looping! */
-					}
-				}
+				waitForNextReactionToStart(nextReaction, waitTime);
 
 				/* re-start loop to check for new reactions. */
 				continue;
@@ -196,6 +189,17 @@ public class Engine extends AbstractExecutionThreadService {
 		}
 	}
 
+	private void waitForNextReactionToStart(Optional<NextReaction> nextReaction, long waitTime) {
+		synchronized (reactions) {
+			try {
+				logger.info(format("Waiting until %tc.", nextReaction.get().getNextTime()));
+				reactions.wait(waitTime);
+			} catch (InterruptedException ie1) {
+				/* we’re looping! */
+			}
+		}
+	}
+
 	private Optional<NextReaction> getNextReaction() {
 		while (isRunning()) {
 			synchronized (reactions) {