CodeNarc Report

[SRC]def DefaultArtifactHandler(ConfigurationContainer config..ctFactory) {

[MSG]Violation in class org.gradle.api.internal.artifacts.dsl.DefaultArtifactHandler. The def keyword is unneeded on constructors

[SRC]private PublishArtifact pushArtifact(org.gradle.api.arti..reClosure) {

[MSG]The org.gradle.api.artifacts.Configuration class was explicitly imported, so specifying the package name is not necessary

[SRC]public def methodMissing(String name, args) {

[MSG]Violation in class org.gradle.api.internal.artifacts.dsl.DefaultArtifactHandler. The def keyword is unneeded when a method is marked public

[SRC]if (!getMetaClass().respondsTo(this, name, args.size())) {

[MSG]Violation in class org.gradle.api.internal.artifacts.dsl.DefaultArtifactHandler. getMetaClass() can probably be rewritten as metaClass

[SRC]def DefaultDependencyHandler(ConfigurationContainer conf..encyFactory,

[MSG]Violation in class org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler. The def keyword is unneeded on constructors

[SRC]if (!getMetaClass().respondsTo(this, name, args.size())) {

[MSG]Violation in class org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler. getMetaClass() can probably be rewritten as metaClass

[SRC]def ModuleFactoryDelegate(ClientModule clientModule, Dep..cyFactory) {

[MSG]Violation in class org.gradle.api.internal.artifacts.dsl.dependencies.ModuleFactoryDelegate. The def keyword is unneeded on constructors

[SRC]node."${childNodeName ?: 'resources'}"() {

[MSG]Violation in class org.gradle.api.internal.file.AntFileCollectionBuilder. Parentheses in the 'null' method call are unnecessary and can be removed.

[SRC]def AntFileCollectionMatchingTaskBuilder(Iterable<Direct..fileTrees) {

[MSG]Violation in class org.gradle.api.internal.file.AntFileCollectionMatchingTaskBuilder. The def keyword is unneeded on constructors

[SRC]def AntFileTreeBuilder(Map<String, File> files) {

[MSG]Violation in class org.gradle.api.internal.file.AntFileTreeBuilder. The def keyword is unneeded on constructors

[SRC]node."${childNodeName ?: 'resources'}"() {

[MSG]Violation in class org.gradle.api.internal.file.AntFileTreeBuilder. Parentheses in the 'null' method call are unnecessary and can be removed.

[SRC]def DefaultAntBuilder(Project gradleProject) {

[MSG]Violation in class org.gradle.api.internal.project.DefaultAntBuilder. The def keyword is unneeded on constructors

[SRC]def Object invokeMethod(String methodName, Object args) {

[MSG]Violation in class org.gradle.api.internal.project.DefaultAntBuilder. The def keyword is unneeded when a method returns the Object type

[SRC]def Object invokeMethod(String methodName, Object args) {

[MSG]Violation in class DefaultAntBuilder. The method invokeMethod contains no logic and can be safely deleted

[SRC]private def doSetProperty(String property, newValue) {

[MSG]Violation in class org.gradle.api.internal.project.DefaultAntBuilder. The def keyword is unneeded when a method is marked private

[SRC]antProject.setUserProperty(MagicNames.ANT_FILE, file.get..olutePath())

[MSG]Violation in class org.gradle.api.internal.project.DefaultAntBuilder. getAbsolutePath() can probably be rewritten as absolutePath

[SRC]def DefaultIsolatedAntBuilder(ClassPathRegistry classPat..erFactory) {

[MSG]Violation in class org.gradle.api.internal.project.DefaultIsolatedAntBuilder. The def keyword is unneeded on constructors

[SRC]Object antBuilder = gradleLoader.loadClass(BasicAntBuild..ewInstance()

[MSG]BasicAntBuilder.class can be rewritten as BasicAntBuilder

[SRC]Object antLogger = gradleLoader.loadClass(AntLoggingAdap..ewInstance()

[MSG]AntLoggingAdapter.class can be rewritten as AntLoggingAdapter

[SRC]antBuilder.project.removeBuildListener(antBuilder.projec..teners()[0])

[MSG]Violation in class org.gradle.api.internal.project.DefaultIsolatedAntBuilder. getBuildListeners() can probably be rewritten as buildListeners

[SRC]def AntBuilderDelegate(builder, antlibClassLoader) {

[MSG]Violation in class org.gradle.api.internal.project.AntBuilderDelegate. The def keyword is unneeded on constructors

[SRC]def void apply(Closure closure) {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]def void apply(Map options) {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]def ScriptHandler getBuildscript() {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]def void buildscript(Closure configureClosure) {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]def StandardOutputCapture getStandardOutputCapture() {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]def LoggingManager getLogging() {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]def Logger getLogger() {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]def String toString() {

[MSG]Violation in class org.gradle.api.internal.project.ProjectScript. The def keyword is unneeded when a method specifies a return type

[SRC]private static Logger logger = LoggerFactory.getLogger(A..ovyCompiler)

[MSG]The field logger is not used within the class org.gradle.api.internal.tasks.compile.AntGroovyCompiler

[SRC]def AntGroovyCompiler(IsolatedAntBuilder ant, ClassPathR..hRegistry) {

[MSG]Violation in class org.gradle.api.internal.tasks.compile.AntGroovyCompiler. The def keyword is unneeded on constructors

[SRC]private static Logger logger = LoggerFactory.getLogger(A..alaCompiler)

[MSG]The field logger is not used within the class org.gradle.api.internal.tasks.scala.AntScalaCompiler

[SRC]def AntScalaCompiler(IsolatedAntBuilder antBuilder) {

[MSG]Violation in class org.gradle.api.internal.tasks.scala.AntScalaCompiler. The def keyword is unneeded on constructors

[SRC]def AntScalaCompiler(IsolatedAntBuilder antBuilder, Iter..nsionDirs) {

[MSG]Violation in class org.gradle.api.internal.tasks.scala.AntScalaCompiler. The def keyword is unneeded on constructors

[SRC]installTask.doLast {

[MSG]The code could be more concise by using a with() or identity() block

[SRC]Delete clean = project.tasks.add(CLEAN_TASK_NAME, Delete.class)

[MSG]Delete.class can be rewritten as Delete

[SRC]String targetTaskName = taskName.substring(prefix.length())

[MSG]Violation in class org.gradle.api.plugins.BasePlugin. The String.substring(int) method can be replaced with the subscript operator

[SRC]Configuration configuration = project.configurations.fin...length())))

[MSG]Violation in class org.gradle.api.plugins.BasePlugin. The String.substring(int) method can be replaced with the subscript operator

[SRC]project.tasks.add(taskName).dependsOn(configuration.getA..figuration))

[MSG]Violation in class org.gradle.api.plugins.BasePlugin. getAllArtifacts() can probably be rewritten as allArtifacts

[SRC]Upload upload = project.getTasks().add(name, Upload.class)

[MSG]Violation in class org.gradle.api.plugins.BasePlugin. getTasks() can probably be rewritten as tasks

[SRC]Upload upload = project.getTasks().add(name, Upload.class)

[MSG]Upload.class can be rewritten as Upload

[SRC]upload.descriptorDestination = new File(project.getBuild.., "ivy.xml")

[MSG]Violation in class org.gradle.api.plugins.BasePlugin. getBuildDir() can probably be rewritten as buildDir

[SRC]ConfigurationContainer configurations = project.getConfigurations();

[MSG]Violation in class org.gradle.api.plugins.BasePlugin. getConfigurations() can probably be rewritten as configurations

[SRC]sourceSets = instantiator.newInstance(DefaultSourceSetCo..nstantiator)

[MSG]DefaultSourceSetContainer.class can be rewritten as DefaultSourceSetContainer

[SRC]return ConfigureUtil.configure(closure, new DefaultManif..eResolver));

[MSG]Violation in class org.gradle.api.plugins.JavaPluginConvention. getProject() can probably be rewritten as project

[SRC]def ProjectReportsPluginConvention(Project project) {

[MSG]Violation in class org.gradle.api.plugins.ProjectReportsPluginConvention. The def keyword is unneeded on constructors

[SRC]def WarPluginConvention(Project project) {

[MSG]Violation in class org.gradle.api.plugins.WarPluginConvention. The def keyword is unneeded on constructors

[SRC]void beforeExecute(Task task) {

[MSG]Violation in class AnnouncingBuildListener. Method parameter [task] is never referenced in the method beforeExecute of class org.gradle.api.plugins.announce.internal.AnnouncingBuildListener

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]private def escape(String value) {

[MSG]Violation in class org.gradle.api.plugins.announce.internal.AppleScriptBackedGrowlAnnouncer. The def keyword is unneeded when a method is marked private

[SRC]} catch (ClassNotFoundException e) {

[MSG]The catch block is empty

[SRC]void send(String title, String message) {

[MSG]Violation in class UnknownAnnouncer. Method parameter [title] is never referenced in the method send of class org.gradle.api.plugins.announce.internal.UnknownAnnouncer

[SRC]void send(String title, String message) {

[MSG]Violation in class UnknownAnnouncer. Method parameter [message] is never referenced in the method send of class org.gradle.api.plugins.announce.internal.UnknownAnnouncer

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]return getClass().getClassLoader().loadClass("org.gradle..conProvider)

[MSG]Violation in class org.gradle.api.plugins.announce.internal.DefaultAnnouncerFactory. getClassLoader() can probably be rewritten as classLoader

[SRC]private def escape(String value) {

[MSG]The method escape is not used within GrowlNotifyBackedAnnouncer.groovy

[SRC]private def escape(String value) {

[MSG]Violation in class org.gradle.api.plugins.announce.internal.GrowlNotifyBackedAnnouncer. The def keyword is unneeded when a method is marked private

[SRC]with(new PrintWriter(sock.getOutputStream(), true)) { out ->

[MSG]Violation in class org.gradle.api.plugins.announce.internal.Snarl. getOutputStream() can probably be rewritten as outputStream

[SRC]else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]import sun.misc.BASE64Encoder

[MSG]The file imports sun.misc.BASE64Encoder, which is not portable and likely to change

[SRC]void send(String title, String message) {

[MSG]Violation in class Twitter. Method parameter [title] is never referenced in the method send of class org.gradle.api.plugins.announce.internal.Twitter

[SRC]getConfigProperties()

[MSG]Violation in class org.gradle.api.plugins.quality.Checkstyle. getConfigProperties() can probably be rewritten as configProperties

[SRC]antBuilder.withClasspath(getCheckstyleClasspath()).execute {

[MSG]Violation in class org.gradle.api.plugins.quality.Checkstyle. getCheckstyleClasspath() can probably be rewritten as checkstyleClasspath

[SRC]ant.checkstyle(config: getConfigFile(), failOnViolation:..pertyName) {

[MSG]Violation in class org.gradle.api.plugins.quality.Checkstyle. getConfigFile() can probably be rewritten as configFile

[SRC]getSource().addToAntBuilder(ant, 'fileset', FileCollecti..ype.FileSet)

[MSG]Violation in class org.gradle.api.plugins.quality.Checkstyle. getSource() can probably be rewritten as source

[SRC]getClasspath().addToAntBuilder(ant, 'classpath')

[MSG]Violation in class org.gradle.api.plugins.quality.Checkstyle. getClasspath() can probably be rewritten as classpath

[SRC]getConfigProperties().each { key, value ->

[MSG]Violation in class org.gradle.api.plugins.quality.Checkstyle. getConfigProperties() can probably be rewritten as configProperties

[SRC]if (!getIgnoreFailures() && ant.project.properties[propertyName]) {

[MSG]Violation in class org.gradle.api.plugins.quality.Checkstyle. getIgnoreFailures() can probably be rewritten as ignoreFailures

[SRC]antBuilder.withClasspath(getCodenarcClasspath()).execute {

[MSG]Violation in class org.gradle.api.plugins.quality.CodeNarc. getCodenarcClasspath() can probably be rewritten as codenarcClasspath

[SRC]ant.codenarc(ruleSetFiles: "file:${getConfigFile()}", ma..ations: 0) {

[MSG]Violation in class org.gradle.api.plugins.quality.CodeNarc. getConfigFile() can probably be rewritten as configFile

[SRC]if (getIgnoreFailures()) {

[MSG]Violation in class org.gradle.api.plugins.quality.CodeNarc. getIgnoreFailures() can probably be rewritten as ignoreFailures

[SRC]antBuilder.withClasspath(getFindbugsClasspath()).execute {

[MSG]Violation in class org.gradle.api.plugins.quality.FindBugs. getFindbugsClasspath() can probably be rewritten as findbugsClasspath

[SRC]getFindbugsClasspath().addToAntBuilder(ant, 'classpath')

[MSG]Violation in class org.gradle.api.plugins.quality.FindBugs. getFindbugsClasspath() can probably be rewritten as findbugsClasspath

[SRC]getPluginClasspath().addToAntBuilder(ant, 'pluginList')

[MSG]Violation in class org.gradle.api.plugins.quality.FindBugs. getPluginClasspath() can probably be rewritten as pluginClasspath

[SRC]getClasses().addToAntBuilder(ant, 'auxAnalyzepath')

[MSG]Violation in class org.gradle.api.plugins.quality.FindBugs. getClasses() can probably be rewritten as classes

[SRC]addUnlessEmpty(ant, getClasspath(), 'auxClasspath')

[MSG]Violation in class org.gradle.api.plugins.quality.FindBugs. getClasspath() can probably be rewritten as classpath

[SRC]addUnlessEmpty(ant, getSource(), 'sourcePath')

[MSG]Violation in class org.gradle.api.plugins.quality.FindBugs. getSource() can probably be rewritten as source

[SRC]def GroovyCodeQualityPluginConvention(Project project) {

[MSG]Violation in class org.gradle.api.plugins.quality.GroovyCodeQualityPluginConvention. The def keyword is unneeded on constructors

[SRC]antBuilder.withClasspath(getJdependClasspath()).execute {

[MSG]Violation in class org.gradle.api.plugins.quality.JDepend. getJdependClasspath() can probably be rewritten as jdependClasspath

[SRC]pathElement(location: getClassesDir())

[MSG]Violation in class org.gradle.api.plugins.quality.JDepend. getClassesDir() can probably be rewritten as classesDir

[SRC]def JavaCodeQualityPluginConvention(Project project) {

[MSG]Violation in class org.gradle.api.plugins.quality.JavaCodeQualityPluginConvention. The def keyword is unneeded on constructors

[SRC]antBuilder.withClasspath(getPmdClasspath()).execute {

[MSG]Violation in class org.gradle.api.plugins.quality.Pmd. getPmdClasspath() can probably be rewritten as pmdClasspath

[SRC]ant.pmd(failOnRuleViolation: !getIgnoreFailures()) {

[MSG]Violation in class org.gradle.api.plugins.quality.Pmd. getIgnoreFailures() can probably be rewritten as ignoreFailures

[SRC]getSource().addToAntBuilder(ant, 'fileset', FileCollecti..ype.FileSet)

[MSG]Violation in class org.gradle.api.plugins.quality.Pmd. getSource() can probably be rewritten as source

[SRC]getRuleSets().each {

[MSG]Violation in class org.gradle.api.plugins.quality.Pmd. getRuleSets() can probably be rewritten as ruleSets

[SRC]getRuleSetFiles().each {

[MSG]Violation in class org.gradle.api.plugins.quality.Pmd. getRuleSetFiles() can probably be rewritten as ruleSetFiles

[SRC]protected void configureTaskDefaults(T task, String baseName) {

[MSG]Violation in class AbstractCodeQualityPlugin. Method parameter [task] is never referenced in the method configureTaskDefaults of class org.gradle.api.plugins.quality.internal.AbstractCodeQualityPlugin

[SRC]protected void configureTaskDefaults(T task, String baseName) {

[MSG]Violation in class AbstractCodeQualityPlugin. Method parameter [baseName] is never referenced in the method configureTaskDefaults of class org.gradle.api.plugins.quality.internal.AbstractCodeQualityPlugin

[SRC]protected void configureForSourceSet(SourceSet sourceSet, T task) {

[MSG]Violation in class AbstractCodeQualityPlugin. Method parameter [sourceSet] is never referenced in the method configureForSourceSet of class org.gradle.api.plugins.quality.internal.AbstractCodeQualityPlugin

[SRC]protected void configureForSourceSet(SourceSet sourceSet, T task) {

[MSG]Violation in class AbstractCodeQualityPlugin. Method parameter [task] is never referenced in the method configureForSourceSet of class org.gradle.api.plugins.quality.internal.AbstractCodeQualityPlugin

[SRC]prunedName = prunedName[0].toLowerCase() + prunedName.substring(1)

[MSG]Violation in class org.gradle.api.plugins.quality.internal.AbstractCodeQualityPlugin. The String.substring(int) method can be replaced with the subscript operator

[SRC]project.tasks['check'].dependsOn { extension.sourceSets...me, null) }}

[MSG]Violation in class org.gradle.api.plugins.quality.internal.AbstractCodeQualityPlugin. The call to collect could probably be rewritten as a spread expression: extension.sourceSets*.getTaskName(taskBaseName, null)

[SRC]private void configureCompileDefaults(final Project proj..avaPlugin) {

[MSG]Violation in class ScalaBasePlugin. Method parameter [javaPlugin] is never referenced in the method configureCompileDefaults of class org.gradle.api.plugins.scala.ScalaBasePlugin

[SRC]JavaBasePlugin javaPlugin = project.plugins.apply(JavaBa..ugin.class);

[MSG]JavaBasePlugin.class can be rewritten as JavaBasePlugin

[SRC]project.convention.getPlugin(JavaPluginConvention.class)..sourceSet ->

[MSG]JavaPluginConvention.class can be rewritten as JavaPluginConvention

[SRC]ScalaCompile scalaCompile = project.tasks.add(taskName, ..pile.class);

[MSG]ScalaCompile.class can be rewritten as ScalaCompile

[SRC]project.tasks.withType(ScalaCompile.class) {ScalaCompile compile ->

[MSG]ScalaCompile.class can be rewritten as ScalaCompile

[SRC]project.getTasks().withType(ScalaDoc.class) {ScalaDoc scalaDoc ->

[MSG]Violation in class org.gradle.api.plugins.scala.ScalaBasePlugin. getTasks() can probably be rewritten as tasks

[SRC]project.getTasks().withType(ScalaDoc.class) {ScalaDoc scalaDoc ->

[MSG]ScalaDoc.class can be rewritten as ScalaDoc

[SRC]project.plugins.apply(ScalaBasePlugin.class);

[MSG]ScalaBasePlugin.class can be rewritten as ScalaBasePlugin

[SRC]project.plugins.apply(JavaPlugin.class);

[MSG]JavaPlugin.class can be rewritten as JavaPlugin

[SRC]project.getTasks().withType(ScalaDoc.class) {ScalaDoc scalaDoc ->

[MSG]Violation in class org.gradle.api.plugins.scala.ScalaPlugin. getTasks() can probably be rewritten as tasks

[SRC]project.getTasks().withType(ScalaDoc.class) {ScalaDoc scalaDoc ->

[MSG]ScalaDoc.class can be rewritten as ScalaDoc

[SRC]ScalaDoc scalaDoc = project.tasks.add(SCALA_DOC_TASK_NAM..laDoc.class)

[MSG]ScalaDoc.class can be rewritten as ScalaDoc

[SRC]projectDef.workDir = sonarProject.workDir

[MSG]The code could be more concise by using a with() or identity() block

[SRC]public @interface IncludeProperties {}

[MSG]The java.lang.annotation.Annotation class was explicitly imported, so specifying the package name is not necessary

[SRC]public @interface SonarProperty {

[MSG]The java.lang.annotation.Annotation class was explicitly imported, so specifying the package name is not necessary

[SRC]project.task("publishArchives", dependsOn: 'assemble', t..ons.class) {

[MSG]PublishPublications.class can be rewritten as PublishPublications

[SRC]project.task("installArchives", dependsOn: 'assemble', t..ons.class) {

[MSG]InstallPublications.class can be rewritten as InstallPublications

[SRC]ConfigureUtil.configure(c, getMaven())

[MSG]Violation in class org.gradle.api.publication.Publications. getMaven() can probably be rewritten as maven

[SRC]model.description = publication.description

[MSG]The code could be more concise by using a with() or identity() block

[SRC]dependency.optional = mavenDep.optional

[MSG]The code could be more concise by using a with() or identity() block

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]private def execute(MavenPublication publication, Instal..port task) {

[MSG]Violation in class org.gradle.api.publication.maven.internal.ant.DefaultMavenPublisher. The def keyword is unneeded when a method is marked private

[SRC]ConfigureUtil.configure(c, getRepository())

[MSG]Violation in class org.gradle.api.publication.maven.internal.model.DefaultMavenPublication. getRepository() can probably be rewritten as repository

[SRC]if (!getApplicationName()) {

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getApplicationName() can probably be rewritten as applicationName

[SRC]return "${GUtil.toConstant(getApplicationName())}_OPTS"

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getApplicationName() can probably be rewritten as applicationName

[SRC]if (!getApplicationName()) {

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getApplicationName() can probably be rewritten as applicationName

[SRC]return "${GUtil.toConstant(getApplicationName())}_EXIT_CONSOLE"

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getApplicationName() can probably be rewritten as applicationName

[SRC]return new File(getOutputDir(), getApplicationName())

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getOutputDir() can probably be rewritten as outputDir

[SRC]return new File(getOutputDir(), getApplicationName())

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getApplicationName() can probably be rewritten as applicationName

[SRC]return new File(getOutputDir(), "${getApplicationName()}.bat")

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getOutputDir() can probably be rewritten as outputDir

[SRC]return new File(getOutputDir(), "${getApplicationName()}.bat")

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getApplicationName() can probably be rewritten as applicationName

[SRC]getOutputDir().mkdirs()

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getOutputDir() can probably be rewritten as outputDir

[SRC]generator.applicationName = getApplicationName()

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getApplicationName() can probably be rewritten as applicationName

[SRC]generator.mainClassName = getMainClassName()

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getMainClassName() can probably be rewritten as mainClassName

[SRC]generator.optsEnvironmentVar = getOptsEnvironmentVar()

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getOptsEnvironmentVar() can probably be rewritten as optsEnvironmentVar

[SRC]generator.exitEnvironmentVar = getExitEnvironmentVar()

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getExitEnvironmentVar() can probably be rewritten as exitEnvironmentVar

[SRC]generator.classpath = getClasspath().collect { "lib/${it.name}" }

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getClasspath() can probably be rewritten as classpath

[SRC]generator.scriptRelPath = "bin/${getUnixScript().name}"

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getUnixScript() can probably be rewritten as unixScript

[SRC]generator.scriptRelPath = "bin/${getUnixScript().name}"

[MSG]The code could be more concise by using a with() or identity() block

[SRC]generator.generateUnixScript(getUnixScript())

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getUnixScript() can probably be rewritten as unixScript

[SRC]generator.generateUnixScript(getUnixScript())

[MSG]The code could be more concise by using a with() or identity() block

[SRC]generator.generateWindowsScript(getWindowsScript())

[MSG]Violation in class org.gradle.api.tasks.application.CreateStartScripts. getWindowsScript() can probably be rewritten as windowsScript

[SRC]generator.generateWindowsScript(getWindowsScript())

[MSG]The code could be more concise by using a with() or identity() block

[SRC]Manifest manifest = getManifest() ?: new DefaultManifest(null)

[MSG]Violation in class org.gradle.api.tasks.bundling.Jar. getManifest() can probably be rewritten as manifest

[SRC]if (getManifest() == null) {

[MSG]Violation in class org.gradle.api.tasks.bundling.Jar. getManifest() can probably be rewritten as manifest

[SRC]ConfigureUtil.configure(configureClosure, getManifest());

[MSG]Violation in class org.gradle.api.tasks.bundling.Jar. getManifest() can probably be rewritten as manifest

[SRC]return ConfigureUtil.configure(configureClosure, getMetaInf())

[MSG]Violation in class org.gradle.api.tasks.bundling.Jar. getMetaInf() can probably be rewritten as metaInf

[SRC]def classpath = getClasspath()

[MSG]Violation in class org.gradle.api.tasks.bundling.War. getClasspath() can probably be rewritten as classpath

[SRC]def classpath = getClasspath()

[MSG]Violation in class org.gradle.api.tasks.bundling.War. getClasspath() can probably be rewritten as classpath

[SRC]getWebXml()

[MSG]Violation in class org.gradle.api.tasks.bundling.War. getWebXml() can probably be rewritten as webXml

[SRC]return ConfigureUtil.configure(configureClosure, getWebInf())

[MSG]Violation in class org.gradle.api.tasks.bundling.War. getWebInf() can probably be rewritten as webInf

[SRC]FileCollection oldClasspath = getClasspath()

[MSG]Violation in class org.gradle.api.tasks.bundling.War. getClasspath() can probably be rewritten as classpath

[SRC]((field.getModifiers() & Modifier.STATIC) == 0) &&

[MSG]Violation in class org.gradle.api.tasks.compile.AbstractOptions. getModifiers() can probably be rewritten as modifiers

[SRC](field.getName() != "metaClass") &&

[MSG]Violation in class org.gradle.api.tasks.compile.AbstractOptions. getName() can probably be rewritten as name

[SRC]def AntGroovydoc(IsolatedAntBuilder ant, ClassPathRegist..hRegistry) {

[MSG]Violation in class org.gradle.api.tasks.javadoc.AntGroovydoc. The def keyword is unneeded on constructors

[SRC]private static Logger logger = LoggerFactory.getLogger(AntScalaDoc)

[MSG]The field logger is not used within the class org.gradle.api.tasks.scala.AntScalaDoc

[SRC]def AntScalaDoc(IsolatedAntBuilder antBuilder) {

[MSG]Violation in class org.gradle.api.tasks.scala.AntScalaDoc. The def keyword is unneeded on constructors

[SRC]def AntScalaDoc(IsolatedAntBuilder antBuilder, Iterable<..nsionDirs) {

[MSG]Violation in class org.gradle.api.tasks.scala.AntScalaDoc. The def keyword is unneeded on constructors

[SRC]return suiteXmlBuilder.getMetaClass()."${name}"

[MSG]Violation in class org.gradle.api.tasks.testing.testng.TestNGOptions. getMetaClass() can probably be rewritten as metaClass

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]return suiteXmlBuilder.getMetaClass().invokeMethod(suite.. name, args)

[MSG]Violation in class org.gradle.api.tasks.testing.testng.TestNGOptions. getMetaClass() can probably be rewritten as metaClass

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]def addToAntBuilder(node, String childNodeName = null) {

[MSG]Violation in class PatternSet. Method parameter [childNodeName] is never referenced in the method addToAntBuilder of class org.gradle.api.tasks.util.PatternSet

[SRC]def addToAntBuilder(Object node, String childNodeName) {

[MSG]Violation in class IntersectionPatternSet. Method parameter [childNodeName] is never referenced in the method addToAntBuilder of class org.gradle.api.tasks.util.IntersectionPatternSet

[SRC]def boolean caseSensitive = true

[MSG]Violation in class org.gradle.api.tasks.util.PatternSet. The def keyword is unneeded when a field type is specified

[SRC]static def setGlobalExcludes(Collection<String> excludes) {

[MSG]Violation in class org.gradle.api.tasks.util.PatternSet. The def keyword is unneeded when a method is marked static

[SRC]static def resetGlobalExcludes() {

[MSG]Violation in class org.gradle.api.tasks.util.PatternSet. The def keyword is unneeded when a method is marked static

[SRC]def boolean equals(Object o) {

[MSG]Violation in class org.gradle.api.tasks.util.PatternSet. The def keyword is unneeded when a method specifies a return type

[SRC]def int hashCode() {

[MSG]Violation in class org.gradle.api.tasks.util.PatternSet. The def keyword is unneeded when a method specifies a return type

[SRC]def IntersectionPatternSet(PatternSet other) {

[MSG]Violation in class org.gradle.api.tasks.util.IntersectionPatternSet. The def keyword is unneeded on constructors

[SRC]return new AndSpec<FileTreeElement>([super.getAsSpec(), ..] as Spec[])

[MSG]Violation in class org.gradle.api.tasks.util.IntersectionPatternSet. getAsSpec() can probably be rewritten as asSpec

[SRC]return new AndSpec<FileTreeElement>([super.getAsSpec(), ..] as Spec[])

[MSG]Violation in class org.gradle.api.tasks.util.IntersectionPatternSet. getAsSpec() can probably be rewritten as asSpec

[SRC]standardOutputCapture = services.get(StandardOutputCapture.class)

[MSG]StandardOutputCapture.class can be rewritten as StandardOutputCapture

[SRC]def Object getScriptTarget() {

[MSG]Violation in class org.gradle.groovy.scripts.BasicScript. The def keyword is unneeded when a method returns the Object type

[SRC]def StandardOutputCapture getStandardOutputCapture() {

[MSG]Violation in class org.gradle.groovy.scripts.BasicScript. The def keyword is unneeded when a method specifies a return type

[SRC]return target.getProperties()

[MSG]Violation in class org.gradle.groovy.scripts.BasicScript. getProperties() can probably be rewritten as properties

[SRC]private static final Logger LOGGER = Logging.getLogger(Script.class)

[MSG]Script.class can be rewritten as Script

[SRC]def void init(Object target, ServiceRegistry services) {

[MSG]Violation in class org.gradle.groovy.scripts.DefaultScript. The def keyword is unneeded when a method specifies a return type

[SRC]loggingManager = services.get(LoggingManager.class)

[MSG]LoggingManager.class can be rewritten as LoggingManager

[SRC]ObjectConfigurationAction action = new DefaultObjectConf..criptTarget)

[MSG]ScriptPluginFactory.class can be rewritten as ScriptPluginFactory

[SRC]ObjectConfigurationAction action = new DefaultObjectConf..criptTarget)

[MSG]ScriptPluginFactory.class can be rewritten as ScriptPluginFactory

[SRC]return services.get(ScriptHandler.class);

[MSG]ScriptHandler.class can be rewritten as ScriptHandler

[SRC]ConfigureUtil.configure(configureClosure, getBuildscript())

[MSG]Violation in class org.gradle.groovy.scripts.DefaultScript. getBuildscript() can probably be rewritten as buildscript

[SRC]def String toString() {

[MSG]Violation in class org.gradle.groovy.scripts.DefaultScript. The def keyword is unneeded when a method specifies a return type

[SRC]getBuildscript()

[MSG]Violation in class org.gradle.initialization.InitScript. getBuildscript() can probably be rewritten as buildscript

[SRC]def String toString() {

[MSG]Violation in class org.gradle.initialization.InitScript. The def keyword is unneeded when a method specifies a return type

[SRC]def String toString() {

[MSG]Violation in class org.gradle.initialization.SettingsScript. The def keyword is unneeded when a method specifies a return type

[SRC]void runSamplesFrom(String dir) {

[MSG]Violation in class AbstractAutoTestedSamplesTest. The method runSamplesFrom is public but not a test method

[SRC]void includeOnly(String includes) {

[MSG]Violation in class AbstractAutoTestedSamplesTest. The method includeOnly is public but not a test method

[SRC]distribution.getTestDir();

[MSG]Violation in class org.gradle.integtests.fixtures.AbstractIntegrationSpec. getTestDir() can probably be rewritten as testDir

[SRC]getTestDir().file(path);

[MSG]Violation in class org.gradle.integtests.fixtures.AbstractIntegrationSpec. getTestDir() can probably be rewritten as testDir

[SRC]executer.withUserHomeDir(distribution.getUserHomeDir())

[MSG]Violation in class org.gradle.integtests.fixtures.AbstractIntegrationSpec. getUserHomeDir() can probably be rewritten as userHomeDir

[SRC]return new GradleBackedArtifactBuilder(executer, getTest..artifacts"))

[MSG]Violation in class org.gradle.integtests.fixtures.AbstractIntegrationSpec. getTestDir() can probably be rewritten as testDir

[SRC]list.each() { runSamplesFromFile(it, runner) }

[MSG]Violation in class org.gradle.integtests.fixtures.AutoTestedSamplesUtil. Parentheses in the 'each' method call are unnecessary and can be removed.

[SRC]current.getTestDir();

[MSG]Violation in class org.gradle.integtests.fixtures.CrossVersionIntegrationSpec. getTestDir() can probably be rewritten as testDir

[SRC]def minVersion = targetGradleVersion.substring(0, target..ength() - 1)

[MSG]Violation in class org.gradle.integtests.fixtures.CrossVersionTestRunner$PreviousVersionExecution. The String.substring(int, int) method can be replaced with the subscript operator

[SRC]boolean reauthenticate(Principal user) {

[MSG]Violation in class HttpServer$TestUserRealm. Method parameter [user] is never referenced in the method reauthenticate of class org.gradle.integtests.fixtures.HttpServer$TestUserRealm

[SRC]boolean isUserInRole(Principal user, String role) {

[MSG]Violation in class HttpServer$TestUserRealm. Method parameter [user] is never referenced in the method isUserInRole of class org.gradle.integtests.fixtures.HttpServer$TestUserRealm

[SRC]boolean isUserInRole(Principal user, String role) {

[MSG]Violation in class HttpServer$TestUserRealm. Method parameter [role] is never referenced in the method isUserInRole of class org.gradle.integtests.fixtures.HttpServer$TestUserRealm

[SRC]void disassociate(Principal user) {

[MSG]Violation in class HttpServer$TestUserRealm. The method disassociate is both empty and not marked with @Override

[SRC]void disassociate(Principal user) {

[MSG]Violation in class HttpServer$TestUserRealm. Method parameter [user] is never referenced in the method disassociate of class org.gradle.integtests.fixtures.HttpServer$TestUserRealm

[SRC]Principal pushRole(Principal user, String role) {

[MSG]Violation in class HttpServer$TestUserRealm. Method parameter [role] is never referenced in the method pushRole of class org.gradle.integtests.fixtures.HttpServer$TestUserRealm

[SRC]void logout(Principal user) {

[MSG]Violation in class HttpServer$TestUserRealm. The method logout is both empty and not marked with @Override

[SRC]void logout(Principal user) {

[MSG]Violation in class HttpServer$TestUserRealm. Method parameter [user] is never referenced in the method logout of class org.gradle.integtests.fixtures.HttpServer$TestUserRealm

[SRC]private static Logger logger = LoggerFactory.getLogger(H..erver.class)

[MSG]HttpServer.class can be rewritten as HttpServer

[SRC]def relativePath = request.pathInfo.substring(path.length() + 1)

[MSG]Violation in class org.gradle.integtests.fixtures.HttpServer$3. The String.substring(int) method can be replaced with the subscript operator

[SRC]TestClassExecutionResult assertTestSkipped(String name) {

[MSG]Violation in class JUnitTestClassExecutionResult. Method parameter [name] is never referenced in the method assertTestSkipped of class org.gradle.integtests.fixtures.JUnitTestClassExecutionResult

[SRC]TestClassExecutionResult assertConfigMethodPassed(String name) {

[MSG]Violation in class JUnitTestClassExecutionResult. Method parameter [name] is never referenced in the method assertConfigMethodPassed of class org.gradle.integtests.fixtures.JUnitTestClassExecutionResult

[SRC]TestClassExecutionResult assertConfigMethodFailed(String name) {

[MSG]Violation in class JUnitTestClassExecutionResult. Method parameter [name] is never referenced in the method assertConfigMethodFailed of class org.gradle.integtests.fixtures.JUnitTestClassExecutionResult

[SRC]import static org.hamcrest.Matchers.*

[MSG]Static imports should appear before normal imports

[SRC]import static org.junit.Assert.assertThat

[MSG]Static imports should appear before normal imports

[SRC]def JUnitTestExecutionResult(TestFile projectDir, String..= 'build') {

[MSG]Violation in class org.gradle.integtests.fixtures.JUnitTestExecutionResult. The def keyword is unneeded on constructors

[SRC]private def findTestClass(String testClass) {

[MSG]Violation in class org.gradle.integtests.fixtures.JUnitTestExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]private def findClasses() {

[MSG]Violation in class org.gradle.integtests.fixtures.JUnitTestExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]def JUnitTestClassExecutionResult(GPathResult testClassN..ClassName) {

[MSG]Violation in class org.gradle.integtests.fixtures.JUnitTestClassExecutionResult. The def keyword is unneeded on constructors

[SRC]private def findTests() {

[MSG]Violation in class org.gradle.integtests.fixtures.JUnitTestClassExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]private def findIgnoredTests() {

[MSG]Violation in class org.gradle.integtests.fixtures.JUnitTestClassExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]artifactNames = names.collect { it.replace('-SNAPSHOT', ..-${build}")}

[MSG]Violation in class org.gradle.integtests.fixtures.MavenModule. The call to collect could probably be rewritten as a spread expression: names*.replace(-SNAPSHOT, -$timestamp-$build)

[SRC]assert dependencies.collect { it.artifactId} as Set == a..ctIds as Set

[MSG]Violation in class org.gradle.integtests.fixtures.MavenScope. The call to collect could probably be rewritten as a spread expression: dependencies*.artifactId

[SRC]def String toString() {

[MSG]Violation in class org.gradle.integtests.fixtures.PreviousGradleVersionExecuter. The def keyword is unneeded when a method specifies a return type

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]def TestFile getGradleHomeDir() {

[MSG]Violation in class org.gradle.integtests.fixtures.PreviousGradleVersionExecuter. The def keyword is unneeded when a method specifies a return type

[SRC]} catch (Throwable e) {}

[MSG]The catch block is empty

[SRC]} catch (Throwable e) {}

[MSG]The catch block is empty

[SRC]} catch (Throwable e) {}

[MSG]The catch block is empty

[SRC]public void showMessage(String message) {

[MSG]Violation in class SFTPServer$1. The method showMessage is both empty and not marked with @Override

[SRC]boolean authenticate(String username, PublicKey key, Ser..n session) {

[MSG]Violation in class SFTPServer$3. Method parameter [username] is never referenced in the method authenticate of class org.gradle.integtests.fixtures.SFTPServer$3

[SRC]boolean authenticate(String username, PublicKey key, Ser..n session) {

[MSG]Violation in class SFTPServer$3. Method parameter [key] is never referenced in the method authenticate of class org.gradle.integtests.fixtures.SFTPServer$3

[SRC]boolean authenticate(String username, PublicKey key, Ser..n session) {

[MSG]Violation in class SFTPServer$3. Method parameter [session] is never referenced in the method authenticate of class org.gradle.integtests.fixtures.SFTPServer$3

[SRC]boolean authenticate(String username, String password, o..n session) {

[MSG]Violation in class DummyPasswordAuthenticator. Method parameter [session] is never referenced in the method authenticate of class org.gradle.integtests.fixtures.DummyPasswordAuthenticator

[SRC]FileUtils.copyURLToFile(fileUrl, new File(configDir.getR..-dsa.key"));

[MSG]Violation in class org.gradle.integtests.fixtures.SFTPServer. getRoot() can probably be rewritten as root

[SRC]sshServer.setFileSystemFactory(new TestNativeFileSystemF..stLogger() {

[MSG]Violation in class org.gradle.integtests.fixtures.SFTPServer. getRoot() can probably be rewritten as root

[SRC]sshServer.setKeyPairProvider(new SimpleGeneratorHostKeyP..-dsa.key"));

[MSG]Violation in class org.gradle.integtests.fixtures.SFTPServer. getRoot() can probably be rewritten as root

[SRC]sshServer.setPasswordAuthenticator(new DummyPasswordAuthenticator());

[MSG]The code could be more concise by using a with() or identity() block

[SRC]sshServer.setPublickeyAuthenticator(new PublickeyAuthenticator() {

[MSG]The code could be more concise by using a with() or identity() block

[SRC]new File(baseDir.getRoot(), filePathToCheck).exists()

[MSG]Violation in class org.gradle.integtests.fixtures.SFTPServer. getRoot() can probably be rewritten as root

[SRC]new TestFile(new File(baseDir.getRoot(), expectedPath))

[MSG]Violation in class org.gradle.integtests.fixtures.SFTPServer. getRoot() can probably be rewritten as root

[SRC]boolean authenticate(String username, String password, o..n session) {

[MSG]The org.apache.sshd.server.session.ServerSession class was explicitly imported, so specifying the package name is not necessary

[SRC]TestClassExecutionResult assertTestsSkipped(String... testNames) {

[MSG]Violation in class TestNgTestClassExecutionResult. Method parameter [testNames] is never referenced in the method assertTestsSkipped of class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult

[SRC]TestClassExecutionResult assertStdout(Matcher<? super St..> matcher) {

[MSG]Violation in class TestNgTestClassExecutionResult. Method parameter [matcher] is never referenced in the method assertStdout of class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult

[SRC]TestClassExecutionResult assertStderr(Matcher<? super St..> matcher) {

[MSG]Violation in class TestNgTestClassExecutionResult. Method parameter [matcher] is never referenced in the method assertStderr of class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult

[SRC]def TestNGExecutionResult(projectDir) {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNGExecutionResult. The def keyword is unneeded on constructors

[SRC]private def findTestClass(String testClass) {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNGExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]private def findTestClasses() {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNGExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]def String testClass

[MSG]Violation in class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult. The def keyword is unneeded when a field type is specified

[SRC]def GPathResult testClassNode

[MSG]Violation in class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult. The def keyword is unneeded when a field type is specified

[SRC]def TestNgTestClassExecutionResult(String testClass, GPa..resultXml) {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult. The def keyword is unneeded on constructors

[SRC]private def findConfigMethod(String testName) {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]private def findConfigMethods() {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]private def findTestMethod(String testName) {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]private def findTestMethods() {

[MSG]Violation in class org.gradle.integtests.fixtures.TestNgTestClassExecutionResult. The def keyword is unneeded when a method is marked private

[SRC]return getFile(baseDir.getAbsolutePath(), file);

[MSG]Violation in class org.gradle.integtests.fixtures.TestNativeFileSystemView. getAbsolutePath() can probably be rewritten as absolutePath

[SRC]String userFileName = physicalName.substring("/".length() - 1);

[MSG]Violation in class org.gradle.integtests.fixtures.TestNativeFileSystemView. The String.substring(int) method can be replaced with the subscript operator

[SRC]String userName = session.getUsername();

[MSG]Violation in class org.gradle.integtests.fixtures.TestNativeFileSystemFactory. getUsername() can probably be rewritten as username

[SRC]void filter(HttpRequest httpRequest) {

[MSG]Violation in class TestProxyServer$1. Method parameter [httpRequest] is never referenced in the method filter of class org.gradle.integtests.fixtures.TestProxyServer$1

[SRC]if (line.matches('Download .+')) {

[MSG]The if statement is empty

[SRC]def UserGuideSamplesRunner(Class<?> testClass) {

[MSG]Violation in class org.gradle.integtests.fixtures.UserGuideSamplesRunner. The def keyword is unneeded on constructors

[SRC]this.dirFilter = getDirFilterPattern()

[MSG]Violation in class org.gradle.integtests.fixtures.UserGuideSamplesRunner. getDirFilterPattern() can probably be rewritten as dirFilterPattern

[SRC]private def cleanup(SampleRun run) {

[MSG]Violation in class org.gradle.integtests.fixtures.UserGuideSamplesRunner. The def keyword is unneeded when a method is marked private

[SRC]private def runSample(GradleRun run) {

[MSG]Violation in class org.gradle.integtests.fixtures.UserGuideSamplesRunner. The def keyword is unneeded when a method is marked private

[SRC]private def compareStrings(String expected, String actua..xtraLines) {

[MSG]Violation in class org.gradle.integtests.fixtures.UserGuideSamplesRunner. The def keyword is unneeded when a method is marked private

[SRC]actual = actual.replaceAll(java.util.regex.Pattern.quote..le/samples')

[MSG]The java.util.regex.Pattern class was explicitly imported, so specifying the package name is not necessary

[SRC]actual = actual.replaceAll(java.util.regex.Pattern.quote..rator), '/')

[MSG]The java.util.regex.Pattern class was explicitly imported, so specifying the package name is not necessary

[SRC]String getPluginId() {

[MSG]Violation in class WellBehavedPluginTest. The method getPluginId is public but not a test method

[SRC]String getMainTask() {

[MSG]Violation in class WellBehavedPluginTest. The method getMainTask is public but not a test method

[SRC]def "plugin does not force creation of build dir during ..uration"() {

[MSG]Violation in class WellBehavedPluginTest. The method plugin does not force creation of build dir during configuration is public but not a test method

[SRC]def "plugin can build with empty project"() {

[MSG]Violation in class WellBehavedPluginTest. The method plugin can build with empty project is public but not a test method

[SRC]buildFile << "apply plugin: '${getPluginId()}'"

[MSG]Violation in class org.gradle.integtests.fixtures.WellBehavedPluginTest. getPluginId() can probably be rewritten as pluginId

[SRC]buildFile << "apply plugin: '${getPluginId()}'"

[MSG]Violation in class org.gradle.integtests.fixtures.WellBehavedPluginTest. getPluginId() can probably be rewritten as pluginId

[SRC]def headersConfiguration = getHeadersConfiguration()

[MSG]Violation in class org.gradle.plugins.binaries.model.internal.ConfigurationBasedNativeDependencySet. getHeadersConfiguration() can probably be rewritten as headersConfiguration

[SRC]protected void store(Node xml) {

[MSG]Violation in class CprojectDescriptor. Method parameter [xml] is never referenced in the method store of class org.gradle.plugins.cpp.cdt.model.CprojectDescriptor

[SRC]task.outputs.file { getOutputFile() }

[MSG]Violation in class org.gradle.plugins.cpp.gpp.GppCompileSpec. getOutputFile() can probably be rewritten as outputFile

[SRC]project.file "$project.buildDir/binaries/${getOutputFileName()}"

[MSG]Violation in class org.gradle.plugins.cpp.gpp.GppCompileSpec. getOutputFileName() can probably be rewritten as outputFileName

[SRC]return "${getBaseName()}.${extension}"

[MSG]Violation in class org.gradle.plugins.cpp.gpp.GppCompileSpec. getBaseName() can probably be rewritten as baseName

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]return getDefaultOutputFileName()

[MSG]Violation in class org.gradle.plugins.cpp.gpp.GppCompileSpec. getDefaultOutputFileName() can probably be rewritten as defaultOutputFileName

[SRC]return OperatingSystem.current().getExecutableName(getBaseName())

[MSG]Violation in class org.gradle.plugins.cpp.gpp.GppCompileSpec. getBaseName() can probably be rewritten as baseName

[SRC]return OperatingSystem.current().getSharedLibraryName(getBaseName())

[MSG]Violation in class org.gradle.plugins.cpp.gpp.GppLibraryCompileSpec. getBaseName() can probably be rewritten as baseName

[SRC]return installName ?: getOutputFileName()

[MSG]Violation in class org.gradle.plugins.cpp.gpp.GppLibraryCompileSpec. getOutputFileName() can probably be rewritten as outputFileName

[SRC]getLibDirName()

[MSG]Violation in class org.gradle.plugins.ear.Ear. getLibDirName() can probably be rewritten as libDirName

[SRC]module = new DefaultEarWebModule(details.path, details.p..dexOf('.')))

[MSG]Violation in class org.gradle.plugins.ear.Ear. The String.substring(int, int) method can be replaced with the subscript operator

[SRC]return ConfigureUtil.configure(configureClosure, getLib())

[MSG]Violation in class org.gradle.plugins.ear.Ear. getLib() can probably be rewritten as lib

[SRC]def EarPluginConvention(FileResolver fileResolver) {

[MSG]Violation in class org.gradle.plugins.ear.EarPluginConvention. The def keyword is unneeded on constructors

[SRC]if (file.getParentFile() != null) {

[MSG]Violation in class org.gradle.plugins.ear.descriptor.internal.DefaultDeploymentDescriptor. getParentFile() can probably be rewritten as parentFile

[SRC]file.getParentFile().mkdirs()

[MSG]Violation in class org.gradle.plugins.ear.descriptor.internal.DefaultDeploymentDescriptor. getParentFile() can probably be rewritten as parentFile

[SRC]new Node(web, nodeNameFor("web-uri", name), path)

[MSG]The instantiated object is not used

[SRC]new Node(web, nodeNameFor("context-root", name), contextRoot)

[MSG]The instantiated object is not used

[SRC]EclipsePlugin delegatePlugin = project.getPlugins().appl..ugin.class);

[MSG]Violation in class org.gradle.plugins.ide.eclipse.EclipseWtpPlugin. getPlugins() can probably be rewritten as plugins

[SRC]EclipsePlugin delegatePlugin = project.getPlugins().appl..ugin.class);

[MSG]EclipsePlugin.class can be rewritten as EclipsePlugin

[SRC]delegatePlugin.getLifecycleTask().dependsOn(getLifecycleTask())

[MSG]Violation in class org.gradle.plugins.ide.eclipse.EclipseWtpPlugin. getLifecycleTask() can probably be rewritten as lifecycleTask

[SRC]delegatePlugin.getLifecycleTask().dependsOn(getLifecycleTask())

[MSG]Violation in class org.gradle.plugins.ide.eclipse.EclipseWtpPlugin. getLifecycleTask() can probably be rewritten as lifecycleTask

[SRC]delegatePlugin.getCleanTask().dependsOn(getCleanTask())

[MSG]Violation in class org.gradle.plugins.ide.eclipse.EclipseWtpPlugin. getCleanTask() can probably be rewritten as cleanTask

[SRC]delegatePlugin.getCleanTask().dependsOn(getCleanTask())

[MSG]Violation in class org.gradle.plugins.ide.eclipse.EclipseWtpPlugin. getCleanTask() can probably be rewritten as cleanTask

[SRC]if (WarPlugin.class.isAssignableFrom(type)) {

[MSG]WarPlugin.class can be rewritten as WarPlugin

[SRC]} else if (EarPlugin.class.isAssignableFrom(type)) {

[MSG]EarPlugin.class can be rewritten as EarPlugin

[SRC]jdt = services.get(Instantiator).newInstance(EclipseJdt,..nsformer()))

[MSG]Violation in class org.gradle.plugins.ide.eclipse.GenerateEclipseJdt. getTransformer() can probably be rewritten as transformer

[SRC]return new Jdt(getTransformer())

[MSG]Violation in class org.gradle.plugins.ide.eclipse.GenerateEclipseJdt. getTransformer() can probably be rewritten as transformer

[SRC]def jdtModel = getJdt()

[MSG]Violation in class org.gradle.plugins.ide.eclipse.GenerateEclipseJdt. getJdt() can probably be rewritten as jdt

[SRC]def allAttributes = attributes.findAll { it.value } + [k.. path: path]

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.AbstractClasspathEntry. getKind() can probably be rewritten as kind

[SRC]def AccessRule(kind, pattern) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.AccessRule. The def keyword is unneeded on constructors

[SRC]def BuildCommand(String name, Map arguments = [:]) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.BuildCommand. The def keyword is unneeded on constructors

[SRC]getLibConfigurations()

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.EclipseWtpComponent. getLibConfigurations() can probably be rewritten as libConfigurations

[SRC]xmlFacet.configure(getFacets())

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.EclipseWtpFacet. getFacets() can probably be rewritten as facets

[SRC]def Facet() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Facet. The def keyword is unneeded on constructors

[SRC]def Facet(Node node) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Facet. The def keyword is unneeded on constructors

[SRC]def Facet(String name, String version) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Facet. The def keyword is unneeded on constructors

[SRC]def Facet(FacetType type, String name, String version) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Facet. The def keyword is unneeded on constructors

[SRC]def Link(String name, String type, String location, Stri..cationUri) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Link. The def keyword is unneeded on constructors

[SRC]def Output(Node node) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Output. The def keyword is unneeded on constructors

[SRC]def Output(String path) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Output. The def keyword is unneeded on constructors

[SRC]node.appendNode('classpathentry', [kind: getKind(), path: path])

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Output. getKind() can probably be rewritten as kind

[SRC]def Project(XmlTransformer xmlTransformer) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded on constructors

[SRC]private def readReferencedProjects() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def readNatures() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def readBuildCommands() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def readLinkedResources() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def addReferencedProjectsToXml() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def addNaturesToXml() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def addBuildSpecToXml() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def addLinkedResourcesToXml() {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.Project. The def keyword is unneeded when a method is marked private

[SRC]def WbDependentModule(node) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.WbDependentModule. The def keyword is unneeded on constructors

[SRC]def WbDependentModule(String deployPath, String handle) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.WbDependentModule. The def keyword is unneeded on constructors

[SRC]def WbProperty(node) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.WbProperty. The def keyword is unneeded on constructors

[SRC]def WbProperty(String name, String value) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.WbProperty. The def keyword is unneeded on constructors

[SRC]def WbResource(node) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.WbResource. The def keyword is unneeded on constructors

[SRC]def WbResource(String deployPath, String sourcePath) {

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.WbResource. The def keyword is unneeded on constructors

[SRC]path = entry.key + filePath.substring(len)

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.internal.FileReferenceFactory. The String.substring(int) method can be replaced with the subscript operator

[SRC]def file = new File(entry.value, path.substring(prefix.length()))

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.internal.FileReferenceFactory. The String.substring(int) method can be replaced with the subscript operator

[SRC]", jarUrl='" + getJarURL() + '\'' +

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.internal.FileReferenceFactory$FileReferenceImpl. getJarURL() can probably be rewritten as jarURL

[SRC]dependencies.collect { it.resolve() }.flatten() as LinkedHashSet

[MSG]Violation in class org.gradle.plugins.ide.eclipse.model.internal.WtpComponentFactory. The call to collect could probably be rewritten as a spread expression: dependencies*.resolve()

[SRC]getModule().mergeXmlModule(xmlModule)

[MSG]Violation in class org.gradle.plugins.ide.idea.GenerateIdeaModule. getModule() can probably be rewritten as module

[SRC]getIdeaProject().mergeXmlProject(xmlModule)

[MSG]Violation in class org.gradle.plugins.ide.idea.GenerateIdeaProject. getIdeaProject() can probably be rewritten as ideaProject

[SRC]getWorkspace().mergeXmlWorkspace(xmlWorkspace)

[MSG]Violation in class org.gradle.plugins.ide.idea.GenerateIdeaWorkspace. getWorkspace() can probably be rewritten as workspace

[SRC]ConfigureUtil.configure(closure, getModule())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModel. getModule() can probably be rewritten as module

[SRC]ConfigureUtil.configure(closure, getProject())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModel. getProject() can probably be rewritten as project

[SRC]ConfigureUtil.configure(closure, getWorkspace())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModel. getWorkspace() can probably be rewritten as workspace

[SRC]ConfigureUtil.configure(closure, getIml())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getIml() can probably be rewritten as iml

[SRC]new File((File) iml.getGenerateTo(), getName() + ".iml")

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getGenerateTo() can probably be rewritten as generateTo

[SRC]new File((File) iml.getGenerateTo(), getName() + ".iml")

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getName() can probably be rewritten as name

[SRC]return new IdeaDependenciesProvider().provide(this, getPathFactory())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getPathFactory() can probably be rewritten as pathFactory

[SRC]def path = { getPathFactory().path(it) }

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getPathFactory() can probably be rewritten as pathFactory

[SRC]def contentRoot = path(getContentRoot())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getContentRoot() can probably be rewritten as contentRoot

[SRC]Set sourceFolders = getSourceDirs().findAll { it.exists(..{ path(it) }

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getSourceDirs() can probably be rewritten as sourceDirs

[SRC]Set testSourceFolders = getTestSourceDirs().findAll { it..{ path(it) }

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getTestSourceDirs() can probably be rewritten as testSourceDirs

[SRC]Set excludeFolders = getExcludeDirs().collect { path(it) }

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getExcludeDirs() can probably be rewritten as excludeDirs

[SRC]def outputDir = getOutputDir() ? path(getOutputDir()) : null

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getOutputDir() can probably be rewritten as outputDir

[SRC]def outputDir = getOutputDir() ? path(getOutputDir()) : null

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getOutputDir() can probably be rewritten as outputDir

[SRC]def testOutputDir = getTestOutputDir() ? path(getTestOut..ir()) : null

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getTestOutputDir() can probably be rewritten as testOutputDir

[SRC]def testOutputDir = getTestOutputDir() ? path(getTestOut..ir()) : null

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getTestOutputDir() can probably be rewritten as testOutputDir

[SRC]getInheritOutputDirs(), outputDir, testOutputDir, depend..etJdkName())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getInheritOutputDirs() can probably be rewritten as inheritOutputDirs

[SRC]getInheritOutputDirs(), outputDir, testOutputDir, depend..etJdkName())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaModule. getJdkName() can probably be rewritten as jdkName

[SRC]getOutputFile().name.replaceFirst(/\.ipr$/, '')

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaProject. getOutputFile() can probably be rewritten as outputFile

[SRC]ConfigureUtil.configure(closure, getIpr())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaProject. getIpr() can probably be rewritten as ipr

[SRC]def modulePaths = getModules().collect {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaProject. getModules() can probably be rewritten as modules

[SRC]getPathFactory().relativePath('PROJECT_DIR', it.outputFile)

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaProject. getPathFactory() can probably be rewritten as pathFactory

[SRC]xmlProject.configure(modulePaths, getJdkName(), getLangu..Wildcards())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaProject. getJdkName() can probably be rewritten as jdkName

[SRC]xmlProject.configure(modulePaths, getJdkName(), getLangu..Wildcards())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaProject. getLanguageLevel() can probably be rewritten as languageLevel

[SRC]xmlProject.configure(modulePaths, getJdkName(), getLangu..Wildcards())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaProject. getWildcards() can probably be rewritten as wildcards

[SRC]ConfigureUtil.configure(closure, getIws())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.IdeaWorkspace. getIws() can probably be rewritten as iws

[SRC]def JarDirectory(path, recursive) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.JarDirectory. The def keyword is unneeded on constructors

[SRC]def Jdk(String jdkName, IdeaLanguageLevel ideaLanguageLevel) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Jdk. The def keyword is unneeded on constructors

[SRC]def Jdk(assertKeyword, jdk15, languageLevel, projectJdkName) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Jdk. The def keyword is unneeded on constructors

[SRC]protected def configure(Path contentPath, Set sourceFold..ludeFolders,

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Module. The def keyword is unneeded when a method is marked protected

[SRC]def ModuleDependency(name, scope) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.ModuleDependency. The def keyword is unneeded on constructors

[SRC]parentNode.appendNode('orderEntry', [type: 'module', 'mo..dExported())

[MSG]Violation in class org.gradle.plugins.ide.idea.model.ModuleDependency. getAttributeMapForScopeAndExported() can probably be rewritten as attributeMapForScopeAndExported

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]result = 31 * result + getScopeHash();

[MSG]Violation in class org.gradle.plugins.ide.idea.model.ModuleDependency. getScopeHash() can probably be rewritten as scopeHash

[SRC]def ModuleLibrary(Collection<Path> classes, Collection<P..ing scope) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.ModuleLibrary. The def keyword is unneeded on constructors

[SRC]Node libraryNode = parentNode.appendNode('orderEntry', [..e('library')

[MSG]Violation in class org.gradle.plugins.ide.idea.model.ModuleLibrary. getAttributeMapForScopeAndExported() can probably be rewritten as attributeMapForScopeAndExported

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]result = 31 * result + getScopeHash()

[MSG]Violation in class org.gradle.plugins.ide.idea.model.ModuleLibrary. getScopeHash() can probably be rewritten as scopeHash

[SRC]expandedUrl = toUrl('file', new File(expandedUrl.substri..nonicalFile)

[MSG]Violation in class org.gradle.plugins.ide.idea.model.PathFactory. The String.substring(int) method can be replaced with the subscript operator

[SRC]def parts = expandedUrl.substring(6).split('!')

[MSG]Violation in class org.gradle.plugins.ide.idea.model.PathFactory. The String.substring(int) method can be replaced with the subscript operator

[SRC]private def toUrl(String scheme, File file) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.PathFactory. The def keyword is unneeded when a method is marked private

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]def Project(XmlTransformer xmlTransformer, pathFactory) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Project. The def keyword is unneeded on constructors

[SRC]private def findProjectRootManager() {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def findWildcardResourcePatterns() {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Project. The def keyword is unneeded when a method is marked private

[SRC]private def findModules() {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Project. The def keyword is unneeded when a method is marked private

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]} else {

[MSG]When an if statement block ends with a return statement the else is unnecessary

[SRC]def Workspace(XmlTransformer withXmlActions) {

[MSG]Violation in class org.gradle.plugins.ide.idea.model.Workspace. The def keyword is unneeded on constructors

[SRC]def filter = { it instanceof SelfResolvingDependency && ..Dependency)}

[MSG]The org.gradle.api.artifacts.ProjectDependency class was explicitly imported, so specifying the package name is not necessary

[SRC]def files = deps.collect { it.resolve() }.flatten()

[MSG]Violation in class org.gradle.plugins.ide.internal.IdeDependenciesExtractor. The call to collect could probably be rewritten as a spread expression: deps*.resolve()

[SRC]def files = deps.collect { it.resolve() }.flatten()

[MSG]Violation in class org.gradle.plugins.ide.internal.IdeDependenciesExtractor. The call to collect could probably be rewritten as a spread expression: deps*.resolve()

[SRC]protected void onApply(Project target) {

[MSG]Violation in class IdePlugin. Method parameter [target] is never referenced in the method onApply of class org.gradle.plugins.ide.internal.IdePlugin

[SRC]String lifecyleTaskName = getLifecycleTaskName();

[MSG]Violation in class org.gradle.plugins.ide.internal.IdePlugin. getLifecycleTaskName() can probably be rewritten as lifecycleTaskName

[SRC]return project.getTasks().getByName(cleanName(worker.getName()));

[MSG]Violation in class org.gradle.plugins.ide.internal.IdePlugin. getTasks() can probably be rewritten as tasks

[SRC]return project.getTasks().getByName(cleanName(worker.getName()));

[MSG]Violation in class org.gradle.plugins.ide.internal.IdePlugin. getName() can probably be rewritten as name

[SRC]Delete cleanWorker = project.getTasks().add(cleanName(wo..elete.class)

[MSG]Violation in class org.gradle.plugins.ide.internal.IdePlugin. getTasks() can probably be rewritten as tasks

[SRC]Delete cleanWorker = project.getTasks().add(cleanName(wo..elete.class)

[MSG]Violation in class org.gradle.plugins.ide.internal.IdePlugin. getName() can probably be rewritten as name

[SRC]Delete cleanWorker = project.getTasks().add(cleanName(wo..elete.class)

[MSG]Delete.class can be rewritten as Delete

[SRC]cleanWorker.delete(worker.getOutputs().getFiles())

[MSG]Violation in class org.gradle.plugins.ide.internal.IdePlugin. getFiles() can probably be rewritten as files

[SRC]cleanWorker.delete(worker.getOutputs().getFiles())

[MSG]Violation in class org.gradle.plugins.ide.internal.IdePlugin. getOutputs() can probably be rewritten as outputs

[SRC]def String moduleName

[MSG]Violation in class org.gradle.plugins.ide.internal.configurer.DeduplicationTarget. The def keyword is unneeded when a field type is specified

[SRC]def Project project

[MSG]Violation in class org.gradle.plugins.ide.internal.configurer.DeduplicationTarget. The def keyword is unneeded when a field type is specified

[SRC]def Closure updateModuleName

[MSG]Violation in class org.gradle.plugins.ide.internal.configurer.DeduplicationTarget. The def keyword is unneeded when a field type is specified

[SRC]String defaultResourceName = getDefaultResourceName();

[MSG]Violation in class org.gradle.plugins.ide.internal.generator.AbstractPersistableConfigurationObject. getDefaultResourceName() can probably be rewritten as defaultResourceName

[SRC]throw new IllegalStateException(String.format("Failed to..getName()));

[MSG]Violation in class org.gradle.plugins.ide.internal.generator.AbstractPersistableConfigurationObject. getName() can probably be rewritten as name

[SRC]protected void load(Node xml) {

[MSG]Violation in class XmlPersistableConfigurationObject. Method parameter [xml] is never referenced in the method load of class org.gradle.plugins.ide.internal.generator.XmlPersistableConfigurationObject

[SRC]protected void store(Node xml) {

[MSG]Violation in class XmlPersistableConfigurationObject. Method parameter [xml] is never referenced in the method store of class org.gradle.plugins.ide.internal.generator.XmlPersistableConfigurationObject

[SRC]void signatureType(SignatureType type) {

[MSG]Violation in class Sign. Method parameter [type] is never referenced in the method signatureType of class org.gradle.plugins.signing.Sign

[SRC]isRequired() || getSignatory() != null

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatory() can probably be rewritten as signatory

[SRC]inputs.property("signatory") { getSignatory()?.keyId?.asHex }

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatory() can probably be rewritten as signatory

[SRC]inputs.files { getSignatures()*.toSign }

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatures() can probably be rewritten as signatures

[SRC]outputs.files { getSignatures()*.toSign }

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatures() can probably be rewritten as signatures

[SRC]if (getSignatory() == null) {

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatory() can probably be rewritten as signatory

[SRC]throw new InvalidUserDataException("Cannot perform signi.. signatory")

[MSG]Violation in class org.gradle.plugins.signing.Sign. getPath() can probably be rewritten as path

[SRC]getSignatures()*.generate()

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatures() can probably be rewritten as signatures

[SRC]def signatureSet = getSignatures()

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatures() can probably be rewritten as signatures

[SRC]new SimpleFileCollection(*getSignatures()*.toSign.findAl.. != null }))

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatures() can probably be rewritten as signatures

[SRC]new SimpleFileCollection(*getSignatures()*.file.findAll(.. != null }))

[MSG]Violation in class org.gradle.plugins.signing.Sign. getSignatures() can probably be rewritten as signatures

[SRC]getDisplayName()

[MSG]Violation in class org.gradle.plugins.signing.SignOperation. getDisplayName() can probably be rewritten as displayName

[SRC]name == null ? (toSignArtifact?.name ?: getFile()?.name) : name

[MSG]Violation in class org.gradle.plugins.signing.Signature. getFile() can probably be rewritten as file

[SRC]extension == null ? getSignatureType()?.extension : extension

[MSG]Violation in class org.gradle.plugins.signing.Signature. getSignatureType() can probably be rewritten as signatureType

[SRC]def toSign = getToSign()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getToSign() can probably be rewritten as toSign

[SRC]def signatureType = getSignatureType()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getSignatureType() can probably be rewritten as signatureType

[SRC]def file = getFile()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getFile() can probably be rewritten as file

[SRC]def toSign = getToSign()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getToSign() can probably be rewritten as toSign

[SRC]def signatureType = getSignatureType()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getSignatureType() can probably be rewritten as signatureType

[SRC]signatureSpec.getSignatory()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getSignatory() can probably be rewritten as signatory

[SRC]signatureSpec.getSignatureType()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getSignatureType() can probably be rewritten as signatureType

[SRC]def toSign = getToSign()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getToSign() can probably be rewritten as toSign

[SRC]def signatory = getSignatory()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getSignatory() can probably be rewritten as signatory

[SRC]def signatureType = getSignatureType()

[MSG]Violation in class org.gradle.plugins.signing.Signature. getSignatureType() can probably be rewritten as signatureType

[SRC]this.configuration = getDefaultConfiguration()

[MSG]Violation in class org.gradle.plugins.signing.SigningExtension. getDefaultConfiguration() can probably be rewritten as defaultConfiguration

[SRC]spec.conventionMapping.map('signatory') { getSignatory() }

[MSG]Violation in class org.gradle.plugins.signing.SigningExtension. getSignatory() can probably be rewritten as signatory

[SRC]spec.conventionMapping.map('signatureType') { getSignatureType() }

[MSG]Violation in class org.gradle.plugins.signing.SigningExtension. getSignatureType() can probably be rewritten as signatureType

[SRC]addSignaturesToConfiguration(signTask, getConfiguration())

[MSG]Violation in class org.gradle.plugins.signing.SigningExtension. getConfiguration() can probably be rewritten as configuration

[SRC]this.addSignaturesToConfiguration(signTask, getConfiguration())

[MSG]Violation in class org.gradle.plugins.signing.SigningExtension. getConfiguration() can probably be rewritten as configuration

[SRC]normalised = keyIdUpped.substring(2)

[MSG]Violation in class org.gradle.plugins.signing.signatory.pgp.PgpKeyId. The String.substring(int) method can be replaced with the subscript operator

[SRC]new File(toSign.path + ".${getExtension()}")

[MSG]Violation in class org.gradle.plugins.signing.type.AbstractSignatureType. getExtension() can probably be rewritten as extension

[SRC]getExtension()

[MSG]Violation in class org.gradle.plugins.signing.type.AbstractSignatureType. getExtension() can probably be rewritten as extension

[SRC]name[++dotIndex..-1] + ".${getExtension()}"

[MSG]Violation in class org.gradle.plugins.signing.type.AbstractSignatureType. getExtension() can probably be rewritten as extension

[SRC]private static final SimpleDateFormat DATE_FORMAT = new .. HH:mm:ss");

[MSG]Violation in class org.gradle.profile.HTMLProfileReport. SimpleDateFormat instances are not thread safe. Wrap the SimpleDateFormat field DATE_FORMAT in a ThreadLocal or make it an instance field

[SRC]return Boolean.class;

[MSG]Boolean.class can be rewritten as Boolean

[SRC]return Long.class;

[MSG]Long.class can be rewritten as Long

[SRC]return Integer.class;

[MSG]Integer.class can be rewritten as Integer

[SRC]return Short.class;

[MSG]Short.class can be rewritten as Short

[SRC]return Byte.class;

[MSG]Byte.class can be rewritten as Byte

[SRC]return Float.class;

[MSG]Float.class can be rewritten as Float

[SRC]return Double.class;

[MSG]Double.class can be rewritten as Double

[SRC]@ExtensionAnnotation(TestPreconditionExtension.class)

[MSG]TestPreconditionExtension.class can be rewritten as TestPreconditionExtension

[SRC]def TestFile baseDir

[MSG]Violation in class org.gradle.util.TestDirHelper. The def keyword is unneeded when a field type is specified

[SRC]def TestDirHelper(TestFile baseDir) {

[MSG]Violation in class org.gradle.util.TestDirHelper. The def keyword is unneeded on constructors

[SRC]while (entry = zipStr.getNextEntry()) {

[MSG]Assignment used as conditional value, which always results in true. Use the == operator instead

[SRC]import static org.hamcrest.Matchers.equalTo

[MSG]Static imports should appear before normal imports

[SRC]import static org.junit.Assert.assertThat

[MSG]Static imports should appear before normal imports

[SRC]import static org.junit.Assert.assertTrue

[MSG]Static imports should appear before normal imports

[SRC]while (entry = zipStr.getNextEntry()) {

[MSG]Violation in class org.gradle.util.TestFileHelper. getNextEntry() can probably be rewritten as nextEntry

[SRC]return perms.substring(1, 10)

[MSG]Violation in class org.gradle.util.TestFileHelper. The String.substring(int, int) method can be replaced with the subscript operator

[SRC]spec.skipped = annotation.value().any() { !it.fulfilled }

[MSG]Violation in class org.gradle.util.TestPreconditionExtension. Parentheses in the 'any' method call are unnecessary and can be removed.

[SRC]feature.skipped = annotation.value().any() { !it.fulfilled }

[MSG]Violation in class org.gradle.util.TestPreconditionExtension. Parentheses in the 'any' method call are unnecessary and can be removed.