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
