Make AbstractModule.binder private, for a hair more encapsulation.

PiperOrigin-RevId: 549090093
google · Aug 2, 2023 · 7f6cff8 · 7f6cff8
1 parent ddb6315
commit 7f6cff8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/core/src/com/google/inject/AbstractModule.java b/core/src/com/google/inject/AbstractModule.java
@@ -53,7 +53,7 @@
  */
 public abstract class AbstractModule implements Module {
 
-  Binder binder;
+  private Binder binder;
 
   @Override
   public final synchronized void configure(Binder builder) {

diff --git a/core/test/com/google/inject/CircularDependencyTest.java b/core/test/com/google/inject/CircularDependencyTest.java
@@ -438,8 +438,8 @@ public void testCircularlyDependentMultipleWays() {
             new AbstractModule() {
               @Override
               protected void configure() {
-                binder.bind(A.class).to(E.class);
-                binder.bind(B.class).to(E.class);
+                bind(A.class).to(E.class);
+                bind(B.class).to(E.class);
               }
             });
     injector.getInstance(A.class);
@@ -452,8 +452,8 @@ public void testDisablingCircularDependencies() {
               @Override
               protected void configure() {
                 binder().disableCircularProxies();
-                binder.bind(A.class).to(E.class);
-                binder.bind(B.class).to(E.class);
+                bind(A.class).to(E.class);
+                bind(B.class).to(E.class);
               }
             });
 

diff --git a/core/test/com/google/inject/JitBindingsTest.java b/core/test/com/google/inject/JitBindingsTest.java
@@ -141,7 +141,7 @@ public void testLinkedToScoped() {
             new AbstractModule() {
               @Override
               protected void configure() {
-                binder.requireExplicitBindings();
+                binder().requireExplicitBindings();
                 bind(Foo.class).to(ScopedFooImpl.class);
               }
             });