Refactor internal generator classes to be final

Author: rbygrave

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

@@ -9,7 +9,7 @@
 import java.util.List;
 import java.util.Map;
 
-class AllScopes {
+final class AllScopes {
 
   private final Map<String, Data> scopeAnnotations = new HashMap<>();
   private final ProcessingContext context;

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

@@ -3,7 +3,7 @@
 import javax.lang.model.element.AnnotationMirror;
 import javax.lang.model.element.Element;
 
-class AnnotationUtil {
+final class AnnotationUtil {
 
   static boolean hasAnnotationWithName(Element element, String matchShortName) {
     for (AnnotationMirror mirror : element.getAnnotationMirrors()) {

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

@@ -6,7 +6,7 @@
 /**
  * Helper that wraps a writer with some useful methods to append content.
  */
-class Append {
+final class Append {
 
   private final Writer writer;
   private int nameIndex;

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

@@ -10,7 +10,7 @@
 /**
  * Read the annotations on the type.
  */
-class AspectAnnotationReader {
+final class AspectAnnotationReader {
 
   private static final String ASPECT = Constants.ASPECT;
   private final TypeElement baseType;

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

@@ -6,7 +6,7 @@
 import javax.lang.model.type.TypeMirror;
 import java.util.*;
 
-class AspectMethod {
+final class AspectMethod {
 
   private final List<AspectPair> aspectPairs;
   private final ExecutableElement method;

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

@@ -2,7 +2,7 @@
 
 import javax.lang.model.element.Element;
 
-class AspectPair implements Comparable<AspectPair> {
+final class AspectPair implements Comparable<AspectPair> {
 
   private final String target;
   private final int ordering;

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

@@ -5,7 +5,7 @@
 import java.util.List;
 import java.util.Set;
 
-class BeanAspects {
+final class BeanAspects {
 
   static final BeanAspects EMPTY = new BeanAspects();
 

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

@@ -9,7 +9,7 @@
 import javax.lang.model.element.TypeElement;
 import java.util.*;
 
-class BeanReader {
+final class BeanReader {
 
   private final TypeElement beanType;
   private final String shortName;

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

@@ -4,7 +4,7 @@
 /**
  * Holds detection and details of request scoped dependencies.
  */
-class BeanRequestParams {
+final class BeanRequestParams {
 
   private final String parentType;
   private RequestScope.Handler reqScopeHandler;

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

@@ -1,6 +1,6 @@
 package io.avaje.inject.generator;
 
-class Constants {
+final class Constants {
 
   static final int ORDERING_DEFAULT = 1000;
 

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

@@ -4,7 +4,7 @@
 import java.util.List;
 import java.util.Set;
 
-class FieldReader {
+final class FieldReader {
 
   private final Element element;
   private final String name;

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

@@ -8,7 +8,7 @@
 /**
  * A type with generic parameters and potentially nested.
  */
-class GenericType {
+final class GenericType {
 
   /**
    * Trim off generic wildcard from the raw type if present.

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

@@ -3,7 +3,7 @@
 import java.util.HashSet;
 import java.util.Set;
 
-class IncludeAnnotations {
+final class IncludeAnnotations {
 
   private static final String[] EXCLUDED_PREFIX = {"javax.annotation.", "javax.inject.", "jakarta.annotation.", "jakarta.inject.", "io.avaje.inject.", "lombok."};
   private static final String[] EXCLUDED_SUFFIX = {".PostConstruct", ".PreDestroy", ".Generated"};

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

@@ -12,7 +12,7 @@
 /**
  * Holds the data as per <code>@DependencyMeta</code>
  */
-class MetaData {
+final class MetaData {
 
   private static final String NEWLINE = "\n";
 

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

@@ -3,7 +3,7 @@
 import javax.lang.model.element.TypeElement;
 import java.util.*;
 
-class MetaDataOrdering {
+final class MetaDataOrdering {
 
   private static final String CIRC_ERR_MSG =
     "To handle circular dependencies consider using field injection " +

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

@@ -2,7 +2,7 @@
 
 import java.util.Collection;
 
-class MetaTopPackage {
+final class MetaTopPackage {
 
   private String topPackage;
 

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

@@ -7,7 +7,7 @@
 /**
  * Looks for lifecycle annotations on methods.
  */
-class MethodLifecycleReader {
+final class MethodLifecycleReader {
 
   private final String initMethod;
   private final String destroyMethod;

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

@@ -15,7 +15,7 @@
 import java.util.List;
 import java.util.Set;
 
-class MethodReader {
+final class MethodReader {
 
   private static final String CODE_COMMENT_BUILD_FACTORYBEAN = "  /**\n   * Create and register %s via factory bean method %s#%s().\n   */";
 

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

@@ -20,7 +20,7 @@
 import java.nio.file.NoSuchFileException;
 import java.util.*;
 
-class ProcessingContext {
+final class ProcessingContext {
 
   private final ProcessingEnvironment processingEnv;
   private final Messager messager;

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

@@ -20,7 +20,7 @@
 import java.util.LinkedHashSet;
 import java.util.Set;
 
-public class Processor extends AbstractProcessor {
+public final class Processor extends AbstractProcessor {
 
   private ProcessingContext context;
   private Elements elementUtils;

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

@@ -7,7 +7,7 @@
 /**
  * Handling detection of request scoped dependencies and appropriate BeanFactory generation.
  */
-class RequestScope {
+final class RequestScope {
 
   private static final String JEX_CONTEXT = "io.avaje.jex.Context";
   private static final String JAVALIN_CONTEXT = "io.javalin.http.Context";

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

@@ -12,7 +12,7 @@
 import java.io.IOException;
 import java.util.*;
 
-class ScopeInfo {
+final class ScopeInfo {
 
   enum Type {
     /**

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

@@ -6,7 +6,7 @@
 import java.util.List;
 import java.util.Map;
 
-class ScopeUtil {
+final class ScopeUtil {
 
   private static final String INJECT_MODULE = "io.avaje.inject.InjectModule";
 

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

@@ -5,7 +5,7 @@
 import java.io.IOException;
 import java.io.Writer;
 
-class SimpleBeanProxyWriter {
+final class SimpleBeanProxyWriter {
 
   private final BeanReader beanReader;
   private final ProcessingContext context;

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

@@ -10,7 +10,7 @@
 /**
  * Write the source code for the bean.
  */
-class SimpleBeanWriter {
+final class SimpleBeanWriter {
 
   private static final String CODE_COMMENT = "/**\n * Generated source - dependency injection builder for %s.\n */";
   private static final String CODE_COMMENT_FACTORY = "/**\n * Generated source - dependency injection factory for request scoped %s.\n */";

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

@@ -13,7 +13,7 @@
 /**
  * Write the source code for the factory.
  */
-class SimpleModuleWriter {
+final class SimpleModuleWriter {
 
   private static final String CODE_COMMENT_FACTORY =
     "/**\n" +

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

@@ -11,7 +11,7 @@
 /**
  * Read the annotations on the type.
  */
-class TypeAnnotationReader {
+final class TypeAnnotationReader {
 
   private final TypeElement beanType;
   private final ProcessingContext context;

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

@@ -7,7 +7,7 @@
 /**
  * Helper for building the registration types.
  */
-class TypeAppender {
+final class TypeAppender {
 
   private final ImportTypeMap importTypes;
   private final Set<String> types = new LinkedHashSet<>();

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

@@ -13,7 +13,7 @@
  * Read points for field injection and method injection
  * on baseType plus inherited injection points.
  */
-class TypeExtendsInjection {
+final class TypeExtendsInjection {
 
   private MethodReader injectConstructor;
   private final List<MethodReader> otherConstructors = new ArrayList<>();

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

@@ -10,7 +10,7 @@
 /**
  * Read the inheritance types for a given bean type.
  */
-class TypeExtendsReader {
+final class TypeExtendsReader {
 
   private static final String JAVA_LANG_OBJECT = "java.lang.Object";
   private final GenericType baseGenericType;

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

@@ -7,7 +7,7 @@
 import java.util.List;
 import java.util.Set;
 
-class TypeReader {
+final class TypeReader {
 
   private final boolean forBean;
   private final TypeElement beanType;

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

@@ -7,7 +7,7 @@
 import javax.lang.model.type.DeclaredType;
 import javax.lang.model.type.TypeMirror;
 
-class Util {
+final class Util {
 
   static final String PROVIDER_PREFIX = "jakarta.inject.Provider<";
   private static final String OPTIONAL_PREFIX = "java.util.Optional<";

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

@@ -1,6 +1,6 @@
 package io.avaje.inject.generator;
 
-class UtilType {
+final class UtilType {
 
   private enum Type {
     LIST,