Re: [perl #41908] [PATCH] 5 feature enhancement of src/library.c (extension guessing), and solutions for the install target problems

2007-05-07 Thread Paul Cochrane
Mike, I don't remember it breaking win32. As I remember it, it was committed the day before 0.4.10 , which wasn't appropriate for a patch like this. Please accept my apologies, you're correct it didn't break anything, just wasn't appropriate so close to a release. Paul

Re: [perl #41908] [PATCH] 5 feature enhancement of src/library.c (extension guessing), and solutions for the install target problems

2007-05-07 Thread Paul Cochrane
Mike, I noticed that the patch was generated before I had turned off the tabs in my editor. I have rebased the patch against 18443 and double checked it for any tab issues. passed full test harness on i686-pc-gnu-linux. Many thanks for rebasing the patch! I've applied it as r18446, thanks :-)

[PATCH] refactor Parrot_get_runtime_prefix, more simple,robust

2007-05-07 Thread Mike Mattie
Hello, I was looking at Parrot_get_runtime_prefix as an example of getting environment variable values the parrot way. The function looked over-complicated so I tried a refactor. The first thing to notice about the function is that it is possible to leak memory with a call. I did not find any lea

Re: [PATCH] refactor Parrot_get_runtime_prefix, more simple,robust

2007-05-07 Thread Mike Mattie
On Mon, 7 May 2007 01:24:32 -0700 Mike Mattie <[EMAIL PROTECTED]> wrote: > Hello, > > I was looking at Parrot_get_runtime_prefix as an example of getting > environment variable values the parrot way. The function looked > over-complicated so I tried a refactor. [snip] > > I am inlining the pa

[perl #42897] [PATCH] spelling fixes in src/library.c

2007-05-07 Thread via RT
# New Ticket Created by Mike Mattie # Please include the string: [perl #42897] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=42897 > A couple of minor spelling fixes in library.c , patch attached --- HEAD/src/library.c 200

Re: [perl #41908] [PATCH] 5 feature enhancement of src/library.c (extension guessing), and solutions for the install target problems

2007-05-07 Thread Mike Mattie
On Mon, 30 Apr 2007 12:22:21 -0700 "Matt Diephouse via RT" <[EMAIL PROTECTED]> wrote: > On Wed Mar 21 03:44:10 2007, codermattie wrote: > > Hello, > > [snip] > I'd like to get this ticket (#41908) resolved. The patch was applied, > so everything is good there, but your reply here has left me > wo

Re: [perl #42897] [PATCH] spelling fixes in src/library.c

2007-05-07 Thread Paul Cochrane
A couple of minor spelling fixes in library.c , patch attached Applied in r18447, thanks!

Re: [perl #42381] [TODO] search library path for parrot bytecode when loading

2007-05-07 Thread Mike Mattie
On Mon, 09 Apr 2007 01:33:14 -0700 Paul Cochrane (via RT) <[EMAIL PROTECTED]> wrote: > # New Ticket Created by Paul Cochrane > # Please include the string: [perl #42381] > # in the subject line of all future correspondence about this issue. > # http://rt.perl.org/rt3/Ticket/Display.html?id=423

[perl #42899] [PATCH] t/codingstd/tabs.t improve diagnostic output

2007-05-07 Thread via RT
# New Ticket Created by Mike Mattie # Please include the string: [perl #42899] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=42899 > Hello, My version of Emacs appears to be leaking tabs into files despite having this in

[perl #42900] [PATCH] src/packfile.c Parrot_load_bytecode, improve error message

2007-05-07 Thread via RT
# New Ticket Created by Mike Mattie # Please include the string: [perl #42900] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=42900 > Hello, I noticed while debugging my patches to the loader that the message for a failur

[perl #42898] [PATCH] src/library.c , honor PARROT_PREFER_SOURCE in extension guessing

2007-05-07 Thread via RT
# New Ticket Created by Mike Mattie # Please include the string: [perl #42898] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=42898 > hello, Building on commit #18446 this patch honors PARROT_PREFER_SOURCE. quick backgrou

[perl #42897] [PATCH] spelling fixes in src/library.c

2007-05-07 Thread James Keenan via RT
Applied to trunk in r18447.

Parrot Bug Summary

2007-05-07 Thread Parrot Bug Summary
Parrot Bug Summary http://rt.perl.org/rt3/NoAuth/parrot/Overview.html Generated at Mon May 7 13:00:03 2007 GMT --- * Numbers * New Issues * Overview of Open Issues * Ticket Status By Version * Requestors with mo

[perl #42899] [PATCH] t/codingstd/tabs.t improve diagnostic output

2007-05-07 Thread Paul Cochrane via RT
On Mon May 07 04:40:24 2007, codermattie wrote: > Hello, > > My version of Emacs appears to be leaking tabs into files despite > having this in my emacs config: > > (setq indent-tabs-mode nil) > > To help track down the problem in emacs/cperl and get it fixed I need > better diagnostics. I have

[perl #42900] [PATCH] src/packfile.c Parrot_load_bytecode, improve error message

2007-05-07 Thread Paul Cochrane via RT
On Mon May 07 06:49:42 2007, codermattie wrote: > Hello, > > I noticed while debugging my patches to the loader that the message >for a failure in .load_bytecode was really > horrible. It was something like this: > > immc: Couldn't find file %s > > At the time I didn't know what immc was, an

[perl #41908] [PATCH] 5 feature enhancement of src/library.c (extension guessing), and solutions for the install target problems

2007-05-07 Thread Paul Cochrane via RT
On Mon Apr 30 12:22:21 2007, mdiep wrote: > On Wed Mar 21 03:44:10 2007, codermattie wrote: > > Hello, > > > > now that 0.4.10 has released I would like to merge rt# 41908 which > > introduces extension guessing. > > The current implementation does not affect the behavior of existing > > code. No n

[perl #42901] Solutions for the install target problems (was RT#41908)

2007-05-07 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #42901] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=42901 > I'm splitting RT#41908 into two pieces, one which is a now-applied patch for extension

Re: [perl #42883] [PATCH] Fix up headerfile guards

2007-05-07 Thread Bernhard Schmalhofer
Mark Glines schrieb: * Fix up the indentation problems caused by the above bullet point. (TODO: why does t/codingstd/c_indent.t require indentation for #if and #ifdef, but not for #ifndef?) As far as I understand c_indent.t, #ifndef is also covered Except that it isn't. If you app

[perl #41583] [PATCH] Tail calls from within vtable methods broken

2007-05-07 Thread Bram Geron
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 This patch copies the context to a new interp->fromc_result_ctx field on a set_returns. As far as I know no new tests fail with this. This patch retains most side-patching (comments etc.) by Jonathan. It also tests in Parrot_free_context whether you

Re: [perl #41583] [PATCH] Tail calls from within vtable methods broken

2007-05-07 Thread Alek Storm
On 5/7/07, Bram Geron <[EMAIL PROTECTED]> wrote: Alek, I made the bulk of the patch before I dug through the other mails and saw your mail. I'm sorry if you were still planning on sending it. Don't feel sorry at all. I got halfway through the patch before I got distracted by other things.