[LOG4J2-2634] AsyncLogger method bytecode should be reduced to 35 bytes for fast inlining

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.11.2
• Fix Version/s: 2.12.0
• Assignee: Carter Kozak
• Reporter: Carter Kozak
• Created: 20/Jun/19 12:27
• Updated: 21/Jun/19 13:57
• Resolved: 20/Jun/19 15:49

Description

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

Issue Links

• links to: GitHub Pull Request #282