Hi Guillaume,
You are correct, I will fix it up soon.

Best Regards
Freeman

On Dec 4, 2007 6:25 PM, Guillaume Nodet <[EMAIL PROTECTED]> wrote:

> 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