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

correct mistypes #7

Closed
wants to merge 1 commit into from
Closed

correct mistypes #7

wants to merge 1 commit into from

Conversation

mishin
Copy link

@mishin mishin commented Jun 25, 2015

missing double quotation mark

@preaction
Copy link
Contributor

Hello,

The Perl 5 Porters, the team responsible for developing Perl 5, use a different tool for accepting patches. Please follow the instructions here: https://metacpan.org/pod/distribution/perl/pod/perlgit.pod#Patch-workflow or here: http://perldoc.perl.org/perlhack.html#SUPER-QUICK-PATCH-GUIDE

To summarize from the above link: Use the perlbug utility (that comes with perl), or send an e-mail to [email protected] with your patch as an attachment. Doing so will ensure you get proper credit for your contribution.

@mishin
Copy link
Author

mishin commented Nov 5, 2015

@preaction thanks for answer

p5p pushed a commit that referenced this pull request Feb 19, 2016
This has large memory savings, test prog,
perl -MTest::More -e"system 'pause'"
before 2196KB Private Bytes Win 7 32 bit to after 2092KB.

-On a CHEK the refcount is a U32 for memory savings on 64 bit CPUs while
 SHEKs are Size_t for refcount because of HE struct, on 32 bit Size_t and
 U32 happen to be the same thing, if there is future integration the
 refcount members will have to be the same type, then duping a SHEK or
 a CHEK is the same code, except that HVhek_COMPILING controls whether to
 aquire OP_REFCNT_LOCK before touching the ref count, in the future with
 atomic operations, the refcount can be manipulated with atomic operations
 regardless if it is a SHEK or CHEK since OP_REFCNT_LOCK lines were removed
-TODO figure out how to do static const CHEKs, hash member must be 0
 since its process specific randomized (rurban's B stores HEKs in RW static
 memory and fixes up the hash #s at runtime), add test and branch
 so that refcount isn't read and written or passed to PerlMemShared_free
 if static flag is on inidicating static const CHEK
-TODO Perl_newGP uses CHEKs not CopFILE, no memcpy and add _< that way
-TODO optimize the former alloca to smallbuf or Safefree or savestack
 newx free
@rjbs rjbs closed this Apr 6, 2016
demerphq added a commit that referenced this pull request Sep 9, 2022
demerphq added a commit that referenced this pull request Sep 9, 2022
demerphq added a commit that referenced this pull request Sep 12, 2022
demerphq added a commit that referenced this pull request Oct 25, 2022
demerphq added a commit that referenced this pull request Nov 5, 2022
demerphq added a commit that referenced this pull request Nov 5, 2022
demerphq added a commit that referenced this pull request Nov 5, 2022
demerphq added a commit that referenced this pull request Nov 5, 2022
demerphq added a commit that referenced this pull request Dec 31, 2022
khwilliamson pushed a commit to khwilliamson/perl5 that referenced this pull request Jan 30, 2023
Test more rare edge cases
demerphq added a commit that referenced this pull request Feb 8, 2023
demerphq added a commit that referenced this pull request Feb 19, 2023
demerphq added a commit that referenced this pull request Feb 20, 2023
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

Successfully merging this pull request may close these issues.

3 participants