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

warnings as XS #9

Open
rurban opened this issue Sep 11, 2015 · 1 comment
Open

warnings as XS #9

rurban opened this issue Sep 11, 2015 · 1 comment
Assignees
Labels
Projects
Milestone

Comments

@rurban
Copy link
Member

@rurban rurban commented Sep 11, 2015

Rewrite warnings.pm which is generated as pure XS. esp. to save memory

A regular XS extension, with a stable set of .xs and .h files, and a warnings_xs.PL regen and definition
script (prev. regen/warnings.pl), which updates the keys in place.
required gperf so far, as with Config as XS.

Seperate into a pure read-only perfect hash, and a user-extendable warnings::register hash.
Check against Carp in XS and PP Carp.

See branches feature/CM-367-cperl-warnings-xs-carp and feature/CM-370-cperl-warnings-xs

@rurban rurban added the enhancement label Sep 11, 2015
@rurban rurban self-assigned this Sep 11, 2015
@rurban
Copy link
Member Author

@rurban rurban commented Sep 11, 2015

Status: One small nasty bug left, in 9enabled

rurban pushed a commit that referenced this issue Jan 6, 2016
See [perl11/p5-Config #9].
Broken with a non-empty inc_version_list_init
rurban pushed a commit that referenced this issue Oct 26, 2016
This macro follows Unicode Corrigendum #9 to allow non-character code
points.  These are still discouraged but not completely forbidden.

It's best for code that isn't intended to operate on arbitrary other
code text to use the original definition, but code that does things,
such as source code control, should change to use this definition if it
wants to be Unicode-strict.

Perl can't adopt C9 wholesale, as it might create security holes in
existing applications that rely on Perl keeping non-chars out.
@rurban rurban added the in progress label Dec 1, 2016
rurban pushed a commit that referenced this issue Jan 3, 2017
This macro follows Unicode Corrigendum #9 to allow non-character code
points.  These are still discouraged but not completely forbidden.

It's best for code that isn't intended to operate on arbitrary other
code text to use the original definition, but code that does things,
such as source code control, should change to use this definition if it
wants to be Unicode-strict.

Perl can't adopt C9 wholesale, as it might create security holes in
existing applications that rely on Perl keeping non-chars out.
@rurban rurban removed the in progress label Mar 30, 2017
@rurban rurban added this to In progress in cperl Dec 16, 2017
rurban added a commit that referenced this issue Sep 3, 2018
Improve tests using FindBin for filter-util.pl [atoomic #9]

Add int casts, assert on 2GB limitation.

No m4 filter in CORE
@rurban rurban added this to the v5.29.1c milestone Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
cperl
  
In progress
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.