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

uses Params::Validate; should use Params::ValidationCompiler #28

Open
karenetheridge opened this issue Feb 7, 2020 · 1 comment
Open

Comments

@karenetheridge
Copy link
Contributor

The documentation at https://metacpan.org/pod/Params::Validate#DESCRIPTION asserts that PVC is preferred to PV (same author for both). I leave this issue here in case someone driving by has the ambition to submit a patch for the change; who knows, at some point it might even be me :)

@rjbs
Copy link
Owner

rjbs commented Jan 20, 2024

This sounds fine if somebody does this and it breaks nothing, but it's worth noting that the big win of Params::ValidationCompiler, as far as I understand it, is efficient re-use. In general, GLD is called once in a process, so making the validator efficient to call 100 times is not much of a win.

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

No branches or pull requests

2 participants