Description: ------------ Whether to report an error vs. warning is wrong. An empty catch statement makes sense, so it should be warning, anything else an error. It's exactly the opposite at the moment.

I didn't write that sniff and I've never used it so I can't comment on the authors intentions. But despite the fact that the author wrote unit tests to enforce errors and warnings in a particular way, I agree that it seems backwards. I can see uses cases for empty catch statements, but not for empty foreach, for, do etc statements. So I committed a fix to the github repo here: https://github.com/squizlabs/PHP_CodeSniffer/commit/44f7cc86ea688f2726ce5c798bf ecf960c5ad5cc I just reversed the severity levels and changed the unit tests.