Skip Menu |

This queue is for tickets about the Parse-RecDescent CPAN distribution.

Report information
The Basics
Id: 53820
Status: resolved
Priority: 0/
Queue: Parse-RecDescent
People
Owner: Nobody in particular
Requestors: OHA [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 1.962.2
Fixed in: (no value)

History Show all quoted text
  Tue Jan 19 09:14:46 2010 OHA [...] cpan.org - Ticket created
the <leftop a /(\+|-)/ a> no longer returns what got matched in the inner regexp, so the ARRAYREF contains only the operands and not the operator in the above case.
  Tue Jan 19 09:49:08 2010 OHA [...] cpan.org - Subject changed from (no value) to '<leftop: a /(-)/ a>'
  Tue Jan 19 09:54:55 2010 OHA [...] cpan.org - Correspondence added
maybe it is a consequence of #49039 so if <leftop: a /(-)/ a> the operator must be returned and if <leftop: a /-/ a> the operator must not be returned as the documentation claim
  Tue Jan 19 09:54:56 2010 OHA [...] cpan.org - Status changed from 'new' to 'open'
  Tue Jan 19 23:43:26 2010 damian [...] conway.org - Correspondence added
Subject: Re: [rt.cpan.org #53820]
Date: Wed, 20 Jan 2010 15:42:41 +1100
To: bug-Parse-RecDescent [...] rt.cpan.org
From: Damian Conway <damian [...] conway.org>
Show quoted text
> the <leftop a /(\+|-)/ a> no longer returns what got matched in the > inner regexp, so the ARRAYREF contains only the operands and not the > operator in the above case.
This is a definite problem. I believe the attached version of RecDescent.pm corrects the error. Please test it and confirm. Thanks for the report, Francesco! Damian

Message body is not shown because sender requested not to inline it.

  Wed Jan 20 04:42:37 2010 OHA [...] cpan.org - Correspondence added
On Tue Jan 19 23:43:26 2010, damian@conway.org wrote: Show quoted text
> > the <leftop a /(\+|-)/ a> no longer returns what got matched in the > > inner regexp, so the ARRAYREF contains only the operands and not the > > operator in the above case.
> > This is a definite problem. I believe the attached version of
RecDescent.pm Show quoted text
> corrects the error. Please test it and confirm.
I confirm this is fixed and i was not able to find any issues. Show quoted text
> Thanks for the report, Francesco!
Thanks for the fix! Oha
  Wed Jan 20 17:05:37 2010 damian [...] conway.org - Correspondence added
Subject: Re: [rt.cpan.org #53820] <leftop: a /(-)/ a>
Date: Thu, 21 Jan 2010 09:04:54 +1100
To: bug-Parse-RecDescent [...] rt.cpan.org
From: Damian Conway <damian [...] conway.org>
Show quoted text
> I confirm this is fixed and i was not able to find any issues.
Excellent. Thank-you. I'll upload the update to CPAN this morning. Damian
  Sun Jan 22 23:13:33 2012 jtbraun [...] CPAN.org - Correspondence added
Issue appears to have been resolved by a previous release according to the notes in the ticket.
  Sun Jan 22 23:13:34 2012 jtbraun [...] CPAN.org - Status changed from 'open' to 'resolved'