catch (Exception exception) { throw new Exception(exception.ToString()); }
I'm not sure what it is about exception handling but it seems to be the most frequently mis-understood/mis-used area of programming C#. The lines above are not nearly has bad as:
catch (Exception exception) { }
which I've seen a lot of too, but it's more of a sigh moment than laugh.