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

Allow XML::Reader to be used without specifying a backend - defaults to XML::Reader #1

Merged
merged 5 commits into from Jan 30, 2018
Merged

Allow XML::Reader to be used without specifying a backend - defaults to XML::Reader #1

merged 5 commits into from Jan 30, 2018

Conversation

@HaraldJoerg
Copy link
Contributor

@HaraldJoerg HaraldJoerg commented Jan 30, 2018

This pull request is motivated by this year's CPAN Pull Request Challenge.

The change allows to 'use XML::Reader;' without specifying a backend. In that case, XML::Reader will now try to load XML::Parser, and if this fails, fall back to XML::Parsepp. This is similar to the behaviour of the modules Text::CSV or JSON which also have a XS and a pure perl implementation.

Tests simulating the absence of modules are included in the xt/author section, so that they will not be executed during an install.

Documentation is amended in English and German - but I fail to speak French, so I have to apologize that the French version is missing.

@klaus03 klaus03 merged commit 9c9abb9 into klaus03:master Jan 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants