checkstyle corrections

Former-commit-id: 2f6fb660cd0152de284b55de3aab9cbb1b22b0b0
This commit is contained in:
Jeremy Long
2014-04-22 08:10:54 -04:00
parent 228bb2fc86
commit 9fe596f3de
22 changed files with 37 additions and 26 deletions

View File

@@ -40,12 +40,12 @@ import org.owasp.dependencycheck.utils.Settings;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public abstract class AbstractSuppressionAnalyzer extends AbstractAnalyzer { public abstract class AbstractSuppressionAnalyzer extends AbstractAnalyzer {
/** /**
* The Logger for use throughout the class * The Logger for use throughout the class
*/ */
private static final Logger LOGGER = Logger.getLogger(AbstractSuppressionAnalyzer.class.getName()); private static final Logger LOGGER = Logger.getLogger(AbstractSuppressionAnalyzer.class.getName());
//<editor-fold defaultstate="collapsed" desc="All standard implementation details of Analyzer"> //<editor-fold defaultstate="collapsed" desc="All standard implementation details of Analyzer">
/** /**
* Returns a list of file EXTENSIONS supported by this analyzer. * Returns a list of file EXTENSIONS supported by this analyzer.

View File

@@ -45,11 +45,12 @@ import org.owasp.dependencycheck.utils.LogUtils;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class DependencyBundlingAnalyzer extends AbstractAnalyzer implements Analyzer { public class DependencyBundlingAnalyzer extends AbstractAnalyzer implements Analyzer {
/** /**
* The Logger. * The Logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(DependencyBundlingAnalyzer.class.getName()); private static final Logger LOGGER = Logger.getLogger(DependencyBundlingAnalyzer.class.getName());
//<editor-fold defaultstate="collapsed" desc="Constants and Member Variables"> //<editor-fold defaultstate="collapsed" desc="Constants and Member Variables">
/** /**
* A pattern for obtaining the first part of a filename. * A pattern for obtaining the first part of a filename.

View File

@@ -26,7 +26,6 @@ import java.util.Properties;
import java.util.TreeMap; import java.util.TreeMap;
import java.util.logging.Level; import java.util.logging.Level;
import java.util.logging.Logger; import java.util.logging.Logger;
import org.owasp.dependencycheck.data.update.NvdCveInfo; import org.owasp.dependencycheck.data.update.NvdCveInfo;
import org.owasp.dependencycheck.data.update.exception.UpdateException; import org.owasp.dependencycheck.data.update.exception.UpdateException;
@@ -36,7 +35,7 @@ import org.owasp.dependencycheck.data.update.exception.UpdateException;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class DatabaseProperties { public class DatabaseProperties {
/** /**
* The Logger. * The Logger.
*/ */

View File

@@ -36,11 +36,12 @@ import java.util.logging.Logger;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public final class DriverLoader { public final class DriverLoader {
/** /**
* The logger. * The logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(DriverLoader.class.getName()); private static final Logger LOGGER = Logger.getLogger(DriverLoader.class.getName());
/** /**
* Private constructor for a utility class. * Private constructor for a utility class.
*/ */

View File

@@ -38,7 +38,7 @@ import java.util.logging.Logger;
* @see java.sql.Driver * @see java.sql.Driver
*/ */
class DriverShim implements Driver { class DriverShim implements Driver {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -29,11 +29,12 @@ import org.owasp.dependencycheck.utils.DownloadFailedException;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class NvdCveUpdater implements CachedWebDataSource { public class NvdCveUpdater implements CachedWebDataSource {
/** /**
* The logger * The logger
*/ */
private static final Logger LOGGER = Logger.getLogger(NvdCveUpdater.class.getName()); private static final Logger LOGGER = Logger.getLogger(NvdCveUpdater.class.getName());
/** /**
* <p> * <p>
* Downloads the latest NVD CVE XML file from the web and imports it into the current CVE Database.</p> * Downloads the latest NVD CVE XML file from the web and imports it into the current CVE Database.</p>

View File

@@ -46,6 +46,10 @@ import org.owasp.dependencycheck.utils.Settings;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class StandardUpdate { public class StandardUpdate {
/**
* Static logger.
*/
private static final Logger LOGGER = Logger.getLogger(StandardUpdate.class.getName()); private static final Logger LOGGER = Logger.getLogger(StandardUpdate.class.getName());
/** /**
* The max thread pool size to use when downloading files. * The max thread pool size to use when downloading files.

View File

@@ -38,11 +38,12 @@ import org.owasp.dependencycheck.utils.Settings;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class CallableDownloadTask implements Callable<Future<ProcessTask>> { public class CallableDownloadTask implements Callable<Future<ProcessTask>> {
/** /**
* The Logger. * The Logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(CallableDownloadTask.class.getName()); private static final Logger LOGGER = Logger.getLogger(CallableDownloadTask.class.getName());
/** /**
* Simple constructor for the callable download task. * Simple constructor for the callable download task.
* *

View File

@@ -32,7 +32,6 @@ import javax.xml.parsers.SAXParserFactory;
import org.owasp.dependencycheck.data.nvdcve.CveDB; import org.owasp.dependencycheck.data.nvdcve.CveDB;
import org.owasp.dependencycheck.data.nvdcve.DatabaseException; import org.owasp.dependencycheck.data.nvdcve.DatabaseException;
import org.owasp.dependencycheck.data.nvdcve.DatabaseProperties; import org.owasp.dependencycheck.data.nvdcve.DatabaseProperties;
import org.owasp.dependencycheck.data.update.StandardUpdate;
import org.owasp.dependencycheck.data.update.exception.UpdateException; import org.owasp.dependencycheck.data.update.exception.UpdateException;
import org.owasp.dependencycheck.data.update.xml.NvdCve12Handler; import org.owasp.dependencycheck.data.update.xml.NvdCve12Handler;
import org.owasp.dependencycheck.data.update.xml.NvdCve20Handler; import org.owasp.dependencycheck.data.update.xml.NvdCve20Handler;
@@ -46,7 +45,7 @@ import org.xml.sax.SAXException;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class ProcessTask implements Callable<ProcessTask> { public class ProcessTask implements Callable<ProcessTask> {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -39,7 +39,7 @@ import org.xml.sax.helpers.DefaultHandler;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class NvdCve20Handler extends DefaultHandler { public class NvdCve20Handler extends DefaultHandler {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -36,7 +36,7 @@ import org.owasp.dependencycheck.utils.FileUtils;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class Dependency implements Comparable<Dependency> { public class Dependency implements Comparable<Dependency> {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -37,7 +37,7 @@ import org.owasp.dependencycheck.utils.UrlStringUtils;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class EvidenceCollection implements Iterable<Evidence> { public class EvidenceCollection implements Iterable<Evidence> {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -30,7 +30,7 @@ import org.owasp.dependencycheck.data.cpe.IndexEntry;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class VulnerableSoftware extends IndexEntry implements Serializable, Comparable<VulnerableSoftware> { public class VulnerableSoftware extends IndexEntry implements Serializable, Comparable<VulnerableSoftware> {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -30,11 +30,12 @@ import org.apache.commons.lang.StringEscapeUtils;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class EscapeTool { public class EscapeTool {
/** /**
* The logger. * The logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(EscapeTool.class.getName()); private static final Logger LOGGER = Logger.getLogger(EscapeTool.class.getName());
/** /**
* URL Encodes the provided text. * URL Encodes the provided text.
* *

View File

@@ -49,11 +49,12 @@ import org.owasp.dependencycheck.utils.Settings;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class ReportGenerator { public class ReportGenerator {
/** /**
* The logger. * The logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(ReportGenerator.class.getName()); private static final Logger LOGGER = Logger.getLogger(ReportGenerator.class.getName());
/** /**
* An enumeration of the report formats. * An enumeration of the report formats.
*/ */

View File

@@ -19,7 +19,6 @@ package org.owasp.dependencycheck.reporting;
import java.util.logging.Level; import java.util.logging.Level;
import java.util.logging.Logger; import java.util.logging.Logger;
import org.apache.velocity.app.Velocity;
import org.apache.velocity.runtime.RuntimeServices; import org.apache.velocity.runtime.RuntimeServices;
import org.apache.velocity.runtime.log.LogChute; import org.apache.velocity.runtime.log.LogChute;
@@ -36,11 +35,12 @@ import org.apache.velocity.runtime.log.LogChute;
* @author Steve Springett <steve.springett@owasp.org> * @author Steve Springett <steve.springett@owasp.org>
*/ */
public class VelocityLoggerRedirect implements LogChute { public class VelocityLoggerRedirect implements LogChute {
/** /**
* The Logger. * The Logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(VelocityLoggerRedirect.class.getName()); private static final Logger LOGGER = Logger.getLogger(VelocityLoggerRedirect.class.getName());
/** /**
* This will be invoked once by the LogManager. * This will be invoked once by the LogManager.
* *

View File

@@ -29,11 +29,12 @@ import org.xml.sax.SAXParseException;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class SuppressionErrorHandler implements ErrorHandler { public class SuppressionErrorHandler implements ErrorHandler {
/** /**
* The logger. * The logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(SuppressionErrorHandler.class.getName()); private static final Logger LOGGER = Logger.getLogger(SuppressionErrorHandler.class.getName());
/** /**
* Builds a prettier exception message. * Builds a prettier exception message.
* *

View File

@@ -40,7 +40,7 @@ import org.xml.sax.XMLReader;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public class SuppressionParser { public class SuppressionParser {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -23,7 +23,6 @@ import java.sql.SQLException;
import java.sql.Statement; import java.sql.Statement;
import java.util.logging.Level; import java.util.logging.Level;
import java.util.logging.Logger; import java.util.logging.Logger;
import org.owasp.dependencycheck.data.nvdcve.CveDB;
import org.owasp.dependencycheck.data.nvdcve.DatabaseException; import org.owasp.dependencycheck.data.nvdcve.DatabaseException;
/** /**
@@ -31,11 +30,12 @@ import org.owasp.dependencycheck.data.nvdcve.DatabaseException;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public final class DBUtils { public final class DBUtils {
/** /**
* The logger. * The logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(DBUtils.class.getName()); private static final Logger LOGGER = Logger.getLogger(DBUtils.class.getName());
/** /**
* Private constructor for a utility class. * Private constructor for a utility class.
*/ */

View File

@@ -36,11 +36,12 @@ import java.util.zip.InflaterInputStream;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public final class Downloader { public final class Downloader {
/** /**
* The logger. * The logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(Downloader.class.getName()); private static final Logger LOGGER = Logger.getLogger(Downloader.class.getName());
/** /**
* Private constructor for utility class. * Private constructor for utility class.
*/ */

View File

@@ -39,7 +39,7 @@ import org.owasp.dependencycheck.Engine;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public final class FileUtils { public final class FileUtils {
/** /**
* The logger. * The logger.
*/ */

View File

@@ -31,11 +31,12 @@ import java.util.logging.SimpleFormatter;
* @author Jeremy Long <jeremy.long@owasp.org> * @author Jeremy Long <jeremy.long@owasp.org>
*/ */
public final class LogUtils { public final class LogUtils {
/** /**
* The logger. * The logger.
*/ */
private static final Logger LOGGER = Logger.getLogger(LogUtils.class.getName()); private static final Logger LOGGER = Logger.getLogger(LogUtils.class.getName());
/** /**
* Private constructor for a utility class. * Private constructor for a utility class.
*/ */