Some constants in test class are not marked as final
----------------------------------------------------
Key: AMQ-3203
URL: https://issues.apache.org/jira/browse/AMQ-3203
Project: ActiveMQ
Issue Type: Improvement
Reporter: Robert Liguori
Priority: Trivial
Consider the following three statements int the MessagePriorityTest test class;
public int MSG_NUM = 600;
public int HIGH_PRI = 7;
public int LOW_PRI = 3;
Each identifier is "named" as a constant, though they are not marked as final.
Quick simple reference:
http://java.about.com/od/javasyntax/a/nameconventions.htm.
I opened this issue as I was surprised it wasn't caught by the quality tools.
Does ActiveMQ have Checkstyle in place... I believe that CXF does, and its
Maven/Checkstyle integration model could be followed here, if need be.
This issue requests that all identifiers named as constants in the ActiveMQ
distribution are actually marked as final, making them actual constants.
Thanks.
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
