Created
December 28, 2013 05:09
-
-
Save hgschmie/8156275 to your computer and use it in GitHub Desktop.
current list of problems in vert.x
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[INFO] --- findbugs-maven-plugin:2.5.3:check (default) @ vertx-core --- | |
[INFO] BugInstance size is 3 | |
[INFO] Error size is 0 | |
[INFO] Total bugs: 3 | |
[INFO] Inconsistent synchronization of org.vertx.java.core.http.impl.PriorityHttpConnectionPool.maxPoolSize; locked 60% of time ["org.vertx.java.core.http.impl.PriorityHttpConnectionPool"] At PriorityHttpConnectionPool.java:[lines 32-176] | |
[INFO] Inconsistent synchronization of org.vertx.java.core.sockjs.impl.Session.dataHandler; locked 50% of time ["org.vertx.java.core.sockjs.impl.Session"] At Session.java:[lines 43-373] | |
[INFO] Inconsistent synchronization of org.vertx.java.core.sockjs.impl.Session.paused; locked 66% of time ["org.vertx.java.core.sockjs.impl.Session"] At Session.java:[lines 43-373] | |
[INFO] --- findbugs-maven-plugin:2.5.3:check (default) @ vertx-hazelcast --- | |
[INFO] BugInstance size is 2 | |
[INFO] Error size is 0 | |
[INFO] Total bugs: 2 | |
[INFO] Inconsistent synchronization of org.vertx.java.spi.cluster.impl.hazelcast.HazelcastClusterManager.hazelcast; locked 55% of time ["org.vertx.java.spi.cluster.impl.hazelcast.HazelcastClusterManager"] At HazelcastClusterManager.java:[lines 45-191] | |
[INFO] Inconsistent synchronization of org.vertx.java.spi.cluster.impl.hazelcast.HazelcastClusterManager.nodeListener; locked 80% of time ["org.vertx.java.spi.cluster.impl.hazelcast.HazelcastClusterManager"] At HazelcastClusterManager.java:[lines 45-191] | |
[INFO] --- findbugs-maven-plugin:2.5.3:check (default) @ vertx-platform --- | |
[INFO] BugInstance size is 25 | |
[INFO] Error size is 0 | |
[INFO] Total bugs: 25 | |
[INFO] Unread public/protected field: org.vertx.java.busmods.BusModBase.eb ["org.vertx.java.busmods.BusModBase"] At BusModBase.java:[lines 32-153] | |
[INFO] org.vertx.java.platform.PlatformLocator.factory isn't final but should be ["org.vertx.java.platform.PlatformLocator"] At PlatformLocator.java:[lines 27-29] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.DefaultPlatformManager.loadModJSONFromClasspath(ModuleIdentifier, ClassLoader): new java.util.Scanner(InputStream) ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.DefaultPlatformManager.loadModuleConfig(ModuleIdentifier, File): new java.util.Scanner(File) ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.DefaultPlatformManager.loadRepos(): new java.io.InputStreamReader(InputStream) ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] Load of known null value in org.vertx.java.platform.impl.DefaultPlatformManager.deployModuleFromFileSystem(File, String, ModuleIdentifier, JsonObject, int, File, boolean, Handler) ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] org.vertx.java.platform.impl.DefaultPlatformManager.loadLanguageMappings() may fail to close stream ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] org.vertx.java.platform.impl.DefaultPlatformManager.loadRepos() may fail to close stream ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] Redundant nullcheck of conf, which is known to be non-null in org.vertx.java.platform.impl.DefaultPlatformManager.doPullInDependencies(File, ModuleIdentifier) ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] Exception is caught when Exception is not thrown in org.vertx.java.platform.impl.DefaultPlatformManager.doMakeFatJar(File, ModuleIdentifier, String) ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] Exception is caught when Exception is not thrown in org.vertx.java.platform.impl.DefaultPlatformManager.unzipModuleData(File, DefaultPlatformManager$ModuleZipInfo, boolean) ["org.vertx.java.platform.impl.DefaultPlatformManager"] At DefaultPlatformManager.java:[lines 58-1745] | |
[INFO] Exceptional return value of java.io.File.mkdirs() ignored in org.vertx.java.platform.impl.DefaultPlatformManager$9.run() ["org.vertx.java.platform.impl.DefaultPlatformManager$9"] At DefaultPlatformManager.java:[lines 344-359] | |
[INFO] new org.vertx.java.platform.impl.Deployment(String, String, ModuleIdentifier, int, JsonObject, URL[], File, String, ModuleReference, boolean, boolean) may expose internal representation by storing an externally mutable object into Deployment.classpath ["org.vertx.java.platform.impl.Deployment"] At Deployment.java:[lines 37-59] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.FatJarStarter.go(String[]): new java.util.Scanner(File) ["org.vertx.java.platform.impl.FatJarStarter"] At FatJarStarter.java:[lines 42-270] | |
[INFO] Exceptional return value of java.io.File.mkdirs() ignored in org.vertx.java.platform.impl.FatJarStarter.unzipJar(File, String) ["org.vertx.java.platform.impl.FatJarStarter"] At FatJarStarter.java:[lines 42-270] | |
[INFO] Nullcheck of deployments at line 419 of value previously dereferenced in org.vertx.java.platform.impl.HAManager.checkFailover(String, JsonObject) ["org.vertx.java.platform.impl.HAManager"] At HAManager.java:[lines 101-499] | |
[INFO] Boxing/unboxing to parse a primitive org.vertx.java.platform.impl.cli.Starter.startPM(boolean, boolean, Args) ["org.vertx.java.platform.impl.cli.Starter"] At Starter.java:[lines 45-564] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.cli.Starter.runVerticle(boolean, boolean, String, Args): new java.util.Scanner(File) ["org.vertx.java.platform.impl.cli.Starter"] At Starter.java:[lines 45-564] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.java.CustomJavaFileObject.openWriter(): new java.io.OutputStreamWriter(OutputStream) ["org.vertx.java.platform.impl.java.CustomJavaFileObject"] At CustomJavaFileObject.java:[lines 33-100] | |
[INFO] org.vertx.java.platform.impl.java.JavaVerticleFactory.createVerticle(String) creates a org.vertx.java.platform.impl.java.CompilingClassLoader classloader, which should be performed within a doPrivileged block ["org.vertx.java.platform.impl.java.JavaVerticleFactory", "org.vertx.java.platform.impl.java.CompilingClassLoader"] At JavaVerticleFactory.java:[lines 28-68]At CompilingClassLoader.java:[lines 37-100] | |
[INFO] Boxing/unboxing to parse a primitive org.vertx.java.platform.impl.resolver.HttpResolution.downloadToFile(String, HttpClientResponse) ["org.vertx.java.platform.impl.resolver.HttpResolution"] At HttpResolution.java:[lines 44-275] | |
[INFO] Boxing/unboxing to parse a primitive org.vertx.java.platform.impl.resolver.HttpResolution.getProxyPort() ["org.vertx.java.platform.impl.resolver.HttpResolution"] At HttpResolution.java:[lines 44-275] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.resolver.HttpResolution.getBasicAuth(): String.getBytes() ["org.vertx.java.platform.impl.resolver.HttpResolution"] At HttpResolution.java:[lines 44-275] | |
[INFO] org.vertx.java.platform.impl.resolver.HttpResolution.suppressDownloadCounter isn't final and can't be protected from malicious code ["org.vertx.java.platform.impl.resolver.HttpResolution"] At HttpResolution.java:[lines 44-275] | |
[INFO] Found reliance on default encoding in org.vertx.java.platform.impl.resolver.MavenLocalRepoResolver.getModuleForMetaData(String, ModuleIdentifier, File, String): new java.util.Scanner(File) ["org.vertx.java.platform.impl.resolver.MavenLocalRepoResolver"] At MavenLocalRepoResolver.java:[lines 31-111] | |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
<?xml version="1.0" encoding="UTF-8"?> | |
<pmd version="5.0.5" timestamp="2013-12-27T20:53:27.877"> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/datagram/impl/DefaultDatagramSocket.java"> | |
<violation beginline="22" endline="22" begincolumn="8" endcolumn="53" rule="DuplicateImports" ruleset="Import Statements" package="org.vertx.java.core.datagram.impl" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/imports.html#DuplicateImports" priority="4"> | |
Avoid duplicate imports such as 'io.netty.channel.socket.InternetProtocolFamily' | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/dns/impl/DefaultDnsClient.java"> | |
<violation beginline="25" endline="25" begincolumn="8" endcolumn="46" rule="DuplicateImports" ruleset="Import Statements" package="org.vertx.java.core.dns.impl" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/imports.html#DuplicateImports" priority="4"> | |
Avoid duplicate imports such as 'org.vertx.java.core.dns.DnsResponseCode' | |
</violation> | |
<violation beginline="174" endline="174" begincolumn="51" endcolumn="53" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.dns.impl" class="DefaultDnsClient" method="reverseLookup" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="175" endline="175" begincolumn="47" endcolumn="49" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.dns.impl" class="DefaultDnsClient" method="reverseLookup" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="176" endline="176" begincolumn="47" endcolumn="49" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.dns.impl" class="DefaultDnsClient" method="reverseLookup" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="182" endline="182" begincolumn="30" endcolumn="32" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.dns.impl" class="DefaultDnsClient" method="reverseLookup" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="185" endline="185" begincolumn="32" endcolumn="34" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.dns.impl" class="DefaultDnsClient" method="reverseLookup" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="221" endline="221" begincolumn="16" endcolumn="83" rule="UnusedPrivateMethod" ruleset="Unused Code" package="org.vertx.java.core.dns.impl" class="DefaultDnsClient" method="lookup" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateMethod" priority="3"> | |
Avoid unused private methods such as 'lookup(String,Handler,int...)'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/dns/impl/netty/DnsResponseDecoder.java"> | |
<violation beginline="54" endline="54" begincolumn="93" endcolumn="95" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.dns.impl.netty" class="DnsResponseDecoder" method="readName" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="85" endline="85" begincolumn="82" endcolumn="84" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.dns.impl.netty" class="DnsResponseDecoder" method="getName" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/dns/impl/netty/decoder/AddressDecoder.java"> | |
<violation beginline="70" endline="71" begincolumn="13" endcolumn="103" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.dns.impl.netty.decoder" class="AddressDecoder" method="decode" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/eventbus/impl/DefaultEventBus.java"> | |
<violation beginline="821" endline="822" begincolumn="7" endcolumn="5" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.eventbus.impl" class="DefaultEventBus" method="cleanupConnection" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/file/impl/DefaultFileSystem.java"> | |
<violation beginline="477" endline="477" begincolumn="11" endcolumn="82" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.file.impl" class="DefaultFileSystem" method="action" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="501" endline="501" begincolumn="11" endcolumn="82" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.file.impl" class="DefaultFileSystem" method="action" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/http/impl/ClientConnection.java"> | |
<violation beginline="245" endline="247" begincolumn="29" endcolumn="5" rule="EmptyIfStmt" ruleset="Empty Code" package="org.vertx.java.core.http.impl" class="ClientConnection" method="close" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyIfStmt" priority="3"> | |
Avoid empty if statements | |
</violation> | |
<violation beginline="333" endline="335" begincolumn="13" endcolumn="3" rule="UselessOverridingMethod" ruleset="Unnecessary" package="org.vertx.java.core.http.impl" class="ClientConnection" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unnecessary.html#UselessOverridingMethod" priority="3"> | |
Overriding method merely calls super | |
</violation> | |
<violation beginline="364" endline="366" begincolumn="12" endcolumn="5" rule="EmptyIfStmt" ruleset="Empty Code" package="org.vertx.java.core.http.impl" class="ClientConnection" method="endRequest" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyIfStmt" priority="3"> | |
Avoid empty if statements | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/http/impl/DefaultHttpClient.java"> | |
<violation beginline="759" endline="760" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.http.impl" class="DefaultHttpClient" method="run" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="805" endline="825" begincolumn="9" endcolumn="9" rule="SwitchStmtsShouldHaveDefault" ruleset="Design" package="org.vertx.java.core.http.impl" class="DefaultHttpClient$ClientHandler" method="doMessageReceived" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SwitchStmtsShouldHaveDefault" priority="3"> | |
Switch statements should have a default label | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/http/impl/DefaultHttpServer.java"> | |
<violation beginline="173" endline="186" begincolumn="17" endcolumn="17" rule="SwitchStmtsShouldHaveDefault" ruleset="Design" package="org.vertx.java.core.http.impl" class="DefaultHttpServer" method="initChannel" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SwitchStmtsShouldHaveDefault" priority="3"> | |
Switch statements should have a default label | |
</violation> | |
<violation beginline="614" endline="614" begincolumn="44" endcolumn="93" rule="UseLocaleWithCaseConversions" ruleset="Design" package="org.vertx.java.core.http.impl" class="DefaultHttpServer$ServerHandler" method="doMessageReceived" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#UseLocaleWithCaseConversions" priority="3"> | |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale | |
</violation> | |
<violation beginline="649" endline="671" begincolumn="9" endcolumn="9" rule="SwitchStmtsShouldHaveDefault" ruleset="Design" package="org.vertx.java.core.http.impl" class="DefaultHttpServer$ServerHandler" method="doMessageReceived" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SwitchStmtsShouldHaveDefault" priority="3"> | |
Switch statements should have a default label | |
</violation> | |
<violation beginline="723" endline="723" begincolumn="11" endcolumn="80" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.http.impl" class="DefaultHttpServer$ServerHandler" method="handshake" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/http/impl/DefaultHttpServerRequest.java"> | |
<violation beginline="95" endline="95" begincolumn="7" endcolumn="45" rule="UnnecessaryFullyQualifiedName" ruleset="Import Statements" package="org.vertx.java.core.http.impl" class="DefaultHttpServerRequest" method="version" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/imports.html#UnnecessaryFullyQualifiedName" priority="4"> | |
Unnecessary use of fully qualified name 'io.netty.handler.codec.http.HttpVersion' due to existing import 'io.netty.handler.codec.http.*' | |
</violation> | |
<violation beginline="269" endline="269" begincolumn="39" endcolumn="63" rule="UseLocaleWithCaseConversions" ruleset="Design" package="org.vertx.java.core.http.impl" class="DefaultHttpServerRequest" method="expectMultiPart" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#UseLocaleWithCaseConversions" priority="3"> | |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale | |
</violation> | |
<violation beginline="318" endline="320" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.http.impl" class="DefaultHttpServerRequest" method="handleEnd" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/http/impl/ServerConnection.java"> | |
<violation beginline="265" endline="267" begincolumn="13" endcolumn="3" rule="UselessOverridingMethod" ruleset="Unnecessary" package="org.vertx.java.core.http.impl" class="ServerConnection" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unnecessary.html#UselessOverridingMethod" priority="3"> | |
Overriding method merely calls super | |
</violation> | |
<violation beginline="292" endline="294" begincolumn="13" endcolumn="3" rule="UselessOverridingMethod" ruleset="Unnecessary" package="org.vertx.java.core.http.impl" class="ServerConnection" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unnecessary.html#UselessOverridingMethod" priority="3"> | |
Overriding method merely calls super | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/impl/DefaultVertx.java"> | |
<violation beginline="56" endline="56" begincolumn="8" endcolumn="34" rule="DuplicateImports" ruleset="Import Statements" package="org.vertx.java.core.impl" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/imports.html#DuplicateImports" priority="4"> | |
Avoid duplicate imports such as 'java.util.concurrent.Future' | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/impl/EventLoopContext.java"> | |
<violation beginline="30" endline="30" begincolumn="31" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.core.impl" class="EventLoopContext" variable="log" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'log'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/json/impl/Base64.java"> | |
<violation beginline="1025" endline="1026" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeObject" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1029" endline="1030" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeObject" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1033" endline="1034" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeObject" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1037" endline="1038" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeObject" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1141" endline="1142" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeBytes" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1145" endline="1146" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeBytes" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1149" endline="1150" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeBytes" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1269" endline="1269" begincolumn="28" endcolumn="29" rule="AddEmptyString" ruleset="Optimization" package="org.vertx.java.core.json.impl" class="Base64" method="decode4to3" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#AddEmptyString" priority="3"> | |
Do not add empty strings | |
</violation> | |
<violation beginline="1270" endline="1270" begincolumn="28" endcolumn="29" rule="AddEmptyString" ruleset="Optimization" package="org.vertx.java.core.json.impl" class="Base64" method="decode4to3" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#AddEmptyString" priority="3"> | |
Do not add empty strings | |
</violation> | |
<violation beginline="1271" endline="1271" begincolumn="28" endcolumn="29" rule="AddEmptyString" ruleset="Optimization" package="org.vertx.java.core.json.impl" class="Base64" method="decode4to3" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#AddEmptyString" priority="3"> | |
Do not add empty strings | |
</violation> | |
<violation beginline="1272" endline="1272" begincolumn="28" endcolumn="29" rule="AddEmptyString" ruleset="Optimization" package="org.vertx.java.core.json.impl" class="Base64" method="decode4to3" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#AddEmptyString" priority="3"> | |
Do not add empty strings | |
</violation> | |
<violation beginline="1386" endline="1388" begincolumn="9" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decode" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1392" endline="1393" begincolumn="13" endcolumn="11" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decode" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1396" endline="1397" begincolumn="13" endcolumn="11" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decode" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1400" endline="1401" begincolumn="13" endcolumn="11" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decode" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1443" endline="1444" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decodeToObject" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1447" endline="1448" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decodeToObject" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1477" endline="1478" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeToFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1506" endline="1507" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decodeToFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1527" endline="1527" begincolumn="7" endcolumn="26" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.core.json.impl" class="Base64" method="decodeFromFile" variable="buffer" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="1528" endline="1528" begincolumn="7" endcolumn="20" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.core.json.impl" class="Base64" method="decodeFromFile" variable="length" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="1529" endline="1529" begincolumn="7" endcolumn="22" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.core.json.impl" class="Base64" method="decodeFromFile" variable="numBytes" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="1557" endline="1558" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decodeFromFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1601" endline="1602" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeFromFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1635" endline="1636" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeFileToFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1639" endline="1640" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="encodeFileToFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1673" endline="1674" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decodeFileToFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1677" endline="1678" begincolumn="9" endcolumn="7" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.json.impl" class="Base64" method="decodeFileToFile" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="1711" endline="1711" begincolumn="26" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.core.json.impl" class="Base64$InputStream" variable="alphabet" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'alphabet'. | |
</violation> | |
<violation beginline="1931" endline="1931" begincolumn="26" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.core.json.impl" class="Base64$OutputStream" variable="alphabet" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'alphabet'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/json/impl/Json.java"> | |
<violation beginline="45" endline="45" begincolumn="7" endcolumn="79" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.json.impl" class="Json" method="encode" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="53" endline="53" begincolumn="7" endcolumn="79" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.json.impl" class="Json" method="encodePrettily" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="63" endline="63" begincolumn="7" endcolumn="70" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.json.impl" class="Json" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/logging/impl/LoggerFactory.java"> | |
<violation beginline="48" endline="49" begincolumn="7" endcolumn="5" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.logging.impl" class="LoggerFactory" method="initialise" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/logging/impl/VertxLoggerFormatter.java"> | |
<violation beginline="33" endline="33" begincolumn="35" endcolumn="70" rule="SimpleDateFormatNeedsLocale" ruleset="Design" package="org.vertx.java.core.logging.impl" class="VertxLoggerFormatter" method="format" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SimpleDateFormatNeedsLocale" priority="3"> | |
When instantiating a SimpleDateFormat object, specify a Locale | |
</violation> | |
<violation beginline="37" endline="37" begincolumn="15" endcolumn="17" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.logging.impl" class="VertxLoggerFormatter" method="format" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="38" endline="38" begincolumn="47" endcolumn="49" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.logging.impl" class="VertxLoggerFormatter" method="format" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="40" endline="40" begincolumn="46" endcolumn="48" rule="AppendCharacterWithChar" ruleset="String and StringBuffer" package="org.vertx.java.core.logging.impl" class="VertxLoggerFormatter" method="format" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#AppendCharacterWithChar" priority="3"> | |
Avoid appending characters as strings in StringBuffer.append. | |
</violation> | |
<violation beginline="40" endline="40" begincolumn="45" endcolumn="49" rule="ConsecutiveLiteralAppends" ruleset="String and StringBuffer" package="org.vertx.java.core.logging.impl" class="VertxLoggerFormatter" method="format" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#ConsecutiveLiteralAppends" priority="3"> | |
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/net/impl/DefaultNetSocket.java"> | |
<violation beginline="171" endline="173" begincolumn="10" endcolumn="3" rule="UselessOverridingMethod" ruleset="Unnecessary" package="org.vertx.java.core.net.impl" class="DefaultNetSocket" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unnecessary.html#UselessOverridingMethod" priority="3"> | |
Overriding method merely calls super | |
</violation> | |
<violation beginline="175" endline="177" begincolumn="10" endcolumn="3" rule="UselessOverridingMethod" ruleset="Unnecessary" package="org.vertx.java.core.net.impl" class="DefaultNetSocket" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unnecessary.html#UselessOverridingMethod" priority="3"> | |
Overriding method merely calls super | |
</violation> | |
<violation beginline="202" endline="204" begincolumn="13" endcolumn="3" rule="UselessOverridingMethod" ruleset="Unnecessary" package="org.vertx.java.core.net.impl" class="DefaultNetSocket" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unnecessary.html#UselessOverridingMethod" priority="3"> | |
Overriding method merely calls super | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/net/impl/ServerID.java"> | |
<violation beginline="40" endline="40" begincolumn="9" endcolumn="45" rule="SimplifyConditional" ruleset="Design" package="org.vertx.java.core.net.impl" class="ServerID" method="equals" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SimplifyConditional" priority="3"> | |
No need to check for null before an instanceof | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/net/impl/TCPSSLHelper.java"> | |
<violation beginline="311" endline="311" begincolumn="17" endcolumn="32" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.core.net.impl" class="TCPSSLHelper" method="createContext" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
<violation beginline="311" endline="311" begincolumn="7" endcolumn="49" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.net.impl" class="TCPSSLHelper" method="createContext" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="359" endline="360" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.net.impl" class="TCPSSLHelper" method="loadStore" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="373" endline="386" begincolumn="7" endcolumn="7" rule="SwitchStmtsShouldHaveDefault" ruleset="Design" package="org.vertx.java.core.net.impl" class="TCPSSLHelper" method="createSslHandler" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SwitchStmtsShouldHaveDefault" priority="3"> | |
Switch statements should have a default label | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/net/impl/VertxHandler.java"> | |
<violation beginline="101" endline="102" begincolumn="13" endcolumn="11" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.net.impl" class="VertxHandler" method="run" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="106" endline="109" begincolumn="12" endcolumn="5" rule="EmptyIfStmt" ruleset="Empty Code" package="org.vertx.java.core.net.impl" class="VertxHandler" method="exceptionCaught" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyIfStmt" priority="3"> | |
Avoid empty if statements | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/net/impl/VertxNetHandler.java"> | |
<violation beginline="65" endline="67" begincolumn="12" endcolumn="5" rule="EmptyIfStmt" ruleset="Empty Code" package="org.vertx.java.core.net.impl" class="VertxNetHandler" method="channelRead" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyIfStmt" priority="3"> | |
Avoid empty if statements | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/parsetools/RecordParser.java"> | |
<violation beginline="50" endline="217" begincolumn="8" endcolumn="1" rule="ClassWithOnlyPrivateConstructorsShouldBeFinal" ruleset="Design" package="org.vertx.java.core.parsetools" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#ClassWithOnlyPrivateConstructorsShouldBeFinal" priority="1"> | |
A class which only has private constructors should be final | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/shareddata/SharedData.java"> | |
<violation beginline="56" endline="56" begincolumn="31" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.core.shareddata" class="SharedData" variable="log" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'log'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/shareddata/impl/Checker.java"> | |
<violation beginline="39" endline="40" begincolumn="35" endcolumn="5" rule="EmptyIfStmt" ruleset="Empty Code" package="org.vertx.java.core.shareddata.impl" class="Checker" method="checkType" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyIfStmt" priority="3"> | |
Avoid empty if statements | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/shareddata/impl/SharedMap.java"> | |
<violation beginline="32" endline="32" begincolumn="31" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.core.shareddata.impl" class="SharedMap" variable="log" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'log'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/EventBusBridge.java"> | |
<violation beginline="317" endline="317" begincolumn="29" endcolumn="77" rule="UnusedPrivateMethod" ruleset="Unused Code" package="org.vertx.java.core.sockjs" class="EventBusBridge" method="getMandatoryObject" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateMethod" priority="3"> | |
Avoid unused private methods such as 'getMandatoryObject(JsonObject,String)'. | |
</violation> | |
<violation beginline="343" endline="343" begincolumn="5" endcolumn="58" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.core.sockjs" class="EventBusBridge" method="doSendOrPub" variable="body" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/BaseTransport.java"> | |
<violation beginline="37" endline="37" begincolumn="1" endcolumn="24" rule="UnusedImports" ruleset="Import Statements" package="org.vertx.java.core.sockjs.impl" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/imports.html#UnusedImports" priority="4"> | |
Avoid unused imports such as 'java.util.Random' | |
</violation> | |
<violation beginline="173" endline="173" begincolumn="26" endcolumn="78" rule="SimpleDateFormatNeedsLocale" ruleset="Design" package="org.vertx.java.core.sockjs.impl" class="BaseTransport" method="handle" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SimpleDateFormatNeedsLocale" priority="3"> | |
When instantiating a SimpleDateFormat object, specify a Locale | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/DefaultSockJSServer.java"> | |
<violation beginline="19" endline="19" begincolumn="1" endcolumn="27" rule="UnusedImports" ruleset="Import Statements" package="org.vertx.java.core.sockjs.impl" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/imports.html#UnusedImports" priority="4"> | |
Avoid unused imports such as 'java.io.IOException' | |
</violation> | |
<violation beginline="344" endline="344" begincolumn="30" endcolumn="82" rule="SimpleDateFormatNeedsLocale" ruleset="Design" package="org.vertx.java.core.sockjs.impl" class="DefaultSockJSServer" method="handle" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SimpleDateFormatNeedsLocale" priority="3"> | |
When instantiating a SimpleDateFormat object, specify a Locale | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/EventSourceTransport.java"> | |
<violation beginline="102" endline="104" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.sockjs.impl" class="EventSourceTransport$EventSourceListener" method="close" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/HtmlFileTransport.java"> | |
<violation beginline="142" endline="144" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.sockjs.impl" class="HtmlFileTransport$HtmlFileListener" method="close" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/JsonCodec.java"> | |
<violation beginline="79" endline="88" begincolumn="9" endcolumn="9" rule="SwitchStmtsShouldHaveDefault" ruleset="Design" package="org.vertx.java.core.sockjs.impl" class="JsonCodec" method="serialize" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#SwitchStmtsShouldHaveDefault" priority="3"> | |
Switch statements should have a default label | |
</violation> | |
<violation beginline="111" endline="111" begincolumn="7" endcolumn="60" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.sockjs.impl" class="JsonCodec" method="encode" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="120" endline="120" begincolumn="7" endcolumn="52" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.sockjs.impl" class="JsonCodec" method="decodeValue" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/JsonPTransport.java"> | |
<violation beginline="88" endline="88" begincolumn="9" endcolumn="37" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.core.sockjs.impl" class="JsonPTransport" method="handle" variable="body" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="112" endline="112" begincolumn="13" endcolumn="57" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.core.sockjs.impl" class="JsonPTransport" method="handle" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="178" endline="180" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.sockjs.impl" class="JsonPTransport$JsonPListener" method="close" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/RawWebSocketTransport.java"> | |
<violation beginline="38" endline="38" begincolumn="31" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.core.sockjs.impl" class="RawWebSocketTransport" variable="log" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'log'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/WebSocketTransport.java"> | |
<violation beginline="89" endline="91" begincolumn="34" endcolumn="13" rule="EmptyIfStmt" ruleset="Empty Code" package="org.vertx.java.core.sockjs.impl" class="WebSocketTransport$WebSocketListener" method="handle" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyIfStmt" priority="3"> | |
Avoid empty if statements | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/sockjs/impl/XhrTransport.java"> | |
<violation beginline="174" endline="176" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.sockjs.impl" class="XhrTransport$XhrPollingListener" method="close" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
<violation beginline="216" endline="218" begincolumn="11" endcolumn="9" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.core.sockjs.impl" class="XhrTransport$XhrStreamingListener" method="close" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-core/src/main/java/org/vertx/java/core/streams/Pump.java"> | |
<violation beginline="40" endline="125" begincolumn="8" endcolumn="1" rule="ClassWithOnlyPrivateConstructorsShouldBeFinal" ruleset="Design" package="org.vertx.java.core.streams" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#ClassWithOnlyPrivateConstructorsShouldBeFinal" priority="1"> | |
A class which only has private constructors should be final | |
</violation> | |
</file> | |
</pmd> | |
<?xml version="1.0" encoding="UTF-8"?> | |
<pmd version="5.0.5" timestamp="2013-12-27T20:53:38.871"> | |
<file name="/home/henning/scratch/vert.x/vertx-hazelcast/src/main/java/org/vertx/java/spi/cluster/impl/hazelcast/HazelcastAsyncMultiMap.java"> | |
<violation beginline="42" endline="42" begincolumn="31" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.spi.cluster.impl.hazelcast" class="HazelcastAsyncMultiMap" variable="log" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'log'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-hazelcast/src/main/java/org/vertx/java/spi/cluster/impl/hazelcast/HazelcastClusterManager.java"> | |
<violation beginline="94" endline="94" begincolumn="5" endcolumn="31" rule="UnnecessaryFullyQualifiedName" ruleset="Import Statements" package="org.vertx.java.spi.cluster.impl.hazelcast" class="HazelcastClusterManager" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/imports.html#UnnecessaryFullyQualifiedName" priority="4"> | |
Unnecessary use of fully qualified name 'com.hazelcast.core.MultiMap' due to existing import 'com.hazelcast.core.*' | |
</violation> | |
</file> | |
</pmd> | |
<?xml version="1.0" encoding="UTF-8"?> | |
<pmd version="5.0.5" timestamp="2013-12-27T20:53:58.568"> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/Args.java"> | |
<violation beginline="58" endline="58" begincolumn="9" endcolumn="78" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="Args" method="getInt" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/DefaultPlatformManager.java"> | |
<violation beginline="439" endline="439" begincolumn="5" endcolumn="52" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="deployVerticle" variable="currentModDir" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="651" endline="651" begincolumn="19" endcolumn="20" rule="UseStringBufferForStringAppends" ruleset="Optimization" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="addDirToZip" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#UseStringBufferForStringAppends" priority="3"> | |
Prefer StringBuffer over += for concatenating strings | |
</violation> | |
<violation beginline="1010" endline="1010" begincolumn="9" endcolumn="99" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="loadModJSONFromClasspath" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="1012" endline="1012" begincolumn="9" endcolumn="98" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="loadModJSONFromClasspath" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="1081" endline="1081" begincolumn="7" endcolumn="98" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="loadModuleConfig" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="1083" endline="1083" begincolumn="7" endcolumn="97" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="loadModuleConfig" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="1085" endline="1085" begincolumn="7" endcolumn="96" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="loadModuleConfig" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="1167" endline="1167" begincolumn="9" endcolumn="11" rule="UseStringBufferForStringAppends" ruleset="Optimization" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="parseIncludeString" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#UseStringBufferForStringAppends" priority="3"> | |
Prefer StringBuffer over += for concatenating strings | |
</violation> | |
<violation beginline="1326" endline="1326" begincolumn="9" endcolumn="72" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="safeMove" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="1426" endline="1426" begincolumn="5" endcolumn="61" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="doDeploy" variable="doneHandler" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="1504" endline="1504" begincolumn="5" endcolumn="72" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.platform.impl" class="DefaultPlatformManager" method="doDeploy" variable="oldTCCL" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/FatJarStarter.java"> | |
<violation beginline="40" endline="272" begincolumn="8" endcolumn="1" rule="ClassWithOnlyPrivateConstructorsShouldBeFinal" ruleset="Design" package="org.vertx.java.platform.impl" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#ClassWithOnlyPrivateConstructorsShouldBeFinal" priority="1"> | |
A class which only has private constructors should be final | |
</violation> | |
<violation beginline="85" endline="85" begincolumn="5" endcolumn="46" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.platform.impl" class="FatJarStarter" method="go" variable="modsDir" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="187" endline="187" begincolumn="9" endcolumn="85" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="FatJarStarter" method="splitCP" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/HAManager.java"> | |
<violation beginline="243" endline="245" begincolumn="33" endcolumn="7" rule="EmptyIfStmt" ruleset="Empty Code" package="org.vertx.java.platform.impl" class="HAManager" method="nodeLeft" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyIfStmt" priority="3"> | |
Avoid empty if statements | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/ModuleClassLoader.java"> | |
<violation beginline="52" endline="52" begincolumn="31" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.platform.impl" class="ModuleClassLoader" variable="log" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'log'. | |
</violation> | |
<violation beginline="98" endline="98" begincolumn="9" endcolumn="47" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl" class="ModuleClassLoader" method="loadClass" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="196" endline="197" begincolumn="7" endcolumn="5" rule="EmptyCatchBlock" ruleset="Empty Code" package="org.vertx.java.platform.impl" class="ModuleClassLoader" method="getResourceAsStream" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/empty.html#EmptyCatchBlock" priority="3"> | |
Avoid empty catch blocks | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/cli/Starter.java"> | |
<violation beginline="45" endline="566" begincolumn="8" endcolumn="1" rule="ClassWithOnlyPrivateConstructorsShouldBeFinal" ruleset="Design" package="org.vertx.java.platform.impl.cli" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#ClassWithOnlyPrivateConstructorsShouldBeFinal" priority="1"> | |
A class which only has private constructors should be final | |
</violation> | |
<violation beginline="314" endline="314" begincolumn="5" endcolumn="37" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.platform.impl.cli" class="Starter" method="runVerticle" variable="hasClasspath" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
<violation beginline="333" endline="333" begincolumn="9" endcolumn="85" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl.cli" class="Starter" method="runVerticle" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/java/CompilingClassLoader.java"> | |
<violation beginline="46" endline="46" begincolumn="17" endcolumn="32" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="CompilingClassLoader" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
<violation beginline="53" endline="53" begincolumn="7" endcolumn="76" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl.java" class="CompilingClassLoader" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="56" endline="56" begincolumn="17" endcolumn="32" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="CompilingClassLoader" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
<violation beginline="72" endline="72" begincolumn="103" endcolumn="124" rule="ConstructorCallsOverridableMethod" ruleset="Design" package="org.vertx.java.platform.impl.java" class="CompilingClassLoader" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#ConstructorCallsOverridableMethod" priority="1"> | |
Overridable method 'resolveMainClassName' called during object construction | |
</violation> | |
<violation beginline="83" endline="83" begincolumn="19" endcolumn="34" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="CompilingClassLoader" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
<violation beginline="86" endline="86" begincolumn="17" endcolumn="32" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="CompilingClassLoader" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/java/JavaSourceContext.java"> | |
<violation beginline="44" endline="44" begincolumn="21" endcolumn="36" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="JavaSourceContext" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
<violation beginline="54" endline="54" begincolumn="7" endcolumn="15" rule="UseStringBufferForStringAppends" ruleset="Optimization" package="org.vertx.java.platform.impl.java" class="JavaSourceContext" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#UseStringBufferForStringAppends" priority="3"> | |
Prefer StringBuffer over += for concatenating strings | |
</violation> | |
<violation beginline="90" endline="90" begincolumn="17" endcolumn="32" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="JavaSourceContext" method="parsePackage" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/java/MemoryFileManager.java"> | |
<violation beginline="56" endline="56" begincolumn="17" endcolumn="32" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="MemoryFileManager" method="getJavaFileForOutput" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/java/PackageHelper.java"> | |
<violation beginline="57" endline="57" begincolumn="9" endcolumn="78" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl.java" class="PackageHelper" method="find" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
<violation beginline="97" endline="97" begincolumn="17" endcolumn="32" rule="AvoidThrowingRawExceptionTypes" ruleset="Strict Exceptions" package="org.vertx.java.platform.impl.java" class="PackageHelper" method="browseJar" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strictexception.html#AvoidThrowingRawExceptionTypes" priority="1"> | |
Avoid throwing raw exception types. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/resolver/BintrayResolution.java"> | |
<violation beginline="28" endline="28" begincolumn="31" endcolumn="33" rule="UnusedPrivateField" ruleset="Unused Code" package="org.vertx.java.platform.impl.resolver" class="BintrayResolution" variable="log" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/unusedcode.html#UnusedPrivateField" priority="3"> | |
Avoid unused private fields such as 'log'. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/resolver/HttpRepoResolver.java"> | |
<violation beginline="61" endline="61" begincolumn="7" endcolumn="91" rule="PreserveStackTrace" ruleset="Design" package="org.vertx.java.platform.impl.resolver" class="HttpRepoResolver" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/design.html#PreserveStackTrace" priority="3"> | |
New exception is thrown in catch block, original stack trace may be lost | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/resolver/HttpResolution.java"> | |
<violation beginline="183" endline="183" begincolumn="11" endcolumn="42" rule="PrematureDeclaration" ruleset="Optimization" package="org.vertx.java.platform.impl.resolver" class="HttpResolution" method="handle" variable="bytesWritten" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/optimizations.html#PrematureDeclaration" priority="3"> | |
Avoid declaring a variable if it is unreferenced before a possible exit point. | |
</violation> | |
</file> | |
<file name="/home/henning/scratch/vert.x/vertx-platform/src/main/java/org/vertx/java/platform/impl/resolver/MavenResolution.java"> | |
<violation beginline="147" endline="147" begincolumn="43" endcolumn="45" rule="StringBufferInstantiationWithChar" ruleset="String and StringBuffer" package="org.vertx.java.platform.impl.resolver" class="MavenResolution" method="getMavenURI" externalInfoUrl="http://pmd.sourceforge.net/pmd-5.0.5/rules/java/strings.html#StringBufferInstantiationWithChar" priority="4"> | |
Do not instantiate a StringBuffer or StringBuilder with a char | |
</violation> | |
</file> | |
</pmd> | |
<?xml version="1.0" encoding="UTF-8"?> | |
<pmd version="5.0.5" timestamp="2013-12-27T20:54:02.808"> | |
</pmd> |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment