[00:00] 🎺🎺🎺 It's time for the monthly Accidental /win Lottery 😍😍😍 We have 6 ballots submitted by 6 users! DRUM ROLL PLEASE!... [00:00] And the winning number is 14! Congratulations to jast! You win a roll of duck tape! [00:01] Perl 6 2017 Advent Day 1 post: "The Grinch of Perl 6: A Practical Guide to Ruining Christmas": https://perl6advent.wordpress.com/2017/12/01/the-grinch-of-perl-6-a-practical-guide-to-ruining-christmas/ [00:06] OMG it's December already! [00:08] Yeah, crazy :) [00:11] Zoffix++ [00:14] 'night o/ [01:20] yesterday's moar vs now's moar does give a little speed improvement for our work [01:20] we might get a performance dip if we don't bump again [05:56] Β¦ roast: e10d781912 | usev6++ | S05-mass/properties-general.t [05:56] Β¦ roast: [jvm] Skip dying Unicode tests [05:56] Β¦ roast: review: https://github.com/perl6/roast/commit/e10d781912 [06:22] i think i found a serious bug in ucd2c.pl. investigating [06:23] basically causes it to not store any properties on any codepoint that is not in UnicodeData.txt [06:23] so any of the not mentioned in that file just get ignored [07:06] squashable6: next [07:06] AlexDaniel, βš πŸ• Next SQUASHathon in β‰ˆ2 hours (2017-12-02 UTC-12⌁UTC+14). See https://github.com/rakudo/rakudo/wiki/Monthly-Bug-Squash-Day [07:21] well i guess it's not that serious for most people. but. a serious issue in the script. most people don't rely on property values of reserved codepoints [07:22] though it could be more important if someone is using an older perl6 and if the property values are more right based on expected usage or something [10:52] <|Tux|> Rakudo version 2017.11-48-gf78fd7c17 - MoarVM version 2017.11-28-gba1fb9542 [10:52] <|Tux|> csv-test-xs-20 0.445 - 0.454 [10:52] <|Tux|> csv-ip5xs 1.046 - 1.052 [10:52] <|Tux|> test-t --race 1.194 - 1.340 [10:52] <|Tux|> test-t 3.013 - 3.044 [10:52] <|Tux|> test 11.543 - 12.189 [10:52] <|Tux|> csv-parser 11.683 - 11.772 [10:52] <|Tux|> csv-ip5xs-20 12.955 - 13.051 [10:52] <|Tux|> test-t-20 --race 19.037 - 19.071 [10:52] <|Tux|> test-t-20 55.079 - 56.239 [10:52] <|Tux|> sorry for the late post [10:52] <|Tux|> jnthn, is this the dip you hoped for? [11:03] [Tux]: looks like it to me, about 5% faster [11:03] further commute& [11:04] oh, that also includes the jitting of the exception ops [11:12] |Tux|: Hm, is that a new low for test-t? :) [11:13] <|Tux|> I think I had one run under 3. /me checks .... [11:13] Looks like a dip over recent days, anyways [11:17] <|Tux|> $ perl -ne'/test-t (2.\d+|3.[01]\d*)/ or next; push @x, [$1,$_]}END{print$_->[1]for sort{$b->[0]<=>$a->[0]}@x' speed.log | tail [11:17] <|Tux|> 2017-10-26 13:58:52 test-t 3.077 [11:17] <|Tux|> 2017-10-14 12:24:16 test-t 3.071 [11:17] <|Tux|> 2017-10-14 12:25:25 test-t 3.071 [11:17] <|Tux|> 2017-10-26 12:50:40 test-t 3.067 [11:17] <|Tux|> 2017-10-26 14:11:11 test-t 3.066 [11:17] <|Tux|> 2017-10-26 11:01:00 test-t 3.064 [11:17] <|Tux|> 2017-10-26 11:48:59 test-t 3.064 [11:17] <|Tux|> 2017-10-19 13:32:36 test-t 3.044 [11:17] <|Tux|> 2017-12-01 11:49:30 test-t 3.044 [11:17] <|Tux|> 2017-12-01 11:46:09 test-t 3.013 [11:17] <|Tux|> so yes, it is a new low [11:20] \o/ [11:20] And quite a jump at that :) [11:21] Was hoping I might just manage to break the 3s barrier. But not quite yet :) [11:23] * |Tux| runs a second double, just to verify [11:25] <|Tux|> test-t 3.021 - 3.041 [12:28] timotimo, jnthn: don't you have a one line fix optimiztion lying around :) [12:38] the exception ops that get jitted still all put a validation in front, i.e. "is the object concrete and is the type right?"; with a proper implementation of these ops it could be a tiny bit faster, but definitely not much [12:39] not enough to get us below 3.0 anyway [12:48] darn [12:48] perhaps it's time to callgrind again [12:51] Β¦ roast/W4anD0eR96-Patch-1: 4355702fc6 | (Alex Chen)++ | S04-phasers/in-loop.t [12:51] Β¦ roast/W4anD0eR96-Patch-1: Test for RT #125488 [12:51] Β¦ roast/W4anD0eR96-Patch-1: review: https://github.com/perl6/roast/commit/4355702fc6 [12:51] RT#125488 [new]: https://rt.perl.org/Ticket/Display.html?id=125488 Some phaser blorsts don't see outer $_ [12:55] so, Str's split method, right? [12:56] it spends 27% of its time inside Seq's sink method [12:56] Β¦ roast: W4anD0eR96++ created pull request #362: Test for RT #125488 [12:56] Β¦ roast: review: https://github.com/perl6/roast/pull/362 [12:56] SPEC#362 [open]: https://github.com/perl6/roast/pull/362 Test for RT #125488 [12:56] which is fine, i expect that's for iterating a loop [13:01] split definitive rings a bell [13:01] *definitively [13:02] that looks to be the kv loop inside split where it goes through the needles [13:03] it's already nqp code it seems [13:04] right [13:05] i'm going to try with a simple iteration and a variable $index that i increment myself [13:07] so you already have a cunning plan :) [13:08] wow [13:08] ? [13:08] the fastest before the change was 3.10s wallclock (105% cpu) [13:08] and now [13:08] the fastest after the change was 2.96s wallclock (104% cpu) [13:09] oh [13:09] impressive [13:09] it is! [13:10] but does it pass a spectest ? [13:11] i shall soon see [13:11] this is exciting [13:14] i find it rather mean to snatch the "going from above to below 3s" thing from liz and jnthn who put considerably more effort into this than i just did [13:16] should jnthn or lizmat take a look at your change or are you confident that it is correct? [13:17] the spec tests shall prove them correct [13:17] the change is basically trivial [13:18] how far are you in the spectest? [13:19] done [13:19] passed :) [13:19] i'd really want to measure more for a good commit message [13:19] but building back and forth ... meh [13:21] :) [13:21] Β¦ roast: 1fbabe9b57 | (Alex Chen)++ | S12-enums/basic.t [13:21] Β¦ roast: Test for RT #116719 [13:21] Β¦ roast: [13:21] Β¦ roast: The very ticket is fixed somewhere. [13:21] Β¦ roast: [13:21] Β¦ roast: It should be mentioned that there was a test with comment RT#116719, [13:21] Β¦ roast: but it is nothing to do with this rt. [13:21] Β¦ roast: review: https://github.com/perl6/roast/commit/1fbabe9b57 [13:21] RT#116719 [open]: https://rt.perl.org/Ticket/Display.html?id=116719 [BUG] Internal error message on Z=> in enum declaration in Rakudo [13:22] don't you have rakudobrew installed as well ? [13:22] i don't [13:23] Β¦ roast: c4ce9a3c1b | (Alex Chen)++ | S12-enums/basic.t [13:23] Β¦ roast: Revert a comment [13:23] Β¦ roast: review: https://github.com/perl6/roast/commit/c4ce9a3c1b [13:23] i'm splitting a 2.3 mb file with three needles for before/after comparison [13:25] huh, there's almost no difference here [13:29] strange [13:35] the reason was i was only calling split a single time [13:35] Text::CSV does it once per line [13:36] oh [13:36] m: say "it only takes { 3.74 * 100 / 4.84 }% as long now" [13:36] rakudo-moar f78fd7c17: OUTPUT: Β«it only takes 77.272727% as long now␀» [13:37] very impressive [13:39] my test has 3 needles, Text::CSV uses 5 [13:41] * timotimo turns around (because cool guys don't look at explosions) [13:41] Β¦ rakudo: 5929887c77 | (Timo Paulssen)++ | src/core/Str.pm [13:41] Β¦ rakudo: make multi-needle split setup cheaper [13:41] Β¦ rakudo: [13:41] Β¦ rakudo: by using a hand-managed index counter rather than the [13:41] Β¦ rakudo: more expensive (but better-looking) .kv iterator. [13:41] Β¦ rakudo: [13:41] Β¦ rakudo: Cuts time to split each line of 100000 by 3 needles down to 77% [13:41] Β¦ rakudo: review: https://github.com/rakudo/rakudo/commit/5929887c77 [13:41] [Tux]: could you time this for me? :) [13:41] timotimo: nice timing :-) [13:46] timotimo++ [13:47] now everyone is hoping that [Tux] will show up :) [13:51] m: say "using 5 needles the percentage is { 4.09 * 100 / 5.14 }%" [13:51] rakudo-moar 5929887c7: OUTPUT: Β«using 5 needles the percentage is 79.571984%␀» [13:51] Just tripped over a bug in Pod::To::HTML... [13:52] coming up on a long keyboard break [13:53] timotimo: do you suffer from carpal tunnel syndrome? [13:54] aye [13:54] "tendovaginitis", such an amusing word if you're still 13 years old in your brain [13:55] started 1.5 months ago [13:55] when i wanted to start my TPF grant [13:55] x_X [13:55] well, ttyl! [13:55] this is common among hardcore gamers [13:55] and pianists i hear [13:55] and pianogamers [13:55] aha, that was new to me [13:56] perhaps not *as* common [13:56] workrave++ [13:59] timotimo: take care of yourself [14:00] first secure your own mask, before attending to Perl6 [14:00] because we want you attending to Perl6 for as long as you're willing :) [14:09] DrForr: Pod::To::HTML, its source code is a headache... [14:12] the thing I don't understand about Pod::To::HTML: wouldn't that be a *perfect* application for object orientation and polymorphism? [14:13] I recommend y'all look at Pandoc for design inspiration [14:20] teatime: i've been cutting back on my perl6 and work times significantly, and quite frankly it's torturous to not do anything [14:22] timotimo: :/ maybe you can treat it as an opportunity to focus more on some other interests / projects [14:22] what can i do without hands? :) [14:23] i can learn to play the harmonica with one of these harnesses that holds it in front of your mouth for you [14:23] Β¦ roast/W4anD0eR96-Patch-1: dfc498b81f | (Alex Chen)++ (committed using GitHub Web editor) | S04-phasers/in-loop.t [14:23] Β¦ roast/W4anD0eR96-Patch-1: Remove test on POST, AlexDaniel++ [14:23] Β¦ roast/W4anD0eR96-Patch-1: [14:23] Β¦ roast/W4anD0eR96-Patch-1: `$_` in POST is set to the return value of the block: [14:23] Β¦ roast/W4anD0eR96-Patch-1: review: https://github.com/perl6/roast/commit/dfc498b81f [14:24] Β¦ roast: 4355702fc6 | (Alex Chen)++ | S04-phasers/in-loop.t [14:24] Β¦ roast: Test for RT #125488 [14:24] Β¦ roast: review: https://github.com/perl6/roast/commit/4355702fc6 [14:24] Β¦ roast: dfc498b81f | (Alex Chen)++ (committed using GitHub Web editor) | S04-phasers/in-loop.t [14:24] Β¦ roast: Remove test on POST, AlexDaniel++ [14:24] Β¦ roast: [14:24] Β¦ roast: `$_` in POST is set to the return value of the block: [14:24] Β¦ roast: review: https://github.com/perl6/roast/commit/dfc498b81f [14:24] Β¦ roast: 48124e0123 | (Alex Chen)++ (committed using GitHub Web editor) | S04-phasers/in-loop.t [14:24] Β¦ roast: Merge pull request #362 from perl6/W4anD0eR96-Patch-1 [14:24] RT#125488 [new]: https://rt.perl.org/Ticket/Display.html?id=125488 Some phaser blorsts don't see outer $_ [14:24] Β¦ roast: [14:24] SPEC#362 [closed]: https://github.com/perl6/roast/pull/362 Test for RT #125488 [14:24] Β¦ roast: Test for RT #125488 [14:24] Β¦ roast: review: https://github.com/perl6/roast/commit/48124e0123 [14:30] wander: It's not tat bad, but I'm refactoring the hell out of it for another project. [14:31] moritz: Yes, an excellent candidate, but in order to do debugging of the internals they wrote it in such way that makes it much more work than necessary. [14:32] Confused me for a few hours until I tracked down the entry point in the perl6 binary and realized why they'd exported what they did. [14:40] * moritz is guilty of a few commits to that repo too [14:59] Β¦ roast: 2a3717c134 | (Alex Chen)++ | S04-phasers/in-loop.t [14:59] Β¦ roast: Add more tests for phasers' in-loop [14:59] Β¦ roast: review: https://github.com/perl6/roast/commit/2a3717c134 [15:04] DrForr: somewhat lack of generality, I'd like pod2html only to generate html body, maybe provide class attr for custom css. it was well designed, but modify for some particular purpose(like fit sth about generate doc site), that's OK. but as where you arrive, someone has to(?) *refactor*(not simply modify) it to adjust his own use [15:05] gather meta message and make the toc are useful, we can export subs(return $toc, @meta, @x-tags) for that [15:05] Isn't it terribly convenient then that what I'm working on is Pod::To::BlogspotHTML which has just a bare element - Blogspot does the heavy HTML lifting for you. [15:11] nowadays web devs use html template generally, maybe generate the whole web page is not that admired. after all, it can provide an adverb to generate a full html using default template. [15:12] render($=pod, :whole) [15:12] our doc site relies heavily on current p2h, anyway [15:17] There's already a :header and :footer argument present, so there's a echanism that we can hijack. [16:41] * |Tux| pops up [16:42] <|Tux|> looks good. have a sec [16:47] <|Tux|> Rakudo version 2017.11-49-g5929887c7 - MoarVM version 2017.11-28-gba1fb9542 [16:47] <|Tux|> csv-test-xs-20 0.453 - 0.476 [16:47] <|Tux|> csv-ip5xs 1.051 - 1.077 [16:47] <|Tux|> test-t --race 1.147 - 1.220 [16:47] <|Tux|> test-t 2.809 - 2.878 [16:47] <|Tux|> test 11.275 - 11.625 [16:47] <|Tux|> csv-parser 11.850 - 11.870 [16:47] <|Tux|> csv-ip5xs-20 12.456 - 12.459 [16:47] <|Tux|> test-t-20 --race 17.585 - 17.675 [16:47] <|Tux|> test-t-20 50.946 - 52.677 [16:47] holy sh*t [16:49] timotimo: have you seen this [16:50] whoops [16:50] :) [16:50] i accidentally 20 msec [16:51] Nice. And you were smart enough not to do it by working on inlining and exceptions :P [16:51] well, your work on inlining and exceptions benefits everyone [16:51] this change only benefits code that uses multi-needle split [16:51] so yeah, it looks great on the benchmark, but few users other than that will see much difference [16:53] looks like the --race version hardly sees a win? but maybe it's just too noisy? [16:53] it's a new low, sure, but 1.147 vs 1.152 isn't that much [16:54] Β¦ roast: 89a96af777 | (Alex Chen)++ | S12-enums/basic.t [16:54] Β¦ roast: Add test for RT#123457 [16:54] Β¦ roast: review: https://github.com/perl6/roast/commit/89a96af777 [16:54] RT#123457 [open]: https://rt.perl.org/Ticket/Display.html?id=123457 enum A "P6opaque: no such attribute '$!signature'" [16:56] timotimo: But look at test-t-20 --race :) [16:56] * |Tux| off to $HOME [16:56] That's a bigger difference [16:57] speed.log doesn't have it [16:58] but my irclogs should have it [16:58] ah, that already saw a nice improvement from jnthn++'s work before mine [17:14] Righty, enough time at the office for me this week :) [17:15] * jnthn bbl [17:39] squashable6: status [17:39] AlexDaniel, πŸ•πŸ• SQUASHathon is in progress! The end of the event in 1 day and β‰ˆ18 hours. See https://github.com/rakudo/rakudo/wiki/Monthly-Bug-Squash-Day [17:39] AlexDaniel, Log and stats: https://gist.github.com/576125f079fd183663e957f37a418e8d [18:51] Geth: ver https://github.com/rakudo/rakudo/commit/b2fbf893dba0df2d67d66b0cb54feb5228de1249 [18:51] Geth: help? [19:15] buggable: speed :4 test-t-20--race [19:16] japhb, β–… ↑ β–ˆ β–‚ dates: 2017-11-10–2017-12-01 [19:16] japhb, β–† β–‡ β–‚β–ˆβ–‚β– β–ƒβ–ˆ β–ˆβ–ƒβ–† β–‚β–ƒβ–‚β–… β–‡ β–ˆ β–„ β–†β–†β–† range: 17.585s–26.624s [19:16] japhb, β–ˆβ–β–ˆβ–‡β–ˆβ–ˆβ–ˆβ–ˆβ–‡β–…β–…β–β–ˆβ–ˆβ–ƒβ–ˆβ–ˆβ–ˆβ–…β–‡β–„β–ˆβ–„β–ˆβ–„β–‡β–ˆβ–ˆβ–ˆβ–ˆβ–ƒβ–ˆβ–ˆβ–ˆβ–†β–ˆβ–‡β–†β–‡β–†β–‡β–ˆβ–ˆβ–ˆ speed: 19% faster (widths: 9/3) [19:16] japhb, β–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–ˆβ–†β–†β–„β–β†“β†“ [19:16] Nice. [19:24] ↓_↓ [19:42] bisectable6: say "ab" ~~ rx:P5/^(a(??{"(?!)"})|(a)(?{1}))b/ && $1 # RT 125027 [19:42] bartolin, Bisecting by exit code (old=2015.12 new=5929887). Old exit code: 1 [19:42] RT#125027 [new]: https://rt.perl.org/Ticket/Display.html?id=125027 [REGEX] Roast rakudo skip/todo test:./S05-modifier/perl5_9.t line:35 reason: 'Quantifier quantifies nothing' [19:42] bartolin, bisect log: https://gist.github.com/996a9c0aeea60fd1061784bc71ceefbd [19:42] bartolin, (2017-09-03) https://github.com/rakudo/rakudo/commit/4c25df745cb98e2cad3b75b53f4fb39a36449371 [19:46] Β¦ roast: e7dd88699a | usev6++ | S05-modifier/perl5_9.t [19:46] Β¦ roast: Unskip now passing test RT #125027 [19:46] Β¦ roast: [19:46] Β¦ roast: Seems to be fixed by rakudo commit 4c25df745c, MasterDuke++ [19:46] Β¦ roast: review: https://github.com/perl6/roast/commit/e7dd88699a [21:02] Β¦ roast: 28555c8449 | usev6++ | S32-str/sprintf-b.t [21:02] Β¦ roast: Do not refer to automatically generated tickets [21:02] Β¦ roast: [21:02] Β¦ roast: All those RT tickets were only created to point to (formerly) failing [21:02] Β¦ roast: tests. They were merged into one ticket (124697), that is closed now. [21:02] Β¦ roast: review: https://github.com/perl6/roast/commit/28555c8449 [21:16] Β¦ roast: 26cacb145a | usev6++ | S32-str/sprintf.t [21:16] Β¦ roast: Add test for RT #123979 [21:16] Β¦ roast: review: https://github.com/perl6/roast/commit/26cacb145a [21:16] RT#123979 [open]: https://rt.perl.org/Ticket/Display.html?id=123979 %064b format doesn't deal with negatives [21:30] .tell jnthn if you could spare a few minutes to take a look at PR 1253 (rakudo) -- i'd be really happy to get your opinion on that one. (it's not urgent in any way) [21:30] bartolin: I'll pass your message to jnthn. [21:58] . [22:27] m: my %h = "24" => 666; my %i := %h.clone; %i{24} = 42; dd %h,%i # this feels wrong to me [22:27] rakudo-moar 5929887c7: OUTPUT: Β«Hash %h = {"24" => 42}␀{"24" => 42}␀» [22:27] binding to a clone of a Hash should disentangle with the original hash, one would expect ? [22:30] I was gonna answer "yes", but given that Pairs are first-class, I actually don't know... [22:31] lizmat: feels wrong to me too. [22:32] would be interesting to know if any spectests break under the other behavior [22:33] squashable6: status [22:33] AlexDaniel, πŸ•πŸ• SQUASHathon is in progress! The end of the event in 1 day and β‰ˆ13 hours. See https://github.com/rakudo/rakudo/wiki/Monthly-Bug-Squash-Day [22:33] AlexDaniel, Log and stats: https://gist.github.com/598e58419879ad2703cc6be855393d3e [22:34] masak: there are no Pairs at that point in the code, just an nqp::hash underneath? Or am I missing something ? [22:35] lizmat: no, probably I am [22:35] lizmat: but it just felt less clear-cut with Pairs (possibly) in the picture [22:36] now, if it were an object hash, you'd have a point :-) [22:36] like, you could conceivably clone just the Hash but none of the Pairs [22:38] I think I understand what's happening: the hash contains containers, which *aren't* cloned [22:38] so you wind up with a new hash with the same old containers [22:38] does the same apply for arrays? [22:50] possibly, haven't checked [22:52] yeah, it's a container not being cloned issue: observe: [22:52] my %h = a => 666; my %i := %h.clone; %i = 42; dd %h, %i [22:52] m: my %h = a => 666; my %i := %h.clone; %i = 42; dd %h, %i [22:52] rakudo-moar 5929887c7: OUTPUT: Β«Hash %h = {:a(42)}␀{:a(42)}␀» [22:52] m: my %h = a => 666; my %i := %h.clone; %i := 42; dd %h, %i [22:52] rakudo-moar 5929887c7: OUTPUT: Β«Hash %h = {:a(666)}␀{:a(42)}␀» [22:53] the second case we bind to the key, thereby removing the container in the cloned hash, and thus not affecting the original hash [22:55] masak: arrays don't have that issue because: " multi method clone(Array:D:) { [self] }" [22:55] which is pretty yuck from a performance point of view [22:57] so I guess it boils down to: what *are* the intended semantics of .clone ? [22:59] m: my Int @a = ^5; my @b := @a.clone; @b[0] = "foo"; dd @b # another consequence of Array.clone's implementation [22:59] rakudo-moar 5929887c7: OUTPUT: Β«["foo", 1, 2, 3, 4]␀» [23:00] m: my Int @a = ^5; my @b := @a.Mu::clone; @b[0] = "foo"; dd @b # appears more correct [23:00] rakudo-moar 5929887c7: OUTPUT: Β«Type check failed in assignment to @a; expected Int but got Str ("foo")␀ in block at line 1␀␀» [23:10] m: my @foo = 1, 2, 3; @foo[1] = |(1, 2, 3); say @foo.perl; say @foo.clone.perl; [23:10] rakudo-moar 5929887c7: OUTPUT: Β«[1, slip(1, 2, 3), 3]␀[1, slip(1, 2, 3), 3]␀» [23:10] oh, ok [23:10] scalar containers, /me remembers [23:10] m: my @foo = 1, 2, 3; @foo[1] := |(1, 2, 3); say @foo.perl; say @foo.clone.perl; [23:10] rakudo-moar 5929887c7: OUTPUT: Β«[1, slip(1, 2, 3), 3]␀[1, slip(1, 2, 3), 3]␀» [23:11] it even works if i replace that scalar container? [23:13] timotimo: that feels like another issue, namely postcircumfix [] not handling Slips correctly ? [23:14] m: my @foo = 1, 2, 3; @foo[1] := |(1, 2, 3); say @foo.perl; say [@foo].perl [23:14] rakudo-moar 5929887c7: OUTPUT: Β«[1, slip(1, 2, 3), 3]␀[1, slip(1, 2, 3), 3]␀» [23:15] looks like circumfix [] isn't handling slips correctly ? [23:15] or is this intended behaviour ? [23:15] good question [23:16] i would have expected my code to mess up and give something different after the clone [23:16] anyways, the disentanglement that moritz did in response to masak's RT #127704 , seems to introduce inconsistencies [23:16] RT#127704 [resolved]: https://rt.perl.org/Ticket/Display.html?id=127704 [BUG] Hash.clone differs from hash assignment; the hash and its clone are spookily entangled [23:20] oddly enough, removing https://github.com/rakudo/rakudo/blob/master/src/core/Array.pm#L50 doesn't seem to break any spectests [23:21] I think the problem really comes from .clone acting differently because of implementation detaiuls [23:21] *details [23:22] the fact that Mu.clone clones all private attributes, and let's you twiddle public attributes [23:22] is fine for non-core objects, where the developer has control over what the attributes are and whether they're private or not [23:23] for core objects, this is different, because they're really implementation details, or am I wrong? [23:24] Β¦ rakudo: tbrowder++ created pull request #1291: Fix table caption RT #126740 [23:24] Β¦ rakudo: review: https://github.com/rakudo/rakudo/pull/1291 [23:24] RT#126740 [new]: https://rt.perl.org/Ticket/Display.html?id=126740 [POD] Pod::Block::Table node caption property is not populated properly [23:24] RAKUDO#1291 [open]: https://github.com/rakudo/rakudo/pull/1291 Fix table caption RT #126740 [23:28] Β¦ roast: tbrowder++ created pull request #363: add tests for fix for RT #126740 [23:28] Β¦ roast: review: https://github.com/perl6/roast/pull/363 [23:28] SPEC#363 [open]: https://github.com/perl6/roast/pull/363 add tests for fix for RT #126740 [23:35] hmmm... looks like someone forgot to implement tests for RT #129762 [23:35] RT#129762 [resolved]: https://rt.perl.org/Ticket/Display.html?id=129762 [BUG] Array.clone does not return a properly independent array