Update JavadocMethodCheck

[maxvetrenko]

http://checkstyle.sourceforge.net/config_javadoc.html#JavadocMethod Update existing Check to allow Override methods without Javadoc. Need to add allowNoJavaDocOnOverridenMethods option

[romani]

instead of hard-coding Override, It might be more useful to update a Check to let user provide names(RegExp or String[]) for annotations that allow missed javadoc.

Example : @Test, @Override, @VisibleForTesting , .....

[maxvetrenko]

@romani, Understood

[romani]

so provide me a example how configuration will looks like

[maxvetrenko]

@romani

<module name="JavadocMethodCheck">>
<property name="ignoreAnnotations" value="Test, Override"/>
</module>

[romani]

What is a reason to force user have "@" in front of each element in list?

[maxvetrenko]

@romani, I agree with you, there is no any reason to force "@". Removed.

[romani]

where is not tags in Java. It is annotation.

[maxvetrenko]

@romani, corrected config.

[romani]

annotationsWithoutJavadoc

do you check JavaDoc for annotations ?

please name option as "ignoreAnnotations".

similar naming could be found at http://checkstyle.sourceforge.net/config_sizes.html#LineLength, http://checkstyle.sourceforge.net/config_misc.html#Regexp

[maxvetrenko]

@romani renamed.