You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by fs...@apache.org on 2019/08/25 19:48:16 UTC

[jmeter] 01/03: Make inner class static

This is an automated email from the ASF dual-hosted git repository.

fschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git

commit 66b90bf6b33094d4b1b1146435ecdccc6592e8dc
Author: Felix Schumacher <fe...@internetallee.de>
AuthorDate: Sun Aug 25 21:19:29 2019 +0200

    Make inner class static
---
 .../java/org/apache/jmeter/gui/action/template/TemplateManager.java   | 2 +-
 .../java/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java | 4 ++--
 .../java/org/apache/jmeter/report/processor/CsvFileSampleSource.java  | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/core/src/main/java/org/apache/jmeter/gui/action/template/TemplateManager.java b/src/core/src/main/java/org/apache/jmeter/gui/action/template/TemplateManager.java
index 637e7c8..4c1699d 100644
--- a/src/core/src/main/java/org/apache/jmeter/gui/action/template/TemplateManager.java
+++ b/src/core/src/main/java/org/apache/jmeter/gui/action/template/TemplateManager.java
@@ -123,7 +123,7 @@ public class TemplateManager {
         return temps;
     }
 
-    public final class LoggingErrorHandler implements ErrorHandler {
+    public static final class LoggingErrorHandler implements ErrorHandler {
         private Logger logger;
         private File file;
 
diff --git a/src/core/src/main/java/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java b/src/core/src/main/java/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
index bd9d3bd..acd8515 100644
--- a/src/core/src/main/java/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
+++ b/src/core/src/main/java/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
@@ -124,7 +124,7 @@ public class HtmlTemplateExporter extends AbstractDataExporter {
      * This class allows to inject graph_options properties to the exported data
      *
      */
-    private class ExtraOptionsResultCustomizer implements ResultCustomizer {
+    private static class ExtraOptionsResultCustomizer implements ResultCustomizer {
         private SubConfiguration extraOptions;
 
         /**
@@ -173,7 +173,7 @@ public class HtmlTemplateExporter extends AbstractDataExporter {
      * This class allows to detect empty graphs
      *
      */
-    private class EmptyGraphChecker implements ResultChecker {
+    private static class EmptyGraphChecker implements ResultChecker {
 
         private final boolean filtersOnlySampleSeries;
         private final boolean showControllerSeriesOnly;
diff --git a/src/core/src/main/java/org/apache/jmeter/report/processor/CsvFileSampleSource.java b/src/core/src/main/java/org/apache/jmeter/report/processor/CsvFileSampleSource.java
index 57b737c..c486480 100644
--- a/src/core/src/main/java/org/apache/jmeter/report/processor/CsvFileSampleSource.java
+++ b/src/core/src/main/java/org/apache/jmeter/report/processor/CsvFileSampleSource.java
@@ -238,7 +238,7 @@ public class CsvFileSampleSource extends AbstractSampleSource {
         produce();
     }
 
-    private class PrivateProducer extends AbstractSampleProcessor implements
+    private static class PrivateProducer extends AbstractSampleProcessor implements
             SampleProducer {
 
         private List<SampleConsumer> sampleConsumers = new ArrayList<>();