Rick Giles wrote:
>I added Simon's ExecutableStatementCountCheck to the metrics package
>which makes at least three packages that contain "counting" checks,
>namely coding, metrics, and sizes. Should all "counting checks" be in
>the same package?
No, I don't think so. The current package structure is quite ok, it
corresponds to the toplevel feature list on the home page ("things
checkstyle can check for are ... "). Counting is not a toplevel feature
of checkstyle :-)
I would consider moving ExecutableStatementCountCheck to the sizes
package, though - functionality is very similar to MethodLengthCheck, so
you'd expect it nearby. That might be personal taste, though...
BTW: Great to see that you and Oleg have been so active recently,
especially since Oliver and myself currently don't have much time available.