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

Consider caching property changed event args for Catel properties #1230

Open
GeertvanHorrik opened this Issue Nov 1, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@GeertvanHorrik
Member

GeertvanHorrik commented Nov 1, 2018

Instead of having to recreate the objects every time, we could re-use the same instances, but not sure whether the hit on performance > saved memory.

Since we are already storing the property data for each Catel property, we could store PropertyChangedEventArgs and pass these instead of always recreating the same value.

@GeertvanHorrik GeertvanHorrik added this to the 5.9.0 milestone Nov 1, 2018

@GeertvanHorrik GeertvanHorrik self-assigned this Nov 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment