Chris Lee

Thanks for the context. Agree that formatting is a back-end concern and re-use (if any) should be pick-from-available-pieces on a best-effort basis (to make those pieces reusable). Possible to provide...

Would welcome any insight into checks / transformations - thanks! So far have managed to canonicalize various source APIs, such as: ``` someLogger.error("test message"); someLogger.error(DummyLog4J2Marker.INSTANCE, "test message"); someLogger.log(Level.ERROR, "test message");...

Thanks. Missing context: that example was from this SLF4J input: ``` logger.info("7. Single parameter, double-escaped anchor: {} \\\\{}", "abc"); ``` ...where SLF4J treats `\{}` as escaping the format anchor and...

Relevant context around use of Automatic-Module-Name: http://branchandbound.net/blog/java/2017/12/automatic-module-name/ https://blog.joda.org/2017/05/java-se-9-jpms-automatic-modules.html

Thanks. To be clear, this isn't about modularizing the code - rather reserving the module name to avoid downstream challenges when Flogger is modularized. Currently, without the Automatic-Module-Name manifest entry,...

The SLF4J back-end would require a dependency on slf4j-api, that may be the ExceptionInInitializer error being unable to find required classes.

Yea, that would do it. No worries. The README documents this requirement: > All code that uses flogger should depend on com.google.flogger:flogger: and com.google.flogger:flogger-system-backend:. > Note: the dependency on flogger-system-backend...

Looking at this further; wondering why `flogger-slf4j-backend` doesn't transitively pull in `flogger-system-backend`. Note the problem isn't limited to slf4j backend, log4j backend has the same behaviour. This set of Gradle...

Thanks for having a look at this! Was looking at SimpleFormatter for a different use case (JSON log output from back-end); observations: 1) Having message formatting for *parameters* is part...

That's already been addressed above: > In the specific code you give, "foo" is free, because it's a class-level constant. When the class gets loaded, all constant strings in the...