#perl6-dev on 20 May 2018 - Raku Programming Language Log

Perl 6 language and compiler development 2.0 | Logs at irclog.perlgeek.de/perl6-dev/today | For toolchain/installation stuff see #perl6-toolchain | For MoarVM see #moarvm
Set by moderator on 19 May 2018.

Geth

¦ rakudo: zoffixznet self-assigned Regression related to splice and something else (Spit module) github.com/rakudo/rakudo/issues/1843

00:12

¦ rakudo: zoffixznet self-unassigned Regression related to splice and something else (Spit module) github.com/rakudo/rakudo/issues/1843

MasterDuke

Zoffix: so far all those examples give the same result, except for `my @a; say @a[List]`

00:15

it used to say `Unable to call postcircumfix @a[ (List) ] with a type object Indexing requires a defined object`, now it says `===SORRY!=== Cannot look up attributes in a List type object`

timotimo

that looks like you're trying to access the list's attributes in the optimizer before checking if it's concrete or not

00:17

MasterDuke

yeah, going to try adding an nqp::isconcrete

00:18 Zoffix joined

Zoffix

MasterDuke: even my @a = ^10; say @a[BEGIN List.new: 1, lazy 1, 2, 3] ? I don't get how it manages to have all the things reified into `$!reified`

00:19

MasterDuke

yep

00:20

Zoffix

m: use nqp; dd nqp::hllizenqp::getattr(List.new(1, lazy 1, 2, 3), List, <$!riefied>))

camelia

5===SORRY!5=== Error while compiling <tmp>
Confused
at <tmp>:1
------> 3use nqp; dd nqp::hllizenqp:7⏏5:getattr(List.new(1, lazy 1, 2, 3), List
expecting any of:
colon pair

Zoffix

m: use nqp; dd nqp::hllize(nqp::getattr(List.new(1, lazy 1, 2, 3), List, <$!riefied>))

camelia

P6opaque: no such attribute '$!riefied' on type List in a List when trying to get a value
in block <unit> at <tmp> line 1

Zoffix

m: use nqp; dd nqp::hllize(nqp::getattr(List.new(1, lazy 1, 2, 3), List, <$!reified>))

camelia

Cannot .elems a lazy list
in block <unit> at <tmp> line 1

Zoffix

m: use nqp; dd nqp::hllize(nqp::getattr(List.new(1, |lazy 1, 2, 3), List, <$!reified>))

camelia

(1, 1, 2, 3)

Zoffix

s: List, 'new', \\(1, |lazy 1, 2, 3)

00:21

SourceBaby

Zoffix, Sauce is at github.com/rakudo/rakudo/blob/849b...t.pm6#L347

Zoffix

k, then

00:22 Zoffix left

MasterDuke

huh, i added `unless nqp::isconcrete($op[1].value) { return $op }` as the first line, and now nothing is getting past there

00:24

nm, wasn't that

00:28

Geth

roast/R1843-cover-splice: c2bf26fcd4 | (Zoffix Znet)++ | S32-array/splice.t
Cover splice past end bugs

Closes github.com/rakudo/rakudo/issues/1843

00:31

01:17 Kaiepi joined 01:57 ilbot3 joined

moderator

Perl 6 language and compiler development 2.0 | Logs at irclog.perlgeek.de/perl6-dev/today | For toolchain/installation stuff see #perl6-toolchain | For MoarVM see #moarvm

Geth

rakudo: jstuder-gh++ created pull request #1845:
Issue 1843: Limit slice end index in splice implementation

02:16

rakudo: f93ee2d1f3 | (Jeremy Studer)++ | src/core/Array.pm6
Limit slice end index in splice implementation

Splice allows the user to enter a number of elems greater than the elems in the list (the extra is just ignored). However, nqp::slice throws an exception if too large an index is submitted. Check that the end index is not greater than the total elems.
zoffix++ for suggestion to reuse existing 'removed' value.

02:31

rakudo: 4537a8e589 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/core/Array.pm6
Merge pull request #1845 from jstuder-gh/splice_slice_end

Limit slice end index in splice implementation
Fixes R#1843 github.com/rakudo/rakudo/issues/1843

synopsebot

R#1843 [open]: github.com/rakudo/rakudo/issues/1843 [regression][tests committed (in a branch)][⚠ blocker ⚠] Regression related to splice and something else (Spit module)

roast: zoffixznet++ created pull request #428:
Cover splice past end bugs

roast: c2bf26fcd4 | (Zoffix Znet)++ | S32-array/splice.t
Cover splice past end bugs

Closes github.com/rakudo/rakudo/issues/1843

02:32

roast: d1355bccda | (Zoffix Znet)++ (committed using GitHub Web editor) | S32-array/splice.t
Merge pull request #428 from perl6/R1843-cover-splice

Cover splice past end bugs
Rakudo fix: github.com/rakudo/rakudo/commit/4537a8e589

rakudo/car-grant-unreduce: d736c9662b | (Zoffix Znet)++ | src/core/Rational.pm6
Remove trailing whitespace

02:53

rakudo/car-grant-unreduce: 6dd20588b6 | (Zoffix Znet)++ | 2 files
Make Rationals fully-immutable

Fixes R#130774: rt.perl.org/Ticket/Display.html?id=130774

03:34 lizmat joined

Geth

rakudo: zoffixznet self-assigned Regression related to splice and something else (Spit module) github.com/rakudo/rakudo/issues/1843
zoffixznet self-unassigned Regression related to splice and something else (Spit module) github.com/rakudo/rakudo/issues/1843

Fix reentrance of array initialization
In a situation like this: sub init() { my @a = 1, (init() unless $++); say @a }; init() we used the same reifier for both running initializations, which caused one array to miss actual values. Fix by creating the reifier object at runtime so each callframe gets its own. Fixes GH #1843

03:41

05:37 shareable6 joined

synopsebot

RAKUDO#1843 [open]: github.com/rakudo/rakudo/issues/1843 [regression][⚠ blocker ⚠] Regression related to splice and something else (Spit module)

[Tux]

Rakudo version 2018.04.1-113-gf98ea64de - MoarVM version 2018.04.1-121-gd634d24cf

csv-ip5xs	0.914	-	0.979
csv-ip5xs-20	9.111	-	9.449
csv-parser	27.044	-	27.309
csv-test-xs-20	0.475	-	0.479
test	9.131	-	9.224
test-t	2.471	-	2.512
test-t --race	0.986	-	1.005
test-t-20	43.893	-	44.297
test-t-20 --race	16.686	-	17.093