http://bugzilla.slf4j.org/show_bug.cgi?id=116





--- Comment #18 from Ceki Gulcu <[EMAIL PROTECTED]>  2008-12-02 19:40:22 ---
Ken,

Thank you for the patch. It looks very good. However, I don't think
FormatModifyingXLogger should extend XLogger. Instead, it should extend
LoggerWrapper as XLogger does. You probably don't want message formatting on
method entry or method exit. Similarly, FormatModifyingXLogger  should have its
own factory, like XLogger does. 


-- 
Configure bugmail: http://bugzilla.slf4j.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dev mailing list
dev@slf4j.org
http://www.slf4j.org/mailman/listinfo/dev

Reply via email to