[perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2008-08-01 Thread Will Coleda via RT
On Fri Jul 25 05:32:49 2008, coke wrote: > On Fri, Jul 25, 2008 at 2:25 AM, Christoph Otto via RT > <[EMAIL PROTECTED]> wrote: > > On Thu Jun 05 19:07:49 2008, coke wrote: > > > >> We can always improve the diagnostic emitted by the PMC compiler. > >> Mismatched strings are going to be an issue whe

[perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2008-07-25 Thread Christoph Otto via RT
On Thu Jun 05 19:07:49 2008, coke wrote: > We can always improve the diagnostic emitted by the PMC compiler. > Mismatched strings are going to be an issue whether they're in a > CONST_STRING declaration or just an assignment to char *. > > So, no, it's not worth fixing up c2str.pl, IMO. So if it

Re: [perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2008-07-25 Thread Will Coleda
On Fri, Jul 25, 2008 at 2:25 AM, Christoph Otto via RT <[EMAIL PROTECTED]> wrote: > On Thu Jun 05 19:07:49 2008, coke wrote: > >> We can always improve the diagnostic emitted by the PMC compiler. >> Mismatched strings are going to be an issue whether they're in a >> CONST_STRING declaration or just

Re: [perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2008-06-05 Thread Will Coleda
On Thu, Jun 5, 2008 at 9:28 PM, chromatic <[EMAIL PROTECTED]> wrote: > On Thursday 05 June 2008 18:19:21 Will Coleda via RT wrote: > >> I have a counter-patch: >> >> Index: tools/build/c2str.pl >> = >> == >> --- tools/build/c2str.pl

Re: [perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2008-06-05 Thread chromatic
On Thursday 05 June 2008 18:19:21 Will Coleda via RT wrote: > I have a counter-patch: > > Index: tools/build/c2str.pl > = > == > --- tools/build/c2str.pl(revision 28124) > +++ tools/build/c2str.pl(working copy) > @@ -1

[perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2008-06-05 Thread Will Coleda via RT
On Wed Jan 02 11:19:02 2008, szbalint wrote: > On Thu Oct 25 08:45:09 2007, ptc wrote: > > In tools/build/c2str.pl there is the todo item: > > > > # TODO maybe cope with escaped \" > > > > Do this. > > I've attached a patch that should complete this TODO. > > The previous attempt failed becaus

[perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2007-11-05 Thread James Keenan via RT
On Thu Nov 01 23:50:36 2007, magnachef wrote: > I have, IMO, finished this TODO. I am not sure who is responsible for > commiting this to the repository. > > I have included the diff below: > > [EMAIL PROTECTED] build]$ diff -U0 c2str.pl c2str.pl.escaped > --- c2str.pl2007-10-30 23:43:57.000

[perl #46909] [TODO] [Perl] Cope with escaped quotes in tools/build/c2str.pl

2007-10-25 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #46909] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46909 > In tools/build/c2str.pl there is the todo item: # TODO maybe cope with escaped \" Do