[infinispan-dev] Logging in Infinispan

[Mircea Markus]

I like the existing one more, as it makes it easier for me too which params goes where. 
On 26 Jan 2011, at 10:16, Manik Surtani wrote:
> So I assume we're all happy with this change?  I've already done the work in a topic branch and ready to push... 
> 
> On 25 Jan 2011, at 22:11, David M. Lloyd wrote:
> 
>> 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
>> _______________________________________________
>> infinispan-dev mailing list
>> infinispan-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/infinispan-dev
> 
> --
> Manik Surtani
> manik at jboss.org
> twitter.com/maniksurtani
> 
> Lead, Infinispan
> http://www.infinispan.org
> 
> 
> 
> 
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev