On Dec 4, 2007 11:09 AM, <[EMAIL PROTECTED]> wrote:

> Modified: servicemix/branches/servicemix-4.0
> /nmr/core/src/main/java/org/apache/servicemix/nmr/core/ServiceMix.java
> URL:
> http://svn.apache.org/viewvc/servicemix/branches/servicemix-4.0/nmr/core/src/main/java/org/apache/servicemix/nmr/core/ServiceMix.java?rev=600866&r1=600865&r2=600866&view=diff
>
> ==============================================================================
> --- 
> servicemix/branches/servicemix-4.0/nmr/core/src/main/java/org/apache/servicemix/nmr/core/ServiceMix.java
> (original)
> +++ 
> servicemix/branches/servicemix-4.0/nmr/core/src/main/java/org/apache/servicemix/nmr/core/ServiceMix.java
> Tue Dec  4 02:09:51 2007
> @@ -46,8 +46,9 @@
>         }
>         if (flows == null) {
>             flows = new FlowRegistryImpl();
> -            flows.register(new StraightThroughFlow(),
> ServiceHelper.createMap(Flow.ID, StraightThroughFlow.class.getName()));
> +
>         }
> +        flows.register(new StraightThroughFlow(), ServiceHelper.createMap
> (Flow.ID, StraightThroughFlow.class.getName()));
>     }
>
>     /**
>
>
>
I don't think this is a good idea.  As this would prevent a proper
configuration of the STFlow by the user (or even the possibility to not use
it).
If you set your own FlowRegistry, you are responsible for populating it, but
if you let the class create it for you, it will be populated with a default
config (in this case, a StraightThroughFlow).

-- 
Cheers,
Guillaume Nodet
------------------------
Blog: http://gnodet.blogspot.com/

Reply via email to