commit 62c1857bf635e8d49124456a5e9eadae571005d2
author Oliver Lietz <olli@apache.org> Sat Jan 26 16:02:27 2019 +0100
committer Oliver Lietz <olli@apache.org> Sat Jan 26 16:02:27 2019 +0100
tree 6eb73a421e89e8340905a0ba0b59ee1e8607259a
parent ea8bc7f1347ad850b538e189fe434fb915b4354d

move length check to ClamUtil

src/main/java/org/apache/sling/clam/internal/ClamUtil.java

diff --git a/src/main/java/org/apache/sling/clam/internal/ClamUtil.java b/src/main/java/org/apache/sling/clam/internal/ClamUtil.java
index bd0ce54..1a40c35 100644
--- a/src/main/java/org/apache/sling/clam/internal/ClamUtil.java
+++ b/src/main/java/org/apache/sling/clam/internal/ClamUtil.java
@@ -75,4 +75,11 @@
         return properties;
     }
 
+    public static boolean checkLength(final long length, final long maxLength) {
+        if (maxLength == -1) {
+            return true;
+        }
+        return length <= maxLength;
+    }
+
 }

src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java

diff --git a/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java b/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java
index b7585d7..85c8364 100644
--- a/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java
+++ b/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java
@@ -34,7 +34,6 @@
 import org.apache.jackrabbit.oak.spi.state.NodeStore;
 import org.apache.sling.api.resource.ResourceResolver;
 import org.apache.sling.api.resource.ResourceResolverFactory;
-import org.apache.sling.clam.internal.ClamUtil;
 import org.apache.sling.commons.threads.ThreadPool;
 import org.apache.sling.commons.threads.ThreadPoolManager;
 import org.apache.sling.event.jobs.JobManager;
@@ -52,6 +51,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import static org.apache.sling.clam.internal.ClamUtil.checkLength;
 import static org.apache.sling.clam.internal.ClamUtil.properties;
 import static org.apache.sling.clam.internal.ClamUtil.scanJobTopic;
 
@@ -177,13 +177,6 @@
         return paths;
     }
 
-    private boolean checkLength(final long length, final long maxLength) {
-        if (maxLength == -1) {
-            return true;
-        }
-        return length <= maxLength;
-    }
-
     private Set<String> concat(final Set<String> a, Set<String> b) {
         final Set<String> set = new HashSet<>(a);
         set.addAll(b);