Camelia, the Perl 6 bug

IRC log for #parrot, 2010-12-24

Parrot | source cross referenced

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

All times shown according to UTC.

Time Nick Message
00:01 Khisanth joined #parrot
00:01 dalek parrot: 8138ebd | jkeenan++ | src/dynpmc/Rules.in:
00:01 dalek parrot: Remove include/pmc/pmc_default.h and include/pmc/pmc_continuation.h throughout, as they are already included in $DYNPMC_H_FILES.  t/codingstd/checkdepends.t should now PASS.
00:02 dalek parrot: review: https://github.com/parrot/parrot/commit/8138ebdf02
00:03 NotFound msg plobsing winxed r722 may be what you want
00:03 aloha OK. I'll deliver the message.
00:03 dalek parrot: c0b71ed | (Matt Rajca)++ | src/embed/pmc.c:
00:03 dalek parrot: Use Parrot_pmc_box_string to box strings
00:03 dalek parrot:
00:03 dalek parrot: Signed-off-by: Christoph Otto <christoph_github@mksig.org>
00:03 dalek parrot: review: https://github.com/parrot/parrot/commit/c0b71ed254
00:03 dalek parrot: 375c610 | (Matt Rajca)++ | / (6 files):
00:03 dalek parrot: Unified boxing of strings
00:03 dalek parrot:
00:03 dalek parrot: Signed-off-by: Christoph Otto <christoph_github@mksig.org>
00:03 dalek parrot: review: https://github.com/parrot/parrot/commit/375c61070a
00:06 plobsing NotFound: pong
00:06 NotFound plobsing: see winxed r722
00:07 plobsing will try. thanks.
00:07 NotFound Uhhh... guys, are you sure that that change to the box opcode makes sense?
00:08 cotto_work NotFound: good catch.
00:09 cotto_work I'm a little disappointed that we don't have a test for that.
00:09 NotFound It passes all tests? If it does, we need a test for that"
00:09 NotFound !
00:09 * cotto_work wins
00:09 NotFound Touché
00:09 Khisanth left #parrot
00:09 plobsing shouldn't string-box use HLL mapping?
00:09 cotto_work Matt_: ping
00:09 cotto_work plobsing: yes
00:09 * NotFound been reading Les trois mousquetaires
00:10 cotto_work Matt_: can you make Parrot_pmc_box_string use Parrot_hll_get_ctx_HLL_type(interp, enum_class_String) for the type and add a test to make sure that hllmapped strings are used for boxing?
00:10 Matt_ cotto_work: will do
00:10 cotto_work excellent.  Thanks.
00:10 Matt_ I ran `make test` and everything passed
00:10 NotFound MMmm.... but the function is for embedding, isn't it?
00:11 NotFound The embedder should not have 'current' HLL.
00:12 NotFound Matt_: we don't have enough tests, that's why we want to improve coverage.
00:13 Yuki`N cotto, is there any work for me to be doing? I'm kinda just stagnating right now.
00:13 cotto_work NotFound: do you think it's a bad idea to use the hllmapped type in an embedding function?
00:14 cotto_work Yuki`N: nothing on the gci list grabs you?
00:14 Yuki`N No, not really.
00:14 Yuki`N There's a task for adding gcno and etc to gitignore
00:14 Yuki`N That's only one point though
00:14 cotto_work I need to nuke that.
00:14 cotto_work kid51 took care of it.
00:14 Yuki`N And all the "test coverage" tasks are somewhat outdated.
00:15 Yuki`N I'm looking to work with some of the new embedding API
00:15 Yuki`N Hopefully if you can point me in the right direction I can help push to get the api ready to be merged back into master.
00:15 Matt_ cotto_work: should I proceed with changing Parrot_pmc_box_string to use Parrot_hll_get_ctx_HLL_type?
00:15 Yuki`N Whiteknight seems to have ignored me every time I bugged him about it though. :3
00:16 NotFound cotto_work: what will be the current HLL? The HLL belongs to a invokable thing or a context, doesn't make sense to the outside world.
00:16 cotto_work NotFound: is there any harm in it?  By default it'll just use the normal string pmc.
00:17 NotFound cotto_work: confusion.
00:17 NotFound ...will be my epitaph
00:17 Khisanth joined #parrot
00:20 cotto_work I don't see the harm but this is whiteknight's domain.  I'd ask him and go with what he says.
00:20 plobsing I was confused I thought that this was touching the box string op. that needs hll map.
00:21 NotFound cotto_work: agrred
00:21 NotFound agreed
00:21 cotto_work plobsing: part of the confusion is that that's what it's doing.
00:22 cotto_work The box op clearly needs hllmap-awareness.  Whether it belongs in the embed api is an open question.
00:23 plobsing I'd argue not.
00:24 kid51 is now known as kid51_at_dinner
00:27 NotFound Apparently we have a test for that ?
00:28 NotFound ok 26 - string boxed to appropriate type for HLL from reg isa MyString
00:29 Matt_ Just modified Parrot_pmc_box_string to use Parrot_hll_get_ctx_HLL_type and all tests run fine
00:29 Matt_ Any next steps for me?
00:29 theory joined #parrot
00:35 Khisanth left #parrot
00:41 cotto_work Matt_: I'd like to know why that test (box_string, end of t/op/box.t) passes.
00:42 cotto_work Yuki`N: if you can think of a task, some can write it up for you.  atm my brain is in too many places to think of something.
00:42 Yuki`N Ah.
00:43 Yuki`N I'm not too sure what needs to be done tbh
00:43 Yuki`N I'll try asking whiteknight again
00:44 cotto_work Yuki`N: how do you feel about packfiles?
00:44 cotto_work http://trac.parrot.org/par​rot/wiki/PackfileTasklist
00:44 Yuki`N Umm, the bytecode container?
00:44 cotto_work that's one of the things it does
00:46 * cotto_work heads home
00:56 kid51_at_dinner is now known as kid51
01:15 nwellnhof left #parrot
01:26 kid51 Here's the result of an up-to-date run of 'make quickcover'
01:26 kid51 http://thenceforward.net/parrot/c​overage/quickcover/coverage.html
01:27 dalek parrot: 2a95633 | jkeenan++ | config/gen/makefiles/root.in:
01:27 dalek parrot: Add src/embed/ to COVER_DIRS and QUICKCOVER_DIRS to guarantee that files in those directories created during coverage analysis are removed by cover-clean.
01:27 dalek parrot: review: https://github.com/parrot/parrot/commit/2a956331e3
01:29 kid51 So on my Linode that run of quickcover took 5:08 (including reconfiguration, make, make test and cover)
01:38 Khisanth joined #parrot
01:42 Khisanth left #parrot
02:24 kennym left #parrot
02:35 kid51 cotto: I just created 3 GCI tasks (as yet unapproved)
02:46 Yuki`N left #parrot
02:48 kid51 left #parrot
02:59 dalek ohm-eta-wink-kzd: 089a4bc | plobsing++ | src/ometa-compiler.dual:
02:59 dalek ohm-eta-wink-kzd: use volatile to ensure values are passed up from closures
02:59 dalek ohm-eta-wink-kzd:
02:59 dalek ohm-eta-wink-kzd: this happens eg: in destructuring binds
02:59 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/089a4bc40d
03:00 dalek ohm-eta-wink-kzd: cdd5f90 | plobsing++ | t/tutorial.Ωη:
03:00 dalek ohm-eta-wink-kzd: eliminate another javascript-ism
03:00 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/cdd5f90d69
03:00 dalek ohm-eta-wink-kzd: 838cb0f | plobsing++ | src/winxed-compiler.dual:
03:00 dalek ohm-eta-wink-kzd: avoid aliasing between keyword 'string' tokens and tokens of type string
03:00 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/838cb0fbcf
03:19 dngor_ is now known as dngor
03:43 dalek ohm-eta-wink-kzd: 95bff41 | plobsing++ | t/tutorial.Ωη:
03:43 dalek ohm-eta-wink-kzd: disambiguate string concatenations
03:43 dalek ohm-eta-wink-kzd:
03:43 dalek ohm-eta-wink-kzd: concatenations is not addition on strings! stop writting languages with this
03:43 dalek ohm-eta-wink-kzd: braindead mindset!
03:44 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/95bff41852
03:44 dalek ohm-eta-wink-kzd: bd25c51 | plobsing++ | t/tutorial.Ωη:
03:44 dalek ohm-eta-wink-kzd: use braces to immitate correct round-trip expression compiler output
03:44 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/bd25c514bf
03:44 dalek ohm-eta-wink-kzd: 332f3d4 | plobsing++ | src/ometa-base.winxed:
03:44 dalek ohm-eta-wink-kzd: use method call in stead of attribute access
03:44 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/332f3d4ddd
03:44 Matt_ left #parrot
04:05 Matt_ joined #parrot
04:11 Andy joined #parrot
04:20 zaur left #parrot
04:21 Khisanth joined #parrot
04:24 theory left #parrot
04:24 plobsing_ joined #parrot
04:28 plobsing left #parrot
04:29 plobsing joined #parrot
04:30 Khisanth left #parrot
04:30 plobsing_ left #parrot
04:33 theory joined #parrot
04:38 allison left #parrot
04:39 theory left #parrot
04:41 Matt_ left #parrot
04:42 cotto plobsing++ for fixing the dependencies/includes earlier today
04:42 plobsing_ joined #parrot
04:46 plobsing left #parrot
05:09 plobsing_ left #parrot
05:34 rurban_ joined #parrot
05:36 plobsing joined #parrot
05:36 rurban left #parrot
05:37 rurban_ is now known as rurban
05:42 dalek ohm-eta-wink-kzd: 3f810c7 | plobsing++ | src/ometa-base.winxed:
05:42 dalek ohm-eta-wink-kzd: delegate to target in OMInputStreamProxy
05:42 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/3f810c7df5
05:42 dalek ohm-eta-wink-kzd: ea320da | plobsing++ | src/ (4 files):
05:42 dalek ohm-eta-wink-kzd: hide js vs winxed join() difference behind macro
05:42 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/ea320da522
05:42 dalek ohm-eta-wink-kzd: 28d4a09 | plobsing++ | src/ometa-base.winxed:
05:42 dalek ohm-eta-wink-kzd: implement {get,set}_pmc_keyed()
05:42 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/28d4a09bda
06:01 Andy left #parrot
06:09 Khisanth joined #parrot
06:09 khisanth_ joined #parrot
06:15 Khisanth left #parrot
06:21 khisanth_ is now known as Khisanth
06:25 plobsing left #parrot
07:05 dd070 joined #parrot
07:07 dd070 left #parrot
07:51 simon_ joined #parrot
07:51 simon_ can PIR subroutines contain subroutines?
07:57 sorear No.
07:58 sorear However, they can be associated with a parent Sub, and can then access the lexicals of the parent
08:01 simon_ oh. I'll get to that.
08:17 fperrad joined #parrot
08:37 cotto msg kid51 The gci task to change the names of coverage-related makfile targets seems really trivial.  Is there something I'm missing?
08:37 aloha OK. I'll deliver the message.
08:40 cotto msg whiteknight Let me know if you want a gci org admin bit.
08:40 aloha OK. I'll deliver the message.
08:40 cotto msg aloha aloha
08:40 aloha OK. I'll deliver the message.
09:03 cotto bacek, ping
09:05 bacek cotto, pong. I'm barely here. It's almost x-mas here :)
09:06 nopaste "cotto" at 192.168.1.3 pasted "is this the codingstd change you're suggesting?" (24 lines) at http://nopaste.snit.ch/27433
09:06 cotto bacek, ^
09:07 bacek cotto, nope. Indent C<goto> as well
09:07 bacek and C<return>
09:07 cotto so indent everything by +2?
09:07 bacek +4
09:08 nopaste "cotto" at 192.168.1.3 pasted "like this?" (24 lines) at http://nopaste.snit.ch/27434
09:08 nopaste "bacek" at 192.168.1.3 pasted "Indent" (10 lines) at http://nopaste.snit.ch/27435
09:08 bacek Like next one :)
09:09 cotto got it
09:09 cotto Is that just to make the labels stick out more?
09:09 bacek yes
09:09 cotto wfm
09:09 bacek for consistency sake
09:11 cotto ok.  I'm glad I asked.
09:13 bacek ok
09:13 bacek afk # MORE BEER!
09:13 contingencyplan joined #parrot
09:25 particle left #parrot
09:26 particle joined #parrot
09:28 rfw left #parrot
09:46 bacek left #parrot
10:14 bacek joined #parrot
10:49 dalek parrot: 11dc7f6 | NotFound++ | t/pmc/exporter.t:
10:49 dalek parrot: clean up a Exporter test and make sure it triggers the intended condition
10:49 dalek parrot: review: https://github.com/parrot/parrot/commit/11dc7f6bb4
10:55 contingencyplan left #parrot
11:00 dalek parrot: 0b29d99 | NotFound++ | src/pmc/exporter.pmc:
11:00 dalek parrot: change a condition that can't be triggered to an assertion
11:00 dalek parrot: review: https://github.com/parrot/parrot/commit/0b29d99d00
11:10 kennym joined #parrot
11:17 dd070 joined #parrot
11:20 dd070 left #parrot
11:27 Kulag left #parrot
11:28 Kulag joined #parrot
11:34 Drossel joined #parrot
11:37 Kulag left #parrot
13:34 rurban_ joined #parrot
13:36 rurban left #parrot
13:37 rurban_ is now known as rurban
13:43 plobsing joined #parrot
13:49 plobsing NotFound++ # volatile worked
14:05 ambs joined #parrot
14:15 NotFound plobsing: good
14:17 bluescreen joined #parrot
15:12 bluescreen left #parrot
15:12 bluescreen joined #parrot
15:12 bluescreen merry xmas!
15:18 tadzik merry!
15:25 fbrito joined #parrot
15:31 fbrito left #parrot
15:40 Matt_ joined #parrot
15:46 fperrad left #parrot
15:49 cottoo joined #parrot
15:49 fbrito joined #parrot
15:54 fperrad joined #parrot
15:54 dip left #parrot
15:54 cotto left #parrot
16:10 kthakore Happy Holidays!!!
16:21 fbrito :)
16:21 fbrito Thank you
16:21 fbrito yeah! aloha is back
16:27 kthakore fbrito: :D!
16:30 fbrito1 joined #parrot
16:36 fbrito left #parrot
16:38 dalek TT #1903 created by plobsing++: tailcall method invocation pops exception handlers
16:38 dalek TT #1903: http://trac.parrot.org/parrot/ticket/1903
16:40 Matt_ left #parrot
16:55 theory joined #parrot
16:59 ambs left #parrot
17:26 Matt_ joined #parrot
17:53 dalek Some 'nwellnhof' person just gave me a 258585 byte commit packet.  They need to be more careful in the future.
17:57 fbrito1 Wow :o
18:12 mikehh we definately need to change that message and how dalek handles branch merges
18:14 sorear I don't think we have much choice in the latter - github just gives us a bunch of commits to display
18:14 sorear I'm welcome to suggestions though
18:15 sorear as for the first, what so you suggest instead?
18:15 cxreg github gives you what format?
18:18 contingencyplan joined #parrot
18:20 mikehh we probably need to add a filter so it just shows the commit message for the merge
18:20 mikehh and the committer
18:23 sorear cxreg: http://help.github.com/post-receive-hooks/
18:24 sorear mikehh: finding "the commit message for the merge" will require some heuristics
18:25 cxreg oh, i didnt realize github had postbacks
18:27 mikehh Add minimum GC threshold option | nwellnhof++ | commit  9b2b65645c30756f4846
18:30 theory_ joined #parrot
18:30 theory left #parrot
18:30 theory_ is now known as theory
18:41 mikehh sorear: actually there seems to be two commits there 1st - Merge branch 'master' into nwellnhof/gc_dynamic_threshold -> Showing 547 changed files with 17,356 additions and 3,729 deletions.  (two conflicts)
18:42 mikehh followed by the Add minimum GC threshold option -> Showing 18 changed files with 144 additions and 188 deletions
18:44 sorear mikehh: 1. how did you get a copy of the commit packet? 2. so the 258kb was mostly just listing changed files?
18:45 fbrito1 left #parrot
18:49 mikehh went to github parrot/parrot - selected branch nwellnhof/gc_dynamic_threshold - looked at commit history
18:50 cottoo is now known as cotto
18:51 mikehh of course it also has the commits in the history that have just been merger as well
18:52 mikehh merged
18:53 sorear looking at the raw packet it contains 1 commit (Add minimum GC threshold option) + 1 commit (Merge branch...) + many commits (The contents of the merge)
18:54 mikehh it looks like nwellnhof merged and then fixed the conflicts (and maybe other things) before pushing
19:02 dalek ohm-eta-wink-kzd: 21a60fc | plobsing++ | src/Ωη.coda.winxed:
19:02 dalek ohm-eta-wink-kzd: properly return in :immediate sub
19:02 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/21a60fccbe
19:02 dalek ohm-eta-wink-kzd: 7a6bffc | plobsing++ | src/ometa-base.winxed:
19:02 dalek ohm-eta-wink-kzd: provide rudimentary parse failure support
19:02 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/7a6bffc463
19:06 cotto ~
19:27 Psyche^ joined #parrot
19:27 Patterner left #parrot
19:27 Psyche^ is now known as Patterner
19:54 rfw joined #parrot
20:08 Matt_ left #parrot
20:21 dukeleto howdy
20:21 dukeleto Yuki`N: pong
20:26 Matt_ joined #parrot
20:29 cotto hi dukeleto
20:29 dukeleto cotto: how goes? Has the gci bottleneck been taken care of?
20:30 cotto yup
20:37 * dukeleto works on his TPF grant blog post
21:19 dukeleto http://leto.net/dukeleto.pl/2010/1​2/parrot-embed-grant-update-1.html
21:34 rurban_ joined #parrot
21:37 rurban left #parrot
21:37 rurban_ is now known as rurban
21:58 theory_ joined #parrot
21:58 theory left #parrot
21:58 theory_ is now known as theory
22:02 theory left #parrot
22:03 Matt_ left #parrot
22:04 theory joined #parrot
22:22 fperrad left #parrot
22:32 bacek left #parrot
22:34 nwellnhof joined #parrot
22:47 theory left #parrot
22:50 theory joined #parrot
23:03 Topic for #parrot is now Parrot 2.11.0 Released | http://parrot.org | Log: irclog.perlgeek.de/parrot/today | Onward and upward with Google Code-In | Please test all nwellnhof/* branches | Merge html_cleanup
23:04 * nwellnhof hijacks the topic
23:06 theory left #parrot
23:06 theory joined #parrot
23:10 jan left #parrot
23:17 particle1 joined #parrot
23:18 Yuki`N joined #parrot
23:21 particle left #parrot
23:32 M_o_C joined #parrot
23:41 nwellnhof Rakudo build time on a 768MB virtual machine:
23:41 nwellnhof master: 4m56.880s
23:42 nwellnhof nwellnhof/gc_dynamic_threshold: 4m11.929s
23:44 sorear nwellnhof: any prayer of success with 384?
23:44 nwellnhof 384?
23:45 nwellnhof MB?
23:45 dukeleto nwellnhof: yes
23:45 dukeleto nwellnhof: or 256mb, even
23:46 nwellnhof my branch should give even more performance improvements with 512 or 384 MB.
23:47 nwellnhof 256 MB is probably not enough to build Rakudo without excessive swapping.
23:48 nwellnhof if run you OOM, my branch doesn't help.
23:51 nwellnhof well, you could lower the dynamic threshold. that might help to build Rakudo in some corner cases.
23:54 nwellnhof in nwellnhof/gc_dynamic_threshold there are now two command line options to tune the GC:
23:55 nwellnhof --gc-min-threshold=KB: the minimum amount of memory the GC will allocate before running a GC. Basically equivalent to the current static threshold. Default is 1/8 of system memory like in master.
23:57 nwellnhof --gc-dynamic-threshold: percentage of actually needed memory that will be allocated before a GC is run. this is only used if it is larger than min-threshold.
23:59 jan joined #parrot

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

Parrot | source cross referenced