Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-06-07 Thread chromatic
On Tuesday 03 June 2008 13:55:29 NotFound wrote: > Looking more carefully at this issue, it seems that those variables > and the code that uses them has no real effect. Without it, make test > pass, make testj pass, make hello and make perl6 builds and runs. > > This patch cleans all. It needs to

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-06-04 Thread François Perrad
NotFound a écrit : Looking more carefully at this issue, it seems that those variables and the code that uses them has no real effect. Without it, make test pass, make testj pass, make hello and make perl6 builds and runs. This patch cleans all. It needs to be tested with make testj in win 32,

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-06-04 Thread François Perrad
2008/5/26 NotFound <[EMAIL PROTECTED]>: > Looks like exec_start.c include jit.h and jit_emit.h but doen't use > it. This patch drops those includes and solve the problem, at least in > i386. > I experiment this patch (on src/exec_start.c). On Windows, with MinGW, r28042 : - linking : OK - but e

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-06-03 Thread NotFound
Looking more carefully at this issue, it seems that those variables and the code that uses them has no real effect. Without it, make test pass, make testj pass, make hello and make perl6 builds and runs. This patch cleans all. It needs to be tested with make testj in win 32, and for completeness i

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-25 Thread François Perrad
2008/5/24 chromatic <[EMAIL PROTECTED]>: > On Friday 23 May 2008 04:06:45 NotFound wrote: > > > Looking better at the Parrot_exec_rel_... vars, they are not intended > > to be exported at all, they are globals for private usage between exec > > and jit, so they don't need the API or DATA things. T

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-25 Thread chromatic
On Sunday 25 May 2008 02:31:44 François Perrad wrote: > on Windows, since r27785, the command 'make hello' can't link. > > Info: resolving _op_jit by linking to __imp__op_jit (auto-import) > src\exec_start.o: In function `emit_disp8_32': > D:/fperrad/Parrot/trunk/src/jit_emit.h:171: undefined refe

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-24 Thread chromatic
On Friday 23 May 2008 04:06:45 NotFound wrote: > Looking better at the Parrot_exec_rel_... vars, they are not intended > to be exported at all, they are globals for private usage between exec > and jit, so they don't need the API or DATA things. This patch cleans > it, make sure they are defined o

[perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-23 Thread Mark Glines via RT
On Fri May 23 07:57:11 2008, julianalbo wrote: > Reopened waiting for testing and commiting the patch to sove the problems. All tests successful on linux/amd64. Mark

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-23 Thread NotFound
Looking better at the Parrot_exec_rel_... vars, they are not intended to be exported at all, they are globals for private usage between exec and jit, so they don't need the API or DATA things. This patch cleans it, make sure they are defined only in jit.c and declared as extern in the others, and a

[perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-23 Thread Seneca Cunningham via RT
On Thu May 22 21:08:01 2008, [EMAIL PROTECTED] wrote: > On Wednesday 21 May 2008 13:04:29 NotFound wrote: > > Changed several casts and removed 'extern' is several places in > > congruence with the changed meaning of the macros. > > Hi, > > Sounds like this part of the patch has some portability

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-22 Thread Mark Glines
On Wednesday 21 May 2008 13:04:29 NotFound wrote: > Changed several casts and removed 'extern' is several places in > congruence with the changed meaning of the macros. Hi, Sounds like this part of the patch has some portability problems. [20:19] <@tetragon> Hrm... exec.c lost the extern on "cha

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-21 Thread chromatic
On Wednesday 21 May 2008 15:07:00 NotFound wrote: > Now it passes allmost all tests. A problem is the PIO_eprintf > function, that is not in any externally usable header. > > There is a coding style fail with the #if indenter check, I'm unable > to locate that problem. I fixed that one. > Fixed

Re: [perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-21 Thread NotFound
Updated patch: Now it passes allmost all tests. A problem is the PIO_eprintf function, that is not in any externally usable header. There is a coding style fail with the #if indenter check, I'm unable to locate that problem. Fixed the perl6nool.pmc, that used a variable named "class". -- Salu2

[perl #54602] [PATCH] Several changes to allow C++ compiling

2008-05-21 Thread via RT
# New Ticket Created by NotFound # Please include the string: [perl #54602] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=54602 > I've been working on several problems that breaks compiling parrot with C++. This patch solv