Simplify nonCirculant conditional

scijava · Apr 4, 2024 · e008a8a · e008a8a
1 parent 3b27e31
commit e008a8a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/scijava-ops-image/src/main/java/org/scijava/ops/image/deconvolve/PadAndRichardsonLucy.java b/scijava-ops-image/src/main/java/org/scijava/ops/image/deconvolve/PadAndRichardsonLucy.java
@@ -226,15 +226,14 @@ public RandomAccessibleInterval<O> apply(RandomAccessibleInterval<I> input,
 		// out of bounds factory will be different depending on if circulant or
 		// non-circulant is used
 		if (obfInput == null) {
-
-			if (!nonCirculant) {
-				obfInput = new OutOfBoundsMirrorFactory<>(
-					OutOfBoundsMirrorFactory.Boundary.SINGLE);
-			}
-			else if (nonCirculant) {
+			if (nonCirculant) {
 				obfInput = new OutOfBoundsConstantValueFactory<>(Util
 					.getTypeFromInterval(input).createVariable());
 			}
+			else {
+				obfInput = new OutOfBoundsMirrorFactory<>(
+					OutOfBoundsMirrorFactory.Boundary.SINGLE);
+			}
 		}
 
 		// default to no acceleration

diff --git a/scijava-ops-image/src/main/java/org/scijava/ops/image/deconvolve/PadAndRichardsonLucyTV.java b/scijava-ops-image/src/main/java/org/scijava/ops/image/deconvolve/PadAndRichardsonLucyTV.java
@@ -219,15 +219,14 @@ public RandomAccessibleInterval<O> apply(RandomAccessibleInterval<I> input,
 		// out of bounds factory will be different depending on if circulant or
 		// non-circulant is used
 		if (obfInput == null) {
-
-			if (!nonCirculant) {
-				obfInput = new OutOfBoundsMirrorFactory<>(
-					OutOfBoundsMirrorFactory.Boundary.SINGLE);
-			}
-			else if (nonCirculant) {
+			if (nonCirculant) {
 				obfInput = new OutOfBoundsConstantValueFactory<>(Util
 					.getTypeFromInterval(input).createVariable());
 			}
+			else {
+				obfInput = new OutOfBoundsMirrorFactory<>(
+					OutOfBoundsMirrorFactory.Boundary.SINGLE);
+			}
 		}
 
 		// default to no acceleration