Is this javadoc ok?