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

Remove TokenizerException #58

Merged
merged 1 commit into from Jun 7, 2019

Conversation

Projects
None yet
2 participants
@kingle
Copy link
Collaborator

commented Jun 7, 2019

  • Removing the private TokenizerException since it seems to just cause more confusion than help -- seems better to have this consistently under TextParseException. It's functionality (filename + line number in the message) still exists in the exception method.
  • Added an initial MasterTest with example zone files to help sanity check records in zone files

@kingle kingle changed the title Remove TokenizerException Remove TokenizerException. Fix #38 Jun 7, 2019

@kingle kingle changed the title Remove TokenizerException. Fix #38 Remove TokenizerException Jun 7, 2019

@ibauersachs ibauersachs merged commit 25fc5e0 into dnsjava:master Jun 7, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+1.5%) to 46.602%
Details
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.