Skip to content

Commit bfa21ee

Browse files
committed
Refactor rename AvajeModule interface to InjectModule
Note that I think this is reasonable in terms of @InjectModule being the same name in that the IDE auto-completion when using the annotation knows it's an annotation. So I don't think we get problematic use like this.
1 parent 1bc0b61 commit bfa21ee

File tree

29 files changed

+110
-110
lines changed

29 files changed

+110
-110
lines changed

inject-generator/src/main/java/io/avaje/inject/generator/Constants.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ final class Constants {
2828
static final String AT_GENERATED_COMMENT = "(\"io.avaje.inject.generator\")";
2929
static final String META_INF_SPI = "META-INF/services/io.avaje.inject.spi.InjectExtension";
3030
static final String META_INF_TESTMODULE = "META-INF/services/io.avaje.inject.test.TestModule";
31-
static final String META_INF_CUSTOM = "META-INF/services/io.avaje.inject.spi.AvajeModule.Custom";
31+
static final String META_INF_CUSTOM = "META-INF/services/io.avaje.inject.spi.InjectModule.Custom";
3232

3333
static final String BEANSCOPE = "io.avaje.inject.BeanScope";
3434
static final String INJECTMODULE = "io.avaje.inject.InjectModule";
@@ -53,7 +53,7 @@ final class Constants {
5353
static final String BEAN_FACTORY2 = "io.avaje.inject.spi.BeanFactory2";
5454
static final String BUILDER = "io.avaje.inject.spi.Builder";
5555
static final String DEPENDENCYMETA = "io.avaje.inject.spi.DependencyMeta";
56-
static final String MODULE = "io.avaje.inject.spi.AvajeModule";
56+
static final String MODULE = "io.avaje.inject.spi.InjectModule";
5757
static final String GENERICTYPE = "io.avaje.inject.spi.GenericType";
5858

5959
static final String CONDITIONAL_DEPENDENCY = "con:";

inject-generator/src/main/java/io/avaje/inject/generator/ExternalProvider.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818

1919
import javax.tools.StandardLocation;
2020

21-
import io.avaje.inject.spi.AvajeModule;
21+
import io.avaje.inject.spi.InjectModule;
2222
import io.avaje.inject.spi.InjectPlugin;
2323

2424
/**
@@ -64,7 +64,7 @@ static void registerModuleProvidedTypes(Set<String> providedTypes) {
6464
return;
6565
}
6666

67-
List<AvajeModule> modules = LoadServices.loadModules(CLASS_LOADER);
67+
List<InjectModule> modules = LoadServices.loadModules(CLASS_LOADER);
6868
if (modules.isEmpty()) {
6969
APContext.logNote("No external modules detected");
7070
return;

inject-generator/src/main/java/io/avaje/inject/generator/LoadServices.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,17 +10,17 @@
1010

1111
final class LoadServices {
1212

13-
static List<AvajeModule> loadModules(ClassLoader classLoader) {
14-
List<AvajeModule> modules = new ArrayList<>();
13+
static List<InjectModule> loadModules(ClassLoader classLoader) {
14+
List<InjectModule> modules = new ArrayList<>();
1515
// load using older Module
1616
ServiceLoader.load(Module.class, classLoader).forEach(modules::add);
17-
// load newer AvajeModule
17+
// load newer InjectModule
1818
final var iterator = ServiceLoader.load(InjectExtension.class, classLoader).iterator();
1919
while (iterator.hasNext()) {
2020
try {
2121
final var spi = iterator.next();
22-
if (spi instanceof AvajeModule) {
23-
modules.add((AvajeModule) spi);
22+
if (spi instanceof InjectModule) {
23+
modules.add((InjectModule) spi);
2424
}
2525
} catch (final ServiceConfigurationError expected) {
2626
// ignore expected error reading the module that we are also writing

inject-generator/src/main/java/io/avaje/inject/generator/ProcessingContext.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ static String loadMetaInfServices() {
5757

5858
private static boolean isInjectModule(String spi) {
5959
var moduleType = APContext.typeElement(spi);
60-
return moduleType != null && moduleType.getSuperclass().toString().contains("AvajeModule");
60+
return moduleType != null && moduleType.getSuperclass().toString().contains("InjectModule");
6161
}
6262

6363
static List<String> loadMetaInfCustom() {

inject-generator/src/main/java/io/avaje/inject/generator/ScopeInfo.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,11 @@ enum Type {
2626
/**
2727
* Default scope.
2828
*/
29-
DEFAULT("AvajeModule"),
29+
DEFAULT("io.avaje.inject.spi.InjectModule"),
3030
/**
3131
* Custom scope.
3232
*/
33-
CUSTOM("AvajeModule.Custom"),
33+
CUSTOM("io.avaje.inject.spi.InjectModule.Custom"),
3434
/**
3535
* Built-in Test scope.
3636
*/

inject-generator/src/main/java/io/avaje/inject/generator/SimpleModuleWriter.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,6 @@ private Set<String> factoryImportTypes() {
211211
importTypes.add(Constants.BEANSCOPE);
212212
importTypes.add(Constants.INJECTMODULE);
213213
importTypes.add(Constants.DEPENDENCYMETA);
214-
importTypes.add(Constants.MODULE);
215214
importTypes.add(Constants.BUILDER);
216215
return importTypes;
217216
}

inject-generator/src/main/java/io/avaje/inject/generator/SimpleOrderWriter.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ private void writePackage() {
5151
+ "import java.util.Set;\n"
5252
+ "import io.avaje.inject.spi.Generated;\n"
5353
+ "import io.avaje.inject.spi.ModuleOrdering;\n"
54-
+ "import io.avaje.inject.spi.AvajeModule;")
54+
+ "import io.avaje.inject.spi.InjectModule;")
5555
.eol();
5656

5757
writer.eol();
@@ -67,7 +67,7 @@ private void writeStartClass() {
6767
writer.append(Constants.AT_GENERATED).eol();
6868
writer.append("public final class %s implements ModuleOrdering {", shortName).eol().eol();
6969

70-
writer.append(" private final AvajeModule[] sortedModules = new AvajeModule[%s];", ordering.size()).eol();
70+
writer.append(" private final InjectModule[] sortedModules = new InjectModule[%s];", ordering.size()).eol();
7171
writer.append(" private static final Map<String, Integer> INDEXES = Map.ofEntries(").eol();
7272
var size = ordering.size();
7373
var count = 0;
@@ -85,7 +85,7 @@ private void writeBuildMethods() {
8585
writer.append(
8686
"\n"
8787
+ " @Override\n"
88-
+ " public List<AvajeModule> factories() {\n"
88+
+ " public List<InjectModule> factories() {\n"
8989
+ " return List.of(sortedModules);\n"
9090
+ " }\n"
9191
+ "\n"
@@ -95,7 +95,7 @@ private void writeBuildMethods() {
9595
+ " }\n"
9696
+ "\n"
9797
+ " @Override\n"
98-
+ " public void add(AvajeModule module) {\n"
98+
+ " public void add(InjectModule module) {\n"
9999
+ " final var index = INDEXES.get(module.getClass().getTypeName());\n"
100100
+ " if (index != null) {\n"
101101
+ " sortedModules[index] = module;\n"

inject-generator/src/main/java/module-info.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import io.avaje.inject.spi.InjectModule;
2+
13
module io.avaje.inject.generator {
24

35
requires java.compiler;
@@ -11,7 +13,7 @@
1113
uses io.avaje.inject.spi.Plugin;
1214
uses io.avaje.inject.spi.Module;
1315
uses io.avaje.inject.spi.InjectPlugin;
14-
uses io.avaje.inject.spi.AvajeModule;
16+
uses InjectModule;
1517

1618
provides javax.annotation.processing.Processor with io.avaje.inject.generator.InjectProcessor;
1719
}

inject-gradle-plugin/src/main/java/io/avaje/inject/plugin/AvajeInjectPlugin.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import static java.util.stream.Collectors.joining;
44
import static java.util.stream.Collectors.toList;
55

6-
import io.avaje.inject.spi.AvajeModule;
6+
import io.avaje.inject.spi.InjectModule;
77
import io.avaje.inject.spi.InjectPlugin;
88
import io.avaje.inject.spi.InjectExtension;
99
import io.avaje.inject.spi.Module;
@@ -94,12 +94,12 @@ private void writeProvidedPlugins(ClassLoader classLoader, FileWriter pluginWrit
9494
private void writeProvidedModules(ClassLoader classLoader, FileWriter moduleWriter) throws IOException {
9595
final Set<String> providedTypes = new HashSet<>();
9696

97-
final List<AvajeModule> avajeModules = new ArrayList<>();
97+
final List<InjectModule> avajeModules = new ArrayList<>();
9898
ServiceLoader.load(Module.class, classLoader).forEach(avajeModules::add);
9999
ServiceLoader.load(InjectExtension.class, classLoader).stream()
100100
.map(Provider::get)
101-
.filter(AvajeModule.class::isInstance)
102-
.map(AvajeModule.class::cast)
101+
.filter(InjectModule.class::isInstance)
102+
.map(InjectModule.class::cast)
103103
.forEach(avajeModules::add);
104104

105105
for (final var module : avajeModules) {

inject-maven-plugin/src/main/java/io/avaje/inject/mojo/AutoProvidesMojo.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
import org.apache.maven.plugins.annotations.ResolutionScope;
3030
import org.apache.maven.project.MavenProject;
3131

32-
import io.avaje.inject.spi.AvajeModule;
32+
import io.avaje.inject.spi.InjectModule;
3333
import io.avaje.inject.spi.InjectExtension;
3434
import io.avaje.inject.spi.InjectPlugin;
3535
import io.avaje.inject.spi.Module;
@@ -133,12 +133,12 @@ private void writeProvidedModules(URLClassLoader newClassLoader, FileWriter modu
133133
final Set<String> providedTypes = new HashSet<>();
134134

135135
final Log log = getLog();
136-
final List<AvajeModule> avajeModules = new ArrayList<>();
136+
final List<InjectModule> avajeModules = new ArrayList<>();
137137
ServiceLoader.load(Module.class, newClassLoader).forEach(avajeModules::add);
138138
ServiceLoader.load(InjectExtension.class, newClassLoader).stream()
139139
.map(Provider::get)
140-
.filter(AvajeModule.class::isInstance)
141-
.map(AvajeModule.class::cast)
140+
.filter(InjectModule.class::isInstance)
141+
.map(InjectModule.class::cast)
142142
.forEach(avajeModules::add);
143143
for (final var module : avajeModules) {
144144
final var name = module.getClass().getTypeName();

0 commit comments

Comments
 (0)