Issue #451: add TypeBodyPadding to checkstyle config

config/checkstyle-checks.xml

@@ -738,9 +738,6 @@
         <module name="JavadocMissingWhitespaceAfterAsterisk"/>
         <module name="JavadocLeadingAsteriskAlign"/>
         <module name="JavadocParagraph"/>
-        <module name="JavadocStyle">
-            <property name="scope" value="public"/>
-        </module>
         <module name="JavadocTagContinuationIndentation"/>
         <module name="JavadocType">
             <!-- avoid errors on tag '@noinspection' -->
@@ -1009,6 +1006,7 @@
         <module name="SingleSpaceSeparator">
           <property name="validateComments" value="false"/>
         </module>
+        <module name="TypeBodyPadding"/>
         <module name="TypecastParenPad"/>
         <module name="WhitespaceAfter">
             <property name="tokens"

net.sf.eclipsecs.checkstyle/src/test/java/net/sf/eclipsecs/checkstyle/ChecksTest.java

@@ -122,4 +122,5 @@ private static String getEclipseCsPath(String packageName, String fileName) thro
             "../net.sf.eclipsecs.checkstyle/metadata/" + packageName.replace(".", "/") + fileName)
                 .getCanonicalPath();
     }
+
 }

net.sf.eclipsecs.checkstyle/src/test/java/net/sf/eclipsecs/checkstyle/utils/CheckUtil.java

@@ -27,6 +27,7 @@
 import com.puppycrawl.tools.checkstyle.utils.ModuleReflectionUtil;
 
 public final class CheckUtil {
+
     private CheckUtil() {
     }
 
@@ -57,4 +58,5 @@ public static Set<Class<?>> getModulesInPackage(Set<Class<?>> modules, String pa
 
         return result;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/CheckstylePlugin.java

@@ -131,4 +131,5 @@ public static void setPlatformLocale(final Locale locale) {
     public ClassLoader getAddonExtensionClassLoader() {
         return mAddonExtensionClassLoader;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/CheckstylePluginPrefs.java

@@ -180,4 +180,5 @@ public static void setInt(String prefId, int value) throws BackingStoreException
         prefs.putInt(prefId, value);
         prefs.flush();
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/builder/Auditor.java

@@ -610,4 +610,5 @@ private String getRuleName(AuditEvent error) {
             return ruleName;
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/builder/CheckerFactory.java

@@ -303,4 +303,5 @@ private static String getLocale() {
         }
         return lang;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/builder/CheckstyleBuilder.java

@@ -413,4 +413,5 @@ private Collection<IResource> getResources(final IContainer container,
     public ISchedulingRule getRule(int kind, Map<String, String> args) {
         return getProject();
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/builder/CheckstyleMarker.java

@@ -29,6 +29,7 @@
  * Represents the Checkstyle file marker.
  */
 public final class CheckstyleMarker {
+
     /** ID for the CheckstyleMarker. */
     public static final String MARKER_ID = CheckstylePlugin.PLUGIN_ID + ".CheckstyleMarker";
 
@@ -80,4 +81,5 @@ public static boolean isCheckstyleMarker(IMarker marker) {
         }
         return isCheckstyleMarker;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/AbstractCheckConfiguration.java

@@ -66,4 +66,5 @@ public void copyConfiguration(ICheckConfiguration target) throws CheckstylePlugi
             CheckstylePluginException.rethrow(ex);
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/CheckConfiguration.java

@@ -195,4 +195,5 @@ public int hashCode() {
         return Objects.hash(getName(), getLocation(), getDescription(), getType(), isGlobal(),
             getResolvableProperties(), getAdditionalData());
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/CheckConfigurationFactory.java

@@ -402,4 +402,5 @@ private static List<ICheckConfiguration> getGlobalCheckConfigurations(Element ro
         }
         return configs;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/CheckConfigurationTester.java

@@ -139,4 +139,5 @@ public List<ResolvableProperty> getUnresolvedProperties() {
             return mUnresolvedProperties;
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/CheckConfigurationXmlWriter.java

@@ -67,4 +67,5 @@ public static void writeCheckConfiguration(Element parentElement,
             addEl.addAttribute(XMLTags.VALUE_TAG, entry.getValue());
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/ConfigurationReader.java

@@ -304,4 +304,5 @@ else if (XMLTags.LAST_ENABLED_SEVERITY_ID.equals(name)) {
             }
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/ConfigurationWriter.java

@@ -271,4 +271,5 @@ else if (childParent.equals(parentInternalName)) {
 
         return childModules;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/GlobalCheckConfigurationWorkingSet.java

@@ -346,4 +346,5 @@ private static Document createCheckConfigurationsDocument(
         }
         return doc;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/ICheckConfiguration.java

@@ -139,4 +139,5 @@ public interface ICheckConfiguration {
      *             Error copying the configuration
      */
     void copyConfiguration(ICheckConfiguration target) throws CheckstylePluginException;
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/ICheckConfigurationWorkingSet.java

@@ -114,4 +114,5 @@ public interface ICheckConfigurationWorkingSet {
      *             unexprected error
      */
     Collection<IProject> getAffectedProjects() throws CheckstylePluginException;
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/Module.java

@@ -322,4 +322,5 @@ else if (Severity.IGNORE == severityLevel) {
     public Map<String, String> getCustomMetaData() {
         return mCustomMetaData;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/ResolvableProperty.java

@@ -120,4 +120,5 @@ public String toString() {
         return MoreObjects.toStringHelper(this).add("propertyName", propertyName)
             .add("value", value).toString();
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/AbstractConfigurationType.java

@@ -258,4 +258,5 @@ public boolean equals(Object obj) {
     public int hashCode() {
         return Objects.hash(mName, mInternalName, mIsCreatable, mIsEditable, mIsConfigurable);
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/BuiltInConfigurationType.java

@@ -79,4 +79,5 @@ protected PropertyResolver getPropertyResolver(ICheckConfiguration config,
 
         return resolver;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/ClasspathVariableResolver.java

@@ -43,4 +43,5 @@ public String resolve(String name) {
         }
         return resolvedPath;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/ConfigurationTypes.java

@@ -174,4 +174,5 @@ public static IConfigurationType getByName(String name) {
         }
         return result;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/IConfigurationType.java

@@ -127,4 +127,5 @@ URL getResolvedConfigurationFileURL(ICheckConfiguration checkConfiguration)
      */
     CheckstyleConfigurationFile getCheckstyleConfiguration(ICheckConfiguration checkConfiguration)
             throws CheckstylePluginException;
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/IContextAware.java

@@ -35,4 +35,5 @@ public interface IContextAware {
      *            the project context
      */
     void setProjectContext(IProject project);
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/InternalConfigurationType.java

@@ -79,4 +79,5 @@ public void notifyCheckConfigRemoved(ICheckConfiguration checkConfiguration)
             }
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/ProjectConfigurationType.java

@@ -85,4 +85,5 @@ public boolean isConfigurable(ICheckConfiguration checkConfiguration) {
         }
         return isConfigurable;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/PropertyUtil.java

@@ -179,4 +179,5 @@ else if (aValue.charAt(pos + 1) != '{') {
             aFragments.add(aValue.substring(prev));
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/RemoteConfigurationType.java

@@ -533,4 +533,5 @@ public String getPassword() {
         }
 
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/ResourceBundlePropertyResolver.java

@@ -60,4 +60,5 @@ public String resolve(String property) {
 
         return value;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/StandardPropertyResolver.java

@@ -90,4 +90,5 @@ else if (mConfigLocation != null
 
         return value;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/configtypes/SystemPropertyResolver.java

@@ -33,4 +33,5 @@ public String resolve(String property) {
         // get the system property
         return System.getProperty(property);
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/meta/CheckUtil.java

@@ -41,6 +41,7 @@
  *
  */
 public final class CheckUtil {
+
     /**
      * Utility class, not intended to be instantiated.
      */
@@ -129,4 +130,5 @@ private static String getTokens(Function<Integer, String> function,
         List<Integer> modifiableTokens) {
         return modifiableTokens.stream().map(function::apply).collect(Collectors.joining(","));
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/meta/ConfigPropertyType.java

@@ -76,4 +76,5 @@ static ConfigPropertyType fromXmlValue(String xmlValue) {
             default -> throw new IllegalArgumentException("Unexpected value: " + xmlValue);
         };
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/meta/IOptionProvider.java

@@ -36,4 +36,5 @@ public interface IOptionProvider {
      * @return the options
      */
     List<String> getOptions();
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/meta/RuleGroupMetadata.java

@@ -124,4 +124,5 @@ public final List<RuleMetadata> getRuleMetadata() {
     public String getGroupId() {
         return mGroupId;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/meta/ThirdPartyRuleGroupParser.java

@@ -55,4 +55,5 @@ static Map<String, ThirdPartyRuleGroupInfo> parse(String metadataContent) {
         }
         return ruleGroups;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/savefilter/CheckerModuleSaveFilter.java

@@ -52,4 +52,5 @@ public void postProcessConfiguredModules(List<Module> configuredModules) {
             configuredModules.add(0, checker);
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/savefilter/ISaveFilter.java

@@ -39,4 +39,5 @@ public interface ISaveFilter {
      *            the configured modules in from this configuration
      */
     void postProcessConfiguredModules(List<Module> configuredModules);
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/savefilter/SaveFilters.java

@@ -93,4 +93,5 @@ public static void process(List<Module> configuredModules) {
             filter.postProcessConfiguredModules(configuredModules);
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/savefilter/SortingSaveFilter.java

@@ -66,4 +66,5 @@ else if (XMLTags.CHECKER_MODULE.equals(internalName2)
             return comp;
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/savefilter/SuppressWarningsHolderSaveFilter.java

@@ -68,4 +68,5 @@ else if (containsSuppressWarningsHolderModule && !needsSuppressWarningsHolderMod
             configuredModules.remove(configuredSuppressWarningsHolder);
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/config/savefilter/TreeWalkerModuleSaveFilter.java

@@ -66,4 +66,5 @@ else if (containsTreeWalkerModule && !containsTreeWalkerDependantModule) {
             configuredModules.remove(configuredTreeWalker);
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/jobs/AbstractCheckJob.java

@@ -28,6 +28,7 @@
  * progress UI.
  */
 public abstract class AbstractCheckJob extends WorkspaceJob implements ISchedulingRule {
+
     /**
      * The job family marker is used by the progress service to provide different icons.
      */

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/jobs/BuildProjectJob.java

@@ -99,4 +99,5 @@ public IStatus run(IProgressMonitor monitor) {
 
         return status;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/jobs/RunCheckstyleOnFilesJob.java

@@ -163,4 +163,5 @@ private static void filter(final List<IFile> files, final IProjectConfiguration
             }
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/FileMatchPattern.java

@@ -157,4 +157,5 @@ public String toString() {
         return MoreObjects.toStringHelper(this).add("patternString", patternString)
             .add("isIncludePattern", isIncludePattern).toString();
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/FileSet.java

@@ -211,4 +211,5 @@ public String toString() {
         return MoreObjects.toStringHelper(this).add("enabled", enabled).add("name", name)
             .add("fileMatchPatterns", fileMatchPatterns).add("checkConfig", checkConfig).toString();
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/IProjectConfiguration.java

@@ -87,4 +87,5 @@ public interface IProjectConfiguration {
      * @return <code>true</code> if checkstyle settings are synched into formatter settings.
      */
     boolean isSyncFormatter();
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/PluginFilters.java

@@ -159,4 +159,5 @@ public static IFilter getByInternalName(String internalName) {
 
         return filter;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/ProjectConfiguration.java

@@ -203,4 +203,5 @@ public String toString() {
             .add("syncFormatter", syncFormatter).add("fileSets", fileSets).add("filters", filters)
             .toString();
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/ProjectConfigurationFactory.java

@@ -390,4 +390,5 @@ private static List<IFilter> getFilters(Element root) {
 
         return filters;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/ProjectConfigurationWorkingCopy.java

@@ -496,4 +496,5 @@ private void writeFilter(IFilter filter, Element docRoot) {
             }
         }
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/filters/AbstractFilter.java

@@ -165,4 +165,5 @@ public String toString() {
             .add("internalName", mInternalName).add("filterDescription", mFilterDescription)
             .add("selected", mSelected).add("readonly", mReadonly).toString();
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/filters/DerivedFilesFilter.java

@@ -59,4 +59,5 @@ public boolean accept(Object element) {
         }
         return goesThrough;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/filters/FilesInSyncFilter.java

@@ -104,4 +104,5 @@ private boolean hasChanges(IFile file, Subscriber subscriber) {
         }
         return hasChanges;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/filters/NonSrcDirsFilter.java

@@ -118,4 +118,5 @@ private List<IPath> getSourceDirPaths(IProject project) {
 
         return sourceDirs;
     }
+
 }

net.sf.eclipsecs.core/src/main/java/net/sf/eclipsecs/core/projectconfig/filters/PackageFilter.java

@@ -155,4 +155,5 @@ public boolean equals(Object o) {
     public int hashCode() {
         return Objects.hash(super.hashCode(), mData);
     }
+
 }