You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-commits@maven.apache.org by ol...@apache.org on 2011/01/13 23:53:04 UTC
svn commit: r1058776 - in
/maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src:
main/java/org/apache/maven/scm/provider/synergy/util/
test/java/org/apache/maven/scm/provider/synergy/consumer/
test/java/org/apache/maven/scm/provider/syner...
Author: olamy
Date: Thu Jan 13 22:53:04 2011
New Revision: 1058776
URL: http://svn.apache.org/viewvc?rev=1058776&view=rev
Log:
[SCM-591] Move to Java 1.5 : fix some generics warning in synergy provider
Modified:
maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/main/java/org/apache/maven/scm/provider/synergy/util/SynergyCCM.java
maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetCompletedTasksConsumerTest.java
maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetTaskObjectsConsumerTest.java
maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/util/SynergyCCMTest.java
Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/main/java/org/apache/maven/scm/provider/synergy/util/SynergyCCM.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/main/java/org/apache/maven/scm/provider/synergy/util/SynergyCCM.java?rev=1058776&r1=1058775&r2=1058776&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/main/java/org/apache/maven/scm/provider/synergy/util/SynergyCCM.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/main/java/org/apache/maven/scm/provider/synergy/util/SynergyCCM.java Thu Jan 13 22:53:04 2011
@@ -192,7 +192,7 @@ public class SynergyCCM
* @return the command line.
* @throws ScmException
*/
- public static Commandline create( List files, String message, String ccmAddr )
+ public static Commandline create( List<File> files, String message, String ccmAddr )
throws ScmException
{
Commandline cl = new Commandline();
@@ -211,9 +211,9 @@ public class SynergyCCM
}
- for ( Iterator i = files.iterator(); i.hasNext(); )
+ for ( Iterator<File> i = files.iterator(); i.hasNext(); )
{
- File f = (File) i.next();
+ File f = i.next();
try
{
cl.createArg().setValue( f.getCanonicalPath() );
@@ -311,7 +311,7 @@ public class SynergyCCM
* @return
* @throws ScmException
*/
- public static Commandline delete( List files, String ccmAddr, boolean replace )
+ public static Commandline delete( List<File> files, String ccmAddr, boolean replace )
throws ScmException
{
Commandline cl = new Commandline();
@@ -326,9 +326,9 @@ public class SynergyCCM
cl.createArg().setValue( "-replace" );
}
- for ( Iterator i = files.iterator(); i.hasNext(); )
+ for ( Iterator<File> i = files.iterator(); i.hasNext(); )
{
- File f = (File) i.next();
+ File f = i.next();
try
{
cl.createArg().setValue( f.getCanonicalPath() );
@@ -508,7 +508,7 @@ public class SynergyCCM
* @return the command line.
* @throws ScmException
*/
- public static Commandline checkoutFiles( List files, String ccmAddr )
+ public static Commandline checkoutFiles( List<File> files, String ccmAddr )
throws ScmException
{
Commandline cl = new Commandline();
@@ -518,9 +518,9 @@ public class SynergyCCM
cl.setExecutable( CCM );
cl.createArg().setValue( CO );
- for ( Iterator i = files.iterator(); i.hasNext(); )
+ for ( Iterator<File> i = files.iterator(); i.hasNext(); )
{
- File f = (File) i.next();
+ File f = i.next();
try
{
cl.createArg().setValue( f.getCanonicalPath() );
@@ -649,9 +649,9 @@ public class SynergyCCM
if ( files.size() > 0 )
{
- for ( Iterator i = files.iterator(); i.hasNext(); )
+ for ( Iterator<File> i = files.iterator(); i.hasNext(); )
{
- File f = (File) i.next();
+ File f = i.next();
try
{
cl.createArg().setValue( f.getCanonicalPath() );
@@ -747,7 +747,8 @@ public class SynergyCCM
{
Properties envVars = CommandLineUtils.getSystemEnvVars();
- for ( Iterator i = envVars.keySet().iterator(); i.hasNext(); )
+ for ( @SuppressWarnings( "rawtypes" )
+ Iterator i = envVars.keySet().iterator(); i.hasNext(); )
{
String key = (String) i.next();
Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetCompletedTasksConsumerTest.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetCompletedTasksConsumerTest.java?rev=1058776&r1=1058775&r2=1058776&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetCompletedTasksConsumerTest.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetCompletedTasksConsumerTest.java Thu Jan 13 22:53:04 2011
@@ -55,7 +55,7 @@ public class SynergyGetCompletedTasksCon
s = in.readLine();
}
- Collection entries = consumer.getTasks();
+ Collection<SynergyTask> entries = consumer.getTasks();
assertEquals( "Wrong number of tasks returned", 1, entries.size() );
@@ -92,11 +92,11 @@ public class SynergyGetCompletedTasksCon
s = in.readLine();
}
- Collection entries = consumer.getTasks();
+ Collection<SynergyTask> entries = consumer.getTasks();
assertEquals( "Wrong number of tasks returned", 2, entries.size() );
- Iterator i = entries.iterator();
+ Iterator<SynergyTask> i = entries.iterator();
SynergyTask task = (SynergyTask) i.next();
assertEquals( 52, task.getNumber() );
assertEquals( "ccm_root", task.getUsername() );
Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetTaskObjectsConsumerTest.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetTaskObjectsConsumerTest.java?rev=1058776&r1=1058775&r2=1058776&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetTaskObjectsConsumerTest.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/consumer/SynergyGetTaskObjectsConsumerTest.java Thu Jan 13 22:53:04 2011
@@ -53,11 +53,11 @@ public class SynergyGetTaskObjectsConsum
s = in.readLine();
}
- Collection entries = consumer.getFiles();
+ Collection<ChangeFile> entries = consumer.getFiles();
assertEquals( "Wrong number of entries returned", 1, entries.size() );
- ChangeFile file = (ChangeFile) entries.iterator().next();
+ ChangeFile file = entries.iterator().next();
assertEquals( "MessageResources.properties", file.getName() );
assertEquals( "3", file.getRevision() );
}
Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/util/SynergyCCMTest.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/util/SynergyCCMTest.java?rev=1058776&r1=1058775&r2=1058776&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/util/SynergyCCMTest.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-synergy/src/test/java/org/apache/maven/scm/provider/synergy/util/SynergyCCMTest.java Thu Jan 13 22:53:04 2011
@@ -70,7 +70,7 @@ public class SynergyCCMTest
{
File f = File.createTempFile( "test", null );
f.deleteOnExit();
- List list = new LinkedList();
+ List<File> list = new LinkedList<File>();
list.add( f );
Commandline cl = SynergyCCM.create( list, "test creation", "CCM_ADDR" );
assertTrue( "CCM_ADDR is not set.", assertContains( cl.getEnvironmentVariables(), "CCM_ADDR=CCM_ADDR" ) );
@@ -151,7 +151,7 @@ public class SynergyCCMTest
{
File f = File.createTempFile( "test", null );
f.deleteOnExit();
- List list = new LinkedList();
+ List<File> list = new LinkedList<File>();
list.add( f );
Commandline cl = SynergyCCM.delete( list, "CCM_ADDR", true );
assertTrue( "CCM_ADDR is not set.", assertContains( cl.getEnvironmentVariables(), "CCM_ADDR=CCM_ADDR" ) );
@@ -246,7 +246,7 @@ public class SynergyCCMTest
{
File f = File.createTempFile( "test", null );
f.deleteOnExit();
- List list = new LinkedList();
+ List<File> list = new LinkedList<File>();
list.add( f );
Commandline cl = SynergyCCM.checkoutFiles( list, "CCM_ADDR" );
assertTrue( "CCM_ADDR is not set.", assertContains( cl.getEnvironmentVariables(), "CCM_ADDR=CCM_ADDR" ) );
@@ -324,7 +324,7 @@ public class SynergyCCMTest
{
File f = File.createTempFile( "test", null );
f.deleteOnExit();
- List list = new LinkedList();
+ List<File> list = new LinkedList<File>();
list.add( f );
Commandline cl = SynergyCCM.checkinFiles( list, "a comment", "CCM_ADDR" );
assertTrue( "CCM_ADDR is not set.", assertContains( cl.getEnvironmentVariables(), "CCM_ADDR=CCM_ADDR" ) );