Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

all() method doesn't consider registered scribes #81

Closed
NeverWise opened this issue Aug 27, 2017 · 1 comment
Closed

all() method doesn't consider registered scribes #81

NeverWise opened this issue Aug 27, 2017 · 1 comment
Labels

Comments

@NeverWise
Copy link

In ChainingParser class the first() method works as expected but the all() method doesn't works.
I think the problem is that the all() method lacks of an if that is present in the first() method.
To be clear this if:

if (index != null) {
	reader.setScribeIndex(index);
}
@mangstadt
Copy link
Owner

Thanks for catching that. Fixed in 254fcec.

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

No branches or pull requests

2 participants