Persistent usage of timber (including low-level classes) is not clear about timber

[JakeWharton]

I inject it everywhere. Haven't found a place where it couldn't be passed in like any other dependency yet. Another option would be to use static injection to put it into a static field somewhere immediately when the object graph is created. Yet another, simpler option:

public static final Timber LOG = BuildConfig.DEBUG ? Timber.DEBUG : Timber.PROD;

[AlexKorovyansky]

Hmm... I like idea inject it everywhere! Other ways looks significantly worse.

For me it's not clear how to inject timber in this case (to Binary and Telnet Connections).

 /* Low level class implemented binary async connection (in Twisted style) */
public class BinaryConnection {

   public BinaryConnection(HostAddress remoteAddress, BinaryPacketReader packetReader)

   ...

}

 /* Low level class implemented telnet async connection (in Twisted style) */
public class TelnetConnection {

   public TelnetConnection(HostAddress remoteAddress) {
        this.binaryConnection = new BinaryConnection(remoteAddress, new DelimitedPacketReader("\r\n"));

   ...

}

Any ideas?

Thank you!

[tbruyelle]

With the help of the new gradle build system, I integrated Timber in the BuildConfig.

Something like that :

buildTypes {
  release {
    buildConfig  "public static final timber.log.Timber LOGGER = timber.log.Timber.PROD;"
  }
  debug {
    buildConfig  "public static final timber.log.Timber LOGGER = timber.log.Timber.DEBUG;"
  } 
}

[JakeWharton]

@tbruyelle Clever!

@korovyansk If you can't inject directly, you need to manually pass around an instance that's injected in a class higher up. You can also use static injection to get it in a static field accessible everywhere.