Hi Tim, I've done the discussed changes + javadocs on Component and Connector.
Best regards, Jerome > -----Message d'origine----- > De : [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] De la > part de Tim Peierls > Envoyé : jeudi 3 janvier 2008 00:15 > À : [email protected] > Objet : Re: ServerServlet broken by concurrency fix in 1.1M1 > > On Jan 2, 2008 4:57 PM, Jerome Louvel <[EMAIL PROTECTED]> wrote: > > > As I don't see any particular drawback, I would also > prefer to mark the > method 'synchronized' for completeness purpose for this > method and similar > ones. > > > > That's fine -- but could you add a comment along the lines of > "synchronized to make compound action (clear, addAll) atomic, > not for visibility"? > > --tim > > >
