Notes for later

ctrueden · ctrueden · commit 1e44cf2b8ac1 · 2019-08-30T10:16:04.000-05:00
diff --git a/pom.xml b/pom.xml
@@ -102,6 +102,11 @@
 
 		<!-- NB: Deploy releases to the SciJava Maven repository. -->
 		<releaseProfiles>deploy-to-scijava</releaseProfiles>
+
+		<!-- NB: Store method parameter names into the bytecode. -->
+		<!-- Overkill, but lets us look up parameter names at runtime. -->
+		<!-- https://www.beyondjava.net/reading-java-8-method-parameter-named-reflection -->
+		<maven.compiler.parameters>true</maven.compiler.parameters>
 	</properties>
 
 	<repositories>
diff --git a/src/main/java/org/scijava/ops/util/Adapt.java b/src/main/java/org/scijava/ops/util/Adapt.java
@@ -39,6 +39,12 @@ public static class Functions {
 		private Functions() {
 		}
 
+		// TODO: when converting, add to a type map, so this computer can have its generic type discerned at will.
+		// The type map also needs to be populated by OpField and OpMethod generic types when they are discovered.
+		// The goal of the type map is to store, for each lambda op instance, what its generic type is.
+		// We know what the generic type is during op discovery, because it's the
+		// generic type of the field (in the case of OpField), or the generic type
+		// parameters of the method (in the case of OpMethod).
 		public static <I, O> Computer<I, O> asComputer(final Function<I, O> function, final Computer<O, O> copy) {
 			return (in, out) -> {
 				final O tmp = function.apply(in);
