diff options
Diffstat (limited to 'src/main')
| -rw-r--r-- | src/main/java/ftbsc/lll/processor/LilleroProcessor.java | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/src/main/java/ftbsc/lll/processor/LilleroProcessor.java b/src/main/java/ftbsc/lll/processor/LilleroProcessor.java index 7a85b8b..eabc2b3 100644 --- a/src/main/java/ftbsc/lll/processor/LilleroProcessor.java +++ b/src/main/java/ftbsc/lll/processor/LilleroProcessor.java @@ -223,13 +223,6 @@ public class LilleroProcessor extends AbstractProcessor { appendMemberFinderDefinition(targetClass, proxyVar, null, constructorBuilder, this.processingEnv, this.mapper); } - //declare it once for efficiency - List<String> injectorNames = - injectors.stream() - .map(ExecutableElement::getSimpleName) - .map(Object::toString) - .collect(Collectors.toList()); - //targets that have matched at least once are put in here Set<ExecutableElement> matchedTargets = new HashSet<>(); @@ -276,6 +269,10 @@ public class LilleroProcessor extends AbstractProcessor { throw new AmbiguousDefinitionException( String.format("Target specified user %s, but name was used by both a finder and injector.", targetAnn.of()) ); + else if(finderCandidates.size() == 0 && injectorCandidates.size() == 0) + throw new AmbiguousDefinitionException( + String.format("Could not find any candidate mathcing target %s", tg.getSimpleName().toString()) + ); else if(finderCandidates.size() == 0 && injectorCandidates.size() != 1) throw new AmbiguousDefinitionException( String.format("Found multiple candidate injectors for target %s::%s!", cl.getSimpleName(), tg.getSimpleName()) @@ -285,6 +282,7 @@ public class LilleroProcessor extends AbstractProcessor { String.format("Found multiple candidate finders for target %s::%s!", cl.getSimpleName(), tg.getSimpleName()) ); else { + matchedTargets.add(tg); if(injectorCandidates.size() == 1) { //matched an injector! toGenerate.put( @@ -307,6 +305,10 @@ public class LilleroProcessor extends AbstractProcessor { } } + //find orphan targets and warn + Set<ExecutableElement> orphanTargets = targets.stream().filter(t -> !matchedTargets.contains(t)).collect(Collectors.toSet()); + //TODO! + //iterate over the map and generate the classes for(String injName : toGenerate.keySet()) { String targetMethodDescriptor = descriptorFromExecutableElement(toGenerate.get(injName).target); |