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 17 May 2018.
Geth rakudo: thundergnat++ created pull request #1840:
Unify Rat and FatRat stringification. Fixes some TODO tests.
00:12
00:21 skids joined
samcv .tell lizmat it's fixed now :) 00:37
yoleaux 17 May 2018 08:04Z <lizmat> samcv: somehow the pictured don't appear on pl6anet.org
samcv: I'll pass your message to lizmat.
Geth rakudo: 82b3a8940e | (Lucas Buchala)++ (committed using GitHub Web editor) | lib/Test.pm6
Test.pm: Tweaks to pass() and flunk()

  * Make pass() always return a True value
  * Give a default empty string argument to flunk()
00:43
rakudo: 5c7f7295f2 | (Zoffix Znet)++ (committed using GitHub Web editor) | lib/Test.pm6
Merge pull request #1839 from lucasbuchala/patch-2

Test.pm: Tweaks to pass() and flunk()
01:14 MasterDuke_ 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
02:46 ufobat_ joined 06:09 shareable6 joined 06:20 robertle joined 07:29 shareable6 joined
[Tux] Rakudo version 2018.04.1-95-g5c7f7295f - MoarVM version 2018.04.1-119-g6bb647d74
csv-ip5xs0.862 - 0.871
csv-ip5xs-208.905 - 9.010
csv-parser25.004 - 25.358
csv-test-xs-200.430 - 0.521
test8.882 - 8.941
test-t2.356 - 2.431
test-t --race1.005 - 1.074
test-t-2041.712 - 42.490
test-t-20 --race14.641 - 15.011
07:51
08:13 lizmat joined
lizmat weekly: www.guido-flohr.net/unicode-regex-p...3905388972 08:26
yoleaux 00:37Z <samcv> lizmat: it's fixed now :)
notable6 lizmat, Noted!
Geth nqp: 87e9f92aa5 | (Elizabeth Mattijsen)++ | tools/build/MOAR_REVISION
Bump MoarVM for latest goodies
08:55
¦ nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...9-g6bb647d
rakudo: be35160d57 | (Elizabeth Mattijsen)++ | tools/build/NQP_REVISION
Bump NQP for latest Moar / JVM goodies
09:26
rakudo: version bump brought these changes: github.com/perl6/nqp/compare/2018....3-g87e9f92
ac6e26e8fb | (Stefan Seifert)++ | 2 files
10:31 FROGGS joined 11:40 ExtraCrispy joined 12:15 ggoebel joined
lizmat Zoffix++ # twitter.com/zoffix/status/997457367657828353 :-) 12:44
tbrowder_ .ask [Coke] do I get a commit bit? 12:52
yoleaux tbrowder_: I'll pass your message to [Coke].
12:55 shareable6 joined 12:57 lucasb joined 12:58 AlexDaniel joined 13:08 robertle joined 14:05 shareable6 joined 14:25 skids joined 14:30 stmuk_ joined
rakudo: 073a7f50f9 | (Stefan Seifert)++ | 2 files
Fix initialization of our scoped arrays

There's no need to create a new Array instance and bind it. We already have an object that we just have to fill with data. Binding a new object worked only for the local variable, but didn't affect the Stash entry.
[Coke] (use p5isms) deep in review, but Is the long term goal to make this the default, rather than having to declare it? 15:03
yoleaux 12:52Z <tbrowder_> [Coke]: do I get a commit bit?
[Coke] tbrowder_: I said your CLA had arrived. anyone who has rakudo bits can give you access. 15:04
(I am not the blocker here)
tbrowder_ thnx 15:05
So, stepping upon soap box, can anyone support my bid for a commit bit for Rakudo? I have a record of successfule 15:08
geekosaur if I had to guess, it's more a matter of someone capable of giving them out noticing and/or having time 15:09
tbrowder_ successful PRs for the improvement of Rakudo Perl 6 pod. I would like to be able to start a public branch to work on a slang for pod 6. 15:10
timotimo doesn't have admin access to the rakudo repo 15:25
15:29 [Tux] joined 15:51 softmoth joined 16:28 stmuk joined
Geth rakudo: zoffixznet self-assigned Regression exposed in Crane and other modules github.com/rakudo/rakudo/issues/1832
54e390f57e | (Zoffix Znet)++ | src/Perl6/Optimizer.nqp

Fixes last bug R#1832 of github.com/rakudo/rakudo/issues/1832
When we rewrite the assignment to the optimized form, we end up returning the results of reification of List::Reifier instead of the actual Array.
Fix by using QAST::Stmts to return our Array variable instead.
16:31
17:05 ufobat_ joined
synopsebot R#1832 [open]: github.com/rakudo/rakudo/issues/1832 [regression][⚠ blocker ⚠] Regression exposed in Crane and other modules
17:11 Ven`` joined
Geth roast: c7af47d480 | (Zoffix Znet)++ | S02-types/array.t
Cover bugs in Array assignment

Closes github.com/rakudo/rakudo/issues/1832 R#1832 Rakudo fix: github.com/rakudo/rakudo/commit/ac6e26e8fb
   github.com/rakudo/rakudo/commit/073a7f50f9
   github.com/rakudo/rakudo/commit/54e390f57e
17:12
AlexDaniel nine++ Zoffix++
I'll retoast then and we'll see where we stand after that
[Coke] tbrowder_: I just got a free minute: I don't have a 'settings' button on github.com/rakudo/rakudo masak? moritz? jnthn? zoffix? 17:21
MasterDuke Zoffix: didn't you have some example that seemed to be slower after that optimization? did you check if that was actually the case? 17:22
17:26 Zoffix joined
Zoffix [Coke]: I think it should actually be an "invite new member" on github.com/orgs/rakudo/people I do have the Settings button, but not the option to add new members to the org 17:26
MasterDuke: it's not slow anymore 17:27
[Coke] Zoffix: I don't have that button either.
Zoffix jnthn would
MasterDuke awesome
[Coke] (forgot to check the org button, thanks for the reminder)
Zoffix and masak and moritz
[Coke] .jnthn when you get a sec, can you add tbrowder to the rakudo org? we have a CLA. thanks. 17:28
.tell jnthn when you get a sec, can you add tbrowder to the rakudo org? we have a CLA. thanks.
yoleaux [Coke]: I'll pass your message to jnthn.
Zoffix ZOFVM: Files=1281, Tests=153297, 155 wallclock secs (21.63 usr 3.32 sys + 3319.47 cusr 171.70 csys = 3516.12 CPU) 17:30
17:30 Zoffix left
MasterDuke [Coke]: while we're on the subject, did you get my CLA? 17:31
[Coke] MasterDuke: nope 17:45
I don't get the emails, I don't think. Please send me a note to will at coleda dot com and remind me to look into it
MasterDuke [Coke]: thanks, done 17:51
[Coke] thanks. 17:52
18:00 shareable6 joined
jnthn . 18:08
yoleaux 17:28Z <[Coke]> jnthn: when you get a sec, can you add tbrowder to the rakudo org? we have a CLA. thanks.
jnthn tbrowder_: Invite sent; let me know if you don't get it 18:09
And welcome :)
18:11 Kaiepi joined 18:24 MasterDuke_ joined
MasterDuke would it be possible to do the array literal re-writing optimization for array slicing (e.g., @a[$b, $c] or @a[@b])? 18:32
and turn it into something like @a[$b],@a[$c] or @a[$b[0]],@a[@b[1]],<...>,@a[@b[+@b]] 18:33
tbrowder_ jnthn: got it and accepted, thanks! 18:34
MasterDuke .tell nine a question you can probably answer irclog.perlgeek.de/perl6-dev/2018-...i_16180792 18:43
yoleaux MasterDuke: I'll pass your message to nine.
18:55 skids joined
releasable6 Next release in ≈23 hours. 2 blockers. Please log your changes in the ChangeLog 19:00
AlexDaniel releasable6: why not include a link to github.com/rakudo/rakudo/wiki/ChangeLog-Draft
releasable6 AlexDaniel, I cannot recognize this command. See wiki for some examples: github.com/perl6/whateverable/wiki/Releasable
19:00 stmuk_ joined
AlexDaniel slaps releasable6 19:01
19:14 robertle joined 19:17 dct joined 19:32 shareable6 joined 19:33 Tison joined
Tison . 19:33
19:33 cognominal joined
Geth rakudo: 3642a879d9 | (Elizabeth Mattijsen)++ | src/core/Array.pm6
Revert "Make @a.splice(offset,length,...) about 20% faster"

This reverts commit a5d46b4273bc5cb011f8f9573dd4e553c5256302.
This appears to cause breakage on the JVM backend (R GH #1838). Since we're close to release, and this was only an optimization, it feels safe to give us some time to figure this one out *after* the 2018.05 release.
19:44
synopsebot RAKUDO#1838 [open]: github.com/rakudo/rakudo/issues/1838 [⚠ blocker ⚠] [JVM] Wrong results for Supply.from-list.rotor after recent optimization for @a.splice
19:48 pmurias joined
lizmat moritz FROGGS re "use p5isms" and undef 19:59
I simplistically thought that adding a "method term:sym<undef>($/) { term:sym<name>($/) }" to Actrions.nqp would magically
codegen "undef" as a call to a sub "undef". 20:00
but it never gets there :-(
moritz if at all, to a sub named term:undef
or term:sym<undef>
you have to check how it's done for 'now'
lizmat ok, but I think the problem is really in the "token term:sym<undef>" 20:01
in the case that .obs just returns, we're not doing the right thing
lizmat tries adding a <.tok> 20:02
moritz: the associated action method is not getting called :-( 20:10
moritz lizmat: probably because there's already a token term:sym<undef> in the grammar? 20:16
lizmat moritz: there is, and it consists of only calls to .obs 20:18
with "use p5isms" these don't throw
moritz lizmat: but still there is a proto token with a branch that matches 'undef' 20:20
lizmat: if you add another branch, still only one of them matches 20:21
lizmat: so maybe leaving the grammar alone, and try only adding an action method for the existing token? 20:22
just spitballing here
lizmat moritz: that was my original approach
but the action method simply doesn't get called :-( 20:23
moritz :(
Geth rakudo: 7bde26fc37 | (Elizabeth Mattijsen)++ | src/Perl6/Grammar.nqp
Make sure "use p5isms" also works for "undef"
20:46
lizmat moritz: my solution in the end 20:47
bartolin lizmat: I think I've found the problem with R#1838 (see my latest comment there). It looks like it was indeed a problem with the jvm implementation of nqp::slice. 20:55
synopsebot R#1838 [open]: github.com/rakudo/rakudo/issues/1838 [JVM] [JVM] Wrong results for Supply.from-list.rotor after recent optimization for @a.splice
lizmat bartolin: cool 20:56
lizmat hopes Jeremy Studer will be able to confirm :-) 20:57
bartolin too ;-)
lizmat goes to get some R&R 21:21
21:37 softmoth joined
Geth rakudo: lucasbuchala++ created pull request #1841:
JSONException -> Rakudo::Internals::JSONException
21:56
rakudo: 1a42564077 | (Lucas Buchala)++ | 2 files
JSONException -> Rakudo::Internals::JSONException

Rename class. No need to pollute global namespace.
21:59
rakudo: b15af2c9de | (Aleks-Daniel Jakimenko-Aleksejev)++ (committed using GitHub Web editor) | 2 files
Merge pull request #1841 from lucasbuchala/jsonx

JSONException -> Rakudo::Internals::JSONException
AlexDaniel lucasb: hey 22:03
lucasb something wrong?
AlexDaniel lucasb: by any chance do you know what is this? github.com/rakudo/rakudo/blob/5ce2....pm6#L2967
oh, that thing is for JSON exceptions! 22:04
ok makes sense
:)
greppable6: JSONException 22:05
greppable6 AlexDaniel, 4 lines, 3 modules: gist.github.com/456ce244502c1ef968...769509b964
AlexDaniel lucasb: well, maybe I didn't quite think it through. PRs to these modules? ↑ 22:06
oh, actually…
these PRs would need to make it work under both 2018.05 *and* previous rakudo versions
lucasb sorry, I had no idea Rakudo::Internals::JSON was relied on outside of core 22:09
this renaming I just did will cause to much trouble before the release? feel free to revert and postpone it 22:10
AlexDaniel I don't think so
just submit a PR to zef 22:11
lucasb ok, I'll do so
with regard to Exceptions::JSON, I think it a little strange for it to be under the Exceptions:: namespace 22:12
AlexDaniel well, and to Net::HTTP also, some modules seem to depend on that
lucasb maybe a better address for it would be "Rakudo::Exceptions::$handler" or something ?
Geth rakudo/revert-1841-jsonx: 60ee289a51 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files
Revert "JSONException -> Rakudo::Internals::JSONException"
22:17
AlexDaniel fwiw have no problem with the revert if ugexe feels strongly about this 22:18
Geth rakudo: ugexe++ created pull request #1842:
Revert "JSONException -> Rakudo::Internals::JSONException"
22:19
rakudo: 60ee289a51 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files
Revert "JSONException -> Rakudo::Internals::JSONException"
22:21
rakudo: ec0422e750 | (Aleks-Daniel Jakimenko-Aleksejev)++ (committed using GitHub Web editor) | 2 files
Merge pull request #1842 from rakudo/revert-1841-jsonx

Revert "JSONException -> Rakudo::Internals::JSONException"
lucasb It's ok. sorry for the disruption :) 22:23
AlexDaniel lucasb: well, we can still make it happen, but much later :)
lucasb: github.com/rakudo/rakudo/pull/1842...-390348001
now that I think about it, even 6 releases is probably not enough, given how often people upgrade zef… 22:24
lucasb: fwiw there were some changes to zef a few minutes ago 22:40
lucasb ugexe++ 22:43
just a (maybe silly) question... 22:44
If JSON is so pervasively used, and even Rakudo requires it, why not ship Rakudo::Internals::JSON as a proper module in the lib/ directory? 22:45
skids Maybe to preserve the ability to have an incomplete/optimized implementation that covers just Rakudo's needs? 22:48
AlexDaniel and really it is only being used by zef 22:49
I mean, the Internals:: version 22:51
you probably meant something different :)
lucasb: btw seriot.ch/parsing_json.php
Kaiepi unrelated, but i want to implement support for wchar_t strings 22:54
what would be the best way to go about it?
i was thinking of adding a trait like `is wide` for native strings to tell nqp/moarvm to treat the strings like they use wchar_t 22:55
geekosaur I'd make a different string type. especially since your 'wide' is narrower than what perl 6 normally does 22:56
(unicode codepoints, full range + synthetics for grapheme support)
22:56 cognominal joined
Kaiepi narrower? 22:56
wchar_t is typically int 22:57
the problem i'm having is i think the wide strings get treated like utf32 strings 22:58
supporting it in moar and the js runtime but i know next to nothing about java 22:59
s/js runtime/\\0 is simple/
23:04 llfourn joined
geekosaur depends on platform, I think. on some it's utf16 23:06
or something that pretends to be that (hello windows)
Kaiepi yeah, it depends on the size of wchar_t
23:11 cognominal joined 23:39 cognominal joined 23:49 cognominal joined