[wildcards] Consider quick fix: convert UNUSED_CATCH_CLAUSE to wildcard

[pq]

Specifically,

f() {
  try {
  } on String catch (exception) {
  }
}

might be fixed to:

f() {
  try {
  } on String catch (_) {
  }
}

in addition to the current fix which removes the clause altogether:

f() {
  try {
  } on String {
  }
}

Question: does such a fix add value?

@kallentu @bwilkerson

[bwilkerson]

The two fixes seem to me to be semantically equivalent. The code will be executed if a String is thrown, and in neither case can the user refer to either the thrown object or the stack trace.

I would want an unnecessary_catch_clause warning on the longer version.

[pq]

I would want an unnecessary_catch_clause warning on the longer version.

In that case we don't want a fix to get us to it. My feeling too. Thanks!