2012/3/9 Simone Tripodi <simonetrip...@apache.org>: > I don't think it is necessary when fixing this kind of issues, unless > you need to heavily change the code, but I don't suspect so ;) > let me know if I can be helpful in some way - more than just helping > on fixing issues :P
I have few doubts regarding checkstyle report, there are many issues related to JavaDocs and some line are too long and so one - I don't treat them as a big problems (especially with current formatting). But there are some strange errors like [2] which says: Name 'INDEXED_PROPERTY_NONE' must match pattern '^[a-z][a-zA-Z0-9]*$'. Variable 'INDEXED_PROPERTY_NONE' must be private and have accessor methods where the INDEXED_PROPERTY_NONE is defined as a constant: public static final int INDEXED_PROPERTY_NONE = 0; It looks like the checkstyle is configured in wrong way. Is there a Commons official checkstyle config ? Or am I free to adjust checkstyle included with OGNL ? [1] http://commons.apache.org/ognl/checkstyle.html [2] http://commons.apache.org/ognl/checkstyle.html#org.apache.commons.ognl.OgnlRuntime.java Regards -- Łukasz mobile +48 606 323 122 http://www.lenart.org.pl/ --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
