Krzysztof I have basically done the same and was about to submit a patch.
It passes all the test. Should i still submit it? On Dec 16, 12:44 am, Krzysztof Koźmic (2) <[email protected]> wrote: > See this. You'll need to change the extension to .patch > > This is my rather quick attempt at removing non-generic collections > from the > codebase.http://wiki.kozmic.pl/GetFile.aspx?File=%2fnon-generic-collections-re... > > On 15 Gru, 14:34, Simon <[email protected]> wrote: > > > and how about in \InversionOfControl\src\Castle.MicroKernel\Lifestyle > > \Pool\DefaultPool.cs > > > it would seem syncronization is redundant here because all usages of > > "inUse" are wrapped in a ReaderWriterLock > > > On Dec 16, 12:15 am, Krzysztof Ko 1/4 mic (2) <[email protected]> wrote: > > > > yes, as far as I can tell it is redundant. > > > > On 15 Gru, 14:07, Simon <[email protected]> wrote: > > > > > In the file > > > > \InversionOfControl\src\Castle.MicroKernel\SubSystems\Naming > > > > \KeySearchNamingSubSystem.cs > > > > > It makes use of Hashtable.Synchronized and ArrayList.Synchronized. > > > > > But since all the methods on KeySearchNamingSubSystem have the > > > > attribute [MethodImpl(MethodImplOptions.Synchronized)] doesnt that > > > > make syncronizing the collections redundant? -- You received this message because you are subscribed to the Google Groups "Castle Project Development List" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en.
