Skip to content
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

Fix dictionary key access #637

Merged
merged 1 commit into from
Feb 20, 2015
Merged

Fix dictionary key access #637

merged 1 commit into from
Feb 20, 2015

Conversation

xumix
Copy link
Contributor

@xumix xumix commented Feb 13, 2015

Looks like someone does not know how to use a dictionary, this fix improves performance 10x for big dictionaries

Looks like someone does not know how to use a dictionary
@Shazwazza
Copy link
Contributor

thx. Yup that is pretty ugly. This legacy code will be deleted entirely in v8 and is obsoleted in 7.3

Shazwazza added a commit that referenced this pull request Feb 20, 2015
Fix dictionary key access
@Shazwazza Shazwazza merged commit 8c13b89 into umbraco:dev-v7 Feb 20, 2015
@xumix
Copy link
Contributor Author

xumix commented Feb 20, 2015

Any chance to integrate this into 6.2.x branch? We are deeply invested in it and can't upgrade to 7.x any time soon.

@xumix xumix deleted the 6.2.5 branch February 20, 2015 09:02
@Shazwazza
Copy link
Contributor

sure will do that now

@Shazwazza
Copy link
Contributor

done in rev: d2662c1 and will be available in 6.2.5

@xumix
Copy link
Contributor Author

xumix commented Feb 20, 2015

Thanks a lot!

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

Successfully merging this pull request may close these issues.

None yet

2 participants