unitycontainer / interception

Unity.Interception package
Apache License 2.0
22 stars 17 forks source link

Reverse changes made by mistake with the wrong commit #37

Closed hypercodeplace closed 4 years ago

hypercodeplace commented 4 years ago

Reverse changes made by mistake with the wrong commit. Part 1. Discussion is here: https://github.com/unitycontainer/interception/pull/29

After this PR is merged, it is needed to merge the next PR: https://github.com/unitycontainer/interception/pull/36

dnfclas commented 4 years ago

CLA assistant check
All CLA requirements met.

ENikS commented 4 years ago

Calling method.GetBaseDefinition() twice (worst case) will degrade performance. I can not agree to this approach. What exactly are you trying to fix here?

If this was in original pr, I must have optimized it out.