Clean up the error reporting in the cargo clippy code

[oli-obk]

Currently we panic in arbitrary places that may or may not have to do with user input. Instead we should move to Result based error reporting for all the cases we consider reachable and not print a stacktrace but a nice error message instead (and call process::exit with a nonzero exit code).

[camsteffen]

Closing as obsolete since much has changed