Brian Lawler <[EMAIL PROTECTED]> writes:

>Hey guys-

>Don't know if you want to accept this patch right before a release, but  
>it has been working in production for us for quite some time.  This is  

Well, why not? ;-) That is, what RCs are for. We discussed it and the
patch looks not very intrusive to me. As we don't use different class
loaders and even parallel threads would generate the same type of
objects, I see no reason why not to apply it.

>-            for (Iterator it = packages.iterator(); it.hasNext();)
>+            for (Iterator it = packages.iterator(); assembler == null  
>&& it.hasNext();)
[...]
>                      assembler = (Assembler) servClass.newInstance();
>                      break; // for()
>                  }

You don't trust the break, do you? ;-)

        Regards
                Henning

-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
[EMAIL PROTECTED]        +49 9131 50 654 0   http://www.intermeta.de/

RedHat Certified Engineer -- Jakarta Turbine Development  -- hero for hire
   Linux, Java, perl, Solaris -- Consulting, Training, Development

"Fighting for one's political stand is an honorable action, but re-
 fusing to acknowledge that there might be weaknesses in one's
 position - in order to identify them so that they can be remedied -
 is a large enough problem with the Open Source movement that it
 deserves to be on this list of the top five problems."
                       -- Michelle Levesque, "Fundamental Issues with
                                    Open Source Software Development"

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to