Geth rakudo: FCO++ created pull request #1029:
add ⁇ ‼ unicode version
02:13
roast: FCO++ created pull request #246:
Add tests for unicode version of ?? !!
02:16
travis-ci Rakudo build errored. Zoffix Znet 'Test Nil line return in REPL ends up as Nil in output 04:59
travis-ci.org/rakudo/rakudo/builds/206298255 github.com/rakudo/rakudo/compare/6...37dd38e061
moritz IOninja: you now have direct access to star 06:39
[Tux] In the original script I now see: 07:25
Tue 14-03-2017 20:00 HAWA De Burcht twee maal verplaatst: een keer naar voren, en een keer terug
perl6 test P6opaque: no such attribute '$storage' in type Map when trying to bind a value
(ignore the middle line :)
This is Rakudo version 2017.02-133-g7966dad5e built on MoarVM version 2017.02-20-g773711e1 07:27
csv-ip5xs 2.809
test-t 4.973 - 5.104
csv-parser 12.822
lizmat Files=1177, Tests=55954, 190 wallclock secs (11.43 usr 4.51 sys + 1131.51 cusr 106.98 csys = 1254.43 CPU) 09:59
IOninja moritz++ thank 11:00
[TuxCM]: is that good or bad? 11:01
Or the error is what you see.
s/Or/Oh/;
IOninja is mildly amused #1029 fails 25 stresstest files before just hanging the run. 11:32
samcv: and yeah, the [ || ... ] and [ | ... ] is used all over the place. Lets you have an nice straight line of "||" before each conditional 11:35
samcv that's fine with me then :) 11:36
jnthn It's parsed "specially" so the leading one is a no-op 11:39
samcv night all 11:40
IOninja night
[TuxCM] IOninja, it is irrelevant: it is the old initial script. I'll have a look later to see if that can be fixed 11:48
IOninja [TuxCM]: which one is the "original script"? `test.pl`? 11:49
[TuxCM] yep
IOninja ./lib/Text/CSV.pm:690: multi method column_names (False) 11:51
That method won't ever be called BTW. You possibly want (Bool:D where {False})
Or not 11:52
[TuxCM] that script was the first functional draft, so it will not ever be production code
IOninja m: multi x($) { say "here" }; multi x(Bool:D where {False}) {say "there"}; x False
camelia ===SORRY!=== Error while compiling <tmp>
Cannot do non-typename cases of type_constraint yet
at <tmp>:1
------> y "here" }; multi x(Bool:D where {False}) {say "there"}; x False
IOninja m: multi x($) { say "here" }; multi x(Bool:D $ where {False}) {say "there"}; x False
camelia here
IOninja m: multi x($) { say "here" }; multi x(Bool:D $ where *.not) {say "there"}; x False
camelia there
IOninja Ah. OK then :)
[TuxCM] «my $line = callframe(1).annotations<line>;» is what is causing the fail 11:54
IOninja Text::CSV is in the repo tho and multi method column_names (False) is in it 11:55
m: callframe(1).annotations<line>
camelia P6opaque: no such attribute '$storage' in type Map when trying to bind a value
in block <unit> at <tmp> line 1
IOninja Thanks.
bisect: callframe(1).annotations<line>
bisectable6 IOninja, Bisecting by exit code (old=2015.12 new=7966dad). Old exit code: 0
lizmat jnthn: so I have a patch that will make .reverse return a Seq, it needs some spectest tweaking, for a 10 element array it is about 30% faster, for a 1000 element array about the same, but reduces the number of GC's to 1/3 of the original 11:56
IOninja Ah, see it. 11:57
Typo on this line: github.com/rakudo/rakudo/blob/7966...ame.pm#L47 11:58
'$storage' neds to be '$!storage'
I'd fix it, but gonna run to work./
lizmat jnthn: do we consider this worth the potential disruption (my @a := @b.reverse potentially breaking)
IOninja: oops, will fix
jnthn lizmat: What did you test? Iterating over it? 11:59
[TuxCM] (canary in the coalmine)++
jnthn (to get the 30% I mean)
Geth rakudo/nom: 9a74cd0e51 | (Elizabeth Mattijsen)++ | src/core/CallFrame.pm
Fix typo, spotted by Zoffix++ Tux++
lizmat my @a = ^10; for ^100000 { for @a.reverse { } } # jnthn 12:00
brb 12:03
Geth rakudo/nom: beda5767ba | (Elizabeth Mattijsen)++ | 4 files
Make List.reverse return a Seq

This appears to have been the intent in the speculation all along. Reason for doing this is mainly to reduce memory pressure, as the previous implementation would *always* create a copy of the reified list. By making it a Seq, it will only create a copy if that is needed. ... (13 more lines)
12:51
roast: 93bd430e1a | (Elizabeth Mattijsen)++ | 4 files
Handle fallout of List.reverse -> Seq migration
12:52
lizmat will handle the 6.c-errata case before the release if these commits are not reverted 12:53
afk for a bit longer& 12:54
Geth roast: 749ed327f6 | (Zoffix Znet)++ | S06-advanced/callframe.t
Test callframe.annotations does not crash

Rakudo fix: github.com/rakudo/rakudo/commit/9a74cd0e51
13:09
rakudo/nom: cf935ca58f | (Zoffix Znet)++ | t/05-messages/01-errors.t
Test Str.Rat.nude on non-numerics doesn't reference guts

RT#130630: rt.perl.org/Ticket/Display.html?id=130630
13:14
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130630
Geth roast: 9c4b563910 | (Zoffix Znet)++ | S03-metaops/cross.t
Test `X` works with lazy RHS

RT#130566: rt.perl.org/Ticket/Display.html?id=130566 Rakudo fix: github.com/rakudo/rakudo/commit/0cd921e351
13:23
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130566
rakudo/nom: c5ab28fe2f | (Zoffix Znet)++ | t/05-messages/01-errors.t
Use same RT ticket labeling format as in roast
13:32
rakudo/nom: 53a6d3aff6 | (Zoffix Znet)++ | t/05-messages/01-errors.t
Test stub code does not reference guts in error when executed

RT#130509: rt.perl.org/Ticket/Display.html?id=130509
13:43
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130509
Geth rakudo/Seq-eqv-List-False: f9eb811dd2 | (Zoffix Znet)++ | src/core/Mu.pm
Make Seq:D eqv List:D return False even when all elements are same
15:23
rakudo: zoffixznet++ created pull request #1030:
Make Seq:D eqv List:D return False even when all elements are same
15:28
roast: 846e330d96 | (Zoffix Znet)++ | S32-list/rotor.t
Test .rotor does not hang with infinite iterable as the cycle

RT#129175: rt.perl.org/Ticket/Display.html?id=129175 Rakudo fix: github.com/rakudo/rakudo/commit/d7...d31da04763
15:41
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=129175
roast: 3fdfcc0ca3 | (Zoffix Znet)++ | S06-signature/slurpy-params.t
Test slurpy positional param does not hang when given inf. list

RT#129175: rt.perl.org/Ticket/Display.html?id=129175 Rakudo fix: github.com/rakudo/rakudo/commit/51...8e8f3d4083
15:46
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=129175
rakudo/nom: f9eb811dd2 | (Zoffix Znet)++ | src/core/Mu.pm
Make Seq:D eqv List:D return False even when all elements are same
16:00
rakudo/nom: dd1dab2e35 | (Zoffix Znet)++ | src/core/Mu.pm
Merge pull request #1030 from rakudo/Seq-eqv-List-False

Make Seq:D eqv List:D return False even when all elements are same
roast: dcdcec2441 | (Zoffix Znet)++ | S03-operators/eqv.t
Test Seqs !eqv Lists

Rakudo fix: github.com/rakudo/rakudo/commit/dd1dab2e35
17:55
[Coke] shouldn't the subtest name be !eqv ? 17:58
IOninja Well, I'm testing `eqv`... the `!` is just a meta operator on it. I'd use `eqv` if we had cpm-nok; 18:01
Geth roast: a66cf6d98d | (Zoffix Znet)++ | S03-operators/eqv.t
Reword test to better indicate intent; [Coke]++
18:03
[Coke] ... I mean in the test title, not in the code. ah well. :)
IOninja I got you, but the test title was accurate and the code was a contortion to make `cmp-ok` succeed :)_ 18:04
[Coke] I disagree, but it's fine. 18:06
IOninja Depends on whether you read `Seq eqv List` as having implied `=== True` after it, I guess 18:08
[Coke] right. :) 18:14
(which i clearly do. :)
Geth rakudo/nom: 9d497e9dbc | (Elizabeth Mattijsen)++ | src/core/Rakudo/Internals.pm
Streamline R:In.MERGESORT* a bit

  - remove one level of nqp::stmts
  - make sure we always get a List with an IterationBuffer as reified
   (well, if we were given an nqp::list, it still might be an nqp::list)
  - remove some unneeded Mu's
22:42
jnthn Input welcome on $*OUT/$*ERR "inheritance": github.com/perl6/doc/issues/1226 22:46
TimToady: Wouldn't mind your take on ^^ also :) 22:47
In that area: I'm also considering whether to implement shell/run/Proc in terms of Proc::Async 22:48
They use the same libuv constructs under the hood, but we have to duplicately maintain a bunch of code in MoarVM 22:49
IOninja 'The docs are not arguably not wrong' wa 22:50
jnthn oops :)
IOninja :)
jnthn Got rid of the first not
I was gonna say "The docs are not wrong" which yeah, they ain't factually, but docs aren't just there to be factual :P
IOninja Sounds like a useful feature to me. 22:54
jnthn Time for me to get some rest :) 'night o/ 22:55
IOninja night