Skip to content

Commit 3b1c84e

Browse files
bcorsoDagger Team
authored andcommitted
[Cleanup] Remove unnecessary switch on BindingType in `SourceFiles#generatedClassNameForBinding()
RELNOTES=N/A PiperOrigin-RevId: 663442605
1 parent f1acd72 commit 3b1c84e

File tree

1 file changed

+8
-17
lines changed

1 file changed

+8
-17
lines changed

java/dagger/internal/codegen/binding/SourceFiles.java

Lines changed: 8 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -149,29 +149,20 @@ public ImmutableMap<DependencyRequest, CodeBlock> frameworkFieldUsages(
149149

150150
/** Returns the generated factory or members injector name for a binding. */
151151
public static ClassName generatedClassNameForBinding(Binding binding) {
152-
switch (binding.bindingType()) {
152+
switch (binding.kind()) {
153+
case ASSISTED_INJECTION:
154+
case INJECTION:
153155
case PROVISION:
154156
case PRODUCTION:
155-
ContributionBinding contribution = (ContributionBinding) binding;
156-
switch (contribution.kind()) {
157-
case ASSISTED_INJECTION:
158-
case INJECTION:
159-
case PROVISION:
160-
case PRODUCTION:
161-
return factoryNameForElement(asExecutable(binding.bindingElement().get()));
162-
163-
case ASSISTED_FACTORY:
164-
return siblingClassName(asTypeElement(binding.bindingElement().get()), "_Impl");
165-
166-
default:
167-
throw new AssertionError();
168-
}
169-
157+
return factoryNameForElement(asExecutable(binding.bindingElement().get()));
158+
case ASSISTED_FACTORY:
159+
return siblingClassName(asTypeElement(binding.bindingElement().get()), "_Impl");
170160
case MEMBERS_INJECTION:
171161
return membersInjectorNameForType(
172162
((MembersInjectionBinding) binding).membersInjectedType());
163+
default:
164+
throw new AssertionError();
173165
}
174-
throw new AssertionError();
175166
}
176167

177168
/**

0 commit comments

Comments
 (0)