You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by dj...@apache.org on 2012/06/05 03:02:12 UTC
svn commit: r1346204 -
/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java
Author: djencks
Date: Tue Jun 5 01:02:12 2012
New Revision: 1346204
URL: http://svn.apache.org/viewvc?rev=1346204&view=rev
Log:
FELIX-3535 finish enabling ServiceMetadata validation
Modified:
felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java
Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java?rev=1346204&r1=1346203&r2=1346204&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java Tue Jun 5 01:02:12 2012
@@ -79,14 +79,7 @@ public class ServiceMetadata {
* @return the implemented interfaces as a string array
*/
public String [] getProvides() {
- String provides[] = new String[m_provides.size()];
- Iterator it = m_provides.iterator();
- int count = 0;
- while (it.hasNext())
- {
- provides[count++] = it.next().toString();
- }
- return provides;
+ return (String[]) m_provides.toArray( new String[m_provides.size()] );
}
/**
@@ -100,5 +93,6 @@ public class ServiceMetadata {
throw componentMetadata
.validationFailure( "At least one provided interface must be declared in the service element" );
}
+ m_validated = true;
}
}