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

Cache fixes #2183

Merged
merged 1 commit into from Jan 18, 2016

Conversation

Projects
None yet
2 participants
@tgeorgiev
Copy link

commented Jan 18, 2016

  • Fixing cache polution when putting an item with the same key multiple times
  • Making shift() to reduce the current size of the cache
Cache fixes
- Fixing cache polution when putting an item with the same key multiple times
- Making shift() to reduce the current size of the cache

yyx990803 added a commit that referenced this pull request Jan 18, 2016

@yyx990803 yyx990803 merged commit a17ab47 into vuejs:dev Jan 18, 2016

1 check passed

ci/circleci Your tests passed on CircleCI!
Details
@yyx990803

This comment has been minimized.

Copy link
Member

commented Jan 18, 2016

wow, didn't know the cache had bugs all along!

@tgeorgiev

This comment has been minimized.

Copy link
Author

commented Jan 18, 2016

yeah, it happens so that I have this other issue that I have templates with whitespace in the beginning that started putting a lot of items in the cache, this is how I found it :)

@tgeorgiev tgeorgiev deleted the tgeorgiev:cache-fix branch Jan 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.