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

ASF subversion and git services commented on LOG4J2-2634:
---------------------------------------------------------

Commit a014c003e378965796bbf0be671e204b18f7aedf in logging-log4j2's branch 
refs/heads/release-2.x from Carter Kozak
[ https://gitbox.apache.org/repos/asf?p=logging-log4j2.git;h=a014c00 ]

LOG4J2-2634: Break up large AsyncLogger methods for inlining


> AsyncLogger method bytecode should be reduced to 35 bytes for fast inlining
> ---------------------------------------------------------------------------
>
>                 Key: LOG4J2-2634
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-2634
>             Project: Log4j 2
>          Issue Type: Improvement
>    Affects Versions: 2.11.2
>            Reporter: Carter Kozak
>            Assignee: Carter Kozak
>            Priority: Minor
>             Fix For: 2.12.0
>
>
> Large methods:
> AsyncLogger.logMessage conditional behavior produces a large method, this can 
> be broken into an abstraction which two implementations.
>  
> AsyncLogger.actualAsyncLog can be inlined in cases where Properties are not 
> set
> RingBufferLogEventHandler.onEvent callback notification makes sense in a 
> separate method so that both are below 35b



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to