From fca796265d58afa628ae80e404a7abca09cef4e7 Mon Sep 17 00:00:00 2001
From: Andrew Jackson <Andrew.Jackson@bl.uk>
Date: Mon, 17 Sep 2018 15:25:45 +0100
Subject: [PATCH] Move sync to after Null check.

---
 .../org/archive/crawler/frontier/WorkQueueFrontier.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/engine/src/main/java/org/archive/crawler/frontier/WorkQueueFrontier.java b/engine/src/main/java/org/archive/crawler/frontier/WorkQueueFrontier.java
index 8fcde273..d08cd566 100644
--- a/engine/src/main/java/org/archive/crawler/frontier/WorkQueueFrontier.java
+++ b/engine/src/main/java/org/archive/crawler/frontier/WorkQueueFrontier.java
@@ -602,17 +602,17 @@ implements Closeable,
                         }
                     }
                     readyQ = getQueueFor(key);
-                synchronized (readyQ) {
                     if (readyQ == null) {
                         // readyQ key wasn't in all queues: unexpected
                         logger.severe("Key " + key
                                 + " in readyClassQueues but not allQueues");
                         break findaqueue;
                     }
+                synchronized (readyQ) {
                     if (readyQ.getCount() == 0) {
                         // readyQ is empty and ready: it's exhausted
-                            readyQ.noteExhausted(); 
-                            readyQ.makeDirty();
+                        readyQ.noteExhausted();
+                        readyQ.makeDirty();
                         readyQ = null;
                         continue;
                     }