logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Remko Popma (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LOG4J2-231) Logger.getParent
Date Tue, 30 Apr 2013 15:20:16 GMT

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

Remko Popma commented on LOG4J2-231:
------------------------------------

Hmm. This code base is a *lot* easier to understand than log4j 1.x though!
                
> Logger.getParent 
> -----------------
>
>                 Key: LOG4J2-231
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-231
>             Project: Log4j 2
>          Issue Type: Question
>          Components: Core, log4j 1.2 emulation
>    Affects Versions: 2.0-beta5
>            Reporter: Remko Popma
>
> Not 100% sure, but this code looked a bit strange:
> (o.a.l.l.core.Logger line 65)
>     public Logger getParent() {
>         final LoggerConfig lc = config.loggerConfig.getParent();
>         if (lc == null) {
>             return null;
>         }
>         if (context.hasLogger(lc.getName())) {
>             return context.getLogger(getName(), getMessageFactory()); // <-------
>         }
>         return new Logger(context, getName(), this.getMessageFactory()); // <-------
>     }
> the last two return statements use the name of *this* logger instead of the parent name
lc.getName().
> Is that correct?
> (this method is not used in core internally but is used in the logtj12-api module, by
Category#getParent)
> I'll try to write a JUnit test for this later, still need to figure out how.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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


Mime
View raw message