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

Added function to stop browsing. #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added function to stop browsing. #7

wants to merge 1 commit into from

Conversation

nleclerc
Copy link

@nleclerc nleclerc commented May 7, 2014

Nothing to add :-)

@jarnoh
Copy link
Owner

jarnoh commented May 7, 2014

Actually, I think calling browse() without parameters does stop browsing already?

@nleclerc
Copy link
Author

nleclerc commented May 7, 2014

Now that you mention it, it probably does but i don't think it makes sense semantically speaking.

Don't you think it's better if browse without parms gives an error and there's a separate stop function ?

Note that i didn't add an error when calling browse without parms, i'm just saying :-)

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.

2 participants