Skip to content

Commit f04fecd

Browse files
committed
refactoring
1 parent 827c1ee commit f04fecd

File tree

4 files changed

+17
-25
lines changed

4 files changed

+17
-25
lines changed

jbbp-plugins/jbbp-gradle/src/main/java/com/igormaznitsa/jbbp/plugin/gradle/AbstractJBBPTask.java

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
package com.igormaznitsa.jbbp.plugin.gradle;
22

3-
import com.igormaznitsa.jbbp.plugin.common.converters.ParserFlags;
43
import com.igormaznitsa.meta.common.utils.GetUtils;
54
import org.apache.commons.io.FileUtils;
65
import org.gradle.api.DefaultTask;
76
import org.gradle.api.GradleException;
87
import org.gradle.api.file.FileVisitDetails;
98
import org.gradle.api.file.FileVisitor;
10-
import org.gradle.api.tasks.*;
9+
import org.gradle.api.tasks.TaskAction;
1110

1211
import javax.annotation.Nonnull;
1312
import javax.annotation.Nullable;
@@ -25,27 +24,18 @@ public AbstractJBBPTask() {
2524
@Nullable
2625
public static String getTextOrFileContent(@Nonnull final JBBPExtension extension, @Nullable final String text, @Nullable final File file) {
2726
String result = null;
28-
if (text!=null) {
27+
if (text != null) {
2928
result = text;
3029
} else if (file != null) {
3130
try {
32-
result = FileUtils.readFileToString(file, GetUtils.ensureNonNull(extension.inEncoding,"UTF-8"));
33-
}catch(IOException ex){
34-
throw new GradleException("Can't read file "+file,ex);
31+
result = FileUtils.readFileToString(file, GetUtils.ensureNonNull(extension.inEncoding, "UTF-8"));
32+
} catch (IOException ex) {
33+
throw new GradleException("Can't read file " + file, ex);
3534
}
3635
}
3736
return result;
3837
}
3938

40-
@TaskAction
41-
public final void doAction() {
42-
JBBPExtension ext = getProject().getExtensions().findByType(JBBPExtension.class);
43-
if (ext == null) {
44-
ext = new JBBPExtension(getProject());
45-
}
46-
doTaskAction(ext);
47-
}
48-
4939
@Nonnull
5040
protected static Set<File> findScripts(@Nonnull final JBBPExtension ext) {
5141
final Set<File> result = new HashSet<File>();
@@ -64,6 +54,15 @@ public void visitFile(final FileVisitDetails fileVisitDetails) {
6454
return result;
6555
}
6656

57+
@TaskAction
58+
public final void doAction() {
59+
JBBPExtension ext = getProject().getExtensions().findByType(JBBPExtension.class);
60+
if (ext == null) {
61+
ext = new JBBPExtension(getProject());
62+
}
63+
doTaskAction(ext);
64+
}
65+
6766
protected abstract void doTaskAction(@Nonnull JBBPExtension extension);
6867

6968
}

jbbp-plugins/jbbp-gradle/src/main/java/com/igormaznitsa/jbbp/plugin/gradle/JBBPCleanTask.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,11 @@
11
package com.igormaznitsa.jbbp.plugin.gradle;
22

33
import com.igormaznitsa.jbbp.plugin.common.converters.JBBPScriptTranslator;
4-
import com.igormaznitsa.jbbp.plugin.common.converters.ParserFlags;
5-
import com.igormaznitsa.jbbp.plugin.common.utils.CommonUtils;
64
import org.gradle.api.GradleException;
75

86
import javax.annotation.Nonnull;
97
import java.io.File;
108
import java.io.IOException;
11-
import java.util.Set;
129

1310
public class JBBPCleanTask extends AbstractJBBPTask {
1411

jbbp-plugins/jbbp-gradle/src/main/java/com/igormaznitsa/jbbp/plugin/gradle/JBBPExtension.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,6 @@
44
import com.igormaznitsa.jbbp.plugin.common.converters.Target;
55
import org.gradle.api.Project;
66
import org.gradle.api.file.ConfigurableFileTree;
7-
import org.gradle.api.file.FileTree;
8-
import org.gradle.api.tasks.Input;
9-
import org.gradle.api.tasks.Optional;
10-
import org.gradle.api.tasks.OutputDirectory;
117

128
import javax.annotation.Nonnull;
139
import java.io.File;
@@ -29,7 +25,7 @@ public class JBBPExtension {
2925
public String inEncoding = "UTF-8";
3026
public String outEncoding = "UTF-8";
3127
public File headCommentFile = null;
32-
public String headComment = null;
28+
public String headComment = null;
3329
public File customTextFile = null;
3430
public String customText = null;
3531
public String superClass = null;
@@ -40,7 +36,7 @@ public class JBBPExtension {
4036
public JBBPExtension(@Nonnull final Project project) {
4137
this.source = project.fileTree("src/jbbp");
4238
this.source.include("**/*.jbbp");
43-
this.output = new File(project.getBuildDir(),"generated-jbbp-dir");
39+
this.output = new File(project.getBuildDir(), "generated-jbbp-dir");
4440
}
4541

4642
}

jbbp-plugins/jbbp-gradle/src/test/groovy/com/igormaznitsa/jbbp/gradle/JBBPPluginTest.groovy renamed to jbbp-plugins/jbbp-gradle/src/test/groovy/com/igormaznitsa/jbbp/plugin/gradle/JBBPPluginTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.igormaznitsa.jbbp.plugin.gradle
1+
22

33
import org.gradle.api.Project
44
import org.gradle.testfixtures.ProjectBuilder

0 commit comments

Comments
 (0)