[
https://issues.apache.org/jira/browse/QPID-8604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17620337#comment-17620337
]
ASF subversion and git services commented on QPID-8604:
-------------------------------------------------------
Commit 5d5ea61ab1f730c0ffdabd5b6f274ecf294877b1 in qpid-broker-j's branch
refs/heads/main from Daniil Kirilyuk
[ https://gitbox.apache.org/repos/asf?p=qpid-broker-j.git;h=5d5ea61ab1 ]
QPID-8604: [Broker-J] long literals (#146)
> [Broker-J] Broker code cleanup: immutable fields, static inner classes, long
> literals
> -------------------------------------------------------------------------------------
>
> Key: QPID-8604
> URL: https://issues.apache.org/jira/browse/QPID-8604
> Project: Qpid
> Issue Type: Improvement
> Components: Broker-J
> Affects Versions: qpid-java-broker-8.0.6
> Reporter: Daniil Kirilyuk
> Priority: Minor
>
> Following issues are covered with this JIRA:
> 1. Private fields whose values never change once they are initialized either
> in the declaration of the field or by a constructor should be marked as
> final. This aids in converting existing classes to immutable classes.
> 2. A non-static inner class has a reference to its outer class, and access to
> the outer class' fields and methods. That class reference makes the inner
> class larger and could cause the outer class instance to live in memory
> longer than necessary.
> 3. There are many places in the broker code where long literals end with
> lowercase 'l' character mimicking the '1'. They should be replaced with
> uppercase 'L'.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
