Re: pdd19 comments

2007-11-05 Thread François PERRAD
At 17:18 02/11/2007 -0400, you wrote: François PERRAD wrote: just few comments about pdd19 : 1) Directive .namespace .namespace is currently valid, and I understand its behavior as : back to the default It's in the docs for IMCC (docs/imcc/calling_conventions.pod), but not in the calling

Re: [perl #46727] [TODO] config/auto/ctags.pm: Write unit tests

2007-11-05 Thread James E Keenan
On Nov 4, 2007, at 11:06 PM, Paul Cochrane via RT wrote: kid51, On 05/11/2007, James Keenan via RT [EMAIL PROTECTED]> wrote: The patch attached refactors configuration step auto::ctags to maximize testability. It also provides 3 test files to replace ptc's original test file. ptc's origin

[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

Parrot Bug Summary

2007-11-05 Thread Parrot Bug Summary
Parrot Bug Summary http://rt.perl.org/rt3/NoAuth/parrot/Overview.html Generated at Mon Nov 5 14:00:02 2007 GMT --- * Numbers * New Issues * Overview of Open Issues * Ticket Status By Version * Requestors with mo

[perl #43299] [TODO] config/auto/gc.pm: Write unit tests

2007-11-05 Thread James Keenan via RT
The way configuration step auto::gc's runstep() method is structured strikes me as peculiar. Can I get some second opinions? Starts out ordinarily enough (I've eliminated some whitespace and comments): if ( !defined($gc) ) { $gc = 'gc'; } elsif ( $gc eq 'libc' ) { if

[perl #43345] [TODO] config/auto/alignptrs.pm: Write unit tests

2007-11-05 Thread James Keenan via RT
Patches committed to trunk in r22731. Resolving ticket.

[perl #43307] [TODO] config/auto/aio.pm: Write unit tests

2007-11-05 Thread James Keenan via RT
Here's another case where I don't understand the logic of a particular configuration step class's runstep() method: auto::aio. (In what follows, I eliminate whitespace, comments and references to verbosity where appropriate.) my $test; my $libs= $conf->data->get('libs'); $conf->d

Re: pdd19 comments

2007-11-05 Thread Bob Rogers
From: François PERRAD <[EMAIL PROTECTED]> Date: Mon, 05 Nov 2007 12:06:39 +0100 >>2) Directive .const with empty string I think it's bug too, I don't understand the limitation : everything except empty string Francois Definitely a bug; the attached patch seems to fix it. If

Re: pdd19 comments

2007-11-05 Thread Bob Rogers
From: Bob Rogers <[EMAIL PROTECTED]> Date: Mon, 5 Nov 2007 22:00:10 -0500 From: François PERRAD <[EMAIL PROTECTED]> Date: Mon, 05 Nov 2007 12:06:39 +0100 >>2) Directive .const with empty string I think it's bug too, I don't understand the limitation : everything ex

Re: pdd19 comments

2007-11-05 Thread Bob Rogers
From: Bob Rogers <[EMAIL PROTECTED]> Date: Mon, 5 Nov 2007 22:05:16 -0500 From: Bob Rogers <[EMAIL PROTECTED]> Date: Mon, 5 Nov 2007 22:00:10 -0500 Definitely a bug; the attached patch seems to fix it. If you would kindly confirm that it works, I'll add a proper tes