Yuriy Sidelnikov created AMQ-5099:
-------------------------------------
Summary: Useless or dead code in OpenWireFormat class.
Key: AMQ-5099
URL: https://issues.apache.org/jira/browse/AMQ-5099
Project: ActiveMQ
Issue Type: Bug
Components: JMS client
Affects Versions: 5.9.0
Reporter: Yuriy Sidelnikov
Priority: Minor
Method marshal has following code:
ByteSequence sequence = null;
// if( ma!=null ) {
// sequence = ma.getCachedMarshalledForm(this);
// }
if (sequence == null) {
....
It makes no sense to check if sequence is null because it is set to null couple
of lines above and possible value change is commented. It would be better to
comment the check or remove both check and commented lines.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
