-
Notifications
You must be signed in to change notification settings - Fork 560
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
swap needs to be removed from perlreapi #16004
Comments
From @toddrCreated by @toddrWe just noticed in the documentation that swap is mentioned in perlreapi.pod but was removed by Nick in c5e13bf Not sure if there are other inconsistencies. Perl Info
|
From @jkeenanOn Wed, 07 Jun 2017 22:59:51 GMT, TODDR wrote:
Please review patch attached. Thank you very much. -- |
From @jkeenan131532-0001-Remove-outdated-references-to-swap.patchFrom 2e5cff95c727e91b4c0cd81e6d23ec50be6069fb Mon Sep 17 00:00:00 2001
From: James E Keenan <jkeenan@cpan.org>
Date: Thu, 8 Jun 2017 08:19:53 -0400
Subject: [PATCH] Remove outdated references to 'swap'.
Per: RT # 131532
---
pod/perlreapi.pod | 5 -----
1 file changed, 5 deletions(-)
diff --git a/pod/perlreapi.pod b/pod/perlreapi.pod
index 52e6b0f..2df337e 100644
--- a/pod/perlreapi.pod
+++ b/pod/perlreapi.pod
@@ -624,7 +624,6 @@ values.
* matching*/
U32 lastparen; /* highest close paren matched ($+) */
U32 lastcloseparen; /* last close paren matched ($^N) */
- regexp_paren_pair *swap; /* Swap copy of *offs */
regexp_paren_pair *offs; /* Array of offsets for (@-) and
(@+) */
@@ -727,10 +726,6 @@ data structure. The Perl engine uses the
C<regexp_internal> structure (see L<perlreguts/Base Structures>) but a custom
engine should use something else.
-=head2 C<swap>
-
-Unused. Left in for compatibility with Perl 5.10.0.
-
=head2 C<offs>
A C<regexp_paren_pair> structure which defines offsets into the string being
--
2.7.4
|
The RT System itself - Status changed from 'new' to 'open' |
From @toddrOn Thu, 08 Jun 2017 05:24:30 -0700, jkeenan wrote:
Looks right to me. Mergeable? |
From @jkeenanOn Fri, 29 Sep 2017 23:59:22 GMT, TODDR wrote:
Commit 5444188; marking ticket Resolved. Thank you very much. |
@jkeenan - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#131532 (status was 'resolved')
Searchable as RT131532$
The text was updated successfully, but these errors were encountered: