-
Notifications
You must be signed in to change notification settings - Fork 933
NH-3964 - Refactor reflection patterns #574
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…flectionHelper with a far better implementation.
d8adb4b
to
e20eff1
Compare
Rebased for resolving a conflict on test project. |
By the way, I may refactor further if you wish: put the methods Hazzik was suggesting here in
Then obsolete |
@fredericDelaporte I was trying to hijack the PR and added refactoring to use |
@fredericDelaporte, if you are going to do more work in this field, please use the same JIRA number. |
Ok. I think |
NH-3964 - This is a sort of follow up to NH-3952: remove usage of
EnumerableHelper
.They were replaced by
ReflectionHelper
, and additionally cached whenever possible and useful.This improvement aims at similarly caching other
MethodInfo
/PropertyInfo
reflected through olderReflectionHelper
usages (impactsLinq
namespace), or any other means (most other NHibernate namespaces).If adequate, those other means of reflecting should be replaced by
ReflectionHelper
.