Make SingleCatchAllException opt-in

Vannevelj / VSDiagnostics

A collection of static analyzers based on Roslyn that integrate with VS

GNU General Public License v2.0

65 stars 16 forks source link

Closed Vannevelj closed 8 years ago

Vannevelj commented 8 years ago

Catch all exceptions aren't all that bad. I think we should disable it by default and let people enable it if they want it.

Hosch250 commented 8 years ago

Perhaps a Hidden would be good? No marker, and only available when the cursor is on the catch token?

Vannevelj commented 8 years ago

There's no code fix, just an analyzer. Does anything happen at all then if you make it hidden?

Hosch250 commented 8 years ago

I don't think so, but I can't say for sure. Disabled sounds good then.

Vannevelj commented 8 years ago

closed in #597