logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matt Sicker (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LOG4J2-547) Update LoggerStream API
Date Tue, 15 Apr 2014 03:49:14 GMT

    [ https://issues.apache.org/jira/browse/LOG4J2-547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13969192#comment-13969192

Matt Sicker commented on LOG4J2-547:

Although I'd recommend creating builders instead of giant constructors, but that's somewhat
of a style opinion I guess.

> Update LoggerStream API
> -----------------------
>                 Key: LOG4J2-547
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-547
>             Project: Log4j 2
>          Issue Type: Improvement
>          Components: API
>    Affects Versions: 2.0-rc1
>            Reporter: Matt Sicker
>            Assignee: Ralph Goers
>             Fix For: 2.0-rc2
>         Attachments: 0001-PrintStream-API-update.patch, MyBenchmark.java, PerfTestCalcLocation.java,
log4j2-547-bbrouwer.patch, log4j2-547-new-module.patch, log4j2-547-remove-streams.patch, log4j2-loggerStream.patch
> I've got some ideas on how to improve the LoggerStream idea that I added a little while
ago. The main thing I'd like to do is extract an interface from it, rename the default implementation
to SimpleLoggerStream (part of the SimpleLogger stuff), and allow log4j implementations to
specify a different implementation if desired.
> In doing this, I'm not sure where specifically I'd prefer the getStream methods to be.
Right now, it's in Logger, but really, it could be in LoggerContext instead. I don't think
I should be required to get a Logger just to get a LoggerStream.
> Now if only the java.io package used interfaces instead of classes. This would be so
much easier to design!

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

View raw message