chore: fixes linting warnings

[scottr1634]

Removes some unused code such as:

• break after continue
• return after throwing an error
• double return statements

Removes new declaration where it is not necessary.

Also comments out some unused variables.

Lastly, suppresses some unused variable warnings, either because they are "Required by the Windows Form Designer" or they are errors in catch blocks that we may want to surface in the future.

[KllrWhle79]

I would advocate for just removing unused code instead of commenting it out. We can always roll back here if we need to get it back.