threading related pull request for dynamic proxy

[Krzysztof Kozmic]

https://github.com/castleproject/Core/pull/60

anyone has any objections against merging this one?

-- 
Krzysztof Kozmic

[hammett]

Makes sense. +1

> --
> You received this message because you are subscribed to the Google Groups
> "Castle Project Development List" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to castle-project-d...@googlegroups.com.
> To post to this group, send email to castle-pro...@googlegroups.com.
> Visit this group at http://groups.google.com/group/castle-project-devel.
> For more options, visit https://groups.google.com/d/optout.



--
Cheers,
hammett
http://www.d-collab.com/
http://www.hammettblog.com/

[Ken Egozi]

looks legit, +1

Ken Egozi.
http://kenegozi.com/blog
http://aspnetwebstack.codeplex.com

http://www.windowsazure.com/en-us/develop/mobile/
http://www.shopyourway.com

http://www.castleproject.org

[Ken Egozi]

with a minor comment on the code (extracting the cacheType variable to outer scope might not be required, though I only looked at the diff, I might be missing something in the bigger picture)

Ken Egozi.
http://kenegozi.com/blog
http://aspnetwebstack.codeplex.com

http://www.windowsazure.com/en-us/develop/mobile/
http://www.shopyourway.com

http://www.castleproject.org

[Krzysztof Kozmic]

It got merged, and using our recently added nightly nugget feed I built a non-scientific experiment running this in a console app on my 4 core (HT) MacBook Pro.  It squarely falls into micro optimisation bucket as any performance difference I noticed (ran a few scenarios, with varying number of iterations and proxy types) is almost negligible.

Here’s one such scenario: https://gist.github.com/kkozmic/8362bc57f24d7adf68e2

-- 
Krzysztof Kozmic

[Ken Egozi]

Might be a silly change but what if you switch the for loops?

Sent from my Windows Phone

---

From: Krzysztof Kozmic
Sent: ‎6/‎26/‎2014 3:32 AM
To: castle-pro...@googlegroups.com

[hammett]

But there's no harm, is there? If there was a downgrade in perf, i'd
be worried (not really)

On Thu, Jun 26, 2014 at 7:32 AM, Krzysztof Kozmic