logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mikael Ståldal (JIRA) <j...@apache.org>
Subject [jira] [Commented] (LOG4J2-1329) API change: let Message.getFormattedMessage() return CharSequence instead of String
Date Mon, 28 Mar 2016 15:12:25 GMT

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

Mikael Ståldal commented on LOG4J2-1329:

So what about my [original proposal|http://mail-archives.apache.org/mod_mbox/logging-log4j-dev/201603.mbox/%3CCANEH4Wxhq=gjNXB=yGzAmMmEY__xCHs_1ARyLMBX1eMy4MZ9vA@mail.gmail.com%3E]

I suggest we add another method to StringBuilderFormattable:
CharSequence getFormattedMessageBuffer(); // feel free to suggest a better name

> API change: let Message.getFormattedMessage() return CharSequence instead of String
> -----------------------------------------------------------------------------------
>                 Key: LOG4J2-1329
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-1329
>             Project: Log4j 2
>          Issue Type: Improvement
>          Components: API
>    Affects Versions: 2.5
>            Reporter: Remko Popma
> I propose that we change {{Message.getFormattedMessage()}} to return a {{CharSequence}}
instead of a String.
> I believe this change would be backwards compatible (all Message implementations that
return a String will work as is), and it opens the way
> for zero-GC Message implementations to return a cached StringBuilder as Mikael [suggested
on the log4j-dev mailing list|http://mail-archives.apache.org/mod_mbox/logging-log4j-dev/201603.mbox/%3CCAAFrFhSs4EqjOERwHObKAZP4wjGWeiCET1nyE1FVndYdePk2yw%40mail.gmail.com%3E].

This message was sent by Atlassian JIRA

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

View raw message