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/ _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs

