Always require a name when we look for ops

Treiblesschorle · ctrueden · commit 81cd296a68a8 · 2019-04-18T12:16:31.000-05:00
diff --git a/src/main/java/org/scijava/ops/OpService.java b/src/main/java/org/scijava/ops/OpService.java
@@ -190,16 +190,6 @@ public <T> T findOpInstance(final String opName, final Nil<T> specialType, final
 		}
 	}
 	
-	public <T> T findOp(final Nil<T> specialType, final Nil<?>[] inTypes, final Nil<?>[] outTypes,
-			final Object... secondaryArgs) {
-		return findOpInstance(null, specialType, inTypes, outTypes, secondaryArgs);
-	}
-
-	public <T> T findOp(final Nil<T> specialType, final Nil<?>[] inTypes, final Nil<?> outType,
-			final Object... secondaryArgs) {
-		return findOpInstance(null, specialType, inTypes, new Nil[] { outType }, secondaryArgs);
-	}
-
 	public <T> T findOp(final String opName, final Nil<T> specialType, final Nil<?>[] inTypes, final Nil<?>[] outTypes,
 			final Object... secondaryArgs) {
 		return findOpInstance(opName, specialType, inTypes, outTypes, secondaryArgs);
diff --git a/src/test/java/org/scijava/ops/OpsTest.java b/src/test/java/org/scijava/ops/OpsTest.java
@@ -75,14 +75,6 @@ public void binaryFunction() {
 				nilDouble//
 		);
 		assert 3.0 == addFunction.apply(1.0, 2.0);
-
-		// look up a specific implementation
-		MathAddDoublesFunction add = ops().findOp( //
-				new Nil<MathAddDoublesFunction>(){},
-				new Nil[] { nilDouble, nilDouble }, //
-				nilDouble//
-		);
-		assert 86.0 == add.apply(10.0, 76.0);
 	}
 
 	@Test
