Re: Coverage analysis for tests of configuration and build tools

2007-05-10 Thread James E Keenan
Steve Peters wrote: On Thu, May 10, 2007 at 04:50:56AM -0400, James E Keenan wrote: The results of a recent run of Devel::Cover along with the tests run with the patch submitted in http://rt.perl.org/rt3/Ticket/Display.html?id=42690 may be seen here: http://thenceforward.net/parrot/coverage/co

Re: [svn:parrot] r18381 - trunk/include/parrot

2007-05-10 Thread jerry gay
On 5/10/07, Nicholas Clark <[EMAIL PROTECTED]> wrote: On Thu, May 10, 2007 at 03:33:41AM -0500, Joshua Isom wrote: > > On May 9, 2007, at 4:01 PM, Nicholas Clark wrote: > >So, !s->strlen does scan as quickly and easily. > > > > To some, but it isn't as easy to just literally read. "Not s's strl

Re: [svn:parrot] r18381 - trunk/include/parrot

2007-05-10 Thread Nicholas Clark
On Thu, May 10, 2007 at 03:33:41AM -0500, Joshua Isom wrote: > > On May 9, 2007, at 4:01 PM, Nicholas Clark wrote: > >So, !s->strlen does scan as quickly and easily. > > > > To some, but it isn't as easy to just literally read. "Not s's strlen" > is a lot different than "STRING_IS_EMTPY". Sin

Re: [perl #42774] [PATCH] Configure.pl --gc=libc doesn't compile

2007-05-10 Thread Andrew Dougherty
On Wed, 9 May 2007, Andy Dougherty wrote: > Ok, I've got it compiling, at least. I'll let it run tests overnight and > submit a cleaned-up patch tomorrow. I've gone back and reviewed the > alignment issue and think I have implemented it sanely. Hmm. Rats. My initial version required more in

Re: [perl #42918] [PATCH] src/library.c use mem_sys_free instead of free()

2007-05-10 Thread chromatic
On Thursday 10 May 2007 02:37:15 Mike Mattie wrote: > quick patch to use the mem_sys_free wrapper instead of using the platform's > free() directly. I only see an OpenPGP signature attachment; can you respond with the patch attached again (or inline, I suppose). If it's a small change, I could

Re: Coverage analysis for tests of configuration and build tools

2007-05-10 Thread Steve Peters
On Thu, May 10, 2007 at 04:50:56AM -0400, James E Keenan wrote: > The results of a recent run of Devel::Cover along with the tests run > with the patch submitted in > http://rt.perl.org/rt3/Ticket/Display.html?id=42690 may be seen here: > http://thenceforward.net/parrot/coverage/configure-build/c

[perl #42918] [PATCH] src/library.c use mem_sys_free instead of free()

2007-05-10 Thread via RT
# New Ticket Created by Mike Mattie # Please include the string: [perl #42918] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=42918 > Hello, quick patch to use the mem_sys_free wrapper instead of using the platform's free

Re: [perl #42905] [PATCH] implement vtable overriding for PDD15, bugfix

2007-05-10 Thread Alek Storm
On 5/8/07, Allison Randal <[EMAIL PROTECTED]> wrote: I actually changed every instance of 'vtable method' to 'vtable function' a few days ago to help clear up the confusion. (A vtable function does have an invocant 'self' but doesn't act as a method in any other way.) But, while I was adding the

[svn:parrot-pdd] r18493 - trunk/docs/pdds/draft

2007-05-10 Thread smash
Author: smash Date: Thu May 10 04:15:31 2007 New Revision: 18493 Modified: trunk/docs/pdds/draft/pdd17_pmc.pod Log: [PDD] * fix pdd17_pmc.pod pod errors Modified: trunk/docs/pdds/draft/pdd17_pmc.pod == --- trunk/doc

Coverage analysis for tests of configuration and build tools

2007-05-10 Thread James E Keenan
The results of a recent run of Devel::Cover along with the tests run with the patch submitted in http://rt.perl.org/rt3/Ticket/Display.html?id=42690 may be seen here: http://thenceforward.net/parrot/coverage/configure-build/coverage.html kid51

Re: [svn:parrot] r18381 - trunk/include/parrot

2007-05-10 Thread Joshua Isom
On May 9, 2007, at 4:01 PM, Nicholas Clark wrote: On Wed, May 09, 2007 at 01:06:49PM -0700, chromatic wrote: On Wednesday 09 May 2007 12:53:57 Nicholas Clark wrote: On Tue, May 01, 2007 at 04:41:22PM -0700, [EMAIL PROTECTED] wrote: + +#define STRING_IS_NULL(s) ((s) == NULL) +#define STRING_