Author: rgrjr
Date: Tue Dec 26 22:15:08 2006
New Revision: 16271
Modified:
trunk/docs/pdds/pdd13_bytecode.pod
Log:
Fix minor typos.
Modified: trunk/docs/pdds/pdd13_bytecode.pod
==
--- trunk/docs/pdds/pdd13_bytecode.p
From: chromatic <[EMAIL PROTECTED]>
Date: Tue, 26 Dec 2006 16:26:03 -0800
On Tuesday 26 December 2006 14:35, Bob Rogers wrote:
> AFAICS, the old code failed because Parrot_switch_to_cs (which is what
> normally sets ctx->contants in a sub call) is not given a chance to run
> in
On Tuesday 26 December 2006 14:35, Bob Rogers wrote:
> AFAICS, the old code failed because Parrot_switch_to_cs (which is what
> normally sets ctx->contants in a sub call) is not given a chance to run
> in this situation. Parrot_loadbc sets interp->code without doing any of
> the other Parrot_swit
From: Jonathan Worthington <[EMAIL PROTECTED]>
Date: Tue, 26 Dec 2006 19:36:58 +
Nikolay Ananiev wrote:
> The attached patch fixes the problem. The code is taken from
> Parrot_call_sub (which works). But I don't know why it was not
> applied to the other two function. Maybe
On Mon Dec 25 22:14:18 2006, [EMAIL PROTECTED] wrote:
> A few style comments here.
> [snip]
>
> > sub dump_vtable {
> > my $self = shift;
> > my $file = shift;
>
> Why two shifts here, when @_ goes unused through the rest of the method?
Fixed in attached patch: Utils.pm.
>
> > s
Jonathan Worthington wrote:
I'd like to poke into this at bit deeper at the moment,
Of course I meant, "in the future". Oh, and it was r16270 that it went
in as.
Don't drink and patch!
Jonathan
Nikolay Ananiev wrote:
The attached patch fixes the problem. The code is taken from Parrot_call_sub
(which works). But I don't know why it was not applied to the other two
function. Maybe it's just a hack?
Perhaps, but I have applied it since it's consistent with
Parrot_call_sub. I'd like t
Lee Duhem wrote:
The attached patch of the first message have some no-ASCII characters,
this patch fix that. Otherwise, there are same.
Thanks, applied in r16269 and closed the related RT ticket.
Jonathan
# New Ticket Created by "Lee Duhem"
# Please include the string: [perl #41139]
# in the subject line of all future correspondence about this issue.
# http://rt.perl.org/rt3/Ticket/Display.html?id=41139 >
The attached patch of the first message have some no-ASCII characters,
this patch fix tha
# New Ticket Created by "Lee Duhem"
# Please include the string: [perl #41140]
# in the subject line of all future correspondence about this issue.
# http://rt.perl.org/rt3/Ticket/Display.html?id=41140 >
Sorry, forget the patch...
Index: src/packfile.c
Sorry, forget the patch...
Index: src/packfile.c
===
--- src/packfile.c (revision 16268)
+++ src/packfile.c (working copy)
@@ -2187,7 +2187,7 @@
default_dump_header(interp, self);
-PIO_printf(interp, "mappings => [\n");
+
The attached patch of the first message have some no-ASCII characters,
this patch fix that. Otherwise, there are same.
James Keenan via RT wrote:
On Sun Dec 03 14:08:31 2006, [EMAIL PROTECTED] wrote:
The files attach refactor tools/build/pmc2c.pl, extracting most of its
functionality into lib/
Parrot/Pmc2c/Utils.pm and provide 8 test files to be stored in t/tools/pmc2cutils/. These
substitute for my fi
James Keenan via RT wrote:
On Sun Dec 03 14:08:31 2006, [EMAIL PROTECTED] wrote:
The files attach refactor tools/build/pmc2c.pl, extracting most of its
functionality into lib/
Parrot/Pmc2c/Utils.pm and provide 8 test files to be stored in t/tools/pmc2cutils/. These
substitute for my fi
chromatic wrote:
On Monday 25 December 2006 21:44, James Keenan via RT wrote:
A few style comments here.
Utils.pm
unshift @{$allargsref->{include}}, (
".",
"$FindBin::Bin/../..",
"$FindBin::Bin/../../src/pmc/"
);
Why no File::Spec here?
There are certain ways i
"Nikolay Ananiev" <[EMAIL PROTECTED]> wrote in message
news:[EMAIL PROTECTED]
> forgot to attach the files. Attached in this message.
The attached patch fixes the problem. The code is taken from Parrot_call_sub
(which works).
But I don't know why it was not applied to the other two function. Maybe
16 matches
Mail list logo
