Fix compiler warnings

spring-batch-core/src/main/java/org/springframework/batch/core/domain/JobInstanceProperties.java

@@ -11,7 +11,6 @@ import java.util.Map;
 import java.util.Map.Entry;
 
 import org.apache.commons.lang.builder.HashCodeBuilder;
-import org.springframework.util.Assert;
 
 /**
  * Value object representing runtime parameters to a batch job.  Because

spring-batch-core/src/main/java/org/springframework/batch/core/runtime/SimpleJobIdentifier.java

@@ -38,7 +38,7 @@ public class SimpleJobIdentifier implements JobIdentifier {
 	 * Default constructor.  Since there it is required that the Identifier at least have a name,
 	 * the default constructor should not be called.
 	 */
-	private SimpleJobIdentifier() {
+	SimpleJobIdentifier() {
 		super();
 	}
 	

spring-batch-core/src/test/java/org/springframework/batch/core/domain/JobInstancePropertiesTests.java

@@ -54,7 +54,7 @@ public class JobInstancePropertiesTests extends TestCase {
 		badLongMap.put("key", "bad long");
 		
 		try{
-			JobInstanceProperties testParameters = new JobInstanceProperties(stringMap, badLongMap, dateMap);
+			new JobInstanceProperties(stringMap, badLongMap, dateMap);
 			fail();
 		}
 		catch(IllegalArgumentException ex){
@@ -68,7 +68,7 @@ public class JobInstancePropertiesTests extends TestCase {
 		badMap.put("key", new Integer(2));
 		
 		try{
-			JobInstanceProperties testParameters = new JobInstanceProperties(badMap, longMap, dateMap);
+			new JobInstanceProperties(badMap, longMap, dateMap);
 			fail();
 		}
 		catch(IllegalArgumentException ex){
@@ -82,7 +82,7 @@ public class JobInstancePropertiesTests extends TestCase {
 		badMap.put("key", new java.sql.Date(System.currentTimeMillis()));
 		
 		try{
-			JobInstanceProperties testParameters = new JobInstanceProperties(stringMap, longMap, badMap);
+			new JobInstanceProperties(stringMap, longMap, badMap);
 			fail();
 		}
 		catch(IllegalArgumentException ex){