[LOG4J2-1321] NameAbbreviator creates temporary objects

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5
• Fix Version/s: 2.6
• Component/s: Pattern Converters
• Epic Link: GC-free
• Assignee: Remko Popma
• Reporter: Remko Popma
• Created: 17/Mar/16 23:01
• Updated: 18/Mar/16 01:47
• Resolved: 18/Mar/16 01:47

Description

PatternLayout's NamePatternConverter and ClassNamePatternConverter delegate to NameAbbreviator.

The current API that NameAbbreviator supports is

String abbreviate(String original)

This creates temporary objects. However, the clients of NameAbbreviator already have a StringBuilder to write into. I propose to modify the NameAbbreviator API to this:

void abbreviate(String original, StringBuilder destination)