[LOG4J2-3310] maybe some log4j-api-scala methods should be changed back to being macros

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: Scala 12.1
• Component/s: Scala API
• Assignee: Unassigned
• Reporter: PJ Fanning
• Created: 04/Jan/22 09:42
• Updated: 10/Feb/22 20:52
• Resolved: 10/Feb/22 20:52

Description

In hindsight, I suspect that I went too far in LOG4J2-3184 with changing methods that used macros into ones that just delegated to base logger at runtime.

These methods are among those that possibly should be macro based, as it might be nice to adjust the pre-compiled code to add the level isEnabled check.

 

def apply(inline level: Level, inline marker: Marker, inline message: Message): Unit

def apply(inline level: Level, inline marker: Marker, inline message: AnyRef): Unit

def apply(inline level: Level, inline marker: Marker, inline message: Message, inline cause: Throwable): Unit

def apply(inline level: Level, inline marker: Marker, inline message: AnyRef, inline cause: Throwable): Unit

def apply(inline level: Level, inline message: Message): Unit

def apply(inline level: Level, inline message: AnyRef): Unit

def apply(inline level: Level, inline message: Message, inline cause: Throwable): Unit

def apply(inline level: Level, inline message: AnyRef, inline cause: Throwable): Unit

def traceEntry(inline params: AnyRef*): EntryMessage

def traceEntry(inline message: Message): EntryMessage

def traceExit(inline entryMessage: EntryMessage): Unit

def traceExit[R](inline entryMessage: EntryMessage, result: R): R

def traceExit[R](inline message: Message, result: R): R

def throwing[T <: Throwable](inline level: Level, inline t: T): T

def catching(inline level: Level, inline t: Throwable): Unit