RE: [perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread Brent Dax
Andy Dougherty: # You'll probably also want the enhanced sprintf tests (or # something like # them.) Currently, the only one that fails for me is the # %5.3f test, which gives 0.500 in stdio, but # 0.5 from Parrot. I finally tracked down this bug. If you look,

Re: the getting started guide

2002-10-09 Thread Dan Sugalski
At 10:07 PM -0700 10/9/02, Robert Spier wrote: > >I'll see about getting some of the internal structures diagrammed >>better, which is the only place things are a little dodgy, but that's >>otherwise fine. > >Simon Wistow <[EMAIL PROTECTED]> dropped me an email and reminded >me about AutoDia http

Warnings on VC++

2002-10-09 Thread Brent Dax
core_ops.c(19) : warning C4005: 'CONST' : macro redefinition C:\Program Files\Microsoft Visual Studio ..NET\Vc7\PlatformSDK\Include\WinDef.h(138) : see previous definition of 'CONST' jit_cpu.c(20) : warning C4005: 'CONST' : macro redefinition C:\Program Files\Microsoft Visual Stud

Re: the getting started guide

2002-10-09 Thread Robert Spier
>>So the html is canonical and you're converting it to pod? I'm >> confused. >I am writing it in my own format. It lets me take notes, write todo >lists, autogenerate the table of contents and glossary. Then I run it >through some perl that currently spits out html and almost make pod. Whatev

Re: the getting started guide

2002-10-09 Thread Robert Spier
>I'll see about getting some of the internal structures diagrammed >better, which is the only place things are a little dodgy, but that's >otherwise fine. Simon Wistow <[EMAIL PROTECTED]> dropped me an email and reminded me about AutoDia http://droogs.org/autodia/ >Now, here's hoping CVS doesn't

Re: the getting started guide

2002-10-09 Thread Erik Lechak
Robert Spier wrote: >>This version is in html. I almost have the pod version ready. >> >> > >So the html is canonical and you're converting it to pod? I'm >confused. > > I am writing it in my own format. It lets me take notes, write todo lists, autogenerate the table of contents and g

Re: [perl #17803] [PATCH] Various tests

2002-10-09 Thread Dan Sugalski
At 7:42 PM -0700 10/8/02, Steve Fink wrote: >Thanks, applied. > >Who came up with the idea of two-argument ne, anyway? That's kind of >bizarre. Definitely bizarre. I think I'd rather not have it, it doesn't make much sense. -- Dan --

Re: the getting started guide

2002-10-09 Thread Dan Sugalski
At 11:28 PM -0400 10/8/02, Erik Lechak wrote: >Well here it is, my first attempt at a "getting started" guide. >It's not ready for public consumption, but I would like to hear some >feedback. The later sections are blank or contain notes in a not >nice format. As time moves on and I learn mor

Re: Eliminate padding warnings

2002-10-09 Thread Tom Hughes
In message <[EMAIL PROTECTED]> Simon Glover <[EMAIL PROTECTED]> wrote: > I've just had a quick look at hash.h (which I should have done in the > first place) and you're quite right. Second attempt at a correct patch > below. > > All tests still pass, but this isn't much comfort, as

Re: Eliminate padding warnings

2002-10-09 Thread Simon Glover
On Wed, 9 Oct 2002, Tom Hughes wrote: > In message <[EMAIL PROTECTED]> > Simon Glover <[EMAIL PROTECTED]> wrote: > > > This one happens because entry is a HASH_ENTRY*, but get_pmc_keyed is > > expecting a PMC*. However, by this point in the function, we've already > > verified that

Re: [perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread Nicholas Clark
On Wed, Oct 09, 2002 at 04:42:36PM -0400, Andy Dougherty wrote: > On Wed, 9 Oct 2002, Andy Dougherty wrote: > > > diff -r -u parrot-orig/t/src/sprintf.t parrot-andy/t/src/sprintf.t > > > + dval = 1.0e6; > > + S = Parrot_sprintf_c(interpreter, "== %g\n", dval); > > +

Re: Eliminate padding warnings

2002-10-09 Thread Tom Hughes
In message <[EMAIL PROTECTED]> Simon Glover <[EMAIL PROTECTED]> wrote: > This one happens because entry is a HASH_ENTRY*, but get_pmc_keyed is > expecting a PMC*. However, by this point in the function, we've already > verified that entry is actually a PMC*, so it should be safe to a

Re: Eliminate padding warnings

2002-10-09 Thread Simon Glover
On Wed, 9 Oct 2002, Nicholas Clark wrote: > On Wed, Oct 09, 2002 at 02:14:50AM -0400, Tanton Gibbs wrote: > > In order, the other significant compiler warnings I see are: > > perlhash.pmc: In function `Parrot_PerlHash_get_pmc_keyed': > perlhash.pmc:192: warning: passing arg 2 of pointer to funct

Re: Eliminate padding warnings

2002-10-09 Thread Nicholas Clark
On Wed, Oct 09, 2002 at 02:14:50AM -0400, Tanton Gibbs wrote: > There were a number of warnings which read something like > "structure padded for alignment of member value in debug.h" > > This can be trivially fixed by reordering the structure members ( I hope). > This patch works fine on cygwin,

RE: [perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread Andy Dougherty
On Wed, 9 Oct 2002, Andy Dougherty wrote: > diff -r -u parrot-orig/t/src/sprintf.t parrot-andy/t/src/sprintf.t > + dval = 1.0e6; > + S = Parrot_sprintf_c(interpreter, "== %g\n", dval); > + printf("%g %s", dval, string_to_cstring(interpreter, S)); Aargh. I fo

RE: [perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread Andy Dougherty
On Wed, 9 Oct 2002, Andy Dougherty wrote: > > # Unfortunately, the compact-looking > > # > > # ch=va_arg((va_list)obj->data, char); > > # > > # gave Sun's compiler indigestion. > Hmm. Just removing the (va_list) cast makes both Sun's compiler and > gcc happy. (That's because va_list is

RE: [perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread Andy Dougherty
On Wed, 9 Oct 2002, Brent Dax wrote: > Andy Dougherty: > # Overall, it continues to look very good. Unfortunately, the > # compact-looking > # > # ch=va_arg((va_list)obj->data, char); > # > # gave Sun's compiler indigestion. I had to split it up into > # the much more pedestrian > # >

Re: the getting started guide

2002-10-09 Thread David Wheeler
On Wednesday, October 9, 2002, at 08:49 AM, Erik Lechak wrote: >> Oh, and spell Piers Cawley's name correctly! :-) >> > > DAMN! I new that would happen. Piers, if your out there listening, > forgive me. Did you misspell it a "Bunch" of times? (Sorry Piers, couldn't resist!) David -- Davi

RE: [perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread Brent Dax
Andy Dougherty: # Overall, it continues to look very good. Unfortunately, the # compact-looking # # ch=va_arg((va_list)obj->data, char); # # gave Sun's compiler indigestion. I had to split it up into # the much more pedestrian # # va_list arg; # arg = (va_list) obj->data; #

Re: the getting started guide

2002-10-09 Thread Erik Lechak
Steve Fink wrote: > >Oh, and spell Piers Cawley's name correctly! :-) > > DAMN! I new that would happen. Piers, if your out there listening, forgive me. Erik

Re: [perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread Andy Dougherty
On Wed, 9 Oct 2002, Brent Dax wrote: [Brent continues to churn out amazing quantities of good stuff] Overall, it continues to look very good. Unfortunately, the compact-looking ch=va_arg((va_list)obj->data, char); gave Sun's compiler indigestion. I had to split it up into the much more p

[perl #17817] [PATCH] Parrot_sprintf-related stuff, part 2

2002-10-09 Thread via RT
# New Ticket Created by Brent Dax # Please include the string: [perl #17817] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt2/Ticket/Display.html?id=17817 > Inspiration struck me as I was working on bug fixes for Parrot_sprintf's patch yesterday.