Just curious — shouldn't we `break` before the line — or `return 0`?

If the code is `23000` but none of the messages match (e.g. the drivers change an error message), then technically it would continue in `42S02`.

Generally, I also realize that when you `return` from a `case`, the `break` would be shown as dead code, but from a code reading perspective, I think these should be added. But that's just my two cents.