You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by fm...@apache.org on 2009/07/09 15:02:13 UTC

svn commit: r792540 - in /felix/trunk/scr/src: main/java/org/apache/felix/scr/impl/ main/java/org/apache/felix/scr/impl/helper/ main/java/org/apache/felix/scr/impl/manager/ main/java/org/apache/felix/scr/impl/metadata/ test/java/org/apache/felix/scr/im...

Author: fmeschbe
Date: Thu Jul  9 13:02:12 2009
New Revision: 792540

URL: http://svn.apache.org/viewvc?rev=792540&view=rev
Log:
FELIX-1314 Split main "impl" package by moving classes to helper, manager,
and metadata subpackages. Adapt unit tests accordingly.

Added:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/Logger.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReadOnlyDictionary.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReflectionHelper.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/AbstractComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentContextImpl.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentFactoryImpl.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DelayedComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ImmediateComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceFactoryComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ComponentMetadata.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/PropertyMetadata.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/PropertyMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ReferenceMetadata.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReferenceMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/XmlHandler.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/XmlHandler.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java   (contents, props changed)
      - copied, changed from r792527, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ReflectionHelperTest.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/ComponentMetadataTest.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ComponentMetadataTest.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java   (contents, props changed)
      - copied, changed from r792521, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/XmlHandlerTest.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/   (props changed)
      - copied from r792521, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/instances/
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances2/   (props changed)
      - copied from r792521, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/instances2/
Removed:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/AbstractComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentContextImpl.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentFactoryImpl.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DelayedComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ImmediateComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/Logger.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/PropertyMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReadOnlyDictionary.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReferenceMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReflectionHelper.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceFactoryComponentManager.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceMetadata.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/XmlHandler.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ComponentMetadataTest.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ReflectionHelperTest.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/XmlHandlerTest.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/instances/
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/instances2/
Modified:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/BundleComponentActivator.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentActivatorTask.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentRegistry.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/MockLogger.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/AcceptMethod.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/BaseObject.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level1Object.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level3Object.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/MethodNameException.java
    felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances2/Level2Object.java

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/BundleComponentActivator.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/BundleComponentActivator.java?rev=792540&r1=792539&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/BundleComponentActivator.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/BundleComponentActivator.java Thu Jul  9 13:02:12 2009
@@ -29,6 +29,14 @@
 import java.util.List;
 import java.util.StringTokenizer;
 
+import org.apache.felix.scr.impl.helper.Logger;
+import org.apache.felix.scr.impl.manager.AbstractComponentManager;
+import org.apache.felix.scr.impl.manager.ComponentFactoryImpl;
+import org.apache.felix.scr.impl.manager.DelayedComponentManager;
+import org.apache.felix.scr.impl.manager.ImmediateComponentManager;
+import org.apache.felix.scr.impl.manager.ServiceFactoryComponentManager;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
+import org.apache.felix.scr.impl.metadata.XmlHandler;
 import org.apache.felix.scr.impl.parser.KXml2SAXParser;
 import org.osgi.framework.BundleContext;
 import org.osgi.service.cm.ConfigurationAdmin;
@@ -42,7 +50,7 @@
  * a single bundle. It will read information from the metadata.xml file
  * descriptors and create the corresponding managers.
  */
-class BundleComponentActivator implements Logger
+public class BundleComponentActivator implements Logger
 {
     // global component registration
     private ComponentRegistry m_componentRegistry;
@@ -315,7 +323,7 @@
      *
      * @return the list of instance references
      */
-    protected List getInstanceReferences()
+    public List getInstanceReferences()
     {
         return m_managers;
     }
@@ -326,7 +334,7 @@
     *
     * @return the BundleContext
     */
-    protected BundleContext getBundleContext()
+    public BundleContext getBundleContext()
     {
         return m_context;
     }
@@ -355,7 +363,7 @@
      * @param name The name of the component to enable or <code>null</code> to
      *      enable all components.
      */
-    void enableComponent( String name )
+    public void enableComponent( String name )
     {
         final AbstractComponentManager[] cm = getSelectedComponents( name );
         if ( cm == null )
@@ -388,7 +396,7 @@
      * @param name The name of the component to disable or <code>null</code> to
      *      disable all components.
      */
-    void disableComponent( String name )
+    public void disableComponent( String name )
     {
         final AbstractComponentManager[] cm = getSelectedComponents( name );
         if ( cm == null )
@@ -431,7 +439,7 @@
         // if all components are selected
         if ( name == null )
         {
-            return ( AbstractComponentManager[] ) m_managers.toArray( new AbstractComponentManager[m_managers.size()] );
+            return (org.apache.felix.scr.impl.manager.AbstractComponentManager[] ) m_managers.toArray( new AbstractComponentManager[m_managers.size()] );
         }
 
         if ( m_componentRegistry.getComponent( name ) != null )
@@ -463,7 +471,7 @@
      *
      * @param task The component task to execute
      */
-    void schedule( ComponentActivatorTask task )
+    public void schedule( ComponentActivatorTask task )
     {
         if ( isActive() )
         {

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentActivatorTask.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentActivatorTask.java?rev=792540&r1=792539&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentActivatorTask.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentActivatorTask.java Thu Jul  9 13:02:12 2009
@@ -20,6 +20,7 @@
 
 
 import org.apache.felix.scr.Component;
+import org.apache.felix.scr.impl.manager.AbstractComponentManager;
 import org.osgi.framework.Bundle;
 import org.osgi.service.log.LogService;
 
@@ -30,7 +31,7 @@
  * implementation. This is mainly used when logging something around the task
  * being run or scheduled.
  */
-abstract class ComponentActivatorTask implements Runnable
+public abstract class ComponentActivatorTask implements Runnable
 {
 
     private final String taskName;

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentRegistry.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentRegistry.java?rev=792540&r1=792539&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentRegistry.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentRegistry.java Thu Jul  9 13:02:12 2009
@@ -29,6 +29,9 @@
 
 import org.apache.felix.scr.Component;
 import org.apache.felix.scr.ScrService;
+import org.apache.felix.scr.impl.manager.AbstractComponentManager;
+import org.apache.felix.scr.impl.manager.ComponentFactoryImpl;
+import org.apache.felix.scr.impl.manager.ImmediateComponentManager;
 import org.osgi.framework.Bundle;
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.Constants;
@@ -220,7 +223,7 @@
     }
 
 
-    Configuration getConfiguration( final BundleContext ctx, final String pid )
+    public Configuration getConfiguration( final BundleContext ctx, final String pid )
     {
         final String filter = "(service.pid=" + pid + ")";
         Configuration[] cfg = getConfigurationInternal( ctx, filter );
@@ -228,7 +231,7 @@
     }
 
 
-    Configuration[] getConfigurations( final BundleContext ctx, final String factoryPid )
+    public Configuration[] getConfigurations( final BundleContext ctx, final String factoryPid )
     {
         final String filter = "(service.factoryPid=" + factoryPid + ")";
         return getConfigurationInternal( ctx, filter );
@@ -265,14 +268,14 @@
 
     //---------- ComponentManager registration support
 
-    long createComponentId()
+    public long createComponentId()
     {
         m_componentCounter++;
         return m_componentCounter;
     }
 
 
-    void checkComponentName( String name )
+    public void checkComponentName( String name )
     {
         if ( m_componentsByName.containsKey( name ) )
         {

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/Logger.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/Logger.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/Logger.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,9 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.helper;
+
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
 
 
 /**
@@ -24,12 +26,12 @@
  * in the XML Parser and ComponentMetadata handling classes and at the same
  * time not be too intrusive for the unit tests.
  */
-interface Logger
+public interface Logger
 {
 
     /**
      * Writes a messages for the given <code>ComponentMetadata</code>.
-     * 
+     *
      * @param level The log level of the messages. This corresponds to the log
      *          levels defined by the OSGi LogService.
      * @param message The message to print

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java
------------------------------------------------------------------------------
    svn:keywords = Author Date Id Revision Rev Url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/Logger.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReadOnlyDictionary.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReadOnlyDictionary.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReadOnlyDictionary.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java Thu Jul  9 13:02:12 2009
@@ -1,4 +1,4 @@
-/* 
+/*
  * Licensed to the Apache Software Foundation (ASF) under one
  * or more contributor license agreements.  See the NOTICE file
  * distributed with this work for additional information
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.helper;
 
 
 import java.util.Dictionary;
@@ -25,7 +25,7 @@
 
 /**
  * The <code>ReadOnlyDictionary</code> is a <code>Dictionary</code> whose
- * {@link #put(Object, Object)} and {@link #remove(Object)} methods have 
+ * {@link #put(Object, Object)} and {@link #remove(Object)} methods have
  * no effect and always return <code>null</code>.
  *
  * @author fmeschbe
@@ -36,7 +36,7 @@
     private Dictionary delegatee;
 
 
-    ReadOnlyDictionary( Dictionary delegatee )
+    public ReadOnlyDictionary( Dictionary delegatee )
     {
         this.delegatee = delegatee;
     }

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReadOnlyDictionary.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java (from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReflectionHelper.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReflectionHelper.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReflectionHelper.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.helper;
 
 
 import java.lang.reflect.InvocationTargetException;
@@ -35,25 +35,25 @@
  * The <code>ReflectionHelper</code> class provides utility methods to find out
  * about binding and activation methods in components.
  */
-final class ReflectionHelper
+public final class ReflectionHelper
 {
 
     // Method instance to implement tristate behaviour on method fields:
     // unchecked (SENTINEL), no method (null), valid method (method object)
-    static final Method SENTINEL;
+    public static final Method SENTINEL;
 
     // class references to simplify parameter checking
-    static final Class COMPONENT_CONTEXT_CLASS = ComponentContext.class;
-    static final Class BUNDLE_CONTEXT_CLASS = BundleContext.class;
-    static final Class MAP_CLASS = Map.class;
-    static final Class INTEGER_CLASS = Integer.class;
+    public static final Class COMPONENT_CONTEXT_CLASS = ComponentContext.class;
+    public static final Class BUNDLE_CONTEXT_CLASS = BundleContext.class;
+    public static final Class MAP_CLASS = Map.class;
+    public static final Class INTEGER_CLASS = Integer.class;
 
     // Helper used to find the best matching activate and modified methods
-    static final ActivatorMethodTester ACTIVATE_ACCEPTED_PARAMETERS = new ActivatorMethodTester( new Class[]
+    public static final ActivatorMethodTester ACTIVATE_ACCEPTED_PARAMETERS = new ActivatorMethodTester( new Class[]
         { COMPONENT_CONTEXT_CLASS, BUNDLE_CONTEXT_CLASS, MAP_CLASS } );
 
     // Helper used to find the best matching deactivate method
-    static final ActivatorMethodTester DEACTIVATE_ACCEPTED_PARAMETERS = new ActivatorMethodTester( new Class[]
+    public static final ActivatorMethodTester DEACTIVATE_ACCEPTED_PARAMETERS = new ActivatorMethodTester( new Class[]
         { COMPONENT_CONTEXT_CLASS, BUNDLE_CONTEXT_CLASS, MAP_CLASS, Integer.TYPE, INTEGER_CLASS } );
 
     static
@@ -101,7 +101,7 @@
      * @throws InvocationTargetException If an unexpected Throwable is caught
      *      trying to access the desired method.
      */
-    static Method getMethod( final Class objectClass, final String name, final MethodTester tester )
+    public static Method getMethod( final Class objectClass, final String name, final MethodTester tester )
         throws NoSuchMethodException, InvocationTargetException
     {
         // whether we accept package private methods
@@ -196,7 +196,7 @@
      * @throws InvocationTargetException If an unexpected Throwable is caught
      *      trying to access the desired method.
      */
-    static Method getMethod( Class clazz, String name, Class[] parameterTypes, boolean acceptPrivate,
+    public static Method getMethod( Class clazz, String name, Class[] parameterTypes, boolean acceptPrivate,
         boolean acceptPackage ) throws NoSuchMethodException, InvocationTargetException
     {
         try
@@ -250,7 +250,7 @@
      * @param acceptPackage Whether a package private method is acceptable
      * @return
      */
-    static boolean accept( Method method, boolean acceptPrivate, boolean acceptPackage )
+    public static boolean accept( Method method, boolean acceptPrivate, boolean acceptPackage )
     {
         // method must be void
         if ( Void.TYPE != method.getReturnType() )
@@ -302,7 +302,7 @@
      * Returns the name of the package to which the class belongs or an
      * empty string if the class is in the default package.
      */
-    static String getPackageName( Class clazz )
+    public static String getPackageName( Class clazz )
     {
         String name = clazz.getName();
         int dot = name.lastIndexOf( '.' );
@@ -311,7 +311,7 @@
 
     //---------- inner classes
 
-    static interface MethodTester
+    public static interface MethodTester
     {
 
         /**
@@ -334,7 +334,7 @@
         Class[] getParameterLists();
     }
 
-    static final class ActivatorMethodTester implements ReflectionHelper.MethodTester
+    public static final class ActivatorMethodTester implements ReflectionHelper.MethodTester
     {
         private final Class[] parameterLists;
         private final Set methods;

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java
------------------------------------------------------------------------------
    svn:keywords = Author Date Id Revision Rev Url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/helper/ReflectionHelper.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java (from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/AbstractComponentManager.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/AbstractComponentManager.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/AbstractComponentManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.manager;
 
 import java.util.ArrayList;
 import java.util.Dictionary;
@@ -27,6 +27,11 @@
 
 import org.apache.felix.scr.Component;
 import org.apache.felix.scr.Reference;
+import org.apache.felix.scr.impl.BundleComponentActivator;
+import org.apache.felix.scr.impl.ComponentActivatorTask;
+import org.apache.felix.scr.impl.ComponentRegistry;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
+import org.apache.felix.scr.impl.metadata.ReferenceMetadata;
 import org.osgi.framework.Bundle;
 import org.osgi.framework.InvalidSyntaxException;
 import org.osgi.framework.ServiceReference;
@@ -40,7 +45,7 @@
  * implementation object's lifecycle.
  *
  */
-abstract class AbstractComponentManager implements Component, ComponentInstance
+public abstract class AbstractComponentManager implements Component, ComponentInstance
 {
     // the ID of this component
     private long m_componentId;
@@ -738,7 +743,7 @@
     }
 
     //**********************************************************************************************************
-    BundleComponentActivator getActivator()
+    public BundleComponentActivator getActivator()
     {
         return m_activator;
     }

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentContextImpl.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentContextImpl.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentContextImpl.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java Thu Jul  9 13:02:12 2009
@@ -16,11 +16,12 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.manager;
 
 
 import java.util.Dictionary;
 
+import org.apache.felix.scr.impl.helper.ReadOnlyDictionary;
 import org.osgi.framework.Bundle;
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.ServiceReference;
@@ -32,7 +33,7 @@
  * Implementation for the ComponentContext interface
  *
  */
-class ComponentContextImpl implements ComponentContext
+public class ComponentContextImpl implements ComponentContext
 {
 
     private AbstractComponentManager m_componentManager;

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentContextImpl.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java (from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentFactoryImpl.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentFactoryImpl.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentFactoryImpl.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.manager;
 
 import java.util.Dictionary;
 import java.util.HashMap;
@@ -24,6 +24,9 @@
 import java.util.IdentityHashMap;
 import java.util.Map;
 
+import org.apache.felix.scr.impl.BundleComponentActivator;
+import org.apache.felix.scr.impl.ComponentRegistry;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
 import org.osgi.framework.Constants;
 import org.osgi.framework.ServiceRegistration;
 import org.osgi.service.cm.Configuration;
@@ -35,7 +38,7 @@
 /**
  * The <code>ComponentFactoryImpl</code> TODO
  */
-class ComponentFactoryImpl extends AbstractComponentManager implements ComponentFactory
+public class ComponentFactoryImpl extends AbstractComponentManager implements ComponentFactory
 {
 
     // The component registry used to retrieve component IDs
@@ -51,7 +54,7 @@
     private final Map m_createdComponents;
 
 
-    ComponentFactoryImpl( BundleComponentActivator activator, ComponentMetadata metadata,
+    public ComponentFactoryImpl( BundleComponentActivator activator, ComponentMetadata metadata,
         ComponentRegistry componentRegistry )
     {
         super( activator, metadata, componentRegistry );
@@ -142,7 +145,7 @@
 
     //---------- ManagedServiceFactory interface ------------------------------
 
-    void updated( String pid, Dictionary configuration )
+    public void updated( String pid, Dictionary configuration )
     {
         if ( getState() == STATE_FACTORY )
         {
@@ -173,7 +176,7 @@
         }
     }
 
-    void deleted( String pid )
+    public void deleted( String pid )
     {
         if ( getState() == STATE_FACTORY && m_configuredServices != null )
         {

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java (from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DelayedComponentManager.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DelayedComponentManager.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DelayedComponentManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java Thu Jul  9 13:02:12 2009
@@ -16,9 +16,12 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.manager;
 
 
+import org.apache.felix.scr.impl.BundleComponentActivator;
+import org.apache.felix.scr.impl.ComponentRegistry;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
 import org.osgi.framework.Bundle;
 import org.osgi.framework.ServiceFactory;
 import org.osgi.framework.ServiceRegistration;
@@ -27,7 +30,7 @@
 /**
  * The <code>DelayedComponentManager</code> TODO
  */
-class DelayedComponentManager extends ImmediateComponentManager implements ServiceFactory
+public class DelayedComponentManager extends ImmediateComponentManager implements ServiceFactory
 {
 
     /**

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DelayedComponentManager.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java (from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.manager;
 
 
 import java.lang.reflect.InvocationTargetException;
@@ -30,6 +30,8 @@
 import java.util.Map;
 
 import org.apache.felix.scr.Reference;
+import org.apache.felix.scr.impl.helper.ReflectionHelper;
+import org.apache.felix.scr.impl.metadata.ReferenceMetadata;
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.Constants;
 import org.osgi.framework.Filter;
@@ -46,7 +48,7 @@
  * declared by a single <code>&lt;reference&gt;</code element in component
  * descriptor.
  */
-class DependencyManager implements ServiceListener, Reference
+public class DependencyManager implements ServiceListener, Reference
 {
     // mask of states ok to send events
     private static final int STATE_MASK = AbstractComponentManager.STATE_UNSATISFIED

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java (from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ImmediateComponentManager.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ImmediateComponentManager.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ImmediateComponentManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.manager;
 
 
 import java.lang.reflect.InvocationTargetException;
@@ -25,6 +25,11 @@
 import java.util.Iterator;
 import java.util.List;
 
+import org.apache.felix.scr.impl.BundleComponentActivator;
+import org.apache.felix.scr.impl.ComponentRegistry;
+import org.apache.felix.scr.impl.helper.ReflectionHelper;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
+import org.apache.felix.scr.impl.metadata.ReferenceMetadata;
 import org.osgi.service.cm.Configuration;
 import org.osgi.service.component.ComponentConstants;
 import org.osgi.service.component.ComponentContext;
@@ -35,7 +40,7 @@
  * The default ComponentManager. Objects of this class are responsible for managing
  * implementation object's lifecycle.
  */
-class ImmediateComponentManager extends AbstractComponentManager
+public class ImmediateComponentManager extends AbstractComponentManager
 {
 
     // The object that implements the service and that is bound to other services
@@ -67,7 +72,7 @@
      * @param activator
      * @param metadata
      */
-    ImmediateComponentManager( BundleComponentActivator activator, ComponentMetadata metadata,
+    public ImmediateComponentManager( BundleComponentActivator activator, ComponentMetadata metadata,
         ComponentRegistry componentRegistry )
     {
         super( activator, metadata, componentRegistry );

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ImmediateComponentManager.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java (from r792527, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceFactoryComponentManager.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceFactoryComponentManager.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceFactoryComponentManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java Thu Jul  9 13:02:12 2009
@@ -16,11 +16,14 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.manager;
 
 
 import java.util.IdentityHashMap;
 
+import org.apache.felix.scr.impl.BundleComponentActivator;
+import org.apache.felix.scr.impl.ComponentRegistry;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
 import org.osgi.framework.Bundle;
 import org.osgi.framework.ServiceFactory;
 import org.osgi.framework.ServiceRegistration;
@@ -33,7 +36,7 @@
 /**
  * The <code>ServiceFactoryComponentManager</code> TODO
  */
-class ServiceFactoryComponentManager extends ImmediateComponentManager implements ServiceFactory
+public class ServiceFactoryComponentManager extends ImmediateComponentManager implements ServiceFactory
 {
 
     // maintain the map of ComponentContext objects created for the

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision url

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ServiceFactoryComponentManager.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ComponentMetadata.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentMetadata.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ComponentMetadata.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ComponentMetadata.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentMetadata.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ComponentMetadata.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ComponentMetadata.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.metadata;
 
 
 import java.util.ArrayList;
@@ -28,6 +28,7 @@
 import java.util.Set;
 import java.util.TreeSet;
 
+import org.apache.felix.scr.impl.helper.Logger;
 import org.osgi.service.component.ComponentException;
 import org.osgi.service.log.LogService;
 
@@ -495,7 +496,7 @@
     /**
      * Method used to verify if the semantics of this metadata are correct
      */
-    void validate( Logger logger )
+    public void validate( Logger logger )
     {
         // nothing to do if already validated
         if ( m_validated )

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ComponentMetadata.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/PropertyMetadata.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/PropertyMetadata.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/PropertyMetadata.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/PropertyMetadata.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/PropertyMetadata.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/PropertyMetadata.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/PropertyMetadata.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.metadata;
 
 import java.util.ArrayList;
 import java.util.List;

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/PropertyMetadata.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ReferenceMetadata.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReferenceMetadata.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ReferenceMetadata.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ReferenceMetadata.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReferenceMetadata.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ReferenceMetadata.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ReferenceMetadata.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.metadata;
 
 import java.util.Set;
 import java.util.TreeSet;

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ReferenceMetadata.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceMetadata.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceMetadata.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/ServiceMetadata.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.metadata;
 
 import java.util.ArrayList;
 import java.util.Iterator;

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/ServiceMetadata.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/XmlHandler.java (from r792521, felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/XmlHandler.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/XmlHandler.java?p2=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/XmlHandler.java&p1=felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/XmlHandler.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/XmlHandler.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/XmlHandler.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.metadata;
 
 
 import java.io.IOException;
@@ -29,6 +29,7 @@
 import java.util.Map;
 import java.util.Properties;
 
+import org.apache.felix.scr.impl.helper.Logger;
 import org.apache.felix.scr.impl.parser.KXml2SAXHandler;
 import org.apache.felix.scr.impl.parser.ParseException;
 import org.osgi.framework.Bundle;
@@ -101,7 +102,7 @@
 
     // creates an instance with the bundle owning the component descriptor
     // file parsed by this instance
-    XmlHandler( Bundle bundle, Logger logger )
+    public XmlHandler( Bundle bundle, Logger logger )
     {
         m_bundle = bundle;
         m_logger = logger;
@@ -109,6 +110,17 @@
 
 
     /**
+    * Called to retrieve the service descriptors
+    *
+    * @return   A list of service descriptors
+    */
+    public List getComponentMetadataList()
+    {
+        return m_components;
+    }
+
+
+    /**
      * Method called when a tag opens
      *
      * @param   uri
@@ -358,17 +370,6 @@
 
 
     /**
-    * Called to retrieve the service descriptors
-    *
-    * @return   A list of service descriptors
-    */
-    List getComponentMetadataList()
-    {
-        return m_components;
-    }
-
-
-    /**
      * @see org.apache.felix.scr.impl.parser.KXml2SAXHandler#characters(java.lang.String)
      */
     public void characters( String text )

Propchange: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/metadata/XmlHandler.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Modified: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/MockLogger.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/MockLogger.java?rev=792540&r1=792539&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/MockLogger.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/MockLogger.java Thu Jul  9 13:02:12 2009
@@ -18,6 +18,9 @@
  */
 package org.apache.felix.scr.impl;
 
+import org.apache.felix.scr.impl.helper.Logger;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
+
 
 public class MockLogger implements Logger
 {

Copied: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java (from r792527, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ReflectionHelperTest.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java?p2=felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java&p1=felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ReflectionHelperTest.java&r1=792527&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ReflectionHelperTest.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.helper;
 
 
 import java.lang.reflect.InvocationTargetException;
@@ -24,12 +24,13 @@
 
 import junit.framework.TestCase;
 
-import org.apache.felix.scr.impl.instances.AcceptMethod;
-import org.apache.felix.scr.impl.instances.BaseObject;
-import org.apache.felix.scr.impl.instances.Level1Object;
-import org.apache.felix.scr.impl.instances.Level3Object;
-import org.apache.felix.scr.impl.instances.MethodNameException;
-import org.apache.felix.scr.impl.instances2.Level2Object;
+import org.apache.felix.scr.impl.helper.ReflectionHelper;
+import org.apache.felix.scr.impl.metadata.instances.AcceptMethod;
+import org.apache.felix.scr.impl.metadata.instances.BaseObject;
+import org.apache.felix.scr.impl.metadata.instances.Level1Object;
+import org.apache.felix.scr.impl.metadata.instances.Level3Object;
+import org.apache.felix.scr.impl.metadata.instances.MethodNameException;
+import org.apache.felix.scr.impl.metadata.instances2.Level2Object;
 
 
 public class ReflectionHelperTest extends TestCase
@@ -139,7 +140,7 @@
         Class dpc = getClass().getClassLoader().loadClass( "DefaultPackageClass" );
         assertEquals( "", ReflectionHelper.getPackageName( dpc ) );
 
-        assertEquals( "org.apache.felix.scr.impl.instances", ReflectionHelper.getPackageName( base.getClass() ) );
+        assertEquals( "org.apache.felix.scr.impl.metadata.instances", ReflectionHelper.getPackageName( base.getClass() ) );
     }
 
 

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java
------------------------------------------------------------------------------
    svn:keywords = Author Date Id Revision Rev Url

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/helper/ReflectionHelperTest.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/ComponentMetadataTest.java (from r792521, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ComponentMetadataTest.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/ComponentMetadataTest.java?p2=felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/ComponentMetadataTest.java&p1=felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ComponentMetadataTest.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/ComponentMetadataTest.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/ComponentMetadataTest.java Thu Jul  9 13:02:12 2009
@@ -16,11 +16,16 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.metadata;
 
 
 import junit.framework.TestCase;
 
+import org.apache.felix.scr.impl.MockLogger;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
+import org.apache.felix.scr.impl.metadata.ReferenceMetadata;
+import org.apache.felix.scr.impl.metadata.ServiceMetadata;
+import org.apache.felix.scr.impl.metadata.XmlHandler;
 import org.osgi.service.component.ComponentException;
 
 

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/ComponentMetadataTest.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Copied: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java (from r792521, felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/XmlHandlerTest.java)
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java?p2=felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java&p1=felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/XmlHandlerTest.java&r1=792521&r2=792540&rev=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/XmlHandlerTest.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl;
+package org.apache.felix.scr.impl.metadata;
 
 
 import java.io.BufferedReader;
@@ -27,6 +27,13 @@
 
 import junit.framework.TestCase;
 
+import org.apache.felix.scr.impl.MockBundle;
+import org.apache.felix.scr.impl.MockLogger;
+import org.apache.felix.scr.impl.metadata.ComponentMetadata;
+import org.apache.felix.scr.impl.metadata.PropertyMetadata;
+import org.apache.felix.scr.impl.metadata.ReferenceMetadata;
+import org.apache.felix.scr.impl.metadata.ServiceMetadata;
+import org.apache.felix.scr.impl.metadata.XmlHandler;
 import org.apache.felix.scr.impl.parser.KXml2SAXParser;
 import org.osgi.service.component.ComponentException;
 import org.xmlpull.v1.XmlPullParserException;

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java
------------------------------------------------------------------------------
    svn:keywords = Author Date Id Revision Rev Url

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/XmlHandlerTest.java
------------------------------------------------------------------------------
    svn:mergeinfo = 

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/
------------------------------------------------------------------------------
    svn:mergeinfo = 

Modified: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/AcceptMethod.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/AcceptMethod.java?rev=792540&r1=792521&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/AcceptMethod.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/AcceptMethod.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl.instances;
+package org.apache.felix.scr.impl.metadata.instances;
 
 
 /**

Modified: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/BaseObject.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/BaseObject.java?rev=792540&r1=792521&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/BaseObject.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/BaseObject.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl.instances;
+package org.apache.felix.scr.impl.metadata.instances;
 
 
 import org.osgi.framework.BundleContext;

Modified: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level1Object.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level1Object.java?rev=792540&r1=792521&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level1Object.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level1Object.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl.instances;
+package org.apache.felix.scr.impl.metadata.instances;
 
 
 import java.util.Map;

Modified: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level3Object.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level3Object.java?rev=792540&r1=792521&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level3Object.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/Level3Object.java Thu Jul  9 13:02:12 2009
@@ -16,12 +16,12 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl.instances;
+package org.apache.felix.scr.impl.metadata.instances;
 
 
 import java.util.Map;
 
-import org.apache.felix.scr.impl.instances2.Level2Object;
+import org.apache.felix.scr.impl.metadata.instances2.Level2Object;
 import org.osgi.service.component.ComponentContext;
 
 

Modified: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/MethodNameException.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/MethodNameException.java?rev=792540&r1=792521&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/MethodNameException.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances/MethodNameException.java Thu Jul  9 13:02:12 2009
@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl.instances;
+package org.apache.felix.scr.impl.metadata.instances;
 
 
 public final class MethodNameException extends RuntimeException

Propchange: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances2/
------------------------------------------------------------------------------
    svn:mergeinfo = 

Modified: felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances2/Level2Object.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances2/Level2Object.java?rev=792540&r1=792521&r2=792540&view=diff
==============================================================================
--- felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances2/Level2Object.java (original)
+++ felix/trunk/scr/src/test/java/org/apache/felix/scr/impl/metadata/instances2/Level2Object.java Thu Jul  9 13:02:12 2009
@@ -16,13 +16,13 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.felix.scr.impl.instances2;
+package org.apache.felix.scr.impl.metadata.instances2;
 
 
 import java.util.Map;
 
-import org.apache.felix.scr.impl.instances.Level1Object;
-import org.apache.felix.scr.impl.instances.MethodNameException;
+import org.apache.felix.scr.impl.metadata.instances.Level1Object;
+import org.apache.felix.scr.impl.metadata.instances.MethodNameException;
 import org.osgi.service.component.ComponentContext;