On Wed, Dec 6, 2017 at 11:21 PM, <ago...@apache.org> wrote:

> Author: agomes
> Date: Wed Dec  6 22:21:19 2017
> New Revision: 1817321
>
> URL: http://svn.apache.org/viewvc?rev=1817321&view=rev
> Log:
> This closes #345 - Updated to latest checkstyle (v8.5), Added many more
> rules to checkstyle, Included checking of test files and more file types.
> Contributed by Graham Russell
>
> Modified:
>     jmeter/trunk/build.properties
>     jmeter/trunk/build.xml
>     jmeter/trunk/checkstyle-suppressions.xml
>     jmeter/trunk/checkstyle.xml
>     jmeter/trunk/docs/images/screenshots/Proxy_Run.png
>     jmeter/trunk/docs/images/screenshots/Select-Templates-Icon.png
>     jmeter/trunk/docs/images/screenshots/Test_Generated.png
>     jmeter/trunk/docs/images/screenshots/Validate-Test-Plan.png
>     jmeter/trunk/docs/images/screenshots/example-recording.png
>     jmeter/trunk/docs/images/screenshots/example-thread-group.png
>     jmeter/trunk/docs/images/screenshots/http-config/http-
> request-defaults.png
>     jmeter/trunk/docs/images/screenshots/template_menu.png
>     jmeter/trunk/docs/images/screenshots/webtest/http-defaults1.png
>     jmeter/trunk/docs/images/screenshots/webtest/http-defaults2.png
>     jmeter/trunk/docs/images/screenshots/webtest/http-request1.png
>     jmeter/trunk/docs/images/screenshots/webtest/http-request2.png
>     jmeter/trunk/docs/images/screenshots/webtest/http_login.png
>     jmeter/trunk/docs/images/screenshots/webtest/threadgroup.png
>     jmeter/trunk/docs/images/screenshots/webtest/threadgroup2.png
>     jmeter/trunk/src/components/org/apache/jmeter/assertions/
> JSONPathAssertion.java
>     jmeter/trunk/src/components/org/apache/jmeter/assertions/
> ResponseAssertion.java
>     jmeter/trunk/src/components/org/apache/jmeter/assertions/
> gui/BeanShellAssertionGui.java
>     jmeter/trunk/src/components/org/apache/jmeter/assertions/
> gui/JSONPathAssertionGui.java
>     jmeter/trunk/src/components/org/apache/jmeter/control/
> ThroughputController.java
>     jmeter/trunk/src/components/org/apache/jmeter/extractor/
> gui/HtmlExtractorGui.java
>     jmeter/trunk/src/components/org/apache/jmeter/extractor/
> json/jsonpath/JSONManager.java
>     jmeter/trunk/src/components/org/apache/jmeter/reporters/
> MailerModel.java
>     jmeter/trunk/src/components/org/apache/jmeter/timers/
> ConstantThroughputTimerBeanInfo.java
>     jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java
>     jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> RenderAsXML.java
>     jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> RenderInBrowser.java
>     jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> ViewResultsFullVisualizer.java
>     jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> backend/BackendListener.java
>     jmeter/trunk/src/core/org/apache/jmeter/NewDriver.java
>     jmeter/trunk/src/core/org/apache/jmeter/assertions/gui/
> AbstractAssertionGui.java
>     jmeter/trunk/src/core/org/apache/jmeter/config/gui/
> SimpleConfigGui.java
>     jmeter/trunk/src/core/org/apache/jmeter/functions/gui/
> FunctionHelper.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/GlobalUndoableEdit.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/UndoHistoryItem.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/action/RemoteStart.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/util/HorizontalPanel.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/util/MenuFactory.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/util/PowerTableModel.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/util/VerticalPanel.java
>     jmeter/trunk/src/core/org/apache/jmeter/gui/util/theme/
> darcula_theme.xml
>     jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleBuilder.java
>     jmeter/trunk/src/core/org/apache/jmeter/report/core/
> SampleMetaDataParser.java
>     jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleWriter.java
>     jmeter/trunk/src/core/org/apache/jmeter/report/
> dashboard/TemplateVisitor.java
>     jmeter/trunk/src/core/org/apache/jmeter/report/
> processor/FilterConsumer.java
>     jmeter/trunk/src/core/org/apache/jmeter/report/
> processor/TaggerConsumer.java
>     jmeter/trunk/src/core/org/apache/jmeter/reporters/
> ResultCollectorHelper.java
>     jmeter/trunk/src/core/org/apache/jmeter/reporters/gui/
> ResultActionGui.java
>     jmeter/trunk/src/core/org/apache/jmeter/resources/
> messages_fr.properties
>     jmeter/trunk/src/core/org/apache/jmeter/samplers/
> AsynchSampleSender.java
>     jmeter/trunk/src/core/org/apache/jmeter/samplers/
> SampleSenderFactory.java
>     jmeter/trunk/src/core/org/apache/jmeter/samplers/
> StatisticalSampleSender.java
>     jmeter/trunk/src/core/org/apache/jmeter/save/CSVSaveService.java
>     jmeter/trunk/src/core/org/apache/jmeter/save/
> ScriptWrapperConverter.java
>     jmeter/trunk/src/core/org/apache/jmeter/save/converters/
> HashTreeConverter.java
>     jmeter/trunk/src/core/org/apache/jmeter/save/converters/
> SampleSaveConfigurationConverter.java
>     jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/
> GenericTestBeanCustomizer.java
>     jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/TableEditor.java
>     jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/TypeEditor.java
>     jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/
> WrapperEditor.java
>     jmeter/trunk/src/core/org/apache/jmeter/testelement/
> property/AbstractProperty.java
>     jmeter/trunk/src/core/org/apache/jmeter/testelement/
> property/NumberProperty.java
>     jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterVariables.java
>     jmeter/trunk/src/core/org/apache/jmeter/util/
> BeanShellBeanInfoSupport.java
>     jmeter/trunk/src/core/org/apache/jmeter/util/Calculator.java
>     jmeter/trunk/src/core/org/apache/jmeter/util/
> CustomX509TrustManager.java
>     jmeter/trunk/src/core/org/apache/jmeter/util/HostNameSetter.java
>     jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java
>     jmeter/trunk/src/core/org/apache/jmeter/util/
> JSR223BeanInfoSupport.java
>     jmeter/trunk/src/core/org/apache/jmeter/util/JsseSSLManager.java
>     jmeter/trunk/src/examples/org/apache/jmeter/examples/
> sampler/gui/ExampleSamplerGui.java
>     jmeter/trunk/src/examples/org/apache/jmeter/examples/
> testbeans/example3/Example3.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/
> DigestEncodeFunction.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/JavaScript.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/
> Jexl2Function.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/
> LogFunction.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/
> SplitFunction.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/
> StringFromFile.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/Uuid.java
>     jmeter/trunk/src/functions/org/apache/jmeter/functions/XPath.java
>     jmeter/trunk/src/jorphan/org/apache/jorphan/collections/Data.java
>     jmeter/trunk/src/jorphan/org/apache/jorphan/gui/MenuScroller.java
>     jmeter/trunk/src/jorphan/org/apache/jorphan/gui/ObjectTableModel.java
>     jmeter/trunk/src/jorphan/org/apache/jorphan/math/StatCalculator.java
>     jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/Functor.java
>     jmeter/trunk/src/jorphan/org/apache/jorphan/util/Converter.java
>     jmeter/trunk/src/junit/org/apache/jmeter/protocol/java/control/gui/
> JUnitTestSamplerGui.java
>     jmeter/trunk/src/junit/test/AfterAnnotatedTest.java
>     jmeter/trunk/src/junit/test/BeforeAnnotatedTest.java
>     jmeter/trunk/src/junit/test/DummyAnnotatedTest.java
>     jmeter/trunk/src/protocol/ftp/org/apache/jmeter/protocol/
> ftp/sampler/FTPSampler.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/control/DNSCacheManager.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/control/HttpMirrorThread.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/control/KerberosManager.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/control/gui/RecordController.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/gui/AuthPanel.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/gui/CookiePanel.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/gui/HeaderPanel.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/modifier/AnchorModifier.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/modifier/RegExUserParameters.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/
> BaseParser.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/
> CssParser.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/
> CssParserCacheLoader.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/
> HTMLParser.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/
> HtmlParsingUtils.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/
> JTidyHTMLParser.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/
> URLCollection.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/proxy/Daemon.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/
> FormCharSetFinder.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/
> HttpRequestHdr.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/proxy/Proxy.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/
> SamplerCreatorFactory.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/sampler/AccessLogSamplerBeanInfo.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/sampler/HTTPHC4Impl.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/
> ConversionUtils.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/
> HTTPConstantsInterface.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/
> HTTPResultConverter.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/util/accesslog/LogFilter.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/util/accesslog/SessionFilter.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/util/accesslog/StandardGenerator.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/util/accesslog/TCLogParser.java
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
> protocol/http/visualizers/RequestViewHTTP.java
>     jmeter/trunk/src/protocol/java/org/apache/jmeter/
> protocol/java/control/gui/BeanShellSamplerGui.java
>     jmeter/trunk/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/
> AbstractJDBCTestElement.java
>     jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/
> jms/client/ClientPool.java
>     jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/
> jms/control/gui/JMSPublisherGui.java
>     jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/
> jms/control/gui/JMSSamplerGui.java
>     jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/
> jms/sampler/JMSSampler.java
>     jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/
> jms/sampler/render/ObjectMessageRenderer.java
>     jmeter/trunk/src/protocol/ldap/org/apache/jmeter/
> protocol/ldap/config/gui/LDAPArguments.java
>     jmeter/trunk/src/protocol/ldap/org/apache/jmeter/
> protocol/ldap/config/gui/LdapExtConfigGui.java
>     jmeter/trunk/src/protocol/mongodb/org/apache/jmeter/
> protocol/mongodb/config/MongoSourceElement.java
>     jmeter/trunk/src/protocol/mongodb/org/apache/jmeter/
> protocol/mongodb/config/MongoSourceElementBeanInfo.java
>     jmeter/trunk/src/protocol/tcp/org/apache/jmeter/protocol/
> tcp/control/gui/TCPSamplerGui.java
>     jmeter/trunk/src/protocol/tcp/org/apache/jmeter/protocol/tcp/sampler/
> BinaryTCPClientImpl.java
>     jmeter/trunk/src/protocol/tcp/org/apache/jmeter/protocol/
> tcp/sampler/TCPSampler.java
>     jmeter/trunk/test/src/org/apache/jmeter/control/
> TestTransactionController.java
>     jmeter/trunk/test/src/org/apache/jmeter/functions/SumFunctionTest.java
>     jmeter/trunk/test/src/org/apache/jmeter/functions/
> TestDigestFunction.java
>     jmeter/trunk/test/src/org/apache/jmeter/functions/
> TestFileToString.java
>     jmeter/trunk/test/src/org/apache/jmeter/protocol/http/
> control/TestHTTPMirrorThread.java
>     jmeter/trunk/test/src/org/apache/jmeter/protocol/http/
> parser/TestBaseParser.java
>     jmeter/trunk/test/src/org/apache/jmeter/protocol/http/
> parser/TestHTMLParser.java
>     jmeter/trunk/test/src/org/apache/jmeter/report/dashboard/
> ApdexPerTransactionTest.java
>     jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java
>     jmeter/trunk/test/src/org/apache/jmeter/resources/
> TestPropertiesFiles.java
>     jmeter/trunk/test/src/org/apache/jmeter/test/ResourceLocator.java
>     jmeter/trunk/test/src/org/apache/jmeter/testelement/
> TestNumberProperty.java
>     jmeter/trunk/test/src/org/apache/jmeter/util/XPathUtilTest.java
>     jmeter/trunk/test/src/org/apache/jmeter/visualizers/
> TestSampleCompareTo.java
>     jmeter/trunk/xdocs/changes.xml
>
> Modified: jmeter/trunk/build.properties
> URL: http://svn.apache.org/viewvc/jmeter/trunk/build.properties?
> rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/build.properties (original)
> +++ jmeter/trunk/build.properties Wed Dec  6 22:21:19 2017
> @@ -80,10 +80,10 @@ bcpkix.jar                  = bcpkix-jdk
>  bcpkix.loc                  = ${maven2.repo}/org/
> bouncycastle/bcpkix-jdk15on/${bcprov.version}
>  bcpkix.md5                  = cb025ef84fb991e14fdf62f6bef7be53
>
> -dec.version                    = 0.1.2
> -dec.jar                        = dec-${dec.version}.jar
> -dec.loc                        = ${maven2.repo}/org/brotli/dec/
> ${dec.version}
> -dec.md5                        = 4b1cd14cf29733941cc536b27e6aedfa
> +dec.version                 = 0.1.2
> +dec.jar                     = dec-${dec.version}.jar
> +dec.loc                     = ${maven2.repo}/org/brotli/dec/
> ${dec.version}
> +dec.md5                     = 4b1cd14cf29733941cc536b27e6aedfa
>
>  caffeine.version            = 2.5.5
>  caffeine.jar                = caffeine-${caffeine.version}.jar
> @@ -140,12 +140,10 @@ commons-pool2.jar             = commons-
>  commons-pool2.loc             = ${maven2.repo}/org/apache/
> commons/commons-pool2/${commons-pool2.version}
>  commons-pool2.md5             = 62727a85e2e1bf6a756f5571d19cc71c
>
> -# Optional for use by checkstyle
> -darcula.version       = e208efb96f70e4be9dc362fbb46f6e181ef501dd
> -darcula.jar           = darcula.jar
> -darcula.loc           = https://github.com/bulenkov/
> Darcula/raw/${darcula.version}/build
> -darcula.md5           = 5afdcd4e299f71fb9dfd1740937bfbea
> -
> +darcula.version            = e208efb96f70e4be9dc362fbb46f6e181ef501dd
> +darcula.jar                = darcula.jar
> +darcula.loc                = https://github.com/bulenkov/
> Darcula/raw/${darcula.version}/build
> +darcula.md5                = 5afdcd4e299f71fb9dfd1740937bfbea
>
>  # dnsjava for DNSCacheManager (http://www.dnsjava.org/ BSD license)
>  dnsjava.version             = 2.1.8
> @@ -185,7 +183,6 @@ httpasyncclient.md5              = 73d4a
>
>  # Apache HttpClient 4.x
>  httpclient.version          = 4.5.3
> -#
>  httpclient.jar              = httpclient-${httpclient.version}.jar
>  httpclient.loc              = ${maven2.repo}/org/apache/
> httpcomponents/httpclient/${httpclient.version}
>  httpclient.md5              = 1965ebb7aca0f9f8faaed3870d8cf689
> @@ -433,10 +430,10 @@ jdom.loc                    = ${maven2.r
>  jdom.md5                    = 140bfed13341fe2039eee0f26a16d705
>
>  # Optional for use by checkstyle
> -checkstyle-all.version       = 6.13
> +checkstyle-all.version       = 8.5
>  checkstyle-all.jar           = checkstyle-${checkstyle-all.
> version}-all.jar
>  checkstyle-all.loc           = http://downloads.sourceforge.
> net/checkstyle/checkstyle/${checkstyle-all.version}/
> checkstyle-${checkstyle-all.version}-all.jar?ts=${
> EPOCHSECONDS}&use_mirror=autoselect
> -checkstyle-all.md5           = ac6e1e81d09bcaf4c0c22181e9bda1d9
> +checkstyle-all.md5           = 6e7d8b1e9a750be769037cc7e3bada40
>
>  # Optional for use by rat
>  rat.version                  = 0.12
> @@ -455,29 +452,29 @@ jacocoant.loc                = ${maven2.
>  jacocoant.md5                = 134a9863234844bf156143dad39564e2
>
>  # Optional for use by JDBC_TESTS.jmx
> -hsqldb.version                  = 2.4.0
> -hsqldb.jar                      = hsqldb-${hsqldb.version}.jar
> -hsqldb.loc                      = ${maven2.repo}/org/hsqldb/
> hsqldb/${hsqldb.version}
> -hsqldb.md5                      = 72cae1d3ef411edc74bc3ff4d12bd47c
> +hsqldb.version               = 2.4.0
> +hsqldb.jar                   = hsqldb-${hsqldb.version}.jar
> +hsqldb.loc                   = ${maven2.repo}/org/hsqldb/
> hsqldb/${hsqldb.version}
> +hsqldb.md5                   = 72cae1d3ef411edc74bc3ff4d12bd47c
>
>  # Optional for use by JMS_TESTS.jmx
> -activemq-all.version                    = 5.15.2
> -activemq-all.jar                        = activemq-all-${activemq-all.
> version}.jar
> -activemq-all.loc                        = ${maven2.repo}/org/apache/
> activemq/activemq-all/${activemq-all.version}
> -activemq-all.md5                        = bd24ae082be11dc969a6e5bc45515a
> b7
> +activemq-all.version         = 5.15.2
> +activemq-all.jar             = activemq-all-${activemq-all.version}.jar
> +activemq-all.loc             = ${maven2.repo}/org/apache/
> activemq/activemq-all/${activemq-all.version}
> +activemq-all.md5             = bd24ae082be11dc969a6e5bc45515ab7
>
>  # Optional for use by FTP_TESTS.jmx
> -mina-core.version               = 2.0.16
> -mina-core.jar                   = mina-core-${mina-core.version}.jar
> -mina-core.loc                   = ${maven2.repo}/org/apache/
> mina/mina-core/${mina-core.version}
> -mina-core.md5                   = fd86528fa9d9ba8fb8c37e3ac28fa45f
> -
> -ftplet-api.version              = 1.1.1
> -ftplet-api.jar                  = ftplet-api-${ftplet-api.version}.jar
> -ftplet-api.loc                  = ${maven2.repo}/org/apache/
> ftpserver/ftplet-api/${ftplet-api.version}
> -ftplet-api.md5                  = a17a7513f5a7e2cd717f7b0fbd15241a
> -
> -ftpserver-core.version                  = 1.1.1
> -ftpserver-core.jar                      = ftpserver-core-${ftpserver-
> core.version}.jar
> -ftpserver-core.loc                      = ${maven2.repo}/org/apache/
> ftpserver/ftpserver-core/${ftpserver-core.version}
> -ftpserver-core.md5                      = 62b0a623ff211013d3056dbdf26139
> b2
> +mina-core.version            = 2.0.16
> +mina-core.jar                = mina-core-${mina-core.version}.jar
> +mina-core.loc                = ${maven2.repo}/org/apache/
> mina/mina-core/${mina-core.version}
> +mina-core.md5                = fd86528fa9d9ba8fb8c37e3ac28fa45f
> +
> +ftplet-api.version           = 1.1.1
> +ftplet-api.jar               = ftplet-api-${ftplet-api.version}.jar
> +ftplet-api.loc               = ${maven2.repo}/org/apache/
> ftpserver/ftplet-api/${ftplet-api.version}
> +ftplet-api.md5               = a17a7513f5a7e2cd717f7b0fbd15241a
> +
> +ftpserver-core.version       = 1.1.1
> +ftpserver-core.jar           = ftpserver-core-${ftpserver-
> core.version}.jar
> +ftpserver-core.loc           = ${maven2.repo}/org/apache/
> ftpserver/ftpserver-core/${ftpserver-core.version}
> +ftpserver-core.md5           = 62b0a623ff211013d3056dbdf26139b2
>
> Modified: jmeter/trunk/build.xml
> URL: http://svn.apache.org/viewvc/jmeter/trunk/build.xml?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/build.xml (original)
> +++ jmeter/trunk/build.xml Wed Dec  6 22:21:19 2017
> @@ -3436,57 +3436,18 @@ run JMeter unless all the JMeter jars ar
>      </antcall>
>    </target>
>
> -  <!-- Special: process to get RSyntaxTextArea-->
> -  <!-- Useless since rsyntaxtextarea artifacts are published to Maven
> Repo since 9th july 2014 (JMeter 2.12) -->
> -  <!--
> -  <target name="_process_rsyntaxtextarea">
> -      <available file="./lib/${rsyntaxtextarea.jar}"
> property="rsyntaxtextarea.exists"/>
> -      <antcall target="_get_rsyntaxtextarea"/>
> -  </target>
> -  -->
> -  <!-- Get RSyntaxTextArea binary zip file, unpack it, pick up jar file
> and remove the zip -->
> -  <!-- Useless since rsyntaxtextarea artifacts are published to Maven
> Repo since 9th july 2014 (JMeter 2.12) -->
> -  <!--
> -  <target name="_get_rsyntaxtextarea" unless="rsyntaxtextarea.exists">
> -    <property name="loc" value="${rsyntaxtextarea.loc}"/>
> -    <property name="zip" value="${rsyntaxtextarea.name}
> _${rsyntaxtextarea.version}.zip"/>
> -    <property name="jar" value="${rsyntaxtextarea.name}.jar"/>
> -    <property name="md5" value="${rsyntaxtextarea.md5}"/>
> -    <property name="path" value="./lib"/>
> -    <get src="${loc}/${zip}"
> -         dest="${build.dir}/${zip}"
> -         usetimestamp="true" ignoreerrors="false"/>
> -    <unzip dest="${build.dir}" src="${build.dir}/${zip}">
> -      <patternset>
> -        <include name="**/${jar}"/>
> -      </patternset>
> -      <mapper type="flatten"/>
> -    </unzip>
> -    <antcall target="_checkMD5">
> -      <param name="file" value="${build.dir}/${jar}"/>
> -      <param name="path" value="${build.dir}"/>
> -      <param name="md5"  value="${md5}"/>
> -    </antcall>
> -    <delete>
> -      <fileset dir="${path}" includes="${rsyntaxtextarea.name}-*.jar"/>
> -    </delete>
> -    <move preservelastmodified="true" overwrite="true"
> -        file="${build.dir}/${jar}" tofile="${path}/${rsyntaxtextarea.jar}"
> verbose="true"/>
> -    <delete file="${build.dir}/${zip}"/>
> -  </target>
> -  -->
> -
>    <!-- Need to define config_loc for use by Eclipse -->
>    <property name="config_loc" value="."/>
>    <target name="checkstyle">
>      <available property="checkstyle.jar.available"
> file="${lib.opt}/${checkstyle-all.jar}"/>
> -    <fail message="This task requires checkstyle, please run
> download_checkstyle target to download checkstyle" unless="checkstyle.jar.
> available"/>
> +    <fail message="This task requires checkstyle, please run
> download_checkstyle target to download checkstyle"
> +          unless="checkstyle.jar.available"/>
>
> -    <antcall target="_process_checkstyle_jars"></antcall>
> +    <antcall target="_process_checkstyle_jars"/>
>      <taskdef resource="com/puppycrawl/tools/checkstyle/ant/
> checkstyle-ant-task.properties"
>               classpath="${lib.opt}/${checkstyle-all.jar}"/>
>      <checkstyle config="checkstyle.xml">
> -      <fileset dir="src" includes="**/*.java"/>
> +      <fileset dir="src" includes="**/*"/>
>        <formatter type="plain"/>
>        <formatter type="xml" toFile="build/checkstyle_errors.xml"/>
>      </checkstyle>
> @@ -3495,7 +3456,8 @@ run JMeter unless all the JMeter jars ar
>
>    <target name="rat">
>      <available property="rat.jar.available" file="${lib.opt}/${rat.jar}"/>
> -    <fail message="This task requires Apache Creadur RAT, please run
> download_rat target to download it" unless="rat.jar.available"/>
> +    <fail message="This task requires Apache Creadur RAT, please run
> download_rat target to download it"
> +          unless="rat.jar.available"/>
>      <taskdef
>        uri="antlib:org.apache.rat.anttasks"
>        resource="org/apache/rat/anttasks/antlib.xml">
>
> Modified: jmeter/trunk/checkstyle-suppressions.xml
> URL: http://svn.apache.org/viewvc/jmeter/trunk/checkstyle-
> suppressions.xml?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/checkstyle-suppressions.xml (original)
> +++ jmeter/trunk/checkstyle-suppressions.xml Wed Dec  6 22:21:19 2017
> @@ -18,10 +18,9 @@ limitations under the License.
>
>  <!DOCTYPE suppressions PUBLIC
>      "-//Puppy Crawl//DTD Suppressions 1.1//EN"
> -    "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd";>
> +    "http://checkstyle.sourceforge.net/dtds/suppressions_1_1.dtd";>
>
>  <suppressions>
> -    <!-- On Windows, it appears that Checkstyle matches files using \
> delims -->
> -    <suppress checks="JavadocPackage" files="[/\\]examples[/\\].*\.
> java"/>
> -    <suppress checks="JavadocPackage" files="[/\\]test[/\\].*\.java"/>
> +    <suppress checks="NewlineAtEndOfFile" files=".*.properties"/>
> +    <suppress checks="FileLength" files="HTTPSamplerBase.java"/>
>  </suppressions>
> \ No newline at end of file
>
> Modified: jmeter/trunk/checkstyle.xml
> URL: http://svn.apache.org/viewvc/jmeter/trunk/checkstyle.xml?
> rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/checkstyle.xml (original)
> +++ jmeter/trunk/checkstyle.xml Wed Dec  6 22:21:19 2017
> @@ -16,12 +16,8 @@
>     limitations under the License.
>  -->
>  <!DOCTYPE module PUBLIC
> -    "-//Puppy Crawl//DTD Check Configuration 1.1//EN"
> -    "http://www.puppycrawl.com/dtds/configuration_1_1.dtd";>
> -
> -<!--
> -  Checkstyle checks configured for Maven.
> --->
> +    "-//Puppy Crawl//DTD Check Configuration 1.3//EN"
> +    "http://checkstyle.sourceforge.net/dtds/configuration_1_3.dtd";>
>
>  <module name="Checker">
>
> @@ -37,84 +33,149 @@
>      <property name="fileExtensions" value="java, groovy"/>
>    </module>
>
> -  <!-- No tabs allowed! -->
> +  <!-- Miscellaneous -->
>    <module name="FileTabCharacter">
> -    <property name="fileExtensions" value="java, groovy, xml, xsd, dtd,
> htm, html, txt"/>
> +    <!-- No tabs allowed! -->
> +    <property name="fileExtensions" value="java, groovy, xml, xsd, dtd,
> htm, html, txt, properties"/>
>    </module>
> -
>    <module name="NewlineAtEndOfFile"/>
> +  <module name="RegexpMultiline">
> +    <!-- No more than 2 blank lines allowed. -->
> +    <property name="format" value="\r?\n[\t ]*\r?\n[\t ]*\r?\n[\t
> ]*\r?\n"/>
> +    <property name="fileExtensions" value="java, xml, properties"/>
> +    <property name="message" value="Unnecessary consecutive lines"/>
> +  </module>
> +  <!--<module name="RegexpOnFilename">-->
> +    <!--<property name="folderPattern" value="[\\/]src[\\/]\w+[\\/]
> java[\\/]"/>-->
> +    <!--<property name="fileNamePattern" value="\.java$"/>-->
> +    <!--<property name="match" value="false"/>-->
> +    <!--<message key="regexp.filepath.mismatch" value="Only java files
> should be located in the ''src/*/java'' folders."/>-->
> +  <!--</module>-->
> +  <module name="UniqueProperties">
> +    <property name="fileExtensions" value="properties"/>
> +  </module>
>
> -  <module name="TreeWalker">
> -
> -    <!-- Operator must be at end of wrapped line -->
> -<!--
> -    <module name="OperatorWrap">
> -      <property name="option" value="eol"/>
> -    </module>
> --->
> +  <!-- Size Violations -->
> +  <module name="FileLength">
> +    <property name="fileExtensions" value="java, groovy"/>
> +  </module>
>
> -    <!-- Interfaces must be types (not just constants) -->
> -    <module name="InterfaceIsType"/>
> +  <module name="TreeWalker">
>
> -    <!-- Must have class / interface header comments -->
> -<!--
> -    <module name="JavadocType"/>
> --->
> -     <!-- Require method javadocs, allow undeclared RTE -->
> -<!--
> -    <module name="JavadocMethod">
> -      <property name="allowUndeclaredRTE" value="true"/>
> -      <property name="allowMissingThrowsTags" value="true"/>
> +    <!-- Annotations -->
> +    <!--<module name="MissingDeprecated"/>-->
> +    <module name="MissingOverride"/>
> +
> +    <!-- Block checks -->
> +    <module name="AvoidNestedBlocks"/>
> +    <module name="EmptyBlock"/>
> +    <module name="EmptyCatchBlock">
> +      <property name="exceptionVariableName" value="expected|ignore"/>
>      </module>
> --->
> -
> -    <!-- Require field javadoc -->
> -<!--
> -    <module name="JavadocVariable"/>
> --->
> +    <!--<module name="LeftCurly"/>-->
> +    <!--<module name="RightCurly"/>-->
> +    <module name="NeedBraces"/>
>
> -    <!-- No public fields -->
> -<!--
> -    <module name="VisibilityModifier">
> -       <property name="protectedAllowed" value="true"/>
> +    <!--Class Design-->
> +    <!--<module name="FinalClass"/>-->
> +    <!--<module name="HideUtilityClassConstructor"/>--> <!--need to
> change 25 class APIs-->
> +    <module name="InterfaceIsType"/> <!-- Interfaces must be types (not
> just constants) -->
> +    <module name="MutableException"/>
> +    <module name="OneTopLevelClass"/>
> +    <module name="ThrowsCount">
> +      <property name="max" value="5"/>
> +      <property name="ignorePrivateMethods" value="false"/>
>      </module>
> --->
>
> +    <!-- Coding -->
> +    <module name="CovariantEquals"/>
>      <!-- Require hash code override when equals is -->
>      <module name="EqualsHashCode"/>
> -
> -    <!-- Disallow unnecessary instantiation of Boolean, String -->
>      <module name="IllegalInstantiation">
> -      <property name="classes" value="java.lang.Boolean,
> java.lang.String"/>
> +      <property name="classes" value="java.lang.Boolean"/>
>      </module>
> -
> -    <module name="AvoidStarImport"/>
> -
> +    <module name="IllegalThrows"/>
> +    <module name="MissingSwitchDefault"/>
> +    <module name="ModifiedControlVariable"/>
> +    <module name="MultipleVariableDeclarations"/>
> +    <module name="NestedForDepth"/>
> +    <module name="NestedIfDepth">
> +      <!-- TODO: reduce this number -->
> +      <property name="max" value="4"/>
> +    </module>
> +    <module name="NestedTryDepth">
> +      <property name="max" value="2"/>
> +    </module>
> +    <module name="OneStatementPerLine"/>
> +    <!--<module name="ParameterAssignment"/>-->
> +    <module name="ReturnCount">
> +      <!-- TODO: reduce these numbers -->
> +      <property name="max" value="9" />
> +      <property name="maxForVoid" value="9"/>
> +    </module>
> +    <module name="SimplifyBooleanExpression"/>
> +    <module name="SimplifyBooleanReturn"/>
> +    <module name="StringLiteralEquality"/>
> +    <module name="SuperFinalize"/>
> +    <module name="UnnecessaryParentheses"/>
> +    <!--<module name="VariableDeclarationUsageDistance"/>-->
> +
> +    <!-- Imports -->
> +    <module name="AvoidStarImport">
> +      <property name="allowClassImports" value="false"/>
> +      <property name="allowStaticMemberImports" value="false"/>
> +    </module>
> +    <module name="RedundantImport"/>
>      <module name="UnusedImports"/>
> +   <module name="ImportOrder">
> +      <property name="groups" value="/^java\./,javax,org,net,com"/>
> +      <property name="ordered" value="true"/>
> +      <property name="separated" value="true"/>
> +      <property name="option" value="top"/>
> +     <property name="sortStaticImportsAlphabetically" value="true"/>
> +    </module>
>
> -    <module name="NeedBraces"/>
> -
> -    <!-- Required for SuppressionCommentFilter below -->
> -    <module name="FileContentsHolder"/>
> +    <!-- JavaDocs -->
> +<!--
> +    <module name="JavadocMethod">
> +      <property name="minLineCount" value="3"/>
> +      <property name="scope" value="protected"/>
> +      <property name="allowUndeclaredRTE" value="true"/>
> +      <property name="allowMissingThrowsTags" value="true"/>
> +      <property name="allowMissingPropertyJavadoc" value="true"/>
> +      <property name="ignoreMethodNamesRegex" value="(get|set|is).*"/>
> +      &lt;!&ndash; Only check methods, i.e. ignore constructors
> &ndash;&gt;
> +      <property name="tokens" value="METHOD_DEF"/>
> +    </module>
> +    -->
> +    <!--<module name="NonEmptyAtclauseDescription"/>-->
>
> +    <!-- Misc -->
> +    <module name="ArrayTypeStyle"/>
> +    <module name="AvoidEscapedUnicodeCharacters"/>
> +    <!--<module name="CommentsIndentation"/>-->
> +    <module name="OuterTypeFilename"/>
> +    <module name="UpperEll"/>
> +
> +    <!-- code metrics -->
> +    <!--<module name="BooleanExpressionComplexity"/>-->
> +    <!--<module name="ClassFanOutComplexity"/>-->
> +    <!--<module name="CyclomaticComplexity"/>-->
> +    <!--<module name="NPathComplexity"/>-->
> +
> +    <!--
> +    Allow comment to suppress checkstyle for a single line
> +    e.g. // CHECKSTYLE IGNORE MagicNumber
> +    -->
> +    <module name="SuppressWithNearbyCommentFilter">
> +      <property name="commentFormat" value="CHECKSTYLE IGNORE (\w+)"/>
> +      <property name="checkFormat" value="$1"/>
> +    </module>
>    </module>
>
> -  <!-- Setup special comments to suppress specific checks from source
> files
> -  CHECKSTYLE\:OFF and CHECKSTYLE\:ON -->
> -  <module name="SuppressionCommentFilter"/>
> -
>    <module name="SuppressionFilter">
>      <!-- config_loc is used by Eclipse plugin -->
>      <property name="file" value="${config_loc}/
> checkstyle-suppressions.xml"/>
>    </module>
>
> -  <!--
> -      Allow comment to suppress checkstyle for a single line
> -      e.g. // CHECKSTYLE IGNORE MagicNumber
> -   -->
> -  <module name="SuppressWithNearbyCommentFilter">
> -    <property name="commentFormat" value="CHECKSTYLE IGNORE (\w+)"/>
> -    <property name="checkFormat" value="$1"/>
> -  </module>
> -
>  </module>
>
> Modified: jmeter/trunk/docs/images/screenshots/Proxy_Run.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/Proxy_Run.png?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/Select-Templates-Icon.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/Select-Templates-Icon.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/Test_Generated.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/Test_Generated.png?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/Validate-Test-Plan.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/Validate-Test-Plan.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/example-recording.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/example-recording.png?rev=1817321&r1=1817320&r2=
> 1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/example-thread-group.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/example-thread-group.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/http-config/http-
> request-defaults.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/http-config/http-request-defaults.png?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/template_menu.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/template_menu.png?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/webtest/http-defaults1.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/webtest/http-defaults1.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/webtest/http-defaults2.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/webtest/http-defaults2.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/webtest/http-request1.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/webtest/http-request1.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/webtest/http-request2.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/webtest/http-request2.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/webtest/http_login.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/webtest/http_login.png?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/webtest/threadgroup.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/webtest/threadgroup.png?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/docs/images/screenshots/webtest/threadgroup2.png
> URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/
> screenshots/webtest/threadgroup2.png?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> Binary files - no diff available.
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/assertions/
> JSONPathAssertion.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/assertions/JSONPathAssertion.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/JSONPathAssertion.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/JSONPathAssertion.java
> Wed Dec  6 22:21:19 2017
> @@ -29,11 +29,11 @@ import org.apache.oro.text.regex.Pattern
>  import org.slf4j.Logger;
>  import org.slf4j.LoggerFactory;
>
> -import com.jayway.jsonpath.JsonPath;
> -
>  import net.minidev.json.JSONArray;
>  import net.minidev.json.JSONObject;
>
> +import com.jayway.jsonpath.JsonPath;
> +
>  /**
>   * This is main class for JSONPath Assertion which verifies assertion on
>   * previous sample result using JSON path expression
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/assertions/
> ResponseAssertion.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/assertions/ResponseAssertion.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/ResponseAssertion.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/ResponseAssertion.java
> Wed Dec  6 22:21:19 2017
> @@ -189,7 +189,7 @@ public class ResponseAssertion extends A
>      }
>
>      private void setTestTypeMasked(int testType) {
> -        int value = getTestType() & ~(TYPE_MASK) | testType;
> +        int value = getTestType() & ~TYPE_MASK | testType;
>          setProperty(new IntegerProperty(TEST_TYPE, value));
>      }
>
> @@ -347,7 +347,7 @@ public class ResponseAssertion extends A
>              return result.setResultForNull();
>          }
>
> -        boolean pass = true;
> +        boolean pass;
>
is this not a change ?

>          boolean hasTrue = false;
>          ArrayList<String> allCheckMessage = new ArrayList<>();
>          try {
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/assertions/
> gui/BeanShellAssertionGui.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/assertions/gui/BeanShellAssertionGui.
> java?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/gui/BeanShellAssertionGui.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/gui/BeanShellAssertionGui.java
> Wed Dec  6 22:21:19 2017
> @@ -28,12 +28,12 @@ import javax.swing.JTextArea;
>  import javax.swing.JTextField;
>
>  import org.apache.jmeter.assertions.BeanShellAssertion;
> +import org.apache.jmeter.gui.util.FilePanelEntry;
>  import org.apache.jmeter.gui.util.JSyntaxTextArea;
>  import org.apache.jmeter.gui.util.JTextScrollPane;
>  import org.apache.jmeter.testelement.TestElement;
>  import org.apache.jmeter.testelement.property.BooleanProperty;
>  import org.apache.jmeter.util.JMeterUtils;
> -import org.apache.jmeter.gui.util.FilePanelEntry;
>
>  public class BeanShellAssertionGui extends AbstractAssertionGui {
>
> @@ -88,8 +88,7 @@ public class BeanShellAssertionGui exten
>          return "bsh_assertion_title"; // $NON-NLS-1$
>      }
>
> -    private JPanel createFilenamePanel()
> -    {
> +    private JPanel createFilenamePanel() {
>
>          JPanel filenamePanel = new JPanel(new BorderLayout());
>          filenamePanel.add(filename, BorderLayout.CENTER);
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/assertions/
> gui/JSONPathAssertionGui.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/assertions/gui/JSONPathAssertionGui.java?
> rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/gui/JSONPathAssertionGui.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/assertions/gui/JSONPathAssertionGui.java
> Wed Dec  6 22:21:19 2017
> @@ -17,6 +17,13 @@
>   */
>  package org.apache.jmeter.assertions.gui;
>
> +import java.awt.BorderLayout;
> +
> +import javax.swing.BorderFactory;
> +import javax.swing.JCheckBox;
> +import javax.swing.event.ChangeEvent;
> +import javax.swing.event.ChangeListener;
> +
>  import org.apache.jmeter.assertions.JSONPathAssertion;
>  import org.apache.jmeter.gui.util.VerticalPanel;
>  import org.apache.jmeter.testelement.TestElement;
> @@ -24,22 +31,12 @@ import org.apache.jmeter.util.JMeterUtil
>  import org.apache.jorphan.gui.JLabeledTextArea;
>  import org.apache.jorphan.gui.JLabeledTextField;
>
> -
> -import javax.swing.JCheckBox;
> -import javax.swing.BorderFactory;
> -import javax.swing.event.ChangeEvent;
> -import javax.swing.event.ChangeListener;
> -import java.awt.BorderLayout;
> -
>  /**
>   * Java class representing GUI for the {@link JSONPathAssertion}
> component in JMeter
>   * @since 4.0
>   */
>  public class JSONPathAssertionGui extends AbstractAssertionGui implements
> ChangeListener {
>
> -    /**
> -     *
> -     */
>      private static final long serialVersionUID = -6008018002423594040L;
>      private JLabeledTextField jsonPath = null;
>      private JLabeledTextArea jsonValue = null;
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/control/
> ThroughputController.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/control/ThroughputController.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java
> Wed Dec  6 22:21:19 2017
> @@ -41,8 +41,9 @@ import org.slf4j.LoggerFactory;
>   * The current implementation executes the first N samples (BYNUMBER)
>   * or the last N% of samples (BYPERCENT).
>   */
> -public class ThroughputController extends GenericController implements
> Serializable, LoopIterationListener,
> -        TestStateListener {
> +public class ThroughputController
> +        extends GenericController
> +        implements Serializable, LoopIterationListener, TestStateListener
> {
>
>      private static final long serialVersionUID = 234L;
>
> @@ -80,19 +81,13 @@ public class ThroughputController extend
>
>      private transient Object counterLock = new Object(); // ensure counts
> are updated correctly
>
> -    /**
> -     * Number of iterations on which we've chosen to deliver samplers.
> -     */
> +    /** Number of iterations on which we've chosen to deliver samplers. */
>      private int numExecutions = 0;
>
> -    /**
> -     * Index of the current iteration. 0-based.
> -     */
> +    /** Index of the current iteration. 0-based. */
>      private int iteration = -1;
>
> -    /**
> -     * Whether to deliver samplers on this iteration.
> -     */
> +    /** Whether to deliver samplers on this iteration. */
>      private boolean runThisTime;
>
>      public ThroughputController() {
> @@ -135,7 +130,7 @@ public class ThroughputController extend
>          JMeterProperty prop = getProperty(MAXTHROUGHPUT);
>          int retVal = 1;
>          if (prop instanceof IntegerProperty) {
> -            retVal = ((IntegerProperty) prop).getIntValue();
> +            retVal = prop.getIntValue();
>          } else {
>              String valueString = prop.getStringValue();
>              try {
> @@ -163,7 +158,7 @@ public class ThroughputController extend
>          JMeterProperty prop = getProperty(PERCENTTHROUGHPUT);
>          float retVal = 100;
>          if (prop instanceof FloatProperty) {
> -            retVal = ((FloatProperty) prop).getFloatValue();
> +            retVal = prop.getFloatValue();
>          } else {
>              String valueString = prop.getStringValue();
>              try {
> @@ -214,8 +209,9 @@ public class ThroughputController extend
>          if (subControllersAndSamplers.isEmpty()) {
>              return true;
>          } else {
> -            return getStyle() == BYNUMBER && getExecutions() >=
> getMaxThroughputAsInt()
> -                && current >= getSubControllers().size();
> +            return getStyle() == BYNUMBER
> +                    && getExecutions() >= getMaxThroughputAsInt()
> +                    && current >= getSubControllers().size();
>          }
>      }
>
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/extractor/
> gui/HtmlExtractorGui.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/extractor/gui/HtmlExtractorGui.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/extractor/gui/HtmlExtractorGui.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/extractor/gui/HtmlExtractorGui.java
> Wed Dec  6 22:21:19 2017
> @@ -125,7 +125,6 @@ public class HtmlExtractorGui extends Ab
>              } else {
>                  htmlExtractor.setExtractor(USE_DEFAULT_EXTRACTOR_IMPL);
>              }
> -
>          }
>      }
>
> @@ -156,8 +155,6 @@ public class HtmlExtractorGui extends Ab
>          add(makeParameterPanel(), BorderLayout.CENTER);
>      }
>
> -
> -
>      private Component makeExtractorPanel() {
>          JPanel panel = new HorizontalPanel();
>          panel.setBorder(BorderFactory.createTitledBorder(
> JMeterUtils.getResString("html_extractor_type"))); //$NON-NLS-1$
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/extractor/
> json/jsonpath/JSONManager.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/extractor/json/jsonpath/JSONManager.
> java?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/extractor/json/jsonpath/JSONManager.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/extractor/json/jsonpath/JSONManager.java
> Wed Dec  6 22:21:19 2017
> @@ -28,14 +28,14 @@ import java.util.Map;
>  import org.slf4j.Logger;
>  import org.slf4j.LoggerFactory;
>
> +import net.minidev.json.JSONArray;
> +import net.minidev.json.JSONObject;
> +
>  import com.jayway.jsonpath.Configuration;
>  import com.jayway.jsonpath.JsonPath;
>  import com.jayway.jsonpath.Option;
>  import com.jayway.jsonpath.PathNotFoundException;
>
> -import net.minidev.json.JSONArray;
> -import net.minidev.json.JSONObject;
> -
>  /**
>   * Handles the extractions
>   * https://github.com/jayway/JsonPath/blob/master/json-
> path/src/test/java/com/jayway/jsonpath/ComplianceTest.java
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/reporters/
> MailerModel.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/reporters/MailerModel.java?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java
> (original)
> +++ jmeter/trunk/src/components/org/apache/jmeter/reporters/MailerModel.java
> Wed Dec  6 22:21:19 2017
> @@ -144,7 +144,7 @@ public class MailerModel extends Abstrac
>      /**
>       * Gets a List of String-objects. Each String is one mail-address of
> the
>       * addresses-String set by <code>setToAddress(str)</code>. The
> addresses
> -     * must be seperated by commas. Only String-objects containing a "@"
> are
> +     * must be separated by commas. Only String-objects containing a "@"
> are
>       * added to the returned List.
>       *
>       * @return a List of String-objects wherein each String represents a
> @@ -176,8 +176,7 @@ public class MailerModel extends Abstrac
>       * Adds a SampleResult for display in the Visualizer.
>       *
>       * @param sample
> -     *            the SampleResult encapsulating informations about the
> last
> -     *            sample.
> +     *            the SampleResult encapsulating information about the
> last sample.
>       */
>      public void add(SampleResult sample) {
>          add(sample, false);
> @@ -247,8 +246,6 @@ public class MailerModel extends Abstrac
>          notifyChangeListeners();
>      }
>
> -
> -
>      /**
>       * Resets the state of this object to its default. But: This method
> does not
>       * reset any mail-specific attributes (like sender, mail-subject...)
> since
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/timers/
> ConstantThroughputTimerBeanInfo.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/timers/ConstantThroughputTimerBeanInf
> o.java?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/components/org/apache/jmeter/timers/
> ConstantThroughputTimerBeanInfo.java (original)
> +++ jmeter/trunk/src/components/org/apache/jmeter/timers/
> ConstantThroughputTimerBeanInfo.java Wed Dec  6 22:21:19 2017
> @@ -19,11 +19,11 @@
>  package org.apache.jmeter.timers;
>
>  import java.beans.PropertyDescriptor;
> +
>  import org.apache.jmeter.testbeans.BeanInfoSupport;
>
>  /**
>   * BeanInfo for the ConstantThroughputTimer.
> - *
>   */
>  public class ConstantThroughputTimerBeanInfo extends BeanInfoSupport {
>
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> Graph.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/visualizers/Graph.java?rev=1817321&r1=
> 1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java
> (original)
> +++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java
> Wed Dec  6 22:21:19 2017
> @@ -220,7 +220,7 @@ public class Graph extends JComponent im
>              }
>              g.drawLine(adjustedWidth, height - data, adjustedWidth,
> height - data - 1);
>              if (log.isDebugEnabled()) {
> -                log.debug("Drawing coords = {}, {}", adjustedWidth,
> (height - data));
> +                log.debug("Drawing coords = {}, {}", adjustedWidth,
> height - data);
>              }
>          }
>
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> RenderAsXML.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/visualizers/RenderAsXML.java?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsXML.java
> (original)
> +++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsXML.java
> Wed Dec  6 22:21:19 2017
> @@ -150,7 +150,7 @@ public class RenderAsXML extends Sampler
>          }
>
>          private void applyToChildren(TreePath parent, Consumer<TreePath>
> method) {
> -            TreeNode node = ((TreeNode) parent.getLastPathComponent());
> +            TreeNode node = (TreeNode) parent.getLastPathComponent();
>              Enumeration<?> e = node.children();
>              while (e.hasMoreElements()) {
>                  TreeNode n = (TreeNode) e.nextElement();
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> RenderInBrowser.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/visualizers/RenderInBrowser.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderInBrowser.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderInBrowser.java
> Wed Dec  6 22:21:19 2017
> @@ -22,15 +22,6 @@ package org.apache.jmeter.visualizers;
>  import java.awt.BorderLayout;
>  import java.awt.Dimension;
>
> -import javafx.application.Platform;
> -import javafx.beans.value.ChangeListener;
> -import javafx.beans.value.ObservableValue;
> -import javafx.concurrent.Worker.State;
> -import javafx.embed.swing.JFXPanel;
> -import javafx.scene.Scene;
> -import javafx.scene.web.WebEngine;
> -import javafx.scene.web.WebView;
> -
>  import javax.swing.BorderFactory;
>  import javax.swing.JLabel;
>  import javax.swing.JOptionPane;
> @@ -41,6 +32,15 @@ import javax.swing.SwingUtilities;
>  import org.apache.jmeter.samplers.SampleResult;
>  import org.apache.jmeter.util.JMeterUtils;
>
> +import javafx.application.Platform;
> +import javafx.beans.value.ChangeListener;
> +import javafx.beans.value.ObservableValue;
> +import javafx.concurrent.Worker.State;
> +import javafx.embed.swing.JFXPanel;
> +import javafx.scene.Scene;
> +import javafx.scene.web.WebEngine;
> +import javafx.scene.web.WebView;
> +
>  /**
>   * {@link ResultRenderer} implementation that uses JAVAFX WebEngine to
> render as browser do
>   * @since 3.2
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> ViewResultsFullVisualizer.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/visualizers/ViewResultsFullVisualizer.
> java?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/visualizers/ViewResultsFullVisualizer.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/visualizers/ViewResultsFullVisualizer.java
> Wed Dec  6 22:21:19 2017
> @@ -485,27 +485,25 @@ implements ActionListener, TreeSelection
>                  log.warn("Error loading result renderer: {}", clazz, e);
>              }
>          }
> -        if(VIEWERS_ORDER.length()>0) {
> -            String[] keys = VIEWERS_ORDER.split(",");
> -            for (String key : keys) {
> -                if(key.startsWith(".")) {
> -                    key = "org.apache.jmeter.visualizers"+key;
> //$NON-NLS-1$
> -                }
> -                ResultRenderer renderer = map.remove(key);
> -                if(renderer != null) {
> -                    selectRenderPanel.addItem(renderer);
> -                } else {
> -                    log.warn(
> -                            "Missing (check spelling error in renderer
> name) or already added(check doublon) "
> -                                    + "result renderer, check property
> 'view.results.tree.renderers_order', renderer name: '{}'",
> -                            key);
> -                }
> -            }
> +        if (VIEWERS_ORDER.length() > 0) {
> +            Arrays.stream(VIEWERS_ORDER.split(","))
> +                    .map(key -> key.startsWith(".")
> +                            ? "org.apache.jmeter.visualizers" + key
> //$NON-NLS-1$
> +                            : key)
> +                    .forEach(key -> {
> +                        ResultRenderer renderer = map.remove(key);
> +                        if (renderer != null) {
> +                            selectRenderPanel.addItem(renderer);
> +                        } else {
> +                            log.warn(
> +                                    "Missing (check renderer name) or
> already added (check doublon) result renderer," +
> +                                            " check property
> 'view.results.tree.renderers_order', renderer name: '{}'",
> +                                    key);
> +                        }
> +                    });
>          }
>          // Add remaining (plugins or missed in property)
> -        for (ResultRenderer renderer : map.values()) {
> -            selectRenderPanel.addItem(renderer);
> -        }
> +        map.values().forEach(renderer -> selectRenderPanel.addItem(
> renderer));
>          nodesModel.setSelectedItem(textObject); // preset to "Text"
> option
>          return selectRenderPanel;
>      }
>
> Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/
> backend/BackendListener.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/
> org/apache/jmeter/visualizers/backend/BackendListener.java?
> rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/BackendListener.java
> (original)
> +++ 
> jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/BackendListener.java
> Wed Dec  6 22:21:19 2017
> @@ -212,10 +212,13 @@ public class BackendListener extends Abs
>                          }
>                          SampleResult sampleResult =
> listenerClientData.queue.take();
>                          if (isDebugEnabled) {
> -                            log.debug("Thread: {} took SampleResult: {},
> isFinal: {}", Thread.currentThread().getName(),
> -                                    sampleResult, (sampleResult ==
> FINAL_SAMPLE_RESULT));
> +                            log.debug("Thread: {} took SampleResult: {},
> isFinal: {}",
> +                                    Thread.currentThread().getName(),
> +                                    sampleResult,
> +                                    sampleResult == FINAL_SAMPLE_RESULT);
>                          }
> -                        while (!(endOfLoop = (sampleResult ==
> FINAL_SAMPLE_RESULT)) && sampleResult != null ) { // try to process as many
> as possible
> +                        // try to process as many as possible
> +                        while (!(endOfLoop = 
> FINAL_SAMPLE_RESULT.equals(sampleResult))
> && sampleResult != null ) {
>                              sampleResults.add(sampleResult);
>                              if (isDebugEnabled) {
>                                  log.debug("Thread: {} polling from queue:
> {}", Thread.currentThread().getName(),
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/NewDriver.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/NewDriver.java?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/NewDriver.java (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/NewDriver.java Wed Dec  6
> 22:21:19 2017
> @@ -20,7 +20,6 @@ package org.apache.jmeter;
>
>  // N.B. this must only use standard Java packages
>  import java.io.File;
> -import java.io.FilenameFilter;
>  import java.io.IOException;
>  import java.io.PrintWriter;
>  import java.io.StringWriter;
> @@ -28,6 +27,7 @@ import java.lang.reflect.Method;
>  import java.net.MalformedURLException;
>  import java.net.URL;
>  import java.security.AccessController;
> +import java.security.PrivilegedAction;
>  import java.text.SimpleDateFormat;
>  import java.util.ArrayList;
>  import java.util.Arrays;
> @@ -63,7 +63,7 @@ public final class NewDriver {
>          final String initial_classpath = System.getProperty(JAVA_CLASS_
> PATH);
>
>          // Find JMeter home dir from the initial classpath
> -        String tmpDir=null;
> +        String tmpDir;
>          StringTokenizer tok = new StringTokenizer(initial_classpath,
> File.pathSeparator);
>          if (tok.countTokens() == 1
>                  || (tok.countTokens()  == 2 // Java on Mac OS can add a
> second entry to the initial classpath
> @@ -74,6 +74,7 @@ public final class NewDriver {
>              try {
>                  tmpDir = jar.getCanonicalFile().
> getParentFile().getParent();
>              } catch (IOException e) {
> +                tmpDir = null;
>              }
>          } else {// e.g. started from IDE with full classpath
>              tmpDir = System.getProperty("jmeter.home","");// Allow
> override $NON-NLS-1$ $NON-NLS-2$
> @@ -96,7 +97,7 @@ public final class NewDriver {
>                  new File(JMETER_INSTALLATION_DIRECTORY + File.separator
> + "lib" + File.separator + "ext"),// $NON-NLS-1$ $NON-NLS-2$
>                  new File(JMETER_INSTALLATION_DIRECTORY + File.separator
> + "lib" + File.separator + "junit")};// $NON-NLS-1$ $NON-NLS-2$
>          for (File libDir : libDirs) {
> -            File[] libJars = libDir.listFiles((FilenameFilter) (dir,
> name) -> name.endsWith(".jar"));
> +            File[] libJars = libDir.listFiles((dir, name) ->
> name.endsWith(".jar"));
>              if (libJars == null) {
>                  new Throwable("Could not access " +
> libDir).printStackTrace(); // NOSONAR No logging here
>                  continue;
> @@ -127,12 +128,8 @@ public final class NewDriver {
>          // ClassFinder needs the classpath
>          System.setProperty(JAVA_CLASS_PATH, initial_classpath +
> classpath.toString());
>          loader = AccessController.doPrivileged(
> -                new java.security.PrivilegedAction<DynamicClassLoader>()
> {
> -                    @Override
> -                    public DynamicClassLoader run() {
> -                        return new DynamicClassLoader(jars.toArray(new
> URL[jars.size()]));
> -                    }
> -                }
> +                (PrivilegedAction<DynamicClassLoader>) () ->
> +                        new DynamicClassLoader(jars.toArray(new
> URL[jars.size()]))
>          );
>      }
>
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/assertions/gui/
> AbstractAssertionGui.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/assertions/gui/AbstractAssertionGui.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/core/org/apache/jmeter/assertions/gui/AbstractAssertionGui.java
> (original)
> +++ 
> jmeter/trunk/src/core/org/apache/jmeter/assertions/gui/AbstractAssertionGui.java
> Wed Dec  6 22:21:19 2017
> @@ -21,7 +21,6 @@ package org.apache.jmeter.assertions.gui
>  import java.util.Arrays;
>  import java.util.Collection;
>
> -
>  import org.apache.jmeter.gui.AbstractScopedJMeterGuiComponent;
>  import org.apache.jmeter.gui.util.MenuFactory;
>
> @@ -32,7 +31,6 @@ import org.apache.jmeter.gui.util.MenuFa
>   * need to use the createScopePanel() to add the panel to the GUI, and
> they also
>   * need to use saveScopeSettings() and showScopeSettings() to keep the
> test element
>   * and GUI in synch.
> - *
>   */
>  public abstract class AbstractAssertionGui extends
> AbstractScopedJMeterGuiComponent {
>
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/config/gui/
> SimpleConfigGui.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/config/gui/SimpleConfigGui.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/config/gui/SimpleConfigGui.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/config/gui/SimpleConfigGui.java
> Wed Dec  6 22:21:19 2017
> @@ -28,6 +28,7 @@ import javax.swing.JButton;
>  import javax.swing.JPanel;
>  import javax.swing.JTable;
>  import javax.swing.ListSelectionModel;
> +
>  import org.apache.jmeter.config.ConfigTestElement;
>  import org.apache.jmeter.gui.util.HeaderAsPropertyRenderer;
>  import org.apache.jmeter.gui.util.PowerTableModel;
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/functions/gui/
> FunctionHelper.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/functions/gui/FunctionHelper.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/functions/gui/FunctionHelper.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/functions/gui/FunctionHelper.java
> Wed Dec  6 22:21:19 2017
> @@ -136,7 +136,7 @@ public class FunctionHelper extends JDia
>
>      private void initializeFunctionList() {
>          String[] functionNames = CompoundVariable.getFunctionNames();
> -        Arrays.sort(functionNames, (o1, o2) ->
> o1.compareToIgnoreCase(o2));
> +        Arrays.sort(functionNames, String::compareToIgnoreCase);
>          functionList = new JLabeledChoice(JMeterUtils.
> getResString("choose_function"), functionNames); //$NON-NLS-1$
>          functionList.addChangeListener(this);
>      }
> @@ -152,13 +152,14 @@ public class FunctionHelper extends JDia
>              this.validate();
>              resultTextArea.setText("");
>              this.repaint();
> -        } catch (InstantiationException | IllegalAccessException e) {
> +        } catch (InstantiationException | IllegalAccessException ex) {
> +            log.info("Exception during stateChanged", ex);
>          }
>      }
>
>      /**
> -     * @throws InstantiationException if function instanciation fails
> -     * @throws IllegalAccessException if function instanciation fails
> +     * @throws InstantiationException if function instantiation fails
> +     * @throws IllegalAccessException if function instantiation fails
>       */
>      protected void initParameterPanel() throws InstantiationException,
> IllegalAccessException {
>          Arguments args = new Arguments();
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/
> GlobalUndoableEdit.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/GlobalUndoableEdit.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/GlobalUndoableEdit.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/GlobalUndoableEdit.java
> Wed Dec  6 22:21:19 2017
> @@ -19,6 +19,7 @@
>  package org.apache.jmeter.gui;
>
>  import java.util.function.Consumer;
> +
>  import javax.swing.undo.AbstractUndoableEdit;
>  import javax.swing.undo.CannotRedoException;
>  import javax.swing.undo.CannotUndoException;
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/UndoHistoryItem.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/UndoHistoryItem.java?rev=1817321&r1=1817320&r2=1817321&
> view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/UndoHistoryItem.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/UndoHistoryItem.java Wed
> Dec  6 22:21:19 2017
> @@ -18,10 +18,10 @@
>
>  package org.apache.jmeter.gui;
>
> -import org.apache.jorphan.collections.HashTree;
> -
>  import java.io.Serializable;
> +
>  import org.apache.jmeter.engine.TreeCloner;
> +import org.apache.jorphan.collections.HashTree;
>
>  /**
>   * Undo history item
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/action/
> RemoteStart.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/action/RemoteStart.java?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/action/RemoteStart.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/action/RemoteStart.java
> Wed Dec  6 22:21:19 2017
> @@ -26,6 +26,7 @@ import java.util.LinkedList;
>  import java.util.List;
>  import java.util.Set;
>  import java.util.StringTokenizer;
> +
>  import org.apache.jmeter.JMeter;
>  import org.apache.jmeter.engine.DistributedRunner;
>  import org.apache.jmeter.gui.GuiPackage;
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/util/
> HorizontalPanel.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/util/HorizontalPanel.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/util/HorizontalPanel.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/util/HorizontalPanel.java
> Wed Dec  6 22:21:19 2017
> @@ -22,8 +22,8 @@
>   */
>  package org.apache.jmeter.gui.util;
>
> -import java.awt.Color;
>  import java.awt.BorderLayout;
> +import java.awt.Color;
>  import java.awt.Component;
>
>  import javax.swing.Box;
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/util/
> MenuFactory.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/util/MenuFactory.java?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/util/MenuFactory.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/util/MenuFactory.java Wed
> Dec  6 22:21:19 2017
> @@ -22,6 +22,7 @@ import java.awt.Component;
>  import java.awt.HeadlessException;
>  import java.io.IOException;
>  import java.io.Serializable;
> +import java.util.Arrays;
>  import java.util.Collection;
>  import java.util.Collections;
>  import java.util.Comparator;
> @@ -55,6 +56,7 @@ import org.apache.jmeter.testbeans.gui.T
>  import org.apache.jmeter.testelement.NonTestElement;
>  import org.apache.jmeter.testelement.TestElement;
>  import org.apache.jmeter.testelement.TestPlan;
> +import org.apache.jmeter.threads.AbstractThreadGroup;
>  import org.apache.jmeter.util.JMeterUtils;
>  import org.apache.jmeter.visualizers.Printable;
>  import org.apache.jorphan.gui.GuiUtils;
> @@ -614,51 +616,38 @@ public final class MenuFactory {
>       * @return whether it is OK to add the dragged nodes to this parent
>       */
>      public static boolean canAddTo(JMeterTreeNode parentNode,
> JMeterTreeNode[] nodes) {
> -        if (null == parentNode) {
> -            return false;
> -        }
> -        if (foundClass(nodes, new Class[]{TestPlan.class})){// Can't add
> a TestPlan anywhere
> +        if (parentNode == null
> +                || foundClass(nodes, new Class[]{TestPlan.class})) {
>              return false;
>          }
>          TestElement parent = parentNode.getTestElement();
>
>          // Force TestFragment to only be pastable under a Test Plan
> -        if (foundClass(nodes, new Class[]{org.apache.jmeter.control.
> TestFragmentController.class})){
> -            if (parent instanceof TestPlan) {
> -                return true;
> -            }
> -            return false;
> +        if (foundClass(nodes, new Class[]{TestFragmentController.class}))
> {
> +            return parent instanceof TestPlan;
>          }
>
>          // Cannot move Non-Test Elements from root of Test Plan or Test
> Fragment
> -        if (!(parent instanceof TestPlan || parent instanceof
> TestFragmentController)
> -                && foundMenuCategories(nodes, NON_TEST_ELEMENTS)) {
> +        if (foundMenuCategories(nodes, NON_TEST_ELEMENTS)
> +                && !(parent instanceof TestPlan || parent instanceof
> TestFragmentController)) {
>              return false;
>          }
>
>          if (parent instanceof TestPlan) {
> -            if (foundClass(nodes,
> -                     new Class[]{Sampler.class, Controller.class}, //
> Samplers and Controllers need not apply ...
> -                     new Class[]{org.apache.jmeter.
> threads.AbstractThreadGroup.class,
> -                             NonTestElement.class
> -                     })  // but AbstractThreadGroup (Controller) and Non
> Test Elements are OK
> -                ){
> -                return false;
> -            }
> -            return true;
> +            return !foundClass(
> +                    nodes,
> +                    new Class[]{Sampler.class, Controller.class}, //
> Samplers and Controllers need not apply ...
> +                    new Class[]{AbstractThreadGroup.class,
> NonTestElement.class});
>          }
>          // AbstractThreadGroup is only allowed under a TestPlan
> -        if (foundClass(nodes, new Class[]{org.apache.jmeter.
> threads.AbstractThreadGroup.class})){
> +        if (foundClass(nodes, new Class[]{AbstractThreadGroup.class})) {
>              return false;
>          }
>          if (parent instanceof Controller) {// Includes thread group;
> anything goes
>              return true;
>          }
>          if (parent instanceof Sampler) {// Samplers and Controllers need
> not apply ...
> -            if (foundClass(nodes, new Class[]{Sampler.class,
> Controller.class})){
> -                return false;
> -            }
> -            return true;
> +            return !foundClass(nodes, new Class[]{Sampler.class,
> Controller.class});
>          }
>
>          // All other
> @@ -689,14 +678,9 @@ public final class MenuFactory {
>       * @return true if nodes is in category
>       */
>      private static boolean foundMenuCategories(JMeterTreeNode[] nodes,
> String category) {
> -        for (JMeterTreeNode node : nodes) {
> -            for (String c : node.getMenuCategories()) {
> -                if (category.equals(c)) {
> -                    return true;
> -                }
> -            }
> -        }
> -        return false;
> +        return Arrays.stream(nodes)
> +                .flatMap(node -> node.getMenuCategories().stream())
> +                .anyMatch(category::equals);
>      }
>
>      /**
> @@ -787,8 +771,8 @@ public final class MenuFactory {
>       * [This is so Thread Group appears before setUp and tearDown]
>       */
>      private static void sortPluginMenus() {
> -        for(Entry<String, List<MenuInfo>> me : menuMap.entrySet()){
> -            Collections.sort(me.getValue(), new
> MenuInfoComparator(!me.getKey().equals(THREADS)));
> +        for (Entry<String, List<MenuInfo>> me : menuMap.entrySet()) {
> +            me.getValue().sort(new MenuInfoComparator(!me.getKey(
> ).equals(THREADS)));
>          }
>      }
>  }
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/util/
> PowerTableModel.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/util/PowerTableModel.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/util/PowerTableModel.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/util/PowerTableModel.java
> Wed Dec  6 22:21:19 2017
> @@ -21,6 +21,7 @@ package org.apache.jmeter.gui.util;
>  import java.lang.reflect.Constructor;
>  import java.lang.reflect.InvocationTargetException;
>  import java.util.ArrayList;
> +import java.util.Arrays;
>  import java.util.List;
>
>  import javax.swing.event.TableModelEvent;
> @@ -149,13 +150,11 @@ public class PowerTableModel extends Def
>
>      public Object[] getRowData(int row) {
>          Object[] rowData = new Object[getColumnCount()];
> -        for (int i = 0; i < rowData.length; i++) {
> -            rowData[i] = model.getColumnValue(i, row);
> -        }
> +        Arrays.setAll(rowData, i -> model.getColumnValue(i, row));
>          return rowData;
>      }
>
> -    private Object createDefaultValue(int i) {
> +    private Object createDefaultValue(int i) { // CHECKSTYLE IGNORE
> ReturnCount
>          Class<?> colClass = getColumnClass(i);
>          try {
>              return colClass.newInstance();
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/util/
> VerticalPanel.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/util/VerticalPanel.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/util/VerticalPanel.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/util/VerticalPanel.java
> Wed Dec  6 22:21:19 2017
> @@ -18,8 +18,8 @@
>
>  package org.apache.jmeter.gui.util;
>
> -import java.awt.Color;
>  import java.awt.BorderLayout;
> +import java.awt.Color;
>  import java.awt.Component;
>
>  import javax.swing.Box;
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/util/theme/
> darcula_theme.xml
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/gui/util/theme/darcula_theme.xml?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/util/theme/darcula_theme.xml
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/util/theme/darcula_theme.xml
> Wed Dec  6 22:21:19 2017
> @@ -17,77 +17,77 @@
>  -->
>  <!DOCTYPE RSyntaxTheme SYSTEM "theme.dtd">
>  <!--
> -    Theme that mimics IntelliJ IDEA's defaults.
> -    See theme.dtd and org.fife.ui.rsyntaxtextarea.Theme for more
> information.
> +   Theme that mimics IntelliJ IDEA's defaults.
> +   See theme.dtd and org.fife.ui.rsyntaxtextarea.Theme for more
> information.
>  -->
>  <RSyntaxTheme version="1.0">
>
> -   <!-- Omitting baseFont will use a system-appropriate monospaced. -->
> -   <!--<baseFont family="..." size="13"/>-->
> -
> -   <!--  General editor colors. -->
> -   <background color="2B2B2B" />
> -   <caret color="bbbbbb" />
> -   <selection fg="D0D0D0" bg="214283" />
> -   <currentLineHighlight color="323232" fade="false" />
> -   <marginLine fg="b0b4b9" />
> -   <markAllHighlight color="ccccff" />
> -   <markOccurrencesHighlight color="ccccff" border="false" />
> -   <matchedBracket fg="D8D8D8" bg="D8D8D8" highlightBoth="true"
> animate="false" />
> -   <hyperlinks fg="6897BB" />
> -   <secondaryLanguages>
> -      <language index="1" bg="fff0cc" />
> -      <language index="2" bg="dafeda" />
> -      <language index="3" bg="ffe0f0" />
> -   </secondaryLanguages>
> -
> -   <!-- Gutter styling. -->
> -   <gutterBorder color="dddddd" />
> -   <lineNumbers fg="D0D0D0" />
> -   <foldIndicator fg="808080" iconBg="ffffff" />
> -   <iconRowHeader activeLineRange="3399ff" />
> -
> -   <!-- Syntax tokens. -->
> -   <tokenStyles>
> -      <style token="IDENTIFIER" fg="BED6FF" />
> -      <style token="RESERVED_WORD" fg="CC7832" bold="true" />
> -      <style token="RESERVED_WORD_2" fg="A9B7C6" bold="true" />
> -      <style token="ANNOTATION" fg="808000" />
> -      <style token="COMMENT_DOCUMENTATION" fg="84B977" italic="true" />
> -      <style token="COMMENT_EOL" fg="808080" italic="true" />
> -      <style token="COMMENT_MULTILINE" fg="629755" italic="true" />
> -      <style token="COMMENT_KEYWORD" fg="84B977" bold="true"
> underline="true" italic="true" />
> -      <style token="COMMENT_MARKUP" fg="84B977" bg="e2ffe2" italic="true"
> />
> -      <style token="DATA_TYPE" fg="BFA4A4" bold="true" italic="true"/>
> -      <style token="FUNCTION" fg="FFC66D" />
> -      <style token="LITERAL_BOOLEAN" fg="CC7832" bold="true" />
> -      <style token="LITERAL_NUMBER_DECIMAL_INT" fg="6897BB" />
> -      <style token="LITERAL_NUMBER_FLOAT" fg="6897BB" />
> -      <style token="LITERAL_NUMBER_HEXADECIMAL" fg="6897BB" />
> -      <style token="LITERAL_STRING_DOUBLE_QUOTE" fg="008000"
> bold="false" />
> -      <style token="LITERAL_CHAR" fg="008000" bold="false" />
> -      <style token="LITERAL_BACKQUOTE" fg="008000" bold="true" />
> -      <style token="MARKUP_TAG_DELIMITER" fg="D8D8D8" bold="true" />
> -      <style token="MARKUP_TAG_NAME" fg="CC7832" bold="true" />
> -      <style token="MARKUP_TAG_ATTRIBUTE" fg="6897BB" bold="true" />
> -      <style token="MARKUP_TAG_ATTRIBUTE_VALUE" fg="008000" bold="true"
> />
> -      <style token="MARKUP_COMMENT" fg="808080" italic="true"/>
> -      <style token="MARKUP_DTD" fg="808080"/>
> -      <style token="MARKUP_PROCESSING_INSTRUCTION"  fg="808080"/>
> -      <style token="MARKUP_CDATA" fg="cc6600"/>
> -      <style token="MARKUP_CDATA_DELIMITER" fg="008080"/>
> -      <style token="MARKUP_ENTITY_REFERENCE" fg="008000"/>
> -      <style token="OPERATOR" fg="D8D8D8" />
> -      <style token="PREPROCESSOR" fg="808080" />
> -      <style token="REGEX" fg="008040" />
> -      <style token="SEPARATOR" fg="D8D8D8" />
> -      <style token="VARIABLE" fg="810ca8" bold="true" />
> -      <style token="WHITESPACE" fg="000000" />
> -
> -      <style token="ERROR_IDENTIFIER" fg="ff0000" />
> -      <style token="ERROR_NUMBER_FORMAT" fg="ff0000" />
> -      <style token="ERROR_STRING_DOUBLE" fg="ff0000" />
> -      <style token="ERROR_CHAR" fg="ff0000" />
> -   </tokenStyles>
> +  <!-- Omitting baseFont will use a system-appropriate monospaced. -->
> +  <!--<baseFont family="..." size="13"/>-->
>
> -</RSyntaxTheme>
> \ No newline at end of file
> +  <!--  General editor colors. -->
> +  <background color="2B2B2B"/>
> +  <caret color="bbbbbb"/>
> +  <selection fg="D0D0D0" bg="214283"/>
> +  <currentLineHighlight color="323232" fade="false"/>
> +  <marginLine fg="b0b4b9"/>
> +  <markAllHighlight color="ccccff"/>
> +  <markOccurrencesHighlight color="ccccff" border="false"/>
> +  <matchedBracket fg="D8D8D8" bg="D8D8D8" highlightBoth="true"
> animate="false"/>
> +  <hyperlinks fg="6897BB"/>
> +  <secondaryLanguages>
> +    <language index="1" bg="fff0cc"/>
> +    <language index="2" bg="dafeda"/>
> +    <language index="3" bg="ffe0f0"/>
> +  </secondaryLanguages>
> +
> +  <!-- Gutter styling. -->
> +  <gutterBorder color="dddddd"/>
> +  <lineNumbers fg="D0D0D0"/>
> +  <foldIndicator fg="808080" iconBg="ffffff"/>
> +  <iconRowHeader activeLineRange="3399ff"/>
> +
> +  <!-- Syntax tokens. -->
> +  <tokenStyles>
> +    <style token="IDENTIFIER" fg="BED6FF"/>
> +    <style token="RESERVED_WORD" fg="CC7832" bold="true"/>
> +    <style token="RESERVED_WORD_2" fg="A9B7C6" bold="true"/>
> +    <style token="ANNOTATION" fg="808000"/>
> +    <style token="COMMENT_DOCUMENTATION" fg="84B977" italic="true"/>
> +    <style token="COMMENT_EOL" fg="808080" italic="true"/>
> +    <style token="COMMENT_MULTILINE" fg="629755" italic="true"/>
> +    <style token="COMMENT_KEYWORD" fg="84B977" bold="true"
> underline="true" italic="true"/>
> +    <style token="COMMENT_MARKUP" fg="84B977" bg="e2ffe2" italic="true"/>
> +    <style token="DATA_TYPE" fg="BFA4A4" bold="true" italic="true"/>
> +    <style token="FUNCTION" fg="FFC66D"/>
> +    <style token="LITERAL_BOOLEAN" fg="CC7832" bold="true"/>
> +    <style token="LITERAL_NUMBER_DECIMAL_INT" fg="6897BB"/>
> +    <style token="LITERAL_NUMBER_FLOAT" fg="6897BB"/>
> +    <style token="LITERAL_NUMBER_HEXADECIMAL" fg="6897BB"/>
> +    <style token="LITERAL_STRING_DOUBLE_QUOTE" fg="008000" bold="false"/>
> +    <style token="LITERAL_CHAR" fg="008000" bold="false"/>
> +    <style token="LITERAL_BACKQUOTE" fg="008000" bold="true"/>
> +    <style token="MARKUP_TAG_DELIMITER" fg="D8D8D8" bold="true"/>
> +    <style token="MARKUP_TAG_NAME" fg="CC7832" bold="true"/>
> +    <style token="MARKUP_TAG_ATTRIBUTE" fg="6897BB" bold="true"/>
> +    <style token="MARKUP_TAG_ATTRIBUTE_VALUE" fg="008000" bold="true"/>
> +    <style token="MARKUP_COMMENT" fg="808080" italic="true"/>
> +    <style token="MARKUP_DTD" fg="808080"/>
> +    <style token="MARKUP_PROCESSING_INSTRUCTION" fg="808080"/>
> +    <style token="MARKUP_CDATA" fg="cc6600"/>
> +    <style token="MARKUP_CDATA_DELIMITER" fg="008080"/>
> +    <style token="MARKUP_ENTITY_REFERENCE" fg="008000"/>
> +    <style token="OPERATOR" fg="D8D8D8"/>
> +    <style token="PREPROCESSOR" fg="808080"/>
> +    <style token="REGEX" fg="008040"/>
> +    <style token="SEPARATOR" fg="D8D8D8"/>
> +    <style token="VARIABLE" fg="810ca8" bold="true"/>
> +    <style token="WHITESPACE" fg="000000"/>
> +
> +    <style token="ERROR_IDENTIFIER" fg="ff0000"/>
> +    <style token="ERROR_NUMBER_FORMAT" fg="ff0000"/>
> +    <style token="ERROR_STRING_DOUBLE" fg="ff0000"/>
> +    <style token="ERROR_CHAR" fg="ff0000"/>
> +  </tokenStyles>
> +
> +</RSyntaxTheme>
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/report/core/
> SampleBuilder.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/core/SampleBuilder.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleBuilder.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleBuilder.java
> Wed Dec  6 22:21:19 2017
> @@ -23,9 +23,6 @@ import java.text.DecimalFormatSymbols;
>  import java.text.NumberFormat;
>  import java.util.Locale;
>
> -import org.apache.jmeter.report.core.Sample;
> -import org.apache.jmeter.report.core.SampleMetadata;
> -
>  /**
>   * This class is meant to simplify the building of samples.
>   * <p>
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/report/core/
> SampleMetaDataParser.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/core/SampleMetaDataParser.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleMetaDataParser.java
> (original)
> +++ 
> jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleMetaDataParser.java
> Wed Dec  6 22:21:19 2017
> @@ -17,8 +17,6 @@
>   */
>  package org.apache.jmeter.report.core;
>
> -import org.apache.jmeter.report.core.SampleMetadata;
> -
>  /**
>   * Simple parser to get a {@link SampleMetadata} instance<br>
>   *
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/report/core/
> SampleWriter.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/core/SampleWriter.java?rev=1817321&
> r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleWriter.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/report/core/SampleWriter.java
> Wed Dec  6 22:21:19 2017
> @@ -19,8 +19,6 @@ package org.apache.jmeter.report.core;
>
>  import java.io.Closeable;
>
> -import org.apache.jmeter.report.core.Sample;
> -
>  /**
>   * Describes sample writer basic operations.<br>
>   * <br>
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/report/
> dashboard/TemplateVisitor.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/dashboard/TemplateVisitor.
> java?rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/TemplateVisitor.java
> (original)
> +++ 
> jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/TemplateVisitor.java
> Wed Dec  6 22:21:19 2017
> @@ -23,12 +23,12 @@ import java.io.IOException;
>  import java.io.OutputStreamWriter;
>  import java.io.Writer;
>  import java.nio.charset.StandardCharsets;
> -import java.nio.file.StandardCopyOption;
>  import java.nio.file.FileAlreadyExistsException;
>  import java.nio.file.FileVisitResult;
>  import java.nio.file.Files;
>  import java.nio.file.Path;
>  import java.nio.file.SimpleFileVisitor;
> +import java.nio.file.StandardCopyOption;
>  import java.nio.file.attribute.BasicFileAttributes;
>
>  import org.apache.commons.io.FileUtils;
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/report/
> processor/FilterConsumer.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/processor/FilterConsumer.java?
> rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/core/org/apache/jmeter/report/processor/FilterConsumer.java
> (original)
> +++ 
> jmeter/trunk/src/core/org/apache/jmeter/report/processor/FilterConsumer.java
> Wed Dec  6 22:21:19 2017
> @@ -19,7 +19,6 @@ package org.apache.jmeter.report.process
>
>  import org.apache.jmeter.report.core.Sample;
>  import org.apache.jmeter.report.core.SamplePredicate;
> -import org.apache.jmeter.report.processor.AbstractSampleConsumer;
>
>  /**
>   * <p>
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/report/
> processor/TaggerConsumer.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/processor/TaggerConsumer.java?
> rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/core/org/apache/jmeter/report/processor/TaggerConsumer.java
> (original)
> +++ 
> jmeter/trunk/src/core/org/apache/jmeter/report/processor/TaggerConsumer.java
> Wed Dec  6 22:21:19 2017
> @@ -22,7 +22,6 @@ import java.util.ArrayList;
>  import org.apache.jmeter.report.core.Sample;
>  import org.apache.jmeter.report.core.SampleBuilder;
>  import org.apache.jmeter.report.core.SampleMetadata;
> -import org.apache.jmeter.report.processor.AbstractSampleConsumer;
>
>  /**
>   * The Class TimeCountConsumer adds a data field to the samples it
> consumes.
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/reporters/
> ResultCollectorHelper.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/reporters/ResultCollectorHelper.java?
> rev=1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- 
> jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollectorHelper.java
> (original)
> +++ 
> jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollectorHelper.java
> Wed Dec  6 22:21:19 2017
> @@ -18,7 +18,6 @@
>
>  package org.apache.jmeter.reporters;
>
> -import org.apache.jmeter.reporters.ResultCollector;
>  import org.apache.jmeter.samplers.SampleResult;
>  import org.apache.jmeter.visualizers.Visualizer;
>
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/reporters/gui/
> ResultActionGui.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/reporters/gui/ResultActionGui.java?rev=
> 1817321&r1=1817320&r2=1817321&view=diff
> ============================================================
> ==================
> --- jmeter/trunk/src/core/org/apache/jmeter/reporters/gui/ResultActionGui.java
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/reporters/gui/ResultActionGui.java
> Wed Dec  6 22:21:19 2017
> @@ -22,15 +22,14 @@ import java.awt.BorderLayout;
>
>  import javax.swing.Box;
>
> -import org.apache.jmeter.reporters.ResultAction;
>  import org.apache.jmeter.gui.OnErrorPanel;
>  import org.apache.jmeter.processor.gui.AbstractPostProcessorGui;
> +import org.apache.jmeter.reporters.ResultAction;
>  import org.apache.jmeter.testelement.OnErrorTestElement;
>  import org.apache.jmeter.testelement.TestElement;
>
>  /**
>   * Create a Result Action Test Element
> - *
>   */
>  public class ResultActionGui extends AbstractPostProcessorGui {
>
>
>
>


-- 
Cordialement.
Philippe Mouawad.

Reply via email to