No, I don't think so - it's been noticed! Rob ----- Original Message -----
From: "James A. N. Stauffer" <stauffer.james@gmail.com> To: "slf4j developers list" <slf4j-dev@qos.ch> Sent: Thursday, 30 May, 2013 2:02:30 PM Subject: Re: [slf4j-dev] slf4j-migrator: format messages with parameterized logging Do I need to do something more than creating the pill request on GitHub? I understand that people have other priorities -- I just wanted to make sure that I didn't put it in a black hole. :-)
James A. N. Stauffer http://jamesjive.blogspot.com Are you good? Take the test at http://www.livingwaters.com/good/
On Thu, May 23, 2013 at 1:00 PM, James A. N. Stauffer < stauffer.james@gmail.com > wrote:
Thanks for the tip. Created.
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 3:17 PM, Robert Elliot < rob@lidalia.org.uk > wrote:
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@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@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@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@qos.ch http://mailman.qos.ch/mailman/listinfo/slf4j-dev _______________________________________________ slf4j-dev mailing list slf4j-dev@qos.ch http://mailman.qos.ch/mailman/listinfo/slf4j-dev
_______________________________________________ slf4j-dev mailing list slf4j-dev@qos.ch http://mailman.qos.ch/mailman/listinfo/slf4j-dev