FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Optimized DependencyObject setValue performance by hshristov · Pull Request #2260 · NativeScript/NativeScript · GitHub

Optimized DependencyObject setValue performance - #2260

Merged
hshristov merged 3 commits into
masterfrom
hhristov/do-perf-improvement
Jun 7, 2016
Merged

hshristov merged 3 commits into
masterfrom
hhristov/do-perf-improvement

Conversation

Copy link
Copy Markdown
Contributor

ResetValue does not remove entry but only reset value for the given ValueSource (Local by default)
Cached all properties from metadata as fields on Property
When binding source value is null/undefined we reset the value of the bound property

ResetValue does not remove entry but only reset value for the given ValueSource (Local by default)
Optimized setValueInternal performance
ns-bot added the cla: yes label Jun 7, 2016

Copy link
Copy Markdown
Contributor Author

cuteness

Copy link
Copy Markdown
Contributor Author

uitests

vakrilov commented Jun 7, 2016

Copy link
Copy Markdown
Contributor

👍

1 similar comment

vchimev commented Jun 7, 2016

Copy link
Copy Markdown
Contributor

👍

hshristov merged commit 743a2ef into master Jun 7, 2016
hshristov deleted the hhristov/do-perf-improvement branch June 7, 2016 12:30
hshristov added this to the 2.1 milestone Jul 1, 2016
hshristov self-assigned this Jul 1, 2016

lock Bot commented Aug 28, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators Aug 28, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL