Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

MutableObjectModelBinder should avoid using PropertyInfo.SetValue to set property values #2259

Closed
pranavkm opened this issue Mar 25, 2015 · 1 comment

Comments

@pranavkm
Copy link
Contributor

We should use cached delegates created using PropertyHelper.MakeFastPropertySetters to set property values instead. Note: We create these delegates as part of creating DefaultMetadataDetailsCache for property entries today.

@pranavkm pranavkm added the Perf label Mar 25, 2015
@rynowak
Copy link
Member

rynowak commented Mar 25, 2015

@Eilon @danroth27 this is something we wanted to do in a follow up to the MMP changes and didn't quite get to yet. There's a few more places to cleanup also beyond the setter in modelbinding.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants