[slf4j-dev] slf4j-migrator: format messages with parameterized logging

[Robert Elliot]

I'm not a committer, so this isn't a rebuke or anything, just a friendly suggestion. You might get more comments & chance of a merge via a pull request on GitHub - easier for everyone. 

Rob

On 21 May 2013, at 19:57, "James A. N. Stauffer" <stauffer.james at gmail.com> wrote:

> Attached fixes/improvements.
> 
> 
> James A. N. Stauffer        http://jamesjive.blogspot.com
> Are you good? Take the test at http://www.livingwaters.com/good/
> 
> 
> On Tue, May 21, 2013 at 10:57 AM, James A. N. Stauffer <stauffer.james at gmail.com> wrote:
>> I have attached two patches that add support for parameterized logging. Comments?
>> 
>> 
>> James A. N. Stauffer        http://jamesjive.blogspot.com
>> Are you good? Take the test at http://www.livingwaters.com/good/
>> 
>> 
>> On Tue, May 21, 2013 at 8:17 AM, James A. N. Stauffer <stauffer.james at gmail.com> wrote:
>>> (I didn't notice any FAQ/guidelines, etc so please direct me to them if the exist so I can follow protocol.)
>>> 
>>> My employer has many projects that use log4j and slf4j is now the approved logging framework. I would like the conversion to make use of the format messages with parameterized logging and remove the calls to is*Enabled().
>>> 
>>> It appears that slf4j-migrator doesn't support that so is there any comment, direction, etc on me adding support for that?
>>> 
>>> James A. N. Stauffer        http://jamesjive.blogspot.com
>>> Are you good? Take the test at http://www.livingwaters.com/good/
> 
> <0003-Fixed-minor-bugs-with-format-conversion.patch>
> _______________________________________________
> slf4j-dev mailing list
> slf4j-dev at qos.ch
> http://mailman.qos.ch/mailman/listinfo/slf4j-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.qos.ch/pipermail/slf4j-dev/attachments/20130521/80991075/attachment.html>