Camelia, the Perl 6 bug

IRC log for #parrot, 2009-10-13

Parrot | source cross referenced

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

All times shown according to UTC.

Time Nick Message
00:02 Tene all of the thread.t tests that fail fail with a segfault.
00:04 Whiteknight threading is definitely something Parrot needs to work on
00:04 Whiteknight there is a lot of cruft floating around, we really need to figure out what all we have
00:07 Tene the run_clone method that's failing isn't defined as a METHOD, it's set up at runtime with register_nci_method()
00:09 Tene looks like it dates back to 'creiss' in 2006
00:09 Tene I've never seen that name before.
00:10 Tene seems like it needs significant updates.
00:16 dalek TT #1106 created by Austin_Hastings++: Cannot presently dump Class pmcs
00:16 Whiteknight what are all these "features" things that people keep wanting?
00:17 cotto_work They're like bugs, except they work.
00:33 preflex joined #parrot
00:36 Whiteknight urg, why are all those threading tests on pcc_reapply failing?
00:37 Whiteknight and why are so many of these stupid failing tests in perl5?
00:39 cotto_work because nobody ever goes there
00:39 cotto_work (not that you didn't already know that)
00:44 Whiteknight the threading system should really be a higher priority then people have made it out to be
00:44 Whiteknight we would be making a big mistake if we shipped 2.0 without good, stable, scalable threading support
00:46 cotto_work That'll be a challenge.
00:56 rhr joined #parrot
01:08 rhr joined #parrot
01:33 rhr joined #parrot
01:34 dalek nqp-rx: 5045605 | pmichaud++ |  (6 files):
01:34 dalek nqp-rx: Handle positional (numbered) captures.
01:34 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/5​045605950a92454b91dc73cf93438b028cfc0c6
01:34 shorten dalek's url is at http://xrl.us/bfr4j3
01:35 kid51 joined #parrot
01:40 dalek nqp-rx: 032c58a | pmichaud++ | t/p6regex/rx_ (5 files):
01:40 dalek nqp-rx: Add remaining test files from PGE (not active/configured yet).
01:40 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/0​32c58a2a79ed6fd8e045421ff9632f9b2b3c503
01:40 shorten dalek's url is at http://xrl.us/bfr4mi
01:42 Whiteknight the failure in t/pmc/codestring.t makes no sense to me
01:42 Whiteknight I just traced through it in gdb and the result value looked correct
01:44 dalek parrot: r41836 | jkeenan++ | branches/auto_libjit:
01:44 dalek parrot: Create a branch to test plobsing's patch in
01:44 dalek parrot: https://trac.parrot.org/parrot/ticket/1105.
01:44 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41836/
01:52 Whiteknight urg, this codestring failure is turning out to be dfficult to debug
01:58 Tene Whiteknight: I looked at it, and I can confirm that when I changed the return(intval -1) to return(intval 42), it successfully returned 42
01:58 Tene so there's a problem with returning a constant -1 from a method, or something.
02:00 Whiteknight well that's horrible
02:01 msmatsko joined #parrot
02:01 Tene Quite!
02:01 Whiteknight okay, well I'm out of time for the night. Goodnight
02:01 Tene Goodnight.
02:02 rhr joined #parrot
02:22 kid51 Why is dalek so slow?
02:23 dalek parrot: r41837 | jkeenan++ | branches/auto_libjit (19 files):
02:23 dalek parrot: Apply in branch plobsing's patch submitted in
02:23 dalek parrot: https://trac.parrot.org/parrot/ticket/1105, with these modifications:
02:23 dalek parrot: 1.  Move config step gen::libjit up to 4th last step.
02:23 dalek parrot: 2.  Add SVN properties and rebuild MANIFEST.
02:23 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41837/
02:26 dalek nqp-rx: 76c694d | pmichaud++ | src/Regex/P6Regex/Actions.pm:
02:26 dalek nqp-rx: Add aliased assertion captures.
02:26 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/7​6c694d64f400d8fa9ec049148b6fa43578016fd
02:26 shorten dalek's url is at http://xrl.us/bfr4ue
02:26 dalek nqp-rx: 4481429 | pmichaud++ | src/Regex/P6Regex/ (2 files):
02:26 dalek nqp-rx: Handle simple binding aliases.
02:26 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/4​48142986da97580857e8994a657cfb13e0cbfb8
02:26 shorten dalek's url is at http://xrl.us/bfr4ug
02:32 mikehh joined #parrot
02:35 janus joined #parrot
02:35 rhr joined #parrot
02:41 Tene dalek isn't slow, it's delayed.
02:41 Tene speed and offset are distinct.
02:52 rhr joined #parrot
02:56 jsut joined #parrot
03:28 JimmyZ joined #parrot
03:29 rhr joined #parrot
04:01 preflex joined #parrot
04:09 dalek nqp-rx: 568cc9b | pmichaud++ | src/ (3 files):
04:09 dalek nqp-rx: Allow PAST::Regex subrule nodes to have arguments to pass to the subrule.
04:09 dalek nqp-rx: Implement simple backreferences.
04:09 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/5​68cc9b7c9f2316c79a57a9b12183ecbe9242ea0
04:09 shorten dalek's url is at http://xrl.us/bfr5ap
04:21 preflex joined #parrot
05:06 dalek nqp-rx: 3de5a4f | pmichaud++ | src/ (2 files):
05:06 dalek nqp-rx: Add alias capturing of non-subrules.
05:06 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/3​de5a4fdac76d93741da3fd12576201276d3fa73
05:06 dalek nqp-rx: f00fd0d | pmichaud++ | src/Regex/P6Regex/Grammar.pm:
05:06 dalek nqp-rx: Allow spaces in character class compositions.
05:06 shorten dalek's url is at http://xrl.us/bfr5f4
05:06 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/f​00fd0d6e7f355532c258ae7df63fbc5159c3c7c
05:06 dalek nqp-rx: 8743056 | pmichaud++ | src/Regex/P6Regex/Actions.pm:
05:06 dalek nqp-rx: Add character class composition.
05:06 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/8​743056645dd50efc011bb310c84118ae9862d29
05:06 shorten dalek's url is at http://xrl.us/bfr5f6
05:06 shorten dalek's url is at http://xrl.us/bfr5f8
05:21 japhb joined #parrot
05:33 KatrinaTheLamia joined #parrot
06:00 uniejo joined #parrot
06:18 KatrinaTheLamia joined #parrot
06:39 kyle_l5l joined #parrot
06:53 dukeleto 'ello
07:31 JimmyZ joined #parrot
07:32 bacek joined #parrot
07:33 bacek o hai
07:35 JimmyZ bacek: hello
07:35 purl salut, JimmyZ.
07:35 bacek G'Day JimmyZ
07:35 bacek what current status of pcc_reapply?
07:38 dukeleto how do I import subs from a PBC (Test/More.pbc) to the current namespace in NQP?
07:41 nopaste "moritz" at 87.176.59.93 pasted "Error summary from pcc_reapply for bacek++" (19 lines) at http://nopaste.snit.ch/18325
07:42 bacek moritz: thanks
07:53 Tene bacek: the next thing that needs fixed, IMO, is parrot_config
07:54 bacek Tene: it's one line fix.
07:54 Tene dukeleto: for now, you use ~3 lines of embedded PIR, or add it to glue.pir
07:54 Tene dukeleto: 'sec
07:55 bacek Tene: config_lib_pasm.in, line 14. Replace "null P1" with "P1 = new 'Undef'"
07:55 Tene bacek: why hasn't that already been committed?
07:55 bacek Tene: I'm not so magical as someone thinks
07:56 Tene bacek: that is to say, I heard someone saying earlier that you had a workaround for the problem.  Is there some problem with that fix, or did I misunderstand?
07:56 nopaste "tene" at 24.10.252.130 pasted "pir import for dukeleto++" (4 lines) at http://nopaste.snit.ch/18326
07:57 Tene dukeleto: if you put that in a sub in glue.pir, fetch the caller namespace and pass it as a second argument to import()
07:57 bacek Tene: it's probably hiding something else. Or just proper fix for previous undefined behaviour. I'm not sure. That's why I didn't commit it
07:57 Tene dukeleto: erm, not a second argument, a named argument :named('targetns')
07:57 baest joined #parrot
08:01 nopaste "tene" at 24.10.252.130 pasted "import sub for dukeleto++" (9 lines) at http://nopaste.snit.ch/18327
08:02 Tene bacek: with that applied, I can build and pass all of the tests in my scheme compiler.
08:03 bacek Tene: SHIP IT!
08:06 dalek parrot: r41838 | tene++ | branches/pcc_reapply/config/gen​/config_pm/config_lib_pasm.in:
08:06 dalek parrot: [pcc] Use undef instead of null in config_lib.  bacek++
08:06 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41838/
08:10 Tene bacek: trying to build rakudo...
08:10 Tene rakudo mostly builds... ten fails at the end.
08:10 nopaste "tene" at 24.10.252.130 pasted "rakudo pcc build failure" (31 lines) at http://nopaste.snit.ch/18328
08:11 Tene although, that's out-of-date rakudo
08:11 moritz Tene: that looks very much like the resig branch, not trunk
08:11 moritz or at least like a Makefile from that branch
08:11 Tene moritz: Oh!  You're right!
08:12 moritz and that's known not to build on linux right now (except with modifications to the makefile)
08:12 Tene Lemme try again, except not stupid this time.
08:13 dukeleto Tene: when I import from within a Q:PIR block, it will be visible from NQP?
08:13 nopaste "tene" at 24.10.252.130 pasted "real rakudo build failure" (284 lines) at http://nopaste.snit.ch/18329
08:14 Tene dukeleto: yes.  If you run the code in my first paste from Q:PIR, it will work.
08:15 dukeleto Tene: i just saw the nopaste
08:16 dalek parrot-plumage: 341bf8b | leto++ | :
08:16 dalek parrot-plumage: [t] Properly test some exit codes using Test::More
08:16 dalek parrot-plumage: review: http://gitorious.org/parrot-plumage/parrot-plumag​e/commit/341bf8b9b3248ce1a6dd3b543bb50dba991dc55f
08:16 shorten dalek's url is at http://xrl.us/bfr5re
08:16 Tene dukeleto: I pasted two times.  Once with code for Q:PIR, and again with code to put in an 'import' sub in glue.pir, if you wanted to do it that way.
08:17 Tene Ooo, failure in imcc!
08:19 * Tene sleep now.
08:20 dukeleto Tene++ # thanks for the import help!
08:21 dalek parrot-plumage: 6c37408 | leto++ | :
08:21 dalek parrot-plumage: [t] Properly import functions from Test::More and add some tests using l...
08:21 dalek parrot-plumage: review: http://gitorious.org/parrot-plumage/parrot-plumag​e/commit/6c37408ac7f35586786c92e8f75fb33d1715bf44
08:21 shorten dalek's url is at http://xrl.us/bfr5ri
08:25 mikehh pcc_reapply - make smoke #28955 at r41838 - 10,691 ok, 17 failed, 262 todo, 560 skipped and 0 unexpectedly succeeded
08:26 dalek parrot-plumage: 375dbe8 | leto++ | :
08:26 dalek parrot-plumage: [t] Add basic tests for usage
08:26 dalek parrot-plumage: review: http://gitorious.org/parrot-plumage/parrot-plumag​e/commit/375dbe842d653cff91c26d82cb0c73f5df689c79
08:26 shorten dalek's url is at http://xrl.us/bfr5rz
09:15 payload joined #parrot
09:57 dalek tracwiki: v9 | kurahaupo++ | ArrayTasklist
09:57 dalek tracwiki: https://trac.parrot.org/parrot/wiki/Ar​rayTasklist?version=9&action=diff
09:57 shorten dalek's url is at http://xrl.us/bfr5yj
09:57 donaldh joined #parrot
09:57 donaldh left #parrot
10:03 dalek nqp-rx: 348b39f | pmichaud++ | src/Regex/constants.pir:
10:03 dalek nqp-rx: Turn some magic constants into symbols.
10:03 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/3​48b39fc4b6f51410d6db1655275985e933fe7da
10:03 dalek nqp-rx: 45fcb57 | pmichaud++ |  (2 files):
10:03 shorten dalek's url is at http://xrl.us/bfr5zg
10:03 dalek nqp-rx: Convert to use symbols instead of magic numbers.
10:03 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/4​5fcb5777aafbc39b64a7974d3a81ee09297ebc7
10:03 shorten dalek's url is at http://xrl.us/bfr5zi
10:03 dalek nqp-rx: bdc43da | pmichaud++ | src/Regex/Cursor.pir:
10:03 dalek nqp-rx: Revise !mark_commit to preserve capture stack.
10:03 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/b​dc43da7cae68c778fe8ffe09da5c37d945c4989
10:03 shorten dalek's url is at http://xrl.us/bfr5zk
10:03 dalek nqp-rx: 1987a95 | pmichaud++ |  (4 files):
10:03 dalek nqp-rx: Add cut-rule token.
10:03 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/1​987a95f3bba6c4f11073954c8ffa6bd41e1937e
10:03 shorten dalek's url is at http://xrl.us/bfr5zn
10:09 dalek nqp-rx: 105b3e0 | pmichaud++ | src/PAST/Compiler-Regex.pir:
10:09 dalek nqp-rx: More replacing magic numbers with symbolic constants.
10:09 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/1​05b3e07f20c7580a5ff648d4a6ff9db88edd42e
10:09 shorten dalek's url is at http://xrl.us/bfr5zt
10:16 mikehh trunk - pre/post-config, smoke (#28589) PASS,fulltest FAIL at r41838 (see TT #1102) - Ubuntu 9.10 (beta) amd64
10:16 mikehh fulltest (testf, testg and testS) (same tests FAIL, same results)
10:16 mikehh t/compilers/imcc/syn/macro.t - Failed tests:  2-4
10:16 mikehh t/compilers/imcc/syn/regressions.t - Failed test:  7
10:49 payload joined #parrot
10:51 payload joined #parrot
10:53 payload joined #parrot
11:29 tetragon joined #parrot
11:29 tetragon_ joined #parrot
11:34 tetragon joined #parrot
11:36 masak joined #parrot
11:46 kid51 joined #parrot
11:47 dalek rakudo: 609cc25 | moritz++ | src/classes/Object.pir:
11:47 dalek rakudo: make the get_integer vtable class .Int; fixes RT #69738
11:47 dalek rakudo: review: http://github.com/rakudo/rakudo/commit/6​09cc25ae44e9b19a9e505796444fb42e645b703
11:47 shorten dalek's url is at http://xrl.us/bfr564
12:01 bluescreen joined #parrot
12:06 mokurai left #parrot
12:32 Coke All these priorities being set are nice. What about my segfaults? =-)
12:33 dalek rakudo: 3d1afed | moritz++ | src/setting/Operators.pm:
12:33 dalek rakudo: give more awesome error message for infinite ranges
12:33 dalek rakudo: Closes RT #69728
12:33 dalek rakudo: review: http://github.com/rakudo/rakudo/commit/3​d1afedfd18a85d570c96937d9abe7f0d6128c49
12:33 shorten dalek's url is at http://xrl.us/bfr6c6
12:37 Coke msg pmichaud (and other sixers) I have a lead on someone in the UK who is interested in putting together some multimedia tutorials and thinking about p6 as a "intro language" curriculum.
12:37 purl Message for pmichaud stored.
12:37 bluescreen joined #parrot
12:37 moritz oh cool
12:40 tokuhirom________ joined #parrot
12:41 whiteknight joined #parrot
12:48 Coke moritz: I am going to give some feedback and encourage him to repost to a list. =-)
12:49 JimmyZ joined #parrot
12:49 moritz Coke: +1
12:49 purl 1
12:50 moritz purl: brain the size of an atom, and all used up for joining IRC channels
12:50 purl moritz: sorry...
12:50 Coke purl, dirac?
12:50 purl it has been said that dirac is at http://sourceforge.net/projects/dirac
12:50 Coke purl, dirac was also paul.
12:50 purl Coke: huh?
12:50 Coke ha! look at comment #2 there.
12:51 Coke (purl's url)
12:51 jonathan lol...review fail
12:54 whiteknight good morning #parrot
13:00 whiteknight fail? "Fuck you" is a very valid review
13:00 whiteknight :)
13:01 masak yeah, but thumbs up + "fuck you"?
13:01 bluescreen joined #parrot
13:03 masak Infinoid: yes, if the commit looks right, please merge.
13:04 * Coke wonders if it's just him or if feather is getting slow.
13:06 Coke if I'm using the sha1 to identify revs and want something human-friendly, is --short mostly safe?
13:06 Coke (e.g. to replace the numbers in column 2 here:) http://github.com/partcl/partcl/blo​b/master/docs/spectest-progress.csv
13:06 shorten Coke's url is at http://xrl.us/bfr6gm
13:13 dalek partcl: 329ce26 | coke++ | tools/tcl_test.pl:
13:13 dalek partcl: Just pull from the known repo where the wiki lives.
13:14 dalek partcl: review: http://github.com/partcl/partcl/commit/3​29ce2643b2d7f1ef8807517187f5ed0042eb807
13:14 dalek partcl: d034e86 | coke++ |  (2 files):
13:14 dalek partcl: We are using git for version control now, can't use svn/git-svn to get a rev.
13:14 dalek partcl: review: http://github.com/partcl/partcl/commit/d​034e8684080752bc58f5c403d1d86b9d5c5a820
13:14 shorten dalek's url is at http://xrl.us/bfr6hn
13:14 shorten dalek's url is at http://xrl.us/bfr6hp
13:14 whiteknight it's so cute when the bots have like a little conversation with each other?
13:14 whiteknight dalek's url?
13:14 whiteknight purl dalek's url?
13:14 purl whiteknight: bugger all, i dunno
13:15 whiteknight joined #parrot
13:26 Coke shorten didn't need to shorten that.
13:26 purl That URL is at http://xrl.us/bfr6iq
13:26 Coke [::facepalm::]
13:27 Coke sadly, xrl.us doesn't reuse short urls.
13:31 barney joined #parrot
13:39 whiteknight purl msg allison I can't get pcc_reapply to build on win32. src/nci_test.c gives me errors about PMCNULL being undefined. I'm wondering if you've seen that error before.
13:39 purl Message for allison stored.
14:01 Coke whiteknight: missing an include?
14:02 Coke (why do we have a _test file in src?)
14:02 whiteknight no idea
14:12 Psyche^ joined #parrot
14:13 shockwave joined #parrot
14:13 dalek tracwiki: v10 | coke++ | ArrayTasklist
14:13 dalek tracwiki: add some comments.
14:13 dalek tracwiki: https://trac.parrot.org/parrot/wiki/Ar​rayTasklist?version=10&action=diff
14:13 dalek tracwiki: v5 | coke++ | AllHackathons
14:13 dalek tracwiki: https://trac.parrot.org/parrot/wiki/Al​lHackathons?version=5&action=diff
14:13 dalek tracwiki: v4 | coke++ | ItsABughunt
14:13 dalek tracwiki: https://trac.parrot.org/parrot/wiki/I​tsABughunt?version=4&action=diff
14:13 shorten dalek's url is at http://xrl.us/bfr6qg
14:13 shorten dalek's url is at http://xrl.us/bfr6qi
14:13 shorten dalek's url is at http://xrl.us/bfr6qk
14:19 schmalbe joined #parrot
14:20 Coke shorten, you're an idiot
14:20 Coke shorten, owner?
14:20 Coke purl, shorten?
14:20 purl shorten is a bot interface to http://metamark.net/ or shortens long urls in the channel or shortens shorter urls in /msgs or made by ask or http://thefeed.no/marcus/archives/000327.html # osx service or deaf to repetition or http://metamark.net/bot
14:21 Coke shorten =~ s/or http://thefeed.no/marcus/archives/000327.html # osx service//
14:23 dalek parrot: r41839 | mikehh++ | branches/pcc_reapply/src/call/args.c:
14:23 dalek parrot: codetest fix - missing function documentation
14:23 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41839/
14:25 mikehh I just set it up so that it passes codetest ( amke make -j test etc) - I think the documentation in src/call/args.c should be reviewed
14:25 mikehh Coke: src/nci_test.c - shared library used for testing the Native Call Interface
14:26 mikehh s/amke/make/ above
14:27 hachi_ joined #parrot
14:27 mikehh Coke: it is in trunk
14:30 jonathan (bug where calling things from c that tailcall leads to segfault in Parrot)--
14:33 Coke mikehh: yes. but why is a testing file in src/
14:33 Coke should be in t/, neh?
14:34 Coke mikehh: argh.
14:34 Coke mikehh: scratch that argh.
14:35 fperrad joined #parrot
14:39 jrtayloriv joined #parrot
14:47 theory joined #parrot
14:59 Coke msg dukeleto - if you can figure out why commit messages are not showing up from github on partcl-commits, I'll give you a cookie.
14:59 purl Message for dukeleto stored.
15:03 allison joined #parrot
15:05 whoppix joined #parrot
15:11 moritz wow, the amount of failed tests in the pcc_reapply branch is impressively low
15:11 moritz 15 or so
15:20 Coke what the holy (*&#. partcl-commits just got spammed by someone. no github email, but spam. :|
15:20 jonathan moritz: Really?
15:20 jonathan Nice!
15:22 Coke msg dukeleto it's working no. no cookie for you! =-)
15:22 purl Message for dukeleto stored.
15:23 payload joined #parrot
15:30 Coke moritz: are you counting failures where N tests were planned but only N-M were run?
15:31 Coke I'm seeing hundredes of failures.
15:31 Coke (in pcc_reapply)
15:31 mikehh Coke: it's been around for at least six years - the first entry in trac svn revision log is r5205 leo
15:32 mikehh Coke in fact that was a change to r5204 - (checked in by leo, 6 years ago)
15:32 Coke mikehh: ... yes.
15:32 Coke that doesn't mean it /belong/s there.
15:32 mikehh probably not :-}
15:32 Coke I just figured if he was looking at the file anyway, now might be a good time to put it where it belongs.
15:34 bluescreen joined #parrot
15:43 Andy joined #parrot
15:49 moritz Coke: I ccounted the errors as shown in failure summary
15:54 Coke moritz; look for lines like:
15:54 nnunley joined #parrot
15:55 Coke Parse errors: Bad plan.  You planned 76 tests but ran 4.
15:55 Coke there's 64 potential failures right there.
15:55 Coke (yes, it could just be one.)
16:03 payload joined #parrot
16:24 cotto_work joined #parrot
16:27 chromatic joined #parrot
16:31 HG` joined #parrot
16:32 kurahaupo joined #parrot
16:32 mikehh joined #parrot
16:34 dalek nqp-rx: 8276d6c | pmichaud++ | src/ (4 files):
16:34 dalek nqp-rx: Add null/fail PAST::Regex nodes (as subtype of 'anchor' pasttype).
16:34 dalek nqp-rx: Add initial parsing and handling for internal modifiers.
16:34 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/8​276d6cde4dcb44282d0d252ae8a306e9a440990
16:34 shorten dalek's url is at http://xrl.us/bfr7gg
16:34 dalek nqp-rx: 3cc84fe | pmichaud++ |  (4 files):
16:34 dalek nqp-rx: Add :ignorecase/:i modifier for literals.
16:34 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/3​cc84fed71b5da6399ac6bedd4900bb62793c245
16:34 shorten dalek's url is at http://xrl.us/bfr7gi
16:41 msmatsko joined #parrot
16:54 einstein joined #parrot
16:58 ruoso joined #parrot
17:04 mikehh got to reboot - bbiab
17:07 Tene Coke: I don't get any failures like that.
17:08 dalek tracwiki: v11 | kurahaupo++ | ArrayTasklist
17:08 dalek tracwiki: notes on thread safety, exponential resizing
17:08 dalek tracwiki: https://trac.parrot.org/parrot/wiki/Ar​rayTasklist?version=11&action=diff
17:08 shorten dalek's url is at http://xrl.us/bfr7pe
17:11 Coke tene: I get tons.
17:11 Coke are you using buildframes=0 ?
17:11 Coke (I was not)
17:11 nopaste "tene" at 24.10.252.130 pasted "My test results summary for Coke++" (14 lines) at http://nopaste.snit.ch/18330
17:12 Tene So I guess I get that once in t/library/coroutine.t for 4 tests.
17:13 nopaste "Coke" at 72.228.52.192 pasted "my test failures for Tene++" (71 lines) at http://nopaste.snit.ch/18331
17:13 Tene The frame builder hasn't ever worked on my platform anyway, coke.
17:13 Coke k.
17:13 Coke Just saying, it's not as rosy here.
17:14 Tene Coke: failures with the frame builder aren't related to algorithmic failures of the pcc refactor.
17:14 Coke I have no idea what these failures are from.
17:14 Coke I just mentioned that as I've seen kid51++ mention it.
17:15 Tene That's the remnants of oldjit.
17:15 Coke if there are failures in the branch that aren't in trunk, I'd say they are at least slightly related. =-)
17:16 Tene the NCI frame builder, which we expect to remove shortly after the upcoming release, hasn't been updated in the branch to use the new calling conventions API.
17:18 dalek tracwiki: v12 | kurahaupo++ | ArrayTasklist
17:18 dalek tracwiki: notes on applicability of uninitialized resizable arrays; more details on "fully general" arrays
17:18 dalek tracwiki: https://trac.parrot.org/parrot/wiki/Ar​rayTasklist?version=12&action=diff
17:18 shorten dalek's url is at http://xrl.us/bfr7re
17:20 Coke ok. So those failures are real, then.
17:21 Coke you just don't happen to be tripping them on your platform.
17:21 Coke yes?
17:22 Tene Depends on who you ask, and what your tolerance for semantic dickery is. ;)
17:22 pmichaud lunchtime, #ps in 68
17:22 Tene (more)
17:25 whiteknight I made an offer last night that if we ripped the frame builder out directly after the release, that I would guarantee a replacement by 1.8
17:25 whiteknight nobody took me up on it
17:25 Tene Yes, they need to be either fixed or the frame builder removed.  The people working on PCC haven't addressed it yet because they've been more worried about getting PCC working correctly first, and everyone hates the frame builder, and the most-likely person to work on it, WK, hates it SO VERY MUCH, and now there's politics, which is perfectly legitimate because he can choose what he spends his very-appreciated volunteer time on, but ...
17:25 Tene ... doesn't help the state of the branch or whatever.
17:26 Coke whiteknight: I don't understand. what's to stop you from working on it in a branch?
17:26 Coke where you can rip it out and replace it and then there's no risk to anyone?
17:26 Tene whiteknight: Your offer was that you would fix the frame builder failures if (what you said)
17:26 Austin joined #parrot
17:27 Tene as I remember it, yes?
17:27 whiteknight Yes. What I really said was that I would fix it now and make the failures go away, AND provide a replacement by 1.8, IF I was allowed to rip it out directly after the release
17:28 Austin heh
17:28 whiteknight What I'm looking for is some kind of commitment that people are actually ready to take the next step
17:28 whiteknight it's a leap of faith to rip it out without having a replacement ready immediately
17:28 Austin Isn't that what branches are for?
17:29 whiteknight Austin: it's not a matter of where I work, it's a matter of when
17:29 whiteknight I have tuits available to do X task in Y timeframe. No tuits after Y timeframe
17:29 Austin Ahh.
17:30 Coke whiteknight: so why not do it in a branch, no faith required?
17:30 Coke then if you happen to miss the mark, it's in a state where a) trunk is unaffected, and b) someone can pick it up.
17:30 Austin Look, just give the baby up for adoption. A child shouldn't have to compete with JIT compilation for daddy's attention...
17:30 Coke so do it now, in a branch. You can start even before the release.
17:30 whiteknight maybe I'm not explaining entirely correctly
17:31 Austin Then you'll have lots of time available.
17:31 Coke I'm certainly confused.
17:31 Austin :)
17:31 Coke I'm willing to believe it's me. =-)
17:31 whiteknight It's a tit-for-tat deal. I'll fix the frame builder test failure now, in exchange, I get to pull out the frame builder after 1.7
17:32 whiteknight This being a FOSS project, and me having no obligations besides my own desires, I'll do something I don't want in exchange for doing something I do want
17:32 pmichaud what prevents you from pulling out the frame builder after 1.7 anyway?
17:32 pmichaud i.e., I'm not seeing the obstacle.
17:32 dalek nqp-rx: 2cd0942 | pmichaud++ | src/Regex/P6Regex/ (2 files):
17:32 dalek nqp-rx: Add :sigspace support.  Fix up comment parsing.
17:32 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/2​cd0942377dfdb668659f5d09bfc67ec1ec71f84
17:32 shorten dalek's url is at http://xrl.us/bfr7ud
17:33 whiteknight pmichaud: people have said we shouldn't pull it out until we have a replacement ready
17:33 Tene pmichaud: chromatic and allison will complain if he removes the frame builder without already having a replacement.  That is, they object to having trunk in a no-frame-builder state at all.
17:33 Austin pmichaud: Do multi-methods hide entire inherited names, or just hide inherited multi signatures? That is, can I have Parent::multi(_, Foo) and derive Child, with Child::multi(_, Bar), and still call Child.multi($foo) okay?
17:33 Tene and WK doesn't want to make the replacement in a branch.
17:33 Coke whiteknight: I think you're making this into a fight when it doesn't have to be.
17:33 whiteknight right, and I'm not going to do what I've been told not to do. Hence, my bargain to change things
17:33 whiteknight I'm not fighting
17:33 Coke why not do all this in a branch, then?
17:33 pmichaud whiteknight: you can certainly remove the frame builder in a branch.
17:33 pmichaud nobody will object to that.
17:34 whiteknight that's not the issue
17:34 pmichaud we're not seeing the issue.
17:34 whiteknight that's fine, it's my issue
17:34 pmichaud Austin: in Parrot I think that multi-methods hide entire inherited names.
17:34 Coke ok. I vote against doing it in trunk, fwiw.
17:34 pmichaud Austin: in Perl 6 they don't.
17:34 Coke I just don't see why this has to be in trunk.
17:35 whiteknight Coke: it doesn't matter where development happens. It can happen in a branch
17:35 Austin pmichaud: Okay. Is this a p6object thing? Is there some obvious way to "uplift" the masked methods?
17:35 Coke ... then I completely fail to see the problem.
17:35 Coke why can you not do this in a branch, then?
17:35 pmichaud Austin: It's parrot.
17:35 whiteknight I can do this in a branch. Location is not the problem
17:35 Coke that is, what's stopping you?
17:35 Austin Coke: I think the issue is commitment to incorporating the changes.
17:36 Coke without knowing what the replacement will look like, how can we agree?
17:36 whiteknight exactly
17:36 whiteknight Coke: because any replacement is better then what we have
17:36 whiteknight an empty file is better
17:36 Austin If he rips out the code, and everybody says, "Well, we are going to work on Unicode in this release. Maybe in a few months" that would suck.
17:36 pmichaud Austin: that particular behavior of methods is entirely Parrot.  P6object doesn't try to modify Parrot's metaobject model in that way.
17:36 Tene Coke: I think it's something like "If I do it in a branch, then people can ignore th elack of a frame builder and potentially keep using the old one; if it's in trunk, then it's everyone's problem."
17:36 Coke right. i don't want a problem.
17:36 whiteknight no problems
17:37 Coke sure, ripping it out without a plan more concrete than 'we'll improve it' seems to be a problem to me.
17:37 Austin pmichaud: Okay. (I sort of conflate p6object w/ parrot, here.)
17:37 whiteknight there is a general plan about replacing the frame builder. I made a bid to fix some bugs in exchange for changing around the timeline a little. No takers, things go the way they are going
17:37 whiteknight no harm, no foul
17:37 Tene Coke: WK sees the existing frame builder as BAD CODE, and removing it completely to be an improvement.  Others disagree.
17:37 whiteknight I'll spend my tuits elsewhere, I have plenty of other projets to work on
17:37 Tene Coke: for example, every architecture other than non-darwin x86 works "fine" witout it.
17:37 Coke elsewhere than parrot, or in parrot?
17:37 whiteknight In Parrot
17:38 whiteknight GC, IO, IMCC/PIRC, Matrixy, etc.
17:38 whiteknight PCC is still going to need lots of love too, after this branch lands
17:38 Tene Coke: and this isn't the first time that oldjit has stalled otherwise-promising development.
17:39 joeri joined #parrot
17:39 Coke Tene: s/oldjit/old crappy placeholder code/ in general.
17:39 pmichaud whiteknight: in my experience, good code gets adopted into trunk pretty quickly, provided it's not a violation of the deprecation policy
17:40 Tene Coke: the main objection comes from chromatic, who sees the precompiled NCI thunks as more evil than the frame builder, even though we still have to use the precompiled thunks on every other platform.
17:40 pmichaud so, either you're saying that what you propose to write won't be good code, or that it violates the deprecation policy
17:40 Tene pmichaud: afaict, removing the frame builder wouldn't violate deprecation policy.
17:40 Austin pmichaud: If I do a find-method on the multi name, I get the parent's multi, right? So could I then clone it and add the extras, then add-method the cloned multi?
17:40 whiteknight no. I'm still not being too clear
17:40 Coke maintaining 2 bits of evil code is worse than 1 bit, and if we're eventually going to be using something else entirely anyway, I fail to see the point of keeping it.
17:40 PerlJam Tene: got a link to some place where chromatic explains that himself?
17:41 PerlJam Tene: (precompiled thunks evil)
17:41 pmichaud Austin: we do things like that in Rakudo, yes (more)
17:41 Tene PerlJam: irc logs from yesterday.  the summary is that *HALF* of our startup time is because of precompiled thunks, and half of the cost of every PMC.
17:41 whiteknight pmichaud: frame builder is causing failures in pcc_reapply. There are two options: fix it or kill it. People would like to fix it so we have a frame builder in place for the 1.7 release (more)
17:41 PerlJam Tene: ah, thanks.
17:41 purl ah, thanks. is it fast enough?
17:41 pmichaud Austin: another possibility would be to have a fallback method in the subclass that is :multi()  (no args) and forwards to the parent method
17:41 chromatic I consider that a regression at least on x86.
17:41 whiteknight I want to kill it. So I say that I will fix it for 1.7, in exchange for being able to kill it directly thereafter
17:42 Austin pmichaud: multi(), or multi(_) on self?
17:42 whiteknight the issue isn't killing it or not, it's when we do it. I'll fix the failures now, in exchange for killing it soon
17:42 pmichaud Austin: either might work.  Depends on the semantics you'd like.
17:42 whiteknight as opposed to me not fixing it now (maybe somebody else will) and we kill it later
17:42 Coke whiteknight: ah. you're not saying you'll write a replacement.
17:42 whiteknight Coke: I will, that's part of my deal sweetener
17:42 Coke you're saying you'll make the branch work so we can kill it more quickly and then not replace it?
17:43 chromatic Exactly.
17:43 Tene Coke: he's also committing to writing a replacement, but wants the frame builder removed regardless.
17:43 whiteknight and I provide a replacement in 1.8
17:43 Coke whiteknight: because we are volunteers, I don't see why you have to do it in this order.
17:43 pmichaud so let me attempt to summarize
17:43 Coke if we assume we need a frame builder, and we rip it out, and then something happens to your plans between 1.7 and 1.8... what then?
17:43 whiteknight that way not a single release goes out the door without a frame builder AND we kill code I hate at the earliest possible moment
17:44 pmichaud actually, I won't attempt it.
17:44 whiteknight Coke: that's where the leap of faith comes in
17:44 whiteknight I didn't say it was a perfect offer
17:45 whiteknight and I'll be working on JIT anyway, just on a different timeframe otherwise
17:45 whiteknight in any case, it's not a big deal
17:46 Coke ok. so, back to my "I have more tests failing in pcc_reapply than tene". chromatic: removing buildframes is a regression?
17:47 whiteknight in terms of performance it's a regression
17:47 chromatic Every failing test in pcc_reapply right now is a regression.
17:47 Coke ok. we have no guarantees on performance, and the HLLs are already pretty slow. what's the slowdown factor?
17:47 chromatic Parrot takes twice as long to start.
17:48 Coke chromatic: if we switch the default sense on --buildframes, do most of those failures go away?
17:48 whiteknight we know that registering hundreds of NCI thunks is a huge slowdown
17:48 chromatic the shared library is 30% larger
17:48 Coke ok. size is not something we guarantee between releases...
17:48 chromatic I don't know about "most" but "many" do.
17:48 cotto_work chromatic, is this a case of things getting worse before they can get better?
17:48 chromatic If we can't fix the frame builder on the branch and it would block merging, then yes cotto_work.
17:49 whiteknight ...or a case of it getting better before it gets awesome?
17:50 dalek tracwiki: v13 | kurahaupo++ | ArrayTasklist
17:50 dalek tracwiki: validity of negative indeces
17:50 dalek tracwiki: https://trac.parrot.org/parrot/wiki/Ar​rayTasklist?version=13&action=diff
17:50 shorten dalek's url is at http://xrl.us/bfr7wu
17:51 allison I'm okay with disabling buildframes for the merge, if we can't fix by the day after 1.7
17:51 allison (though fixing it would be nice)
17:51 whiteknight we're merging after 1.7?
17:52 whiteknight (not that I expect we could reasonably be in before)
17:52 allison I can't really test fixing it, since buildframes doesn't do anything on the platforms I have access to
17:52 allison yes, because a bigger concern is the speed of CallSignature
17:52 whiteknight ok
17:52 allison this gives chromatic time to work on it before we merge, or at least before 1.8
17:53 chromatic I don't mind trying to fix the frame builder in the branch, but I haven't for two reasons.
17:53 allison so we don't have a release hit by that slowdown
17:53 chromatic 1) I think CallSignature is a bigger priority
17:53 Tene allison: I could give you access to an x86 box.
17:53 allison agreed
17:53 whiteknight do we have any numbers on the slowdown?
17:53 chromatic 2) I'm not confident of doing it before the branch passes tests appropriately
17:53 whiteknight like, how bad it is?
17:53 chromatic 2.5 times slower, by the fib.pir benchmark.
17:53 allison 2) also agreed
17:53 whiteknight chromatic: Agreed, fixing tests is higher priority
17:53 whiteknight 2.5 times !?!?
17:53 whiteknight holy crap
17:53 purl only in the Vatican, my friend
17:53 chromatic That's for passing only one positional argument.
17:54 chromatic It gets worse O(n) per arguments.
17:54 dalek tracwiki: v14 | coke++ | ArrayTasklist
17:54 dalek tracwiki: https://trac.parrot.org/parrot/wiki/Ar​rayTasklist?version=14&action=diff
17:54 shorten dalek's url is at http://xrl.us/bfr7xr
17:54 whiteknight Oi vey!
17:54 chromatic That's almost all GC pressure.
17:54 whiteknight well, that's something we need to address ASAP
17:54 whiteknight I didn't realize it was so bad
17:55 allison whiteknight: see the CallSignature refactor items on the tasklist page
17:55 chromatic I'm working on a CallSignature refactor in a branch.
17:55 chromatic I'll poke at that today.
17:55 allison (added to flesh out what I said in a recent conversation with chromatic)
17:55 kurahaupo Earlier it was mentioned that a "general purpose array PMC" that could hold any value without needing a PMC would be a win. As I'm looking at refactoring the PMC's that implement arrays in various ways, would you like me to work on that as a starting point?
17:55 whiteknight I can take a look at the refactor too
17:56 Austin pmichaud: Is there a way to do "find_method" on a class, without having an instance? Barring that, is there a way to access the MRO from PIR? I know there's a C method, but I can't find any exposure of it.
17:57 pmichaud yes, it's possible to find the MRO
17:57 dalek tracwiki: v14 | whiteknight++ | CallingConventionsTasklist
17:57 dalek tracwiki: Move the CallSignature refactors section up a little bit because it's higher priority then other stuff
17:57 dalek tracwiki: https://trac.parrot.org/parrot/wiki/CallingCo​nventionsTasklist?version=14&action=diff
17:57 shorten dalek's url is at http://xrl.us/bfr7zg
17:57 Austin (without reimplementing C3?)
17:57 Austin :)
17:58 pmichaud the question of find_method on class objects came up in design meeting a couple of weeks ago -- let me check
17:58 Coke kurahaupo: who mentioned that?
17:58 Coke I would imagine that in order to handle any of SNIP types... you'd need something like a PMC.
17:58 Coke (so why not just use that if you need multiple types?)
17:59 Austin Coke: Probably to avoid the cost of creating the pmc.
17:59 * kurahaupo goes hunting in #parrot logs ...
17:59 Coke Austin: ok. so now you have... what. a C struct?
17:59 Coke that can contain any of the 4 types?
17:59 whiteknight chromatic: refactors look good, but we're definitely going to want to make sure that CallSignatures and the arguments therein are accessible from PIR
17:59 Coke and isn't GC'able?
17:59 whiteknight otherwise jonathan and pmichaud might murder us
18:00 Austin I'd guess something like the  flyweight  pattern, with the array tracking type externally.
18:00 allison whiteknight: the array and hash interfaces are the defined way to access the arguments
18:00 Coke Austin: if the type is tracked externally, what's the array contain, void *s ?
18:00 allison whiteknight: as in, the refactor has to support the same interface, but the internals can be anything
18:01 whiteknight allison: so long as we give them a way, I just want to make sure we don' lose sight of that
18:01 whiteknight gotcha
18:01 Austin Coke: I have no idea. A union, maybe?
18:01 kurahaupo coke: discussion between allison and chromatic circa 2009-10-10T07:43Z
18:01 Coke Austin: ah, that's what I meant when i said struct.
18:01 Coke whoops.
18:01 pmichaud Austin: (find_method)  --- there's a find_method method (note, not opcode) that walks the MRO of a class object
18:01 Austin Coke: Your guess is as good a mine. :$
18:02 Austin pmichaud: Aha! You rock. Thanks.
18:02 Coke I am unconvinced that a constructing such a beast is worth it, unless it means we can kill all the *<type>* variants.
18:02 moritz kurahaupo: you can make links to individual lines on http://irclog.perlgeek.de/parrot/2009-10-10 by clicking on the time in the left column
18:02 pmichaud src/pmc/class.pmc:1960   (or thereabouts, I haven't svn up'd in a while)
18:03 Austin pmichaud: Based on what you've told me so far, I'm guessing that parrot multi-method resolution stops with the first *name* that it finds in the MRO, rather than searching for other like-named multi's with closer matching arglists. Does this sound reasonable?
18:04 pmichaud there's also a way to find out all of the parent classes (mro order) by using 'all_parents' with the inspect opcode.
18:04 Coke kurahaupo: that conversation appears to be related specifically to CallSignature.
18:04 pmichaud Austin: that's been my experience, yes.
18:04 kurahaupo discussion starts roughly at http://irclog.perlgeek.de/p​arrot/2009-10-10#i_1588983
18:04 pmichaud Austin: it stops resolution at the first name encountered.
18:04 Austin pmichaud: Really? I looked for all_parents, but didn't find it exported.
18:04 Coke I could be mistaken, though.
18:05 pmichaud Austin: $P0 = inspect classobj, 'all_parents'    #  I think
18:05 pmichaud P6object uses it, so you could look there for an example.
18:05 Austin Nope, I'm wrong. It's not in the POD, but it is in the code. (What I get for running perldoc instead of less, I suppose.)
18:05 hachi joined #parrot
18:06 pmichaud oh yes, it's not in the POD.  I remember noticing that when I went looking for it a few weeks ago also :)
18:06 Austin :)
18:06 mikehh joined #parrot
18:07 kurahaupo coke: yes, discussion was about improving performance of CallSignatures
18:08 whiteknight do we have any leads or ideas about fixing the remaining test failures on pcc_reapply?
18:08 whiteknight I had them running in the debugger last night, and didn't even understand the problem
18:12 whiteknight for the one test, t/pmc/codestring.t, the one method was clearly returning -1, but that was not the resul
18:12 whiteknight I suspect it's an issue of sign extension, but haven't tried a fix
18:12 dalek nqp-rx: f650cd5 | pmichaud++ | STATUS:
18:12 dalek nqp-rx: Update STATUS.
18:12 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/f​650cd5859cee42816b4c6597c6b16ae0c0c91b9
18:12 shorten dalek's url is at http://xrl.us/bfr74y
18:13 allison whiteknight: my guess on the threads failures is that there's some code in the threads implementation that's still handling args/returns in the old way
18:14 whiteknight so I guess we need to march into the belly of that beast somewhere then
18:15 allison whiteknight: indeed
18:16 allison t/op/lexicals_28.pir seems to be a failure in the argument or return handling logic
18:16 allison that is, it's not correctly detecting the end of the positional arguments or results array
18:16 whiteknight yeah, I stared at that file for a while and couldn't figure out what was going wrong
18:17 allison t/library/coroutine.t has a different error now
18:17 allison more specific
18:17 allison "flattened return on a non-array"
18:17 whiteknight urg
18:18 allison it may simply be a case of something accessing the flags for the wrong element
18:18 whiteknight or the new system accessing things in a different order from the old system
18:18 dalek partcl: c27357b | coke++ | tools/spec_info.pl:
18:18 dalek partcl: git revisions aren't numeric.
18:18 allison (which seems to happen whenever the index is updated without re-looking up the flags
18:18 dalek partcl: review: http://github.com/partcl/partcl/commit/c​27357be555551298e942dadfd3580b55d130182
18:18 shorten dalek's url is at http://xrl.us/bfr75r
18:18 dalek partcl: 9bade3a | coke++ | docs/spectest- (2 files):
18:18 dalek partcl: Record another spec test run.
18:18 dalek partcl: Huge number of new, undiagnosed failures.
18:18 dalek partcl: review: http://github.com/partcl/partcl/commit/9​bade3a520f0b97b898ee48ff1e12709eb355e72
18:18 shorten dalek's url is at http://xrl.us/bfr75t
18:18 whiteknight which, for named returns filling unfilled positionals, is a very real possibility
18:19 bacek joined #parrot
18:19 allison whiteknight: I've been going around restricting the scope of the flags variables, so they're only created when needed, and not lingering around to trap some unwary line of code
18:20 allison whiteknight: ordering conflicts is another possibility, aye
18:20 whiteknight especially if old PIR code is expecting certain orders of things
18:21 allison t/op/calling.t is three very different erros
18:21 allison errors
18:21 whiteknight It creates a bit of an issue because named arguments are stored in an order-less hash, but are used to fill positional arguments
18:21 pmichaud btw, within the next few days I expect I'll be at a point where benchmarking nqp-rx would be very useful
18:21 pmichaud s/benchmarking/profiling/
18:21 allison #59 is caused by calling get_results directly inside the label of a continuation
18:21 whiteknight that should be .get_results(), right?
18:21 allison I have some code that almost fixes that,
18:21 pmichaud so if someone can do some profiling, or let me know how to do it, that might be very interesting/helpful.
18:22 whiteknight pmichaud: I probably can help
18:22 cotto_work pmichaud, add -Rprofiling to the parrot cli
18:22 whiteknight although I shouldn't be the only one, I'm a profiling n00b
18:22 * pmichaud tries it on existing pge test.
18:22 allison whiteknight: yes, but I suspect get_results or .get_results would have the same effect in this case. It's an order of operations problem
18:22 pmichaud (svn up, rebuild, sigh)
18:23 cotto_work the final bit of output will tell you the next step
18:23 allison t/op/calling_61.pir looks like a return handling bug
18:23 pmichaud cotto_work: thanks
18:23 pmichaud will report soon (after building)
18:23 cotto_work pmichaud, I hope you find code useful.
18:24 pmichaud if not, you'll hear about it :)
18:24 cotto_work great
18:25 whiteknight mikehh: when did the 9.10 beta come out?
18:27 allison whiteknight: t/op/calling_47.pir looks like it's not returning values from tailcall properly
18:28 whiteknight was that the failure in tailcall from an NCI method?
18:28 * whiteknight doesn't have his dev machine here so he cant look at all these
18:28 mikehh I got it Friday 2nd but didn't install until this week
18:28 whiteknight and it doesn't build on win32 right now
18:28 dalek TT #1107 created by Austin_Hastings++: Opcode get_class has failed assertion with garbage input
18:29 Austin What do you think, Lewis - is this snake poisonous?    Well, Clark, go ahead and step on it and then we'll know.
18:29 whiteknight "garbage input"? Well, there's your problem
18:29 allison whiteknight: it's a PIR Sub, don't think it's NCI related
18:30 whiteknight allison: okay, I must be thinking of the wrong one then
18:30 chromatic #ps in 1
18:30 dalek tracwiki: v1 | chromatic++ | StructPruning
18:30 dalek tracwiki: Created page with suggestions for parrot_interp_t and PackFile
18:30 dalek tracwiki: https://trac.parrot.org/parrot/wiki/St​ructPruning?version=1&amp;action=diff
18:30 shorten dalek's url is at http://xrl.us/bfr78u
18:31 whiteknight #ps in 0
18:31 pmichaud does pprof2cg.pl generally take a long time?
18:31 schmalbe left #parrot
18:36 pmichaud cotto_work: profiler seems to work just fine, thanks!
18:36 chromatic It takes a long time, yes.
18:37 cotto_work good to hear
19:04 payload joined #parrot
19:13 joeri joined #parrot
19:24 Tene Testing cardinal against pcc branch...
19:26 Tene cardinal compiles fine with pcc branch.
19:26 allison Tene: sweet!
19:26 Tene running tests now
19:26 Tene allison: if you can come up with any ideas about the rakudo build failures, I'll look at them tonight.
19:27 allison Tene: do you have a nopaste of the failures?
19:27 Tene Yes...
19:27 Tene http://nopaste.snit.ch/18329
19:27 dalek nqp-rx: fee0b48 | pmichaud++ | build/Makefile.in:
19:27 dalek nqp-rx: Add test_loud Makefile target.
19:27 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/f​ee0b4825e6c5159659b69efda92373b93e519b9
19:27 shorten dalek's url is at http://xrl.us/bfr8hz
19:27 dalek nqp-rx: 302f7f2 | pmichaud++ | src/ (2 files):
19:27 dalek nqp-rx: Add ability to use ? and ! with enumerated character lists <?[abc]> and <![def]>
19:27 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/3​02f7f2488ff9887161ecfb51af9309785915e32
19:27 shorten dalek's url is at http://xrl.us/bfr8h3
19:28 Tene Really busy with $work right now, so can't actually look at anything.
19:28 chromatic Any thoughts on whether CallSignature should use a single array for INSP types or one array for each?
19:28 Tene I'd personally attempt the former first.
19:28 allison Tene: okay, will leave you a message for later if I turn anything up
19:28 bacek good morning
19:29 allison chromatic: we have to preserve the order of all elements, not just the relative order of each type
19:29 allison chromatic: so I'd lean toward a single data structure
19:29 bacek chromatic: single array is better. Just because get_foo_keyed_int will be simpler
19:30 chromatic We have to preserve the order of all elements?
19:30 bacek for "manual" handling with HLL
19:30 allison as in, we need to know that a subroutine was called with a PNSINS order if it was
19:31 Coke I remember back when parrot /didn't/ keep track of that!
19:31 Coke fun times.
19:31 allison recovering that from a P, NN, I, SS order is a pain
19:31 chromatic I don't remember when Parrot ever kept track of that.
19:31 Coke ... order of arguments?
19:31 chromatic Right.
19:31 chromatic Unless you mean for PCCINVOKE char * signatures.
19:31 Coke huh. I think you and I might be talking about different things, then.
19:31 allison yes, as we learned from experience back when arguments were stored directlyin registers
19:32 dukeleto Coke: what is up with the partcl test suite ?
19:32 allison chromatic: it does currently track that order in the call state struct
19:32 Coke dukeleto: http://github.com/partcl/partcl/commit/9​bade3a520f0b97b898ee48ff1e12709eb355e72
19:32 shorten Coke's url is at http://xrl.us/bfr75t
19:32 Coke I'm trying to track down if that's me or parrot.
19:32 chromatic Okay, I'll test for a single array now then.
19:32 Coke (running against the last known good run of parrot at r41575
19:33 chromatic What happens when the first element in the array is an INTVAL and someone wants it as a Float, String, or PMC?
19:33 allison chromatic: to put it really simply, if it's one array, we can just iterate over that array for argument processing, if it's multiple arrays, we'll have to do clever bookeeping and swapping between the two
19:33 allison s/two/four/
19:33 chromatic I don't understand what you mean by bookkeeping.
19:34 allison chromatic: automatically boxed as a PMC
19:34 allison converted as a Float or String
19:34 Coke mannnn is svn slow.
19:34 chromatic Conversion on access then.
19:34 allison chromatic: keeping track of how far along we are in all 4 arrays while iterating
19:35 chromatic Oh, you iterate over it with an Iterator instead of keeping an external index?
19:35 Tene chromatic: I think she's thinking that you would use compact arrays instead of sparse.
19:35 * Coke does an svn up -r<foo> and gets a checksum mismatch on a file. that's a new one.
19:35 Tene cardinal passes its test suite as expected with pcc branch.  yay.
19:36 Coke and it's repeatable. nice.
19:36 chromatic I was going to use arrays, but then I decided that linked lists would be much, much simpler and almost never a performance issue.
19:36 chromatic Whereas reallocating and copying to make arrays work well is more complex than we need right now.
19:36 chromatic I can write a correct and performant array system, but... meh.
19:36 allison Tene: yes, I was
19:37 chromatic Mostly my question was "Does PNSINS mean 0:P 1:N 2:S 3:I 4:N 5:S?" and it sounds like yes.
19:37 allison yes
19:38 whiteknight problem comes from named arguments though
19:38 whiteknight I suggest that they be stored in the array directly, too. No need for a hash
19:38 allison whiteknight: not necesarily, the could be stored in the same place, but have another layer of index added
19:38 whiteknight that's what I was thinking
19:39 allison as in, string index
19:39 allison and, flagged in such a way that they don't get pulled into an iteration over positional args
19:39 whiteknight that way, using named args for a missing positional would happen in a consistent order AND be relatively quick
19:39 chromatic Or the hash value could be the struct in the array.
19:40 whiteknight if named values don't fill in for positional values in a consistent way, that behavior is worthless
19:40 bacek allison: is interp->current_args (and other stuff) not used anymore?
19:40 szabgab joined #parrot
19:40 allison bacek: yes, there are quite a few c structs that can be pruned in the branch
19:41 allison mainly interp and context
19:41 bacek allison: good. It still accessed in Parrot_mmd_arg_tuple_func
19:42 bacek looks like it's leftover which should be removed.
19:42 allison bacek: aye, it's not set in set_args anymore, so it will be null
19:43 whiteknight I suspect Parrot_mmd_arg_tuple_func should use CallSignature instead
19:43 bacek allison: can I just rip out this function?
19:43 allison bacek: what calls it?
19:43 whiteknight but, if it's been using the wrong thing all this time and hasn't caused failures, maybe it needs to die
19:43 bacek whiteknight: there is _from_sigobj variant
19:43 iblechbot joined #parrot
19:43 bacek allison: looks like no one
19:43 allison bacek: then rip away
19:44 bacek allison: multisub...
19:45 payload left #parrot
19:46 allison bacek: it's called from Parrot_mmd_sort_manhattan
19:46 bacek allison: sort_manhattan called only from MultiSub afaik
19:47 bacek but it should call _by_sig_pmc anyway
19:47 allison bacek: yes, those other calls should be updated to be the same as 'invoke'
19:47 bacek allison: yes, ripping now.
19:47 whiteknight KILL!
19:48 allison bacek: I'm guessing that the other vtable functions aren't tested
19:48 allison bacek: because if they were, they'd be getting failures now
19:48 bacek allison: indeed...
19:48 purl indubitably
19:48 whiteknight maybe we need more tests for MultiSub then
19:48 bacek MultiSub is good candidate for weekly test target.
19:49 chromatic It was a target a couple of weeks ago.
19:49 treed Tene: Minus the three tests that have been failing for a while?
19:49 whiteknight hello treed
19:49 treed 'lo
19:49 * treed just got back from lunch.
19:50 treed or are those passing with the PCC branch too?
19:51 * treed has a meeting in 9 minutes.
19:53 Tene treed: three tests tat failed to compile.
19:54 treed yeah, same as before
19:54 dalek parrot: r41840 | bacek++ | branches/pcc_reapply/src/call/args.c:
19:54 dalek parrot: [cage] Don't use const pointers in from_continuation functions. They are not const
19:54 treed at some point, I'd like some help in tracking that down
19:54 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41840/
19:54 treed it really seems like a parrot problem
19:54 allison Tene: the Rakudo error is really odd, it's a mismatch in the number of arguments passed to an op
19:55 allison Tene: but likely related to the warning "warning: excess elements in array initializer"
19:55 bacek time to go. See you soon.
19:57 dalek parrot: r41841 | bacek++ | branches/pcc_reapply (7 files):
19:57 dalek parrot: Prune parrot_interp_t struct. Also remove old version of mmd_sort_manhattan function which doesn't work anymore
19:57 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41841/
19:58 chromatic INCOMIGN
19:58 chromatic INCOMING
19:58 purl somebody said INCOMING was https://pause.perl.org/incoming/
20:01 dalek parrot: r41842 | chromatic++ | branches/pcc_optimize_sig/t/pmc/callsignature.t:
20:01 Coke msg dukeleto looks like it's not a parrot problem. (old parrot that worked before fails with current partcl)
20:01 purl Message for dukeleto stored.
20:01 dalek parrot: [t] Added tests for push/pop and indexed access of INSP types in CallSignature.
20:01 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41842/
20:01 dalek parrot: r41843 | chromatic++ | branches/pcc_optimize_sig/t/pmc/callsignature.t:
20:01 dalek parrot: [t] Added CallSignature tests for storing all indexed values in a single array
20:01 dalek parrot: (or at least appearing that way) and destructive access via shift VTABLE entry.
20:01 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41843/
20:01 dalek parrot: r41844 | chromatic++ | branches/pcc_optimize_sig/t/pmc/callsignature.t:
20:01 dalek parrot: [t] Added boxing/unboxing tests for positionals stored in CallSignature PMC.
20:01 dalek parrot: review: https://trac.parrot.org/parrot/changeset/41844/
20:03 dukeleto Coke: yay
20:05 Coke not for me! =--)
20:06 Austin pmichaud: I have a sub called Class::find_method. This invokes 7 $class.find_method($name)  after chasing around for a bit setting up $class and $name. Apparently, this is recursive, despite Class:: being a module rather than a class in NQP. Is this a bug, or am I missing something?
20:09 * Austin sings "When you're chewing on life's gristle, don't grumble: give a whistle! And this'll help things turn out for the best."
20:09 joeri left #parrot
20:10 moritz always look on the bright side of life *whistle* *whistle*
20:10 pmichaud in NQP, "Class" isn't usually a Parrot class object.
20:11 pmichaud but, more to the point, I'm not sure exactly what you're trying to do or expect.
20:11 Austin (incidentally, this record's available in the foyer...)
20:11 mokurai joined #parrot
20:11 * whiteknight has to leave. Later
20:11 chromatic That's too bad.  The Holy Grail was under his seat.
20:12 * pmichaud re-reads Austin's description, looking for understanding.
20:12 Austin Pmichaud: What I'm trying to do is resolve the class object, then call find_method on it. And that all works, so I refactored it out of another sub into a "find_method" sub that I could call.
20:13 pmichaud and by "class object" you mean "Parrot class object", as opposed to P6object/NQP protoobjects?
20:14 Austin I mean whatever comes back from get_class. I'm stealing heavily from P6object.pir
20:15 Austin So when $class isa 'String', I call get_class, then that doesn't work, so I split('::') on it, then call get_namespace, then call get_class on that.
20:15 Coke dukeleto - bug introduced here (2d301e4f8217ec0c7dd76e6445aa23c391ce32f8 is first bad commit
20:15 Austin And *that* is in $class. So then I call $class.get_class() and I wind up back in my sub, rather than off in C someplace like I expected.
20:15 Coke (which is, of course, a relatively invasive change.)
20:15 pmichaud okay, if you're using get_class, that's a Parrot object
20:16 Austin Sorry. $class.find_method   -- not get_class
20:16 pmichaud if you put a sub into the Class namespace, either imcc or the method objects might be confusing it
20:16 Austin That's what I was thinking - this might be a bug.
20:17 Austin Because the generated PIR does not declare this sub as :method, so why dispatch to it?
20:17 pmichaud because Parrot still thinks it's a good idea to have methods listed in the namespace
20:17 Tene Austin: my first suspicion is that it's due to :methods going into the namespace
20:18 pmichaud and for built-in PMC types (such as Class), they might look in the namespaces for methods no matter what.
20:18 Tene so when you put yours into the namespace, maybe it ends up clobbering th eold method
20:18 Austin Hunh.
20:19 Tene that's just a guess
20:19 Austin So the method call on builtin::Class is doing a namespace lookup?
20:19 pmichaud found the bug
20:19 Tene but, probably related to the :methods in namespaces thing.
20:19 pmichaud seems to be unrelated
20:20 pmichaud nopaste coming
20:20 * dukeleto starting writing small parrot blog post which is now becoming epic
20:21 nopaste "pmichaud" at 72.181.176.220 pasted "subs in namespaces automatically treated like methods #1" (16 lines) at http://nopaste.snit.ch/18332
20:21 pmichaud note that foo isn't declared :method
20:22 moritz this is quite like perl 5 :/
20:23 Austin This is a bug, right?
20:23 pmichaud I'd say yes.
20:23 pmichaud And i suspect it happens only for the builtin PMC types
20:23 pmichaud testing
20:23 Austin You want to fill out the ticket, or shall I?
20:24 pmichaud you can.  my tickets tend to get warnocked.
20:24 Austin :)
20:24 nopaste "pmichaud" at 72.181.176.220 pasted "works for non-PMC types" (18 lines) at http://nopaste.snit.ch/18333
20:28 dukeleto whoa, I wish I would have found this guide to NQP sooner: http://en.wikibooks.org/wiki/Parr​ot_Virtual_Machine/Not_Quite_Perl
20:28 shorten dukeleto's url is at http://xrl.us/bfr8th
20:30 Austin Thanks, pmichaud++ for the quick examples. TT#1108
20:33 dalek TT #1108 created by Austin_Hastings++: Subs in "built-in PMC" namespaces are treated as PMC methods
20:35 * Austin sings "Time has passed, I'd forgotten; Mother Nature does wonderful things. I thought nothing could stop me from lovin' you, but time changes everything."
20:38 Austin Hmm.. new issue - MultiSub does not support get_namespace. :(
20:42 Austin W00t! Trampolines work.
20:44 * dukeleto likes jumping on trampolines
20:45 * Austin likes automatically generating them.
20:52 dukeleto parrot hacktivity blog post: http://leto.net/dukeleto.pl/2009/​10/parrot-hacktivity-report.html
20:52 shorten dukeleto's url is at http://xrl.us/bfr8yp
20:52 dalek TT #1109 created by Austin_Hastings++: MultiSub PMC does not support get_namespace method
20:58 Whiteknight joined #parrot
21:02 bluescreen joined #parrot
21:04 TonyC joined #parrot
21:04 chromatic Helpful tip: don't include the parameter name when writing the return type in a C function signature.
21:17 hudnix joined #parrot
21:18 AndyA joined #parrot
21:21 dalek rakudo: 54cfe42 | pmichaud++ | docs/spectest-progress.csv:
21:21 dalek rakudo: spectest-progress.csv update: 452 files, 27836 (72.8% of 38211) pass, 0 fail
21:21 dalek rakudo: review: http://github.com/rakudo/rakudo/commit/5​4cfe421ee2693a7c9b5a980ef4dc927313cb627
21:21 shorten dalek's url is at http://xrl.us/bfr85s
21:27 KatrinaTheLamia joined #parrot
21:27 AndyA joined #parrot
21:40 dalek parrot-plumage: e2f0d58 | leto++ | :
21:40 dalek parrot-plumage: [t] Add basic tests for plumage info
21:40 dalek parrot-plumage: review: http://gitorious.org/parrot-plumage/parrot-plumag​e/commit/e2f0d58a4b2c1025577bbe410e2e9634c98b5dc3
21:40 shorten dalek's url is at http://xrl.us/bfr88p
21:53 chromatic Oh right, pointer tagging only works if you tag the pointers.
21:53 Whiteknight joined #parrot
21:56 dalek parrot-plumage: 87306d9 | leto++ | :
21:56 dalek parrot-plumage: [t] Add some tests for default behavior and when input args are not valid
21:56 dalek parrot-plumage: review: http://gitorious.org/parrot-plumage/parrot-plumag​e/commit/87306d91bce14bc233af51d5b7a1a614c7430b50
21:56 shorten dalek's url is at http://xrl.us/bfr9an
22:04 dukeleto how does one properly mock tests which fetch/configure/install software (such as plumage)? sounds like I need to include dummy svn/git repos in my test suite
22:08 darbelo joined #parrot
22:12 dalek parrot-plumage: 20d64f5 | leto++ | :
22:12 dalek parrot-plumage: [t] More tests for invalid projects
22:12 dalek parrot-plumage: review: http://gitorious.org/parrot-plumage/parrot-plumag​e/commit/20d64f58e35c1d933292ca05225f4de6845202f9
22:12 shorten dalek's url is at http://xrl.us/bfr9dd
22:15 patspam joined #parrot
22:24 dalek tracwiki: v27 | dukeleto++ | CallingConventionsOverview
22:24 dalek tracwiki: https://trac.parrot.org/parrot/wiki/CallingCo​nventionsOverview?version=27&amp;action=diff
22:24 shorten dalek's url is at http://xrl.us/bfr9er
22:50 dalek parrot-plumage: 7a0c81a | leto++ | :
22:50 dalek parrot-plumage: [tasks] Remove 'teach dalek about gitorious' from tasklist, darbelo++, i...
22:50 dalek parrot-plumage: review: http://gitorious.org/parrot-plumage/parrot-plumag​e/commit/7a0c81a7fa897e7382923bbc332d35915f990260
22:50 shorten dalek's url is at http://xrl.us/bfr9hw
22:51 japhb dukeleto++ # Constant plumage commits
22:53 Coke hurm. should git revert -n <sha1> leave me in a state where there's nothing to commit?
22:54 Tene Coke: no.
22:54 Whiteknight dukeleto: ping
22:54 Tene Coke: 'revert' is for making an 'undo' commit.
22:54 Tene commit the reverse of some different commit.
22:54 Tene you want reset
22:54 Tene git reset --hard HEAD
22:55 Tene will do
22:55 Coke Yes. I want to revert a commit.
22:55 Coke but I don't want to commit it immediately. I just want to undo <sha1>
22:56 Tene Coke: -n will apply the change to your working directory *and* the index.
22:56 Tene so it's like you then did git add ...
22:58 Tene Coke: if you want to un-add everything, then do your git- revert and then run git reset --mixed HEAD
22:59 Coke I think I got it. I just did a reset and then did a revert sans -n. I can always remove that commit before git push if it doesn't work out.
23:00 Coke ... but it does. shame. no test failures in 'make test', but it killed spectest.
23:01 Coke is there an easy way to see what commits I have locally that aren't in the repo I cloned from?
23:02 Coke diff master?
23:03 darbelo Did partcl's svn branches get pulled into github?
23:03 Coke darbelo: looks like, yes.
23:03 darbelo Guess I'll have to get back to work then :)
23:05 chromatic Testing CallSignature is difficult because if there are bugs, tests won't work.
23:05 Tene Coke: git log master..
23:06 Coke that shows my local master, yes?
23:06 Tene Coke: that's the abbreviated form of master..HEAD
23:06 Coke <blank stare>
23:07 Tene Coke: try origin/master.. instead then
23:08 Coke ah. 'git cherry origin' looks promising.
23:09 Coke Tene++
23:14 dalek partcl: b455eb2 | coke++ | runtime/ (5 files):
23:14 dalek partcl: Revert "PIR cleanup and some refactoring of call_chain manipulation"
23:14 dalek partcl: This reverts commit 2d301e4f8217ec0c7dd76e6445aa23c391ce32f8.
23:14 dalek partcl: review: http://github.com/partcl/partcl/commit/b​455eb2039facb7d65723ee07414329e09ea2ecd
23:14 shorten dalek's url is at http://xrl.us/bfr9kn
23:15 Coke yay, git, saving me from myself.
23:22 Whiteknight I have a local git repo. How do I put it on github?
23:22 Whiteknight their documentation seems to describe every other operation under the sun
23:26 dalek nqp-rx: 982f190 | pmichaud++ | src/Regex/P6Regex/Actions.pm:
23:26 dalek nqp-rx: Clean up handling of initial negated character class subrule.
23:26 dalek nqp-rx: review: http://github.com/perl6/nqp-rx/commit/9​82f1901f8cb3658c094e7d83faf45c35174f203
23:26 shorten dalek's url is at http://xrl.us/bfr9na
23:28 pmichaud Whiteknight: just a sec
23:29 pmichaud it's http://github.com/guides/i​mport-an-existing-git-repo
23:30 Whiteknight that's exactly what I was looking for! Thanks pmichaud++
23:35 Whiteknight Yay! I have a repo on github!
23:35 darbelo Whiteknight: Where? Where?
23:36 chromatic Ha, CallSignature is starting to pass its tests.
23:36 Whiteknight http://github.com/Whiteknight/matrixy
23:37 darbelo Matrixy? I haden't heard of it before. How awesome is it?
23:40 * darbelo barely remembers his MATLAB
23:45 Whiteknight It needs a lot of work, I haven't really touched it in months
23:47 Whiteknight I need to come up with a clever name for a linear algebra library bundle for Parrot
23:47 Whiteknight because that's the next repo I'm creating
23:47 darbelo I've been wanting to do some numerical work on parrot for a while, throw that url at me when it's created.
23:47 Whiteknight darbelo: help me come up with a good name
23:48 Whiteknight and by "help me", I mean "do it for me" :)
23:48 darbelo "plap"
23:48 Whiteknight sexy
23:48 Austin Convect
23:48 darbelo Parrot's Linear Algebra Package
23:48 Austin conure , vectors, convex
23:48 * Whiteknight searches wikipedia for some kind of bird that has something to do with matrix multiplication...
23:49 Whiteknight http://github.com/Whitekni​ght/parrot-linear-algebra
23:50 * Whiteknight got tired of trying to be clever
23:53 Austin l-a-p-d ?
23:54 Whiteknight yeah, and I could get a bunch of T-Shirts that say LAPD on them
23:59 * darbelo wants to get parrot 'native' big numbers.

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

Parrot | source cross referenced