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

Add example #7

Merged
merged 3 commits into from
Feb 19, 2017
Merged

Add example #7

merged 3 commits into from
Feb 19, 2017

Conversation

breznak
Copy link
Contributor

@breznak breznak commented Oct 5, 2016

  • rename the test to LRUCache11Test.cpp
  • add SampleMain.cpp + fix it

Fixes #5

@mohaps
Copy link
Owner

mohaps commented Oct 18, 2016

I have been meaning to add some more comprehensive examples and tests to this codebase. :) Thanks for hurrying that up

@ekg
Copy link

ekg commented Oct 20, 2016

Something looks in conflict here. Mind resolving the conflict and pushing to your branch again? I think it will update the PR.

@breznak
Copy link
Contributor Author

breznak commented Oct 22, 2016

@mohaps done. I had to rebase and force push, please verify the change from your last PR to SampleMain is now in LRU..Test (as that is the copy/rename of the file) Thank you

@breznak
Copy link
Contributor Author

breznak commented Feb 16, 2017

Ping on the open PRs? This one is ready, on the other one I'd need help.

@mohaps mohaps merged commit d9c740c into mohaps:master Feb 19, 2017
@breznak breznak deleted the add_example branch February 19, 2017 22:06
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.

Rename SampleMain.cpp to LRUCache11.cpp
3 participants