jmeter-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fschumac...@apache.org
Subject svn commit: r1721339 - /jmeter/trunk/src/core/org/apache/jmeter/report/config/ConfigurationUtils.java
Date Tue, 22 Dec 2015 10:50:56 GMT
Author: fschumacher
Date: Tue Dec 22 10:50:56 2015
New Revision: 1721339

URL: http://svn.apache.org/viewvc?rev=1721339&view=rev
Log:
Remove check for null and document the need for a non null argument in the javadoc. The code
will now throw a NPE instead of ArgumentNullException.

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/report/config/ConfigurationUtils.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/report/config/ConfigurationUtils.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/config/ConfigurationUtils.java?rev=1721339&r1=1721338&r2=1721339&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/config/ConfigurationUtils.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/config/ConfigurationUtils.java Tue Dec
22 10:50:56 2015
@@ -43,17 +43,13 @@ public class ConfigurationUtils {
      * @param value
      *            the string value
      * @param clazz
-     *            the class of the property
+     *            the class of the property (must not be {@code null})
      * @return the converted string
      * @throws ConfigurationException
      *             when unable to convert the string
      */
     public static <TProperty> TProperty convert(String value,
         Class<TProperty> clazz) throws ConfigurationException {
-        if (clazz == null) {
-            throw new ArgumentNullException("clazz");
-        }
-
         TProperty result;
         if (clazz.isAssignableFrom(String.class)) {
             @SuppressWarnings("unchecked") // OK because checked above



Mime
View raw message