Camelia, the Perl 6 bug

IRC log for #parrot, 2012-08-15

Parrot | source cross referenced

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

All times shown according to UTC.

Time Nick Message
00:20 PacoAir joined #parrot
01:56 dalek parrot: f2fb998 | jkeenan++ | t/src/checkdepend.t:
01:56 dalek parrot: [codingstd] Conform to cuddled-else and no-hard-tabs standards.
01:56 dalek parrot: review: https://github.com/parrot/parrot/commit/f2fb9980b0
02:18 kid51 joined #parrot
05:38 dalek rakudo/sigilless: bf892d1 | moritz++ | src/core/Int.pm:
05:38 dalek rakudo/sigilless: switch Int to sigilless
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/bf892d1d92
05:38 dalek rakudo/sigilless: 61d67fa | moritz++ | src/core/Rat.pm:
05:38 dalek rakudo/sigilless: use sigilless params in Rat
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/61d67fac68
05:38 dalek rakudo/sigilless: 0d98c32 | moritz++ | src/core/Complex.pm:
05:38 dalek rakudo/sigilless: switch Complex to sigillless params
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/0d98c32890
05:38 dalek rakudo/sigilless: 504e994 | moritz++ | src/core/ (5 files):
05:38 dalek rakudo/sigilless: switch Attribute, Enumeration, Hash, LoL and Order to sigilless params
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/504e994257
05:38 dalek rakudo/sigilless: 234ad61 | moritz++ | src/core/List.pm:
05:38 dalek rakudo/sigilless: switch List to sigilless params
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/234ad61b3b
05:38 dalek rakudo/sigilless: 1f6fe81 | moritz++ | src/core/ (3 files):
05:38 dalek rakudo/sigilless: switch Numeric to sigilless params
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/1f6fe8154a
05:38 dalek rakudo/sigilless: 221379e | moritz++ | src/core/ (4 files):
05:38 dalek rakudo/sigilless: switch Numeric, Real and Cursor to sigilless params
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/221379e6ba
05:38 dalek rakudo/sigilless: 3d21d3f | moritz++ | src/core/ (2 files):
05:38 dalek rakudo/sigilless: switch Mu to sigilless params (mostly)
05:38 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/3d21d3ff04
06:00 fperrad joined #parrot
07:15 brrt joined #parrot
07:56 lucian joined #parrot
08:17 dalek parrot/threads: d215d68 | nine++ | src/scheduler.c:
08:17 dalek parrot/threads: Move thrading initialization to Parrot_cx_init_scheduler
08:17 dalek parrot/threads:
08:17 dalek parrot/threads: In embedding situations, control flow may bypass
08:17 dalek parrot/threads: Parrot_cx_begin_execution (using Parrot_api_pmc_invoke directly). Moving
08:17 dalek parrot/threads: threading init to Parrot_cx_init_scheduler makes sure it is run for the
08:17 dalek parrot/threads: first interp.
08:17 dalek parrot/threads: review: https://github.com/parrot/parrot/commit/d215d68d7e
08:17 dalek parrot/threads: 3f147ee | nine++ | lib/Parrot/Pmc2c/PMC/Proxy.pm:
08:17 dalek parrot/threads: Add some description to Parrot::Pmc2c::PMC::Proxy
08:17 dalek parrot/threads: review: https://github.com/parrot/parrot/commit/3f147eeccf
08:17 dalek parrot/threads: 276a2f8 | nine++ | src/scheduler.c:
08:17 dalek parrot/threads: Merge branch 'threads' of github.com:niner/parrot into threads
08:17 dalek parrot/threads:
08:17 dalek parrot/threads: Conflicts:
08:17 dalek parrot/threads: src/scheduler.c
08:17 dalek parrot/threads: review: https://github.com/parrot/parrot/commit/276a2f836a
08:17 dalek parrot/threads: 25acd5a | nine++ | / (10 files):
08:17 dalek parrot/threads: Merge branch 'threads' of github.com:parrot/parrot into threads
08:17 dalek parrot/threads: review: https://github.com/parrot/parrot/commit/25acd5aeba
09:08 dalek rakudo/sigilless: 30fab97 | moritz++ | src/core/ (8 files):
09:08 dalek rakudo/sigilless: switch more of the setting to sigilless params (Int, Junction, Parcel, Range, Rational, Routine, stubs)
09:08 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/30fab97578
09:08 dalek rakudo/sigilless: 018777f | moritz++ | src/core/Array.pm:
09:08 dalek rakudo/sigilless: switch Array to sigilless params
09:08 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/018777f389
09:09 dalek rakudo/sigilless: ddb2dd2 | moritz++ | src/core/operators.pm:
09:09 dalek rakudo/sigilless: Switch operators.pm to sigilless params
09:09 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/ddb2dd2b74
09:09 dalek rakudo/sigilless: 76b036d | moritz++ | src/core/metaops.pm:
09:09 dalek rakudo/sigilless: Switch metaops to sigilless params
09:09 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/76b036d529
09:57 dalek rakudo/sigilless: 105cec6 | moritz++ | src/ (2 files):
09:57 dalek rakudo/sigilless: switch SAFE and CORE to sigilless params
09:57 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/105cec6170
09:57 dalek rakudo/sigilless: f75de7d | moritz++ | src/core/Mu.pm:
09:57 dalek rakudo/sigilless: remove the last use of \$foo from the setting
09:57 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/f75de7db26
09:57 dalek rakudo/sigilless: dfff492 | moritz++ | docs/ROADMAP:
09:57 dalek rakudo/sigilless: update ROADMAP
09:57 dalek rakudo/sigilless:
09:57 dalek rakudo/sigilless: sigilless params are done; sigilless variables are still TODO
09:57 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/dfff492edf
09:57 dalek rakudo/sigilless: 86bc703 | moritz++ | src/Perl6/Grammar.pm:
09:57 dalek rakudo/sigilless: allow typed \ and | in signatures without a name
09:57 dalek rakudo/sigilless: review: https://github.com/rakudo/rakudo/commit/86bc703828
11:09 nopaste joined #parrot
11:10 tuxit joined #parrot
11:27 schmoo joined #parrot
11:46 dalek Heuristic branch merge: pushed 34 commits to rakudo/nom by moritz
11:51 PacoAir joined #parrot
11:55 dalek rakudo/nom: 449f30f | moritz++ | docs/ChangeLog:
11:55 dalek rakudo/nom: update ChangeLog
11:55 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/449f30fc0c
12:03 dalek rakudo/nom: de84a76 | jnthn++ | docs/ChangeLog:
12:03 dalek rakudo/nom: Some more ChangeLog bits.
12:03 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/de84a7695a
12:10 whiteknight joined #parrot
12:11 whiteknight good morning, #parrot
12:11 whiteknight Very interesting link about Java performance: http://paulbuchheit.blogspot.com/2​007/06/java-is-faster-than-c.html#
12:12 whiteknight I don't have a good explanation for the difference, yet
12:14 whiteknight The timings are clearly ignoring jvm startup time.
12:50 brrt hi whiteknight
13:00 PacoAir_ joined #parrot
13:11 bluescreen joined #parrot
13:26 rurban The jvm caches its compiled code on disk, so it's measuring warm-start. Novae re-did the bench and there java was slower.
13:51 rurban whiteknight: We missed you yesterday in #ps. Did you skim through the logs?
13:56 brrt its not /that/ surprising given the amount of information java has that gcc doesn't
13:56 brrt and whats more
13:57 brrt sbenchmarks like these don't measure the 'whole system' thing
13:57 brrt where you may have many different objects / structures
13:57 brrt many different algorithsm
13:57 brrt and the calling conventions / memory layout / etc may make a huge difference
14:05 whiteknight rurban: Busy with work yesterday. I didn't see the logs. Anything interesting?
14:06 whiteknight brrt: Yeah, but that code has lots of primitive floats and ints. Those kinds of things don't really benefit from type-specialization since the C code was using floats and ints too
14:06 brrt hmm fair enough
14:06 brrt still
14:07 brrt it isn't /really/ surprising the jvm creates machine code with floats and ints that is as fast that gcc creates
14:07 brrt which is what is happening
14:07 whiteknight My best guess is that the timing code in the C version causes a slow syscall, while the Date class in the jvm can probably use a cached value and click counts
14:08 whiteknight I don't know that for certain, but that's my best guess
14:08 whiteknight of course, if you take that call out, there's no way to compare timings anymore
14:11 whiteknight rurban: I sent kid51 a message last night. If all his testing looks good, I'll merge io_cleanup1
14:12 whiteknight if I get the all-clear, I can merge tonight
14:15 brrt ... thats actually not unrealistic
14:28 rurban I am testing now nine's threads changes. io_cleanup1 smoked well.
14:35 whiteknight good
14:36 whiteknight I am looking forward to merging io_cleanup1, because as the name implies there's more work to do
14:36 whiteknight but future branches will be much smaller
14:36 whiteknight I  may try to start 6model first, before I start trying to re-do some of the IO pmcs
14:37 whiteknight yay! kid51 sent his approval to the list
14:37 whiteknight time to merge
14:38 alester joined #parrot
14:39 brrt yay!
14:39 alester rurban: This is awesome. http://blogs.perl.org/users/rurb​an/2012/08/threadsanitizer.html
14:39 alester It's great to see all the nitty-gritty details.
14:40 rurban yes. The new llvm based version is even fast, for huge projects.
14:40 rurban But the current version v1 is super-simple to use.
14:46 rurban nine: threads passed ok all tests on solaris. http://smolder.parrot.org/app/​projects/report_details/30613
14:46 whiteknight rurban: Where are threads still failing? Windows/cygwin?
14:48 rurban cygwin yes. still a few deadlocks. currently in t/op/time.t in the same loop as ever.
14:48 whiteknight okay
14:48 whiteknight At this point, I think threads won't merge before the release no matter what
14:48 whiteknight A week is not a lot of time to get something so big integrated and tested
14:48 rurban I agree.
14:49 rurban io_cleanup1 should be enough. Did you talk with moritz?
14:49 whiteknight I have not talked to moritz lately
14:50 rurban there's one TODO multiseperator test in io_cleanup still failing
14:50 whiteknight in Rakudo?
14:50 rurban t/pmc/filehandle.t 12
14:50 moritz if a rakudo spectest of current nom with the parrot-iocleanup1 branch merged with in passes, I'm fine with merging
14:50 whiteknight moritz: I'll give that a try tonight
14:51 moritz s:2nd/with//
14:51 rurban I thought you already implemented  record_separator, multiple chars.
14:52 davidfetter joined #parrot
14:52 rurban Oh, the test is wrong. There is no fh open
14:52 whiteknight rurban: Oh that's fun
14:56 dalek Heuristic branch merge: pushed 38 commits to parrot/whiteknight/io_cleanup1 by Whiteknight
14:57 whiteknight I'll run Rakudo specttest tonight on that branch, and merge immediately if all looks good
14:58 whiteknight and if my laptop freezes up halfway though, again, I'll say some cursewords and wait till tomorrow
14:59 darbelo joined #parrot
15:00 rurban I'll try also on my fast build machine. But I'm leaving in a few hours to Europe.
15:00 whiteknight oh fun, what are you doing in Europe?
15:01 rurban BTW: I already fixed that wrong test t/pmc/filehandle_12.pir
15:01 moritz YAPC::EU maybe? :-)
15:01 whiteknight oh, I didn't know YAPC:EU was soon
15:02 moritz upcoming week
15:02 moritz Monday to Wednesday
15:04 rurban p5p is having some meeting before in Perl, Germany
15:08 dalek parrot/whiteknight/io_cleanup1: 0fb668d | rurban++ | t/pmc/filehandle.t:
15:08 dalek parrot/whiteknight/io_cleanup1: fix t/pmc/filehandle_12.pir: open the filehandle
15:08 dalek parrot/whiteknight/io_cleanup1:
15:08 dalek parrot/whiteknight/io_cleanup1: multiple chars record_separator should work IMHO, but do not yet.
15:08 dalek parrot/whiteknight/io_cleanup1: review: https://github.com/parrot/parrot/commit/0fb668d5d0
15:09 * moritz now spectests rakudo on io_cleanup1
15:09 dmalcolm joined #parrot
15:12 rurban whiteknight: do you want a ticket for the failing t/pmc/filehandle_12.pir case? I thought it should already work.
15:13 whiteknight yes, please
15:18 rurban handle.pmc still uses the old io_readline_s method. That's the problem.
15:25 moritz rakudo fails all 16 socket tests on newest whiteknight/io_cleanup1
15:25 dalek parrot/whiteknight/io_cleanup1: 34e880a | rurban++ | t/pmc/filehandle.t:
15:25 dalek parrot/whiteknight/io_cleanup1: [GH #812] Test to support io multi-char record_seperator
15:25 dalek parrot/whiteknight/io_cleanup1: review: https://github.com/parrot/parrot/commit/34e880a0f1
15:26 whiteknight heh, great
15:26 moritz all the tests say "Got: ''"
15:26 whiteknight okay, I'll fix it
15:26 whiteknight rurban: io_readline_s is the new method
15:26 dngor joined #parrot
15:27 rurban Oh, I got it mixed up.
15:32 rurban Only filehandle::readline_interactive is missing the delim, and dynoplibs/io.ops
15:32 whiteknight ah, okay
15:32 whiteknight that method has slightly different semantics with the prompt and all
15:33 rurban well, if someone changes the filehandle record_seperator, readline_interactive should use it I think
15:34 whiteknight okay, I can update that
15:35 rurban But there's special CRLF logic afterwards
15:35 whiteknight I think we've got to keep that method the same as it was
15:35 rurban maybe leave it in asis.
15:35 rurban agreed
15:35 whiteknight yeah, that's what I'm thinking
15:38 rurban http://pastebin.com/TsRXmmji
15:39 whiteknight ah, perfect
15:40 rurban wrong, there's no SELF in .ops
15:40 whiteknight almost perfect
15:40 whiteknight s/SELF/$2/
15:40 whiteknight actually, we can probably move that logic into Parrot_io_readline
15:40 whiteknight and keep the op body the same, for now
15:51 dalek mod_parrot/new-route: f93ec00 | (Bart Wiegmans)++ | / (4 files):
15:51 dalek mod_parrot/new-route: Add test for mime-based loading
15:51 dalek mod_parrot/new-route: review: https://github.com/bdw/mod​_parrot/commit/f93ec00369
16:01 brrt left #parrot
16:04 rurban But that test actually never calls Parrot_io_readline(), only _s()
16:06 rurban better check the failing socket tests
16:08 dalek rakudo/nom: e326148 | moritz++ | src/core/Match.pm:
16:08 dalek rakudo/nom: avoid codepoints above 255 in the setting
16:08 dalek rakudo/nom:
16:08 dalek rakudo/nom: reverts the factor ~2 parse slowdown of the setting introduced in 94aa2b7
16:08 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/e326148fd9
16:15 jashwanth joined #parrot
16:19 rurban Parrot_io_readline http://pastebin.com/6z07rnDt
16:23 bluescreen joined #parrot
16:39 darbelo joined #parrot
16:39 rurban oops, the GH #812 test was wrong.
16:41 dalek parrot/whiteknight/io_cleanup1: b694441 | rurban++ | t/pmc/filehandle.t:
16:41 dalek parrot/whiteknight/io_cleanup1: [GH #812] fix t/pmc/filehandle_12.pir
16:41 dalek parrot/whiteknight/io_cleanup1: review: https://github.com/parrot/parrot/commit/b6944414e0
16:44 whiteknight so that test passes now?
16:44 rurban yes
16:44 rurban forgot a seek
16:44 whiteknight oh, okay
16:45 whiteknight I love how some of these tests are wrong, and might have been wrong for years
16:45 rurban I'm also adding the terminator attr to the old Parrot_io_readline()
16:45 rurban Thats' the problem with skipped or TODO tests
16:50 whiteknight okay
16:50 whiteknight readline is the worst thing ever
16:51 tadzik :D
16:51 tadzik what about it?
16:56 dalek parrot/whiteknight/io_cleanup1: 8812d14 | rurban++ | src/io/api.c:
16:56 dalek parrot/whiteknight/io_cleanup1: [GH #812] use record_separator in fallback io.ops readline method also
16:56 dalek parrot/whiteknight/io_cleanup1: review: https://github.com/parrot/parrot/commit/8812d14a0b
16:57 rurban done, smoking now.
16:58 rurban have you found the socket errors in rakudo?
16:58 whiteknight I can't look until I get home tonight
16:58 rurban oki
17:00 rurban I'll also have to leave soon to get my plane. See you in ~17 hrs.
17:01 whiteknight see you then
17:07 dalek parrot/whiteknight/io_cleanup1: b1171e6 | rurban++ | config/gen/makefiles/root.in:
17:07 dalek parrot/whiteknight/io_cleanup1: [GH #812] Makefile add pmc/pmc_handle.h dependency to src/io/api.o
17:07 dalek parrot/whiteknight/io_cleanup1:
17:07 dalek parrot/whiteknight/io_cleanup1: [CAGE] fixes t/src/checkdepends.t
17:07 dalek parrot/whiteknight/io_cleanup1: review: https://github.com/parrot/parrot/commit/b1171e6541
17:10 benabik joined #parrot
17:31 rurban cygwin is now only failing t/op/time.t in threads http://smolder.parrot.org/app/​projects/report_details/30614
17:41 benabik_ joined #parrot
17:51 dalek rakudo/nom: b18ea07 | moritz++ | src/Perl6/ (2 files):
17:51 dalek rakudo/nom: sigilless variable declaration
17:51 dalek rakudo/nom:
17:51 dalek rakudo/nom: Currently only works for 'my' variables, and type constraints are ignored
17:51 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/b18ea07b40
18:24 lucian joined #parrot
18:41 dalek rakudo/nom: c076f5d | moritz++ | src/Perl6/Grammar.pm:
18:41 dalek rakudo/nom: fix copy&past-o from STD.pm6
18:41 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/c076f5d3ce
18:41 dalek rakudo/nom: eb2f531 | moritz++ | src/Perl6/Actions.pm:
18:41 dalek rakudo/nom: fix a copy&past-o in method declare_param
18:41 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/eb2f531a6a
18:41 dalek rakudo/nom: dc19a70 | moritz++ | src/Perl6/Actions.pm:
18:41 dalek rakudo/nom: enable type checks for "my Str \x = ..." term definitions, jnthn++
18:41 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/dc19a705fd
19:29 lucian joined #parrot
19:32 Khisanth joined #parrot
20:01 Khisanth joined #parrot
21:01 dalek rakudo/nom: 30134cf | coke++ | t/fudgeandrun:
21:01 dalek rakudo/nom: add t/fudgeandrun from niecza++
21:01 dalek rakudo/nom:
21:01 dalek rakudo/nom: simplifies testing of individual roast files.
21:01 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/30134cf780
23:28 kid51 joined #parrot
23:31 kid51_ joined #parrot
23:41 benabik joined #parrot
23:42 * kid51_ wonders if this storm is what's causing my irc connection to continually drop
23:43 kid51_ Hmm, we don't have aloha for msgs, do we?
23:43 benabik aloha: aloha?
23:43 aloha benabik: aloha is simple Bot::BasicBot::Pluggable hosted on https://github.com/bacek/aloha
23:43 benabik aloha: ping
23:43 benabik aloha: ping?
23:43 aloha benabik: ping is to get someone's attention, pong is to indicate that the pingee is paying attention
23:44 kid51_ msg whiteknight We are getting "uninitialized value" warnings in the 'make' output on io_cleanup1 branch.
23:44 aloha OK. I'll deliver the message.
23:56 dalek rakudo/nom: efde6e5 | tadzik++ | tools/build/Makefile.in:
23:56 dalek rakudo/nom: Don't forget to install precompiled Pod::To::Text
23:56 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/efde6e57bb

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

Parrot | source cross referenced