Guys,
Our current Log impl allows for string formatting, using params like
log.info("This is a {0} with some {1}", "string", "words")
originally inspired by the Seam logger. IMO we should switch to using a more standard
form of parameterisation, as used by the JDKs String.format() - as well as in many other
languages including C and Python:
log.info("This is a %s with some %s", "string", "words")
This will be more consistent with String.format() used elsewhere (e.g., generating
exception messages). A quick search and replace in the codebase will take care of this.
Opinions?
Cheers
Manik
--
Manik Surtani
manik(a)jboss.org
twitter.com/maniksurtani
Lead, Infinispan
http://www.infinispan.org