Question

I’m hoping this is straightforward. I work on a large code-base, the overall quality is good, but occasionally you get some of these:

try
{
   // Calls a .NET remoting method.
}
catch
{
   throw;
}

Note there is no finally logic and the catch does not specify any exceptions or do anything other than what I’ve provided above. However, I know that catching and re-throwing can alter the call-stack in the exception details. What I’m not sure about is if this behaviour is here specifically because of a .NET remoting call.

Is it safe to remove this try-catch? So far as I can see, it is, but I thought I’d double check for any odd behaviour first.

Answer 1

As far as I know, catch (Exception ex) { throw ex } resets the stack-trace. And just catch { throw; } does not.

So if you don’t perform any additional logic on error, e.g. logging, I don’t know any reason to not remove that catch.