Ah.  Yeah.  Ok.   I added the supertype thing after I name it.  I'll take
another look when I get near my computer.

Regards,
Christian
Sent from my iPhone.

On Jan 13, 2013, at 17:32, Jesse Wilson <notifications@github.com> wrote:

In compiler/src/main/java/dagger/internal/codegen/InjectProcessor.java:

>      }
> -    if (supertype != null) {
> -      writer.statement("injectMembersBindings.add(%s)", "supertype");
> +    return interfaces.toArray(new String[0]);
> +  }
> +
> +  private void writeImports(JavaWriter writer, boolean hasFields, boolean isProvider)
> +      throws IOException {
> +    writer.emitImport(Binding.class);
> +    writer.emitImport(Linker.class);
> +    if (hasFields) {
> +      writer.emitImport(MembersInjector.class);

The boolean is called hasFields but the code that calls this uses
"hasFields || supertype != null". Why not just name the boolean
"importMembersInjector"

—
Reply to this email directly or view it on
GitHub<https://github.com/square/dagger/pull/133/files#r2630052>.