[infinispan-dev] Logging in Infinispan

[David M. Lloyd]

Use the %1$s form.

On 01/25/2011 04:08 PM, Emmanuel Bernard wrote:
> In some languages, param 2 needs to be put before param 1. How do you solve that wo positional params?
>
> On 25 janv. 2011, at 15:52, Pete Muir wrote:
>
>>
>> On 25 Jan 2011, at 20:29, Manik Surtani wrote:
>>
>>> 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?
>>
>> I like this style more FWIW, I think it's easier to alter messages.
>> _______________________________________________
>> infinispan-dev mailing list
>> infinispan-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/infinispan-dev
>
>
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev


-- 
- DML