On 04/15/2010 12:55 PM, Caleb James DeLisle wrote:
I would like to propose changing checkstyle.xml to allow checkstyle to be enabled and disabled using inline comments. I propose this with some reservation because this can be a slippery slope but I would rather see the problems isolated in the file than the file excluded. Also I think this can be used to enforce checkstyle on changes made to code in big files in the old core.
I don't like it very much, since it adds non-code elements in the source file. A source file should contain as little languages as possible, and we already have Java + Javadoc + HTML + English comments, adding checkstyle config would be too much. Can't the error be solved in other ways? What exactly is the problem? -- Sergiu Dumitriu http://purl.org/net/sergiu/