ClassUtils: refactor setValue method

ctrueden · ctrueden · commit 6d8223414634 · 2014-07-06T19:41:06.000-05:00
diff --git a/src/main/java/org/scijava/util/ClassUtils.java b/src/main/java/org/scijava/util/ClassUtils.java
@@ -363,8 +363,8 @@ public static void setValue(final Field field, final Object instance,
 	{
 		try {
 			field.setAccessible(true);
-			field.set(instance, ConversionUtils
-				.convert(value, field.getGenericType()));
+			final Type fieldType = field.getGenericType();
+			field.set(instance, ConversionUtils.convert(value, fieldType));
 		}
 		catch (final IllegalAccessException e) {
 			throw new IllegalArgumentException("No access to field: " +

Original file line number	Diff line number	Diff line change
`@@ -363,8 +363,8 @@ public static void setValue(final Field field, final Object instance,`
`363`	`363`	`{`
`364`	`364`	`try {`
`365`	`365`	`field.setAccessible(true);`
`366`		`- field.set(instance, ConversionUtils`
`367`		`- .convert(value, field.getGenericType()));`
	`366`	`+ final Type fieldType = field.getGenericType();`
	`367`	`+ field.set(instance, ConversionUtils.convert(value, fieldType));`
`368`	`368`	`}`
`369`	`369`	`catch (final IllegalAccessException e) {`
`370`	`370`	`throw new IllegalArgumentException("No access to field: " +`