FELIX-4076 - Useless locking on getRequiredHandler
Remove useless synchronized
git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@1486736 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java b/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java
index 2606230..3214127 100644
--- a/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java
+++ b/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java
@@ -366,7 +366,7 @@
}
/**
- * Computes the list of missing handlers. This method is called with the monitor lock.
+ * Computes the list of missing handlers.
* @return the list of missing handlers.
* @see org.apache.felix.ipojo.Factory#getMissingHandlers()
*/
@@ -392,12 +392,11 @@
/**
* Gets the list of required handlers.
- * This method is synchronized to avoid the concurrent modification
- * of the required handlers.
+ * The required handler list cannot change.
* @return the list of required handlers.
* @see org.apache.felix.ipojo.Factory#getRequiredHandlers()
*/
- public synchronized List<String> getRequiredHandlers() {
+ public List<String> getRequiredHandlers() {
List<String> list = new ArrayList<String>();
for (RequiredHandler req : m_requiredHandlers) {
list.add(req.getFullName());