Camelia, the Perl 6 bug

IRC log for #parrot, 2008-04-22

Parrot | source cross referenced

| Channels | #parrot index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
07:48 irclogbot joined #parrot
07:48 Topic for #parrotis now #parrot Parrot 0.6.1 "Bird of Paradise" Released | http://parrotcode.org
07:55 moritz joined #parrot
08:16 Zaba_ joined #parrot
08:32 ruoso joined #parrot
08:40 IllvilJa joined #parrot
08:40 mj41 joined #parrot
08:46 Zaba joined #parrot
09:13 wknight8111 joined #parrot
09:26 Psyche^ joined #parrot
09:58 dalek r27115 | fperrad++ | trunk:
09:58 dalek : [gettext]
09:58 dalek : - fix build on MinGW32
09:58 dalek : - more consistent with gmp.pm and others
09:58 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27115
10:33 kid51 joined #parrot
10:34 kid51 Does anyone know what packages are needed to install OpenGL on Debian Linux?
10:34 Zaba what do you mean by installing OpenGL?
10:36 kid51 Installing it such that configuration steps auto::opengl and gen::opengl detect it and work it into Parrot -- as we've been doing the past several days.
10:37 kid51 We established that OpenGL "comes with" Mac OS X -- so I've used it successfully on my iBook.  But I tried to do so yesterday on an Ubuntu box and failed to get to the spinning triangle.
10:38 kid51 When I did apt-cache search opengl, many selections came up but it wasn't entirely clear what packages I needed to add for this purpose.
10:38 Zaba you need opengl headers, I think
10:39 Zaba well and of course drivers appropriate for hardware which provide libGL, which you most likely have already.
10:41 Zaba /bin/sh: line 1:  6886 Segmentation fault      ./miniparrot config_lib.pasm > runtime/parrot/include/config.fpmc
10:46 Zaba hmm, works after make clean
10:46 Zaba (as usual)
10:50 rotty kid51: apparently, you need at least mesa-common-dev (as this contains GL/gl.h)
10:52 tetragon kid51: freeglut-dev is a package that provides the required header
10:53 kid51 Thanks.
10:56 kid51 tetragon:  Just got an error on Mac similar to the one you posted in 53170.  I just posted my build log.
10:57 tetragon I'm abot to test the patch
11:04 Zaba hmm
11:07 Zaba in perl6, are there exception objects?
11:10 tetragon kid51: I got r27101 to build with chromatic's patch
11:18 Zaba hmm
11:19 * Zaba runs rakudo on spec test
11:20 tetragon kid51: It built, but I got the same crash during test
11:23 tetragon kid51: t/op/sprintf.t test 174 is crashing with chromatic's patch
11:24 Zaba_ joined #parrot
11:27 * tetragon looks up where to set the watchpoint
11:30 Zaba rakudo is not very good on spec tests so far, it seems..
11:31 * Zaba would really like to contribute if he just knew how to code in pir properly
11:35 dalek r27116 | fperrad++ | trunk:
11:35 dalek : [digest]
11:35 dalek : - rename gen::digest to gen::crypto
11:35 dalek : - add tests for gen::crypto
11:35 dalek :  Courtesy of kid51
11:35 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27116
11:47 kid51 Zaba:  You're not alone in that!
11:47 Zaba I mostly code perl5 myself
11:48 kid51 me too; it meets all my personal and day-job needs
11:48 Zaba what puzzles me is that there are so much ??s in failed tests list
11:48 Zaba kid51, though I really like perl6 from what I read about/in it
11:48 kid51 Which failed tests lists?  Those for parrot, or those for perl6?
11:49 Zaba perl6, spectest
11:50 kid51 I only began to go into languages/perl6/ recently, so I mostly don't know what I'm talking about.  But I think that most of those tests were imported from the now-suspended Pugs project.  So we expect to have many failures there as yet.
11:50 Zaba yes, there are many
11:50 Zaba but only few actually state total tests/which of them failed
11:50 kid51 Unlike the Parrot tests, where if we have a failure (e.g., the one tetragon just reported), it needs immediate attention.
11:51 Zaba perl6 mostly fails with 'syntax error's on those pugs tests
11:51 Zaba kid51, so was pugs officially suspended and is no more being worked on?
11:56 pmichaud (explanation of '??')   if there's a "syntax error" in a test, then we can't run it to find out how many tests there are, thus ?? gets reported.
11:56 pmichaud where "syntax error" can mean either that it's really an error or that rakudo simply doesn't know how to parse that particular construct yet
11:57 Zaba seems like it was mostly rakudo
11:57 pmichaud sure, that's probably the case
11:57 Zaba for example, "}\n\n{" seems to have confused it in some of those tests
11:57 pmichaud but yes, if a file has a syntax error in it, we never even get to see the "1..23" that tells us there are 23 tests that were supposed to run.
11:57 Zaba I see.
11:58 pmichaud however, given the new fudge and test organization, we really ought to be able to drastically reduce the ??s
11:58 Zaba sounds quite good :>
11:58 pmichaud so this is a very useful observation
11:59 Zaba { say "test"; } { say "moep"; }
11:59 pmichaud that's not valid Perl 6
11:59 Zaba this is not a valid perl6 code, is it?
11:59 Zaba but with a semicolon there between } and { it is, right?
11:59 pmichaud yes.
12:00 Zaba so sub { something } more code requires a semicolon after } in perl6 too, right?
12:01 pmichaud if it's being followed by something on the same line, yes
12:01 pmichaud sub foo { something } \n { say "bar"; }    # ok
12:02 pmichaud sub foo { something } { say "bar"; }   # error
12:02 pmichaud sub foo { something }; { say "bar"; }  # ok
12:02 Zaba I see.
12:03 Zaba that makes perfect sense.. more sense than perl5's way even
12:03 Zaba does rakudo implement type checking currently?
12:03 kid51 pmichaud:  At our buildfest in NYC last week, someone who got to Hello World asked, "So where do I find the Perl 6 syntax?" -- and I didn't have a quick answer.
12:04 pmichaud kid51:  closest answer at the moment is the synopses, I think
12:04 pmichaud Zaba: rakudo has limited support for type checking -- jonathan's been adding it
12:04 Zaba pmichaud, limited in what?
12:05 Coke tetragon: "make -j2" should work. If you find that a "make" works when "make -j2" don't, open a bug ticket.
12:07 pmichaud limited in that it doesn't quite work everywhere, and some constructs it doesn't understand yet
12:07 pmichaud for example, I don't know if it understands things like  Dog|Cat yet
12:08 tetragon Coke: I haven't found that -j2 affects anything except backscroll readablilty
12:12 GeJ tewk++
12:14 GeJ I was trying to create some bindings this week-end for some Enlightenment library and spent a couple of hours how to create the dlfunc calls.
12:15 * kid51 to $job
12:16 GeJ ncigen seems really interesting.
12:20 Zaba ncigen?
12:22 wknight8111 joined #parrot
12:24 tetragon joined #parrot
12:38 GeJ http://code.google.com/soc/2008/perl​/appinfo.html?csaid=DF96A07990EB6EC1
12:38 shorten GeJ's url is at http://xrl.us/bjons
12:45 wknight8111 seen chromatic
12:45 purl chromatic was last seen on #parrot 3 days and 9 hours ago, saying: Nice work.  [Apr 18 21:01:53 2008]
12:50 shamu joined #parrot
12:51 contingencyplan joined #parrot
12:58 dalek r27117 | fperrad++ | trunk:
12:58 dalek : [digest]
12:58 dalek : - allows SHA256 & SHA512 when OpenSSL 0.9.8
12:58 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27117
12:59 Infinoid msg kid51 http://packages.ubuntu.com/freeglut3-dev (if you haven't already sorted that out)
12:59 purl Message for kid51 stored.
13:01 bpphillips-1 joined #parrot
13:01 Coke Infinoid: another patch for you regarding the ops syntax : http://rt.perl.org/rt3/Tic​ket/Display.html?id=52570
13:02 jhorwitz joined #parrot
13:05 Infinoid Aaw, for me?  You shouldn't have.
13:09 Coke =-0
13:09 Coke er, =-)
13:09 Infinoid I think this has been an especially volatile week for parrot development
13:09 Infinoid lots of big changes, lots of stuff going on
13:09 Coke While I disagree, I let it accept ":flag1:flag2" for now. I just happened to write them all with the space.
13:10 Coke For the first time in some time, I had to resolve a conflict on that patch, so yah. =-)
13:10 Infinoid patch fails to apply cleanly, rejects hunks in src/ops/pmc.ops and src/ops/debug.ops, even with r27116 (the rev the patch was apparently generated from)
13:10 Infinoid but I'm still able to read through the patch
13:11 Infinoid I like it.
13:15 skids joined #parrot
13:16 sjansen joined #parrot
13:20 Coke I may need to re-up.
13:21 Infinoid the only addition I'd consider is making $flags a hashref populated by the parsing code, and using exists($flags{pic}) instead of regexes
13:21 Infinoid seems cleaner, and makes things look more like pmc2c.
13:23 Zaba Infinoid, that indeed looks cleaner
13:25 Infinoid well, exists($$flags{pic}) or exists($flags->{pic}) if you prefer... whatever.
13:30 Coke Infinoid: I think that can go in as a separate patch.
13:32 Infinoid agreed
13:34 Zaba hmm
13:34 Zaba some crypto tests fail here
13:34 Zaba (parrot)
13:42 Coke the failures are just copyrights.
13:45 Zaba t/codingstd/trailing_space.t    1   256     1    1  1
13:45 Zaba t/steps/auto_crypto-03.t        5  1280    17    5  10-12 15-16
13:46 Zaba hm
13:46 Zaba there's some trailing whitespace in config/gen/crypto.pm line 72
13:48 * Infinoid fixes that
13:50 dalek r27118 | infinoid++ | trunk:
13:50 dalek : [digets] Remove some trailing whitespace introduced by r27117.
13:50 dalek : Zaba++ for reporting it.
13:50 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27118
13:50 Zaba and about crypto.. mhm
13:51 Zaba I assume the problem's on my side
13:51 Infinoid not necessarily.  r27117 changed crypto
13:53 iblechbot joined #parrot
13:53 Zaba shall I pastebin prove -v output?
13:54 Andy joined #parrot
13:54 Coke ... DOH.
13:54 Coke I'm trying to figure out why that file is showing binary. the mime type is "plain/text"
13:57 dalek r27119 | coke++ | trunk:
13:57 dalek : The mime type is 'text/plain', not 'plain/text'
13:57 dalek : (ugh, do we need to update the file_metadata test?)
13:57 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27119
14:00 ccube joined #parrot
14:21 * Coke wonders what mime type "text/script" is supposed to be.
14:26 ambs joined #parrot
14:27 ambs seen alias
14:27 purl alias was last seen on #perl 7 hours and 54 minutes ago, saying: job done
14:27 ambs seen merlyn
14:27 purl merlyn was last seen on #moose 32 days and 16 hours ago, saying: ... http://methodsandmessages.vox.com/lib​rary/post/the-year-of-smalltalk.html  [Mar 20 15:09:20 2008]
14:29 rdice joined #parrot
14:41 dalek r27120 | coke++ | trunk:
14:41 dalek : A file of type "plain/text" (instead of text/plain)
14:41 dalek : was added to the repo recently, confusing svn
14:41 dalek : into thinking it was binary.
14:41 dalek : Test for this by checking all the mime types (which we already had to do
14:41 dalek : anyway..) against an explicit list of those we allow... which is basically
14:41 dalek : all the sane types I found in the repo.
14:41 dalek : Remove the one remaining obviously wrong file type this exposed.
14:41 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27120
14:47 dalek r27121 | coke++ | trunk:
14:47 dalek : [codingstd] Nothing to see here, move along.
14:47 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27121
15:05 mj41 joined #parrot
15:15 jalbo joined #parrot
15:16 jalbo Hello.
15:19 Coke jalbo: hio
15:21 Tene jalbo: hi
15:23 gryphon joined #parrot
15:28 Theory joined #parrot
15:31 Zaba hmm
15:31 Zaba t/distro/file_metadata.t    2   512     5    2  3-4
15:32 Zaba http://rafb.net/p/xLNi9D82.html
15:32 jalbo I'm having same error.
15:33 Zaba seems like some svn attributes are messed..
15:33 * Infinoid just fixed that :)
15:33 dalek r27122 | infinoid++ | trunk:
15:33 dalek : [tcl] Fix file metadata for pkgIndex.tcl.
15:33 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27122
15:34 nopaste "Infinoid" at 96.238.213.50 pasted "crypto failures on linux/amd64" (36 lines) at http://nopaste.snit.ch/12760
15:34 Infinoid Zaba: are those the same failures you are seeing?
15:34 Zaba wait
15:34 jalbo Doing testing now...
15:34 * Zaba too
15:35 Zaba Infinoid, yes, exactly
15:35 Infinoid hmm.  not actually a failure in crypto, just in the config tests for crypto
15:36 Infinoid oh, and r27118 =~ s/digets/digest/
15:37 jalbo No, I was still in previous build, updating now.
15:38 Coke Infinoid: what change did you make there?
15:39 Coke as I just ran that test locally and got no errors pre commit.
15:39 Infinoid Coke: I followed the advice given by file_metadata.t:
15:39 Infinoid svn ps svn:keywords "Author Date Id Revision" languages/tcl/library/msgcat/pkgIndex.tcl
15:39 Infinoid svn ps svn:eol-style native languages/tcl/library/msgcat/pkgIndex.tcl
15:40 particle iirc those don't fail until the file has been committed
15:40 Infinoid I've noticed that file_metadata.t ignores my new files unless I update MANIFEST first
15:40 Coke That's not a new file.
15:40 Infinoid well, I don't know then
15:40 Coke particle: odd, because svn pg should return the *local* values, and therefore should fail.
15:41 Infinoid particle: mk_manifest_and_skip.pl will cause it to start noticing your files without having to commit
15:41 Infinoid I've been doing that for the new stub pmcs I've been adding to the pdd13pbc branch, and it's been working well.
15:41 Coke I'm not sure that's the right fix for files that are copied wholesale into the repo, but I'm pretty sure that one isn't going to matter, so nevermind.
15:41 particle when's the oscon hackathon? weekend before, or after?
15:41 Coke (the patch to .tcl)
15:41 Coke Infinoid: thanks for the cleanup
15:42 Infinoid np, nice that the test told me what to do :)
15:43 pmichaud oscon hackathon (if any) is after oscon, iirc
15:44 pmichaud before oscon is too difficult to arrange with other oscon events
15:44 pmichaud (at any rate, I can't make it before oscon, as I'll be vacationing in northern calif and oregon :-)
15:45 Infinoid pmichaud: if you come through the Lake Tahoe area, I'll buy you a beer.
15:45 Infinoid hmm.  maybe codingstd/distro tests which know how to fix it (like file_metadata.t and trailing_space.t) should have an option to do so.  t/codingstd/trailing_space.t --cleanup
15:46 Infinoid t/distro/file_metadata.t --jfdi
15:47 jalbo Updated to 27122, no metadata errors now.
15:48 jalbo t/steps/auto_crypto-03.t                   (Wstat: 1280 Tests: 17 Failed: 5)
15:48 jalbo Failed test number(s):  10-12, 15-16
15:48 jalbo Non-zero exit status: 5
15:48 pmichaud Infinoid: mmmm, Lake Tahoe.  So far we not going quite that far east.
15:48 Infinoid fair enough.  its not the most convenient place to drive to
15:48 pmichaud but it's awful pretty when you get there :-)
15:49 Coke Infinoid: I'd rather not have it do that. If we're going to do that, let's move to commit hooks.
15:49 Infinoid Coke: even better :)
15:50 pmichaud I just wrote a short shell script that fixes metadata settings
15:50 pmichaud so when file_metadata.t carps, I just invoke the shell command to fix it :-)
15:55 Infinoid file_metadata.t is nice for giving you the commands you need to run
15:55 Infinoid unfortunately, the trailing space and tab tests require manual editing, and I guess I'm just lazy.
15:56 particle we can employ perltidy and a 'make tidy' target
15:56 particle and obtw why don't you write pirtidy while you're at it
15:57 particle we already have a parser, in languages/pir
16:02 slightlyoff joined #parrot
16:03 Infinoid eh.  a simple s/\s+$//; s/\t/        /g is sufficient
16:11 Coke there is a pirtidy.pl already.
16:12 Coke (but it's written in perl, not PIR0
16:17 wknight8111 does it need to be written in PIR?
16:32 particle no
16:32 particle *but*
16:32 particle it'd be nice to have something that can take a parse tree and spit out re-formatted source
16:32 particle given a ruleset, for *any* hll
16:33 particle it could be written in nqp
16:39 cognominal should that mean annotation convention for the grammar?
16:40 cognominal like "is indent" or "has indent"
16:41 particle i'd like to have a library, written in nqp or pir, which takes rules (like perltidy) and applies them to a parse tree generated from PCT
16:41 particle so, that means it could be used for ruby, js, perl 6, etc
16:41 Infinoid "does prettyprint"
16:41 Infinoid or maybe that's something else
16:42 particle so is, has, does are all at the language's syntax level
16:42 particle that's not something i care about *yet*
16:42 particle but certainly could be useful in perl 6 to specify somehow
16:43 cognominal hum can we derive a grammar without changing rules but adding 'deparsing rules' annotations
16:43 particle as long as there's a way to store them in the parse tree, yes
16:43 particle then use an action file to emit the prettified source
16:44 cognominal btw, I am working on a parsing tree trimmer to make the parsing tree shorter and more readable
16:46 cognominal 160 line for a 27 chars programs does not make it  very useful grammar debugging tool.
16:48 particle nice
16:54 particle who made a mess of the imcc parser?
16:55 particle lots of warnings
16:57 Infinoid I regenerated imclexer.c a few days ago
16:58 Zaba joined #parrot
16:58 chromatic joined #parrot
17:00 nopaste "particle" at 24.19.3.148 pasted "messy imcc parser warnings" (43 lines) at http://nopaste.snit.ch/12761
17:02 nopaste "jalbo" at 213.96.228.50 pasted "Drop :const in isa and getclass, http://rt.perl.org/rt3//Publ​ic/Bug/Display.html?id=53066" (153 lines) at http://nopaste.snit.ch/12762
17:02 jalbo I'm testing this changes, all tests pass here.
17:05 Infinoid is that new?  imcparser.c hasn't been touched in a while
17:06 particle i don't know, i haven't built parrot in a while
17:06 particle compilers\imcc\instructions.c(141) : error C2059: syntax error : '}'
17:07 particle it could be that i'm noticing it only because my build failed shortly afterwards
17:08 particle i don't think c likes having empty curly brackets
17:08 cjfields joined #parrot
17:28 particle coverity... what's going on with that lately?
17:29 chromatic No scans since mid-February.
17:29 chromatic I've sent David a few messages here and there, but haven't heard back.
17:30 sjansen joined #parrot
17:31 Coke particle: c c or c chromatic?
17:33 particle ok, as long as somebody's on top of it
17:33 chromatic I'm on top of the big nothing happening anyway.
17:35 particle i know i can count on you.
17:35 particle care to fix my build?
17:35 Coke anyone aside from Infinoid have feedback on the ops syntax update?
17:36 pmichaud I haven't been following it closely, but :flag1 :flag2 is my preference
17:37 Coke in general, no one seems to complain about having to not write goto NEXT(); =-)
17:37 wknight8111 where can we get more information about the update? I haven't read anything about it
17:37 chromatic particle, what's your build problem?  Is it the same as tetragon's?
17:38 Coke wknight8111: mailing list: :"[PATCH] Simplify ops syntax"
17:38 wknight8111 thanks coke
17:38 Coke it's a minor thing. i figured if it was hated, someone would have spoken up.
17:38 particle compilers\imcc\instructions.c(141) : error C2059: syntax error : '}'
17:40 chromatic Hm, an empty array initializer.
17:40 pmichaud oh, that's mine.
17:40 pmichaud I did that when removing restoreall/saveall
17:40 particle ah. that's who i should blame
17:41 pmichaud I'll fix it... just a sec.
17:41 chromatic You can delete more code in there.  Nice.
17:41 pmichaud well, I didn't want to get rid of it entirely in this case
17:41 pmichaud in case we ever re-introduce an opcode that needs special handling and is of the 'read' sort.
17:42 pmichaud otoh, if we wanted to get rid of cleari, clearn, clearp, clears then this might be a good time to do it :-)
17:44 dalek r27123 | jkeenan++ | trunk:
17:44 dalek : Temporarily disabling tests in response to Mark Glines' report of test failures in http://rt.perl.org/rt3/Tic​ket/Display.html?id=53126.
17:44 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27123
17:45 Infinoid Zaba++'s report, actually.  I'm just a routing protocol.
17:45 chromatic pmichaud, I meant the loop over reads.
17:45 pmichaud right, I was planning to leave it there in the code.  But I'll comment it out.
17:46 chromatic Why comment it out?  It's dead code.
17:46 pmichaud it might come back if we get another special "read" op?
17:46 chromatic Is that likely?
17:47 pmichaud I hope not.  But I guess we could eliminate r_special as well, then?
17:47 pmichaud (just looking into that)
17:47 chromatic Looks like it.
17:48 pmichaud so, we'd just be left with w_special then for the clear[isnp] ops.  Sounds good to me.
17:48 * pmichaud grabs his trusty Acme Code Ripper (TM)
17:50 pmichaud removing r_special does seem to rip out a fair amount of code
17:50 pmichaud I'll do it in two stages -- one to get code compiling again, another to do a more thorough cleaning.
17:51 chromatic Perfect.
17:51 purl if it seems perfect, your understanding is less than perfect.
17:51 chromatic One fewer global in IMCC is also very good.
17:51 cognominal Coke: "anyone aside from Infinoid have feedback on the ops syntax update?". Are these flags yet  used anywhere. If yes were?
17:51 Coke cognominal: I'm adding in :flow; the only other one I see in use is :pic
17:51 pmichaud (just a rebuild and make test away from committing.)
17:52 Coke cognominal: mainly trying to add :flow, not remove other dead flags.
17:52 cognominal I don't say they are dead, I ask if they are used yet.
17:53 cognominal I see their use for a restricted parrot.
17:53 particle coke: after i build parrot successfully based on patrick's fix, commit it and i'll test
17:53 particle the security pdd is on the list of things todo
17:55 ambs joined #parrot
17:55 cognominal well, in the mean time there is obvious stuff like a restricted parrot cannot open files and sockets
17:56 Infinoid is that something I'll have to care about, for the new .pbc file writing code?
17:56 chromatic No.
17:56 Infinoid great.
17:57 dalek r27124 | allison++ | trunk:
17:57 dalek : [pdd] Add interface specification to Strings PDD.
17:57 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27124
17:58 dalek r27125 | allison++ | trunk:
17:58 dalek : [pmc] Correcting bad documentation in the String PMC found while working on the
17:58 dalek : Strings PDD.
17:58 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27125
17:59 allison joined #parrot
18:01 mj41 hi, does anybody use http://tt.perl6.cz/ ? or is it still useless? :-)
18:02 ambs joined #parrot
18:02 pmichaud re-testing... ran into a test failure (svn up hopefully fixes it)
18:04 dalek r27126 | allison++ | trunk:
18:04 dalek : [pdd] Removing obsolete documentation from datatypes PDD, since the updated
18:04 dalek : content exists in other PDDs.
18:04 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27126
18:05 pmichaud particle: r27127
18:05 dalek r27127 | pmichaud++ | trunk:
18:05 dalek : [imcc]:
18:05 dalek : * Fix empty const array introduced by removing saveall/restoreall.
18:05 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27127
18:05 * particle rebuilds
18:06 pmichaud someone want to double-check me.... is the "ins_reads2" function in instructions.c actually *used* anywhere?
18:07 pmichaud oh, I found it.
18:07 pmichaud okay.
18:07 pmichaud oh, no I didn't.  back to earlier question.. is it actually *used* anywhere?
18:08 pmichaud aha.  It's not used anywhere because I dead-code removed it last night.
18:08 pmichaud cool, more ripping.  :-)
18:08 chromatic Always nice.
18:10 pmichaud hmmm, that's not it.  checking more.
18:10 pmichaud looks like that function was never called.
18:11 barney joined #parrot
18:13 dalek r27128 | allison++ | trunk:
18:13 dalek : [pdd] Launching the Strings PDD out of draft.
18:13 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27128
18:16 Ivatar joined #parrot
18:21 particle parrot's failing a lot of tests now
18:23 chromatic Did you make realclean recently?
18:23 dalek r27129 | pmichaud++ | trunk:
18:23 dalek : [imcc]:
18:23 pmichaud there was a PBC_COMPAT update last night as a result of eliminating saveall/restoreall
18:23 dalek : * More dead code removal.  The ins_reads2 function is never called,
18:23 dalek :   so we get rid of it and the r_special array.
18:23 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27129
18:24 Infinoid All tests successful here on r27127
18:24 * Infinoid does it again
18:24 pmichaud all tests successful here on r27129.
18:27 chromatic #ps in 3
18:30 particle Failed 45/580 test scripts. 216/11212 subtests failed.
18:32 ambs particle, what arch?
18:32 * Coke guesses winders.
18:32 particle msvc
18:32 ambs Coke, yeah, I guessed that as well after asking
18:32 Coke chromatic: that's not quite what I meant, but by all means, you can lead this dance. =-)
18:33 Coke watch out, I will CRUSH your toes if you're not careful.
18:33 chromatic I'm actually a good dancer.
18:33 chromatic Well, okay.  Adequate.
18:35 ambs yay!
18:35 ambs /bin/sh: line 1:  8911 Segmentation fault      ./miniparrot config_lib.pasm > runtime/parrot/include/config.fpmc
18:35 Coke ugh, I hate that error.
18:35 chromatic is this after a make realclean?
18:36 Coke (if so, did you build with -j ?)
18:36 davidfetter joined #parrot
18:36 ambs chromatic, not yet. doing it now
18:37 Coke .flanders!
18:37 chromatic Yeah, if PBC files depended on PBC_COMPAT we might be able to avoid this somewhat.
18:38 Coke generated PBC file? I can do that.
18:38 Coke (and static ones as well.)
18:38 Infinoid (we have static PBC files?)
18:38 chromatic the native PBC tests
18:39 Coke we're skipping those atm, neh?
18:39 chromatic Yeah, too much PBC_COMPAT churn.
18:39 Ivatar hmm, I had problems with miniparrot yesterday... seems to be fixed now.
18:39 chromatic I was getting sick of fixing them for kid51.
18:39 ambs and yes, make clean solved it
18:39 nopaste "particle" at 24.19.3.148 pasted "win32 build output and test failures (long)" (5856 lines) at http://nopaste.snit.ch/12763
18:39 Infinoid I suppose I'll have to update those when I change the .pbc file format
18:40 Coke now that "make -j" works, perhaps we can address the next level of "why do I have to make realclean".
18:40 ambs hehehe
18:40 Coke (one of which is the generated pbc fils.)
18:40 chromatic s/one/most/
18:40 Coke pmichaud: if you record it on my voicemail I can transcribe it for you. =-)
18:40 ccube Why is the Array class a subclass of ResizablePMCArray instead of the other way around?
18:41 pmichaud Array is fixed size
18:41 pmichaud it's easier to create a fixed size object from a variable-sized one than vice-versa :-)
18:41 Coke I don't think it used to be that way, diddit?
18:41 chromatic As long as I can remember, so at least ten minutes.
18:41 ccube So to check for a container I need to isa against ResizablePMCArray instead of Array
18:42 pmichaud better is probably "does array"
18:42 Coke array.pmc's pmcclass doesn't extend anything.
18:42 pmichaud as opposed to isa.  there might be multiple container base classes, as opposed to one master one.
18:42 ccube From a pedagogical standpoint the more fundamental class should have a more fundamental name
18:42 Coke RPA extends FPA which doesn't extend anything.
18:43 ccube Call what is currently ResizablePMCArray Array, and call Array FixedArray
18:43 Coke ccube: can we argue about one thing at a time? easier for us old timers. =-)
18:43 Coke array does not subclass RPA, that I can see.
18:44 ccube Well what is a better way to check if can create an iterator on a given PMC?
18:44 pmichaud $I0 = does $P0, 'array'
18:45 ccube I was checking by isa 'Array' and isa 'Hash'
18:45 Coke don't do that.
18:45 ccube does that work for hashes too
18:45 Coke does $P0, 'hash'
18:45 pmichaud $I0 = does $P0, 'hash'
18:45 ccube isa is deprecated?
18:45 pmichaud no, but isa checks class hierachy, as opposed to a role
18:45 Coke I don't think so.
18:45 particle ccube: get_iter $P0
18:46 particle ...ask the pmc for it's iterator
18:46 pmichaud whether or not a PMC can have an iterator on it is part of its interface or role, not what class it happens to be
18:46 ccube ok, does that return NULL pmc when you can't iterate over $P0?
18:46 ambs All tests successful, 26 tests and 570 subtests skipped.
18:47 ambs (27129)
18:48 Coke (that's easy enough to check, btw.)
18:48 particle ccube: look it up
18:48 ccube yep
18:48 ccube i'm checking
18:49 ccube get_iter is experimental ?
18:49 particle is it? it's been around forever
18:49 particle i guess we can move it out of experimental.
18:49 ccube in src/ops/experimental.ops - not in docs
18:50 particle lemme check the pdd
18:50 pmichaud I think get_iter is experimental, otherwise I'd have been using it more often instead of   $P0 = new 'Iterator', $P1
18:50 particle =item nextkey_keyed
18:50 particle PMC* nextkey_keyed(INTERP, PMC* self, PMC* key, INTVAL what)
18:50 particle PMC* nextkey_keyed_int(INTERP, PMC* self, INTVAL key, INTVAL what)
18:50 particle PMC* nextkey_keyed_str(INTERP, PMC* self, STRING* key, INTVAL what)
18:50 particle Advance to the next position while iterating through an aggregate. [NOTE: this
18:50 particle feature needs review together with the Iterator PMC.]
18:50 particle that note at the bottom means it's not settled yet
18:51 particle so, use get_iter, until we replace it
18:53 ccube get_iter doesn't appear to be an op. it's not in include/parrot/oplib/ops.h
18:54 Coke at this point, somethings presence in experimental.ops isn't really an indication of its status.
18:54 chromatic Yeah, no one's moved them around in ages.
18:54 particle right
18:55 ccube It isn't an op at all, looks like a vtable method
18:55 particle the pmc pdd includes nextkey_keyed, though
18:55 particle get_iter is an op
18:55 Coke it's just "iter".
18:55 Coke particle: doesn't seem to be.
18:55 particle ah, right.
18:55 particle 'iter'
18:55 particle get_iter is the vtable function, iter is the op
18:55 particle $P1 = iter $P0
18:56 particle stupid me--
19:01 ccube hmm, iter called on a string PMC returns a true iterator, then my code fails when I try to reference index it
19:02 Coke ISTR String used to support iterator, doing char-by-char.
19:02 Coke what does it do on an Int?
19:02 ccube don't know yet
19:03 ccube but if you can't call shift on it they I believe you shouldn't get an object from iter that tests true
19:03 particle iterator pmc is a ball of mud
19:03 chromatic Just for fun, if someone has time, try getting a PMC keyed iterator on a NameSpace.
19:03 ccube That's what I'm trying to do
19:04 ccube I am writing a simple class information program that is running into all sorts of problems
19:04 chromatic The NameSpace PMC tests don't test that.
19:05 ccube it found that nice bug yesterday where key_string was passing the buck to Parrot_Key_get_string and Parrot_Key_get_string was passing the buck back to key_string - blew the stack
19:06 ccube a hazard of the buck passing style of programming
19:06 chromatic Yeah, I tried to fix that up, and now I think there's something funky in NameSpace's key handling.
19:06 dalek r27130 | infinoid++ | trunk:
19:06 dalek : [PMC] Back out vtable consting patches r27028 and r27029, per design discussion in #parrotsketch.
19:06 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27130
19:07 dalek r27131 | infinoid++ | trunk:
19:07 dalek : [core] Un-const a caller of VTABLE_isa.
19:07 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27131
19:08 ccube Is a there a tool that can tell you when functions are unintentionally calling each other recursively?
19:09 chromatic It's not really unintentional.  As far as I can tell, it's just NameSpace producing a bad Key.
19:09 ccube Well if intentional, you need to create a bottom
19:10 chromatic If NameSpace produced the right type of Key, it wouldn't happen.
19:10 chromatic I think it's a bug.
19:10 chromatic I don't know exactly where it is, but there's my prediction.
19:10 ccube yes, I would call blowing the stack a bug
19:10 Infinoid r27131 broke the build, forgot to update the prototype too.  fixing...
19:10 ccube Why are you calling a function that's just going to call you without changing anything?
19:11 chromatic It's not going to call back directly, ccube.  It's a bug.
19:12 dalek r27132 | infinoid++ | trunk:
19:12 dalek : [core] Remove const from the prototype too.  Infinoid--
19:12 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27132
19:13 Coke particle: you going to announce the SOC stuff?
19:14 particle on #ps?
19:14 Coke <nod>
19:14 particle okies.
19:16 Coke particle: can I go ahead and commit the ops syntax stuff?
19:16 particle coke: ok, can't make anything worse
19:17 ccube hmm, Hash  reports that it doesn't hash or Hash, NameSpace neither, ResizablePMCArray says it doesn't array or Array
19:17 ccube so it looks like I'm back to isa
19:17 dalek allison@perl.org | milestones:
19:17 dalek link: http://www.perlfoundation.org​/parrot/index.cgi?milestones
19:18 Coke ccube... can you provide a test, please?
19:19 nopaste "coke" at 72.228.52.192 pasted "ccube : this prints '1' for me." (5 lines) at http://nopaste.snit.ch/12764
19:20 nopaste "ccube" at 74.70.96.161 pasted "does fails" (80 lines) at http://nopaste.snit.ch/12765
19:20 Coke ccube: try "array", not "Array"
19:20 ccube tried that first
19:20 Coke "hash" not "Hash"
19:20 ccube tried that first too
19:20 Coke see my test.
19:20 Coke try that.
19:22 ccube maybe it works for a newly generated Hash, but not one you get in the tree that results from inspect
19:22 particle s/does/provides/ maybe?
19:22 chromatic No, provides is only for PMCs in the .pmc file.
19:23 nopaste "chromatic" at 63.105.17.30 pasted "does on RPA, Array, Hash, and OrderedHash" (26 lines) at http://nopaste.snit.ch/12766
19:23 Coke particle: committed.
19:23 purl The chicken is involved, but the pig is *committed*.
19:24 dalek r27133 | coke++ | trunk:
19:24 dalek : Apply patch in RT #52570 to simplify ops a bit, and make the syntax for
19:24 dalek : flags/classes a bit more rigorous.
19:24 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27133
19:25 nopaste "coke" at 72.228.52.192 pasted "ccube: this prints "Hash\n1"" (8 lines) at http://nopaste.snit.ch/12767
19:26 particle Infinoid: what if you build a static parrot, and want to run postgres?
19:27 Infinoid particle: both schemes use "loadlib", so if "loadlib" doesn't exist, you're SOL either way
19:27 Infinoid I guess my question is whether those things should be pir-only or not, assuming that's possible
19:27 dalek allison@perl.org | milestones:
19:27 dalek link: http://www.perlfoundation.org​/parrot/index.cgi?milestones
19:28 ccube coke, try it on the Hash you get from the PMC returned by inspect, like I am doing (I have done s/Array/array/ and s/Hash/hash/)
19:28 Coke ccube: I did that.
19:28 Coke as I demonstrated in the example.
19:28 Coke http://nopaste.snit.ch/12767
19:29 Coke so inspect seems to be returning a 'Hash', which does 'hash'.
19:29 chromatic Does loadlib not exist in a static Parrot?
19:29 Coke (parrotsketch) is someone translating us into spanish?
19:30 ccube yes, but with $P1 = inspect $P0 try it on $P2 = $P1['methods']
19:30 particle si y no.
19:30 Coke ccube: see, these details help. =-)
19:30 particle huh? what?
19:31 particle ccube: can you nopaste an example?
19:31 particle holy unreachable code, batman!
19:31 ccube take the program I pasted earlier and change Array to array and Hash to hash and try it
19:31 Coke ccube: which class are you inspecting?
19:31 ccube Any
19:31 ccube but I have tried Class and Array
19:31 ccube and NameSpace
19:31 purl hmmm... NameSpace is XSLT, other stuff isn't
19:31 Coke if I do 'methods' I get a null PMC back.
19:32 pmichaud Array might not have any methods on it.
19:33 ccube I wouldn't be asking you guys to debug my program for me if I didn't think it would help parrot
19:33 Infinoid purl, forget NameSpace
19:33 purl Infinoid: I forgot namespace
19:33 pmichaud Also, Array isn't really a "Class".  It's a PMCProxy :-)
19:33 pmichaud If this sounds confusing, it's because it is.  :-|
19:33 szbalint chromatic: yeah, those kinds of blog posts are good. Keep them coming!
19:33 Coke ccube: what do you expect that program to output?
19:34 particle coke: i get millions of warnings when building ops files
19:34 pmichaud I'm not sure if it's possible to inspect Class, Array, or Namespace
19:34 ccube print what inspect returns and if the object is a container print what it contains recursively (up to MaxDepth)
19:34 particle Class has an inspect method
19:34 pmichaud ccube:  perhaps Data::Dumper would do this, also?
19:34 ccube (that is the objects in the inspect hash)
19:35 ccube I am trying to learn parrot not dump data
19:35 particle runtime/parrot/include/Data/Dumper.pir
19:35 particle load_library 'Data/Dumper.pbc' or whatever
19:35 ccube I am trying to find out how given ops work
19:36 pmichaud ccube:  understood -- was just pointing that Data::Dumper does much the same thing.  You could look at how it does things.
19:36 particle data::dumper doesn't use inspect, for sure
19:36 ccube so I shouldn't use inspect?
19:36 particle there are tests  for inspect for class and object pmcs
19:37 Coke ccube: so the bug is that if you inspect something, and it has a methods key, you're not seeing any values?
19:37 ccube no - inspect returns a hash container and I call my display_container function on that if any of the VALUES in the container are containers, then recursively display them
19:38 Coke and it's doing that. what's the problem?
19:38 ccube that's the intention
19:38 Coke methods is a Hash, you find that. then what breaks?
19:38 ccube for me the version I pasted does not disply NameSpace or methods
19:38 nopaste "particle" at 24.19.3.148 pasted "win32 build output after coke's commit" (7177 lines) at http://nopaste.snit.ch/12768
19:38 ccube when I was using isa, it did (at least for Hashes)
19:39 Coke the version you pasted is broken.
19:39 ccube In what respect ?
19:39 Coke if you fix the 'Array' -> 'array', and 'Hash' -> 'hash' for does, how is your output then not correct?
19:39 ccube I said that I had done that locally
19:40 ccube same result
19:40 Coke yes, but then you said 'the code you pasted'. the more precise you are, the easier it is to help you. =-)
19:40 Coke Ok. I see .const string Indent='     '
19:40 Coke .const int MaxDepth = 6
19:40 Coke .sub main :main
19:40 Coke .param pmc argv
19:40 Coke .local int argc
19:40 Coke argc = argv
19:40 Coke if argc >= 2 goto ok
19:40 Coke die "Need to provide a classname"
19:40 Coke ok:
19:40 Coke .local string classname
19:40 Coke classname = argv[1]
19:40 Coke $P0 = get_class classname
19:40 Coke $I0 = defined $P0
19:40 Coke if $I0 goto valid
19:40 Coke die "Invalid classname"
19:40 Coke valid:
19:40 Coke $P1 = inspect $P0
19:40 Coke $I0 = 0
19:40 Coke display_container($P1, $I0)
19:40 * particle points to NOPASTE
19:40 Coke .end
19:40 Coke GAH.
19:41 Coke sorry.
19:41 Coke I see: "methods: Hash". What is wrong with this?
19:41 Coke it was an accident. I know about nopaste.. thanks for your understanding. :P
19:41 pmichaud $I2 = does $P2, 'Hash'
19:41 pmichaud I see no $P2 here.
19:42 Coke (that certainly changes the output. =-)
19:42 ccube coke, yes I FIRST tried 'array' and 'hash' as you said, that didn't work so I tried 'Array' and "Hash', that's what I pasted, the I went back to 'array' and 'hash' again
19:42 Coke pmichaud has found your typo. Enjoy.
19:42 pmichaud ccube:  $I2 = does $P2, 'Hash'   # I see no $P2 here.
19:42 pmichaud (and yes, 'Hash' should be 'hash')
19:43 ccube ok, that let me fix that
19:43 pmichaud also note that
19:43 pmichaud $P1 = box[$S0]
19:43 pmichaud is likely wrong if box is an array container as opposed to a hash container
19:44 pmichaud if box is a Hash or hash-like object, then shifting over the iterator returns keys in the hash
19:44 Infinoid japhb: I was not criticizing your approach, quite the opposite.  I'm just wondering whether NCI will improve to the point that this stuff becomes easier :)
19:44 pmichaud but if box is an Array or array-like object, then shifting over the iterator returns the elements of the container
19:45 Infinoid japhb: sounds like OpenGL is an interesting test case for NCI.
19:45 pmichaud and so when box is an array-like container, we can't use the shifted elements of the iterator as keys to the container
19:45 ccube gee, i thought the point of an iterator was to act the same over all containers
19:46 ccube so I have to have a separate display_hash and display_array?
19:46 chromatic Yeah, but hashes and arrays are somewhat different.
19:47 pmichaud iterating over a hash is effectively the same as iterating over hash.keys()
19:47 japhb Infinoid: Oh, I didn't at *all* take your question as criticism.  I read it as an honest set of questions.
19:48 japhb Infinoid: Yes, it appears I keep pushing the NCI limits.  :-)
19:48 pmichaud japhb++ # push limits
19:48 particle tewk++ # expand limits
19:49 ccube ok guys, thanks for the help
19:49 Infinoid japhb: if it makes you feel any better, libfuse will be fun too.  It was a PITA with perl5 - it wants to call multiple callbacks in multiple threads at once, from threads it creates internally, without calling a thread-context initializer function first
19:49 Coke particle: please resend your nopaste link.
19:49 particle nopaste: "particle" at 24.19.3.148 pasted "win32 build output after coke's commit" (7177 lines) at http://nopaste.snit.ch/12768
19:49 japhb Infinoid: EWWW.
19:49 Infinoid so I had to play nasty games with TLS and store away a copy of perl's context structure, to dup as needed
19:50 japhb Infinoid: it sounds like libfuse's design is pretty hateful to bindings writers.
19:51 chromatic Any shared library that doesn't use opaque pointers is hateful to bindings writers.
19:51 Infinoid I am amazed that it works at all.  The bindings explicitly disabled multithreaded operation for years, which kinda sucks for a filesystem. :)
19:51 japhb quite
19:53 particle every thread has a different operating system
19:53 japhb As it is, I haven't solved the threading issue with the GLUT callbacks -- GLUT very deeply assumes global state is a design feature for its target audience
19:53 particle err, filesystem.
19:53 particle (joking while eating)--
19:54 Infinoid hopefully not :)
19:54 japhb must go AFK for a bit, bbl if you need me ...
19:59 Coke particle: unless ($flags =~ /\b:flow\b) ; # fails to match when $flags contains the string ":flow". hurmm.m
19:59 Coke (so it's always running the condition)
19:59 Coke meaning it's always adding goto NEXT(), which is why you have your warnings.
20:00 particle \: ??
20:02 Infinoid removing the leading \b makes it match
20:02 pmichaud \b:flow\b is probably wrong, yes
20:02 pmichaud as that would mean that there has to be a word character prior to the :
20:02 Infinoid the : itself is a good delimiter for the left side :)
20:02 Coke ... what Infinoid said.
20:02 Coke testing now.
20:03 Coke a holdover from the "," version.
20:03 pmichaud if you want to enforce spaces, then   \s:flow\b
20:03 Coke (and it still builds, compiles, and passes all tests.
20:03 Coke pmichaud: can't.
20:05 dalek r27134 | coke++ | trunk:
20:05 dalek : Fix re that checked for :flow tag (holdover from previous iteration).
20:05 dalek : The RE failed, which means the goto NExT() was added everywhere, which results
20:05 dalek : in unreachable code on any ops that were tagged :flow.
20:05 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27134
20:06 * particle rebuilds
20:06 * particle reconfigs, then rebuilds
20:07 particle same
20:08 particle c:\usr\local\parrot\trunk\src\ops\io.ops(71) : warning C4702: unreachable code
20:08 particle c:\usr\local\parrot\trunk\src\ops\io.ops(71) : warning C4702: unreachable code
20:08 particle c:\usr\local\parrot\trunk\src\ops\io.ops(72) : warning C4702: unreachable code
20:08 particle c:\usr\local\parrot\trunk\src\ops\io.ops(71) : warning C4702: unreachable code
20:08 particle c:\usr\local\parrot\trunk\src\ops\io.ops(72) : warning C4702: unreachable code
20:08 particle c:\usr\local\parrot\trunk\src\ops\io.ops(73) : warning C4702: unreachable code
20:08 particle etc
20:08 Coke are you still getting one at core.ops(53) ?
20:09 particle no
20:09 Coke then those are different. hang on.
20:09 particle the first is core.ops(154)
20:09 Coke do you swear these errors didn't used to occur? =-)
20:09 particle i so swear.
20:11 Coke Parrot_load_bytecode_sc seems to end with two goto NEXT()'s.
20:11 particle the mail arrives, and for the 3rd day in a row, i visit catalogchoice.com
20:11 ambs lol
20:11 Coke aha.
20:12 Coke particle: I think the generated code is being passed to the multiple variants of an opcode.
20:12 Coke so when it's added once, it's then copied and then added again.
20:13 Coke hoisting the :flow check out of the loop will probably do it. checking...
20:13 particle i'll check the .c file
20:14 Coke yup, that seems to generate proper code.
20:15 ruoso joined #parrot
20:16 Coke particle: re-up, try again.
20:16 dalek r27135 | coke++ | trunk:
20:16 dalek : [ops]
20:16 dalek : Only add the implicit goto NEXT(); once per opcode, instead of accumulating
20:16 dalek : it once per variant.
20:16 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27135
20:17 particle c:\usr\local\parrot\trunk\src\ops\object.ops(81) : warning C4702: unreachable code
20:17 Coke (and thanks for catching the warnings) - I am used to seeing warnings, so I'm not sure I would have noticed this ("make test passes? whee!")
20:17 particle many fewer warnings, but still ~20
20:17 Coke probably YA bug.
20:17 particle most in object.ops, some in string.ops
20:18 Coke ah. some of these aren't marked as ":flow" when they should be!
20:18 Coke can you nopaste the list? will make it easier to track down.
20:18 nopaste "particle" at 24.19.3.148 pasted "win32 build output after r27135" (83 lines) at http://nopaste.snit.ch/12769
20:20 * japhb BAK, if there was any remaining NCI/OpenGL discussion
20:21 particle is it done yet? :P
20:22 Coke be so nice if those warnings told me the OP name. =-)
20:22 japhb particle: thbbt.  :-)
20:23 japhb You can blame stomach flu for lack of progress, unfortunately
20:28 Coke particle: that should kill some more, let me know what's left. :|
20:29 particle object.ops 565, 570, 575, 580, 585
20:29 dalek r27136 | coke++ | trunk:
20:29 dalek : [ops]
20:29 dalek : Mark some :flow ops as such.
20:29 dalek : Delete some superfluous goto NEXT();
20:29 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27136
20:29 Coke particle: that's the copyright section. :P
20:30 particle don't blame me
20:30 particle ops2c must be reporting faulty numbers
20:30 particle however, i can tell you they don't warn in core_ops_switch
20:30 particle just core_ops
20:31 Coke I'm guessing its: opcode_t *
20:31 Coke Parrot_pic_infix___ic_p_p (opcode_t *cur_opcode, PARROT_INTERP)  {
20:31 Coke PANIC(interp, "How did you do that");return 0;
20:31 Coke }
20:31 rdice joined #parrot
20:31 dalek r27137 | kjs++ | trunk:
20:31 dalek : [src] localize variables by adding 'else' clause, and remove some newlines within 1 statement, after which it's still well under 80 columns.
20:31 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27137
20:31 particle maybe. return 0 should go
20:32 Coke hah! I bet that was broken before! =-)
20:32 Coke particle: concur. You wanna commit that one?
20:32 Coke lib/Parrot/Op.pm line 394
20:33 Coke Though that whole thing seems fishy in the first place.
20:34 particle testing now
20:34 particle no warnings with that change
20:34 Coke particle:++
20:34 Coke particle++ #even.
20:35 particle coke++ # DRY - making op bodies smaller
20:35 particle however, i kinda feel the attribute should be the exception and not the rule
20:35 Coke sorry it took a few iterations. Thanks for the feedback.
20:35 Coke particle: it is!
20:35 particle really? i thought most ops had goto NEXT()
20:35 Coke goto NEXT(); is added by default. if it is present, it's not added.
20:36 particle ah.
20:36 particle perfect.
20:36 purl La perfection est atteinte non quand il ne reste rien � ajouter, mais quand il ne reste rien � enlever
20:36 Coke at least better.
20:36 Coke next step is to look at all the flags we are declaring. I am tempted to make it a fixed list and die on the build if one that isn't kosher is specified.
20:36 Coke ... and then we can remove the ones that are crufty and will never be implemented.
20:37 particle speak with allison if it looks like any are security related
20:37 particle i'd like to have an op security model that groups ops into categories
20:37 particle then we can specify by category which ops to enable/disable or flag as unsafe
20:38 particle as well as being able to specify by name
20:38 dalek r27138 | kjs++ | trunk:
20:38 dalek : [src] localize variables and align assignment operators.
20:38 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27138
20:40 dalek r27139 | particle++ | trunk:
20:40 dalek : [ops2c] silence final msvc warnings by removing unreachable code in generated PANIC() macro - coke++
20:40 dalek diff: http://www.parrotvm.org/svn​/parrot/revision?rev=27139
21:02 jonathan This week's "why I missed parrotsketch" excuse: I was on a plane, and too stingy to pay for expensive airport wifi to pre-file.
21:02 Coke jonathan: FAIL!
21:02 Coke seen obra?
21:02 purl obra was last seen on #perl 14 days and 2 hours ago, saying: what dist did you try to install that showed the problem?  [Apr  8 11:58:05 2008]
21:03 jonathan how does one get purl to tell them the messages?
21:03 Coke messages?
21:03 purl To access purl's messages, msg me with the word "messages".
21:03 jonathan Cool, thanks.
21:04 jonathan Infinoid: will get to your questions when more than 5% of my braincells are working, which should hopefully happen after I've slept.
21:04 Infinoid no worries, I've got enough to keep me busy
21:05 particle jonathan: expect loads of test failures with trunk head
21:05 particle i'm hoping to find out why soon
21:05 particle but i'm sleepy too
21:08 jonathan particle: wasn't planning on doing anything tonight...
21:08 jonathan Last few days have generally been very busy seeing people, and then sleeping early on a night because I've felt a tad under the weather.
21:08 jonathan Managed to catch a cold within a couple of days of being in the UK.
21:09 particle Failed 45/580 test scripts. 216/11212 subtests failed.
21:18 ambs good night
21:18 purl sleep well too
21:18 ambs left #parrot
21:41 AndyA joined #parrot
22:02 mire joined #parrot
22:03 particle seems that "print $S" isn't causing parrot to error on win32
22:03 particle instead, it segfaults.
22:04 Infinoid bonus!
22:14 Infinoid limited time offer, low low prices.
22:18 particle hrm. interesting.
22:19 nopaste "particle" at 24.19.3.148 pasted "bt of seggy with parrot t/compilers/imcc/syn/symbols_1.pir" (14 lines) at http://nopaste.snit.ch/12770
22:20 Infinoid intl.dll?  gettext?
22:21 * Infinoid wonders if he broke particle with r27065
22:21 particle you did
22:21 particle i've attempted to fix locally
22:23 Infinoid hmm.  you could always reintroduce the typo... :)
22:24 Limbic_Region joined #parrot
22:26 wknight8111 it's an extension of "if it ain't broke, don't fix it"
22:26 wknight8111 "if fixing it breaks it, don't fix it"
22:27 Infinoid fixing the typo fixed RT #53112
22:27 Infinoid and apparently, broke msvc.
22:29 Patterner "FixedIntegerArray: Can't resize! current instr.: 'parrot;PGE::Util;__onload' pc -1 ((unknown file):-1)"   parrot Perl6Grammar.pir PGE/builtins.pg
22:30 particle Patterner: did you realclean lately?
22:30 Patterner yes
22:30 Patterner r27139
22:31 particle harumph.
22:33 particle ah. this looks much better with the typo in place
22:33 particle ok, now i know what to fix
22:38 Limbic_Region thanks to whomever fixed the Win32/Cygwin bug preventing the build
22:38 Limbic_Region both Win32/MinGW and Win32/Cygwin are now stuck in nqp after parrot.exe is successfully linked
22:39 Limbic_Region which means perl6.exe can't be built either :-(
22:59 kid51 joined #parrot
23:05 tetragon joined #parrot
23:08 * tetragon realises that she still has a gdb session open on the r27081 crash
23:10 kid51 tetragon:  Was there any resolution of that problem we were looking at this morning?
23:11 tetragon The crash?  Not as of last I checked (this morning)
23:20 tetragon Really should finish looking up everything I need to be able to set a usable watchpoint for chromatic
23:22 cognominal left #parrot
23:28 cognominal joined #parrot
23:36 skids joined #parrot
23:36 tetragon Crash still happening for me on a clean r27139

| Channels | #parrot index | Today | | Search | Google Search | Plain-Text | summary

Parrot | source cross referenced