RFR: 8305920 : Replace String concatenation in a loop in SimpleAttributeSet.toString with StringBuilder

Alexey Ivanov aivanov at openjdk.org
Tue Apr 18 10:47:54 UTC 2023


On Fri, 14 Apr 2023 16:50:04 GMT, Phil Race <prr at openjdk.org> wrote:

> Why is this important ? How often is toString() used on this class ?

I agree it's not too important, `SimpleAttributeSet.toString` is unlikely to be used often, however, it may when one is debugging code and dumping document structure as well as attributes applied.

So, it a low-risk clean-up PR… at the same time it doesn't add much value because the method rarely used.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13479#issuecomment-1512857557



More information about the client-libs-dev mailing list