#144 Cache scoped items

[MaciejWanat]

I've done a bit more than I suggested in https://github.com/pamidur/aspect-injector/issues/144, so here's the list of things I've done:

• I think suggestion with parameter of PerInstance is great! The only difference I made is that I believe default behavior should NOT be per instance. If you think it should be otherwise, it will be easy to change
• I've changed way of calculating hash of arguments. Previous way was incorrect, and if you had function accepting two same type arguments, their order didn't matter (see the test).
• GetKey now has two overloads - with and without instance. I also changed it to be protected instead private, so anyone can define their own way of creating a key, in they way they find suitable, by overriding these methods.
• Added tests for following changes
• Fixed some minor things, removed unused code, typos, renamed to match convention

I got a bit confused in line 58, because there are no tests for multiple triggers. So I figured I will use Any to define which key to use, but I'm not sure whether its correct.

I hope you like the changes :)

[MaciejWanat]

This failed test is passing when ran directly, doesn't when tests are run altogether.

[MaciejWanat]

Using method.DeclaringType.FullName ends up with behaving in really strange way, so I've changed it to method, which should work just as good - tests passes correctly.

[MaciejWanat]

@pamidur when is this change expected to be realeased?

[pamidur]

Monday-Tuesday I think, I have a few more fixes in mind.

[pamidur]

@MaciejWanat, one thing I will change is PerInstance by default equals true, to be backward compatible with previous versions

[MaciejWanat]

@pamidur good idea, I was thinking about the same suggestion

[pamidur]

I also found an issue in AI with properties from base classes in attributes, I need to fix before the release

[pamidur]

Released as https://www.nuget.org/packages/Aspects.Cache/1.0.2