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

document rejecting unicode and binary names #5

Open
rurban opened this issue Mar 31, 2015 · 1 comment
Open

document rejecting unicode and binary names #5

rurban opened this issue Mar 31, 2015 · 1 comment

Comments

@rurban
Copy link

rurban commented Mar 31, 2015

Package::Stash has a wrong understanding of the Perl API, it uses a crazy regex filter to disallow certain package names. perl5.16 allows the full set of binary characters as package name, even embedded \0 (it is now officially "binary (un)safe"), and it also allows unicode names.

This needs to be documented that it is ASCII only.

But I would rather call this drastic limitation a bug, and the regex filter should be rather left out, and I would provide a Package::Stash::ASCII module with this filter enabled.

Not talking about the wrong setting of the utf8 flag on the names. This is a seperate ticket already.

@ribasushi
Copy link

See also 1st paragraph of p5sagit/Sub-Name#4 (comment) and p5sagit/Sub-Name#4 (comment)

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