Skip to content

Conversation

@rice668
Copy link

@rice668 rice668 commented Jun 29, 2017

Remove unused LRUCache class since it was created in 2014.

@aljoscha
Copy link
Contributor

Thanks for working on this! While on it, could you also remove LRUCacheMap and LRUCacheMapTest? I think these are similarly unused.

@rice668
Copy link
Author

rice668 commented Jul 12, 2017

Sure, Thanks, I had seen those before. LRUCacheMap and LRUCacheMapTest will be removed in this PR soon.

@rice668
Copy link
Author

rice668 commented Jul 12, 2017

Hello, @aljoscha PR had been updated. Please helps check it again, thanks!

@aljoscha
Copy link
Contributor

Thanks! I merged 😃

Could you please close this PR?

@rice668
Copy link
Author

rice668 commented Jul 13, 2017

Yep. I will close this PR. 咯咯~

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants