[AMQ-5138] Useless code in VMTransport class

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.9.0
• Fix Version/s: 5.10.0
• Component/s: Broker
• Assignee: Unassigned
• Reporter: Yuriy Sidelnikov
• Created: 07/Apr/14 10:42
• Updated: 07/Apr/14 14:58
• Resolved: 07/Apr/14 14:58

Description

In VMTransport class taskRunner is set to null two times needlessly on the next line and couple of lines below:
taskRunner = null;
messageQueue = null;

if (mq != null)

{ mq.clear(); }

// Allow pending deliveries to finish up, but don't wait
// forever in case of an stalled onCommand.
if (tr != null) {
try

{ tr.shutdown(TimeUnit.SECONDS.toMillis(1)); }

catch(Exception e) {
}
taskRunner = null;
}