todo

assira.lambda/src/main/java/net/ranides/assira/reflection/impl/CPInspector.java

@@ -24,11 +24,6 @@ import java.util.stream.StreamSupport;
  */
 public final class CPInspector {
 
-    // @todo (assira.lambda #1) ConstantPool wrapper identical to UnsafeUtils
-    // wrapper should use compatibility constructs from link below to construct correct wrapper for corresponding JDK
-    // https://github.com/jhalterman/typetools/pull/49/files
-
-
     private CPInspector() {
         /* utility class */
     }
@@ -128,6 +123,8 @@ public final class CPInspector {
 
     // Cały cyrk z tym wrapperem tylko po to, aby nie mieć 20 warningów z powodu użycia klasy "sun.reflect".
     // Mamy jedno użycie (tutaj). A reszta kodu używa typu niejawnie, używając pola "v"
+    //
+    // https://git.ranides.net/projects/assira/issues/25
     private static final class CPP {
 
         public final jdk.internal.reflect.ConstantPool v;

assira/src/test/java/net/ranides/assira/functional/CompositorTest.java

@@ -760,8 +760,6 @@ public class CompositorTest {
         assertEquals("#2", function2.apply("e", "z"));
     }
 
-    // TODO coverage: #source() #consumer()
-
     @AllArgsConstructor
     private static final class AtomicDouble extends Number {
 

assira/src/test/java/net/ranides/assira/reflection/impl/AClassesTest.java

@@ -161,7 +161,6 @@ public class AClassesTest {
         List<String> exp2 = IOStrings.readLines(TestFiles.FIELDS_2, Charsets.UTF8);
         List<String> exp3 = IOStrings.readLines(TestFiles.FIELDS_3, Charsets.UTF8);
 
-        // @todo (assira #1) JDK dependent!
         TestReflective.assertEquals(exp1, iclist().fields().names());
         TestReflective.assertEquals(exp2, iclist().fields().require(PRIVATE).names());
         TestReflective.assertEquals(exp3, iclist().fields().require(DECLARED).names());

assira/src/test/java/net/ranides/assira/reflection/impl/RConstructorTest.java

@@ -159,11 +159,6 @@ public class RConstructorTest {
     @Ignore("very bad reflection")
     @Test
     public void testMatchGeneric() {
-        // @todo (assira #6) reflective: generics: match
-        // mamy konstruktor public <R extends Number> Record(R param1, R param2) {
-        // nie wiem czy istnieje jakakolwiek szansa, żeby w oparciu o type-bounds match zaimplementować
-        // chyba takie "super-elastic" metody będą po prostu bezużyteczne w przypadku reflective IClass
-        // normalnie to dużo prostsze przypadki nie przechodzą, więc płaczu nie ma
         IConstructors<?> list1 = ic.constructors().require(IArguments.typeinfo(Number.class, Number.class));
         IConstructors<?> list2 = ic.constructors().matches(IClasses.typeinfo(Number.class, Number.class));
         IConstructors<?> list3 = ic.constructors().accepts(0,0);

assira/src/test/java/net/ranides/assira/reflection/impl/RWClassTest.java

@@ -32,17 +32,10 @@ public class RWClassTest {
         assertEquals(Arrays.asList(), ic1.interfaces().stream().distinct().list());
         assertEquals("? super java.util.ArrayList<java.lang.Float>", ic1.reflective().toString());
         assertEquals(Object.class, ic1.raw());
-        // @todo (assira #6) reflective: generics: isInstance dla wildcards
-        // rzecz w tym, że my tu mamy unresolved params, bo przekazujemy objekt (i type erasure wchodzi)
-        // a to oznacza, że NIE powinno tutaj nic matchować na ten moment
-        // o ile nie zrobimy obsługi prawdziwej dla atrybutu IAttribute.RESOLVED
-        // tylko w takim wypadku możemy być "tolerancyjniejsi" względem erasure'owanych typów
-        // i je przepuszczać
-        
-        // test dla wildcardów powinien opierać się na isSuper/isSubclass dla TypeToken jako argumentu
-        // bo on zawiera całe info na temat typu testowanego
-//        System.out.printf("%s%n", ic1);
-//        assertTrue(ic1.isInstance(new ArrayList<Float>())); 
+
+        // https://git.ranides.net/projects/assira/issues/24
+        // System.out.printf("%s%n", ic1);
+        // assertTrue(ic1.isInstance(new ArrayList<Float>()));
 
         assertFalse(ic1.isInstance(new ArrayList<Double>()));
         assertFalse(ic1.isInstance(new java.util.LinkedList<Float>()));

assira1/src/main/java/net/ranides/assira/collection_map/CrossCompositeMap.java

@@ -17,7 +17,6 @@ import java.util.Map.Entry;
  * @param <KY> 
  * @param <V> 
  * @author ranides
- * @todo (migration) assira
  */
 @SuppressWarnings("unchecked")
 public class CrossCompositeMap<KX, KY, V> implements CrossMap<KX, KY, V>  {

assira1/src/main/java/net/ranides/assira/collection_map/CrossEnumMap.java

@@ -16,7 +16,6 @@ import java.util.Map;
  * @param <KY> 
  * @param <V> 
  * @author ranides
- * @todo (migration) assira
  */
 @SuppressWarnings("PMD.ShortVar")
 public final class CrossEnumMap<KX extends Enum<KX>, KY extends Enum<KY>, V> implements CrossMap<KX, KY, V> {

assira1/src/main/java/net/ranides/assira/collection_map/CrossHashMap.java

@@ -17,7 +17,6 @@ import java.util.Map.Entry;
  * @param <KY> 
  * @param <V> 
  * @author ranides
- * @todo (migration) assira
  */
 @SuppressWarnings("PMD.ShortVar")
 public class CrossHashMap<KX, KY, V> implements CrossMap<KX, KY, V> {

assira1/src/main/java/net/ranides/assira/collection_map/CrossMap.java

@@ -13,7 +13,6 @@ package net.ranides.assira.collection.map;
  * @param <KY> 
  * @param <V> 
  * @author ranides
- * @todo (migration) assira
  */
 @SuppressWarnings("PMD.ShortVar")
 public interface CrossMap<KX, KY, V> {