Hi Simon,
Simon Pepping wrote:
On Tue, Oct 20, 2009 at 10:04:09AM -, vhenneb...@apache.org wrote:
Author: vhennebert
Date: Tue Oct 20 10:04:09 2009
New Revision: 827023
URL: http://svn.apache.org/viewvc?rev=827023view=rev
Log:
Fixed checkstyle issues.
Factorized duplicated code into getXMLReader method.
Modified:
xmlgraphics/fop/trunk/src/java/org/apache/fop/cli/InputHandler.java
/**
- * Create a catalog resolver and use it for XML parsing and XSLT URI
resolution
+ * Creates a catalog resolver and use it for XML parsing and XSLT URI
resolution.
* Try the Apache Commons Resolver, and if unsuccessful,
* try the same built into Java 6
*/
The above text uses inconsistent language: creates, use, try.
Right. The original purpose of the change was to add the missing period
at the end of the first sentence, then I saw the missing ‘s’ [1] and
mechanically added it without checking the rest.
[1] http://java.sun.com/j2se/javadoc/writingdoccomments/index.html#styleguide
Thanks for double-checking,
Vincent
