[AMQNET-433] Correctly rethrow exceptions without swallowing the stack trace

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 1.6.0
• Component/s: ActiveMQ
• Assignee: Jim Gomes
• Reporter: Daniel Marbach
• Created: 26/Apr/13 12:24
• Updated: 30/Apr/13 05:46
• Resolved: 26/Apr/13 19:40

Description

When looking through the code I saw a lot of the following code snippets:

try
{

}
catch(AnyException ex)

{ // do something throw ex; } This WILL rewrite the stack trace and is not considered best practice. I suggest you change the appropriate places in the code to: try { }

catch(AnyException ex)
{
// do something
throw;
}