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
Bleadperl eae48c89 breaks TIMB/Devel-NYTProf-4.05.tar.gz #10764
Comments
From @andkgit bisect: commit eae48c8 refactor and regularise label/statement grammar perl -V: Summary of my perl5 (revision 5 version 13 subversion 6) configuration: Characteristics of this binary (from libperl): -- |
From @timbunceThere's something odd happening here. Just before eae48c8 (at ff0c75a): $ perl -d:Trace <<END
but at eae48c8 this happens:
Tim. |
The RT System itself - Status changed from 'new' to 'open' |
From @timbunceZefram, any thoughts on this? Tim. On Sat, Oct 30, 2010 at 01:50:39PM +0100, Tim Bunce wrote:
|
From zefram@fysh.orgTim Bunce wrote:
Going to look at it, haven't had time yet. -zefram |
From zefram@fysh.orgTim Bunce wrote:
I believe this is fixed by a one line change (plus `make regen_perly`): Inline Patchdiff --git a/perly.y b/perly.y
index 7407b21..ed844d9 100644
--- a/perly.y
+++ b/perly.y
@@ -448,7 +448,6 @@ barestmt: PLUGSTMT
/* a block is a loop that happens once */
$$ = newWHILEOP(0, 1, (LOOP*)(OP*)NULL,
(OP*)NULL, $1, $2, 0);
- PL_parser->copline = NOLINE;
}
| PACKAGE WORD WORD '{' remember
{
Looking at related constructs, there's a change to the line number The nature of the changes in eae48c8 render it relatively at risk -zefram |
From @cpansproutOn Sat Nov 06 09:26:43 2010, zefram@fysh.org wrote:
Thank you. Applied as 166f120.
Tim Bunce’s tests from |
@cpansprout - Status changed from 'open' to 'resolved' |
From @cpansproutTim Bunce said:
Thank you. Applied, with some adjustments, as fc6b470. |
Migrated from rt.perl.org#78586 (status was 'resolved')
Searchable as RT78586$
The text was updated successfully, but these errors were encountered: