Wow, I just got back a huge project in C# from outsourced developers and while going through my code review my analysis tool revealed bunches of what it considered bad stuff
My feeling is that any empty Catch Block needs a comment.
Possibly it's valid to ignore certain errors, but you need to document your reasons.
Also, you wouldn't want to make it a generic "catch (Exception e) { }".
You should catch only the specific error type that's expected there and is known to be safely ignored.