github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
MasterDuke re github.com/rakudo/rakudo/issues/2158, reverting github.com/MoarVM/MoarVM/pull/631 fixes it, but i'm not sure why 01:02
if i stick an fprintf of nread here github.com/MoarVM/MoarVM/blob/mast...ops.c#L554 i get ~15 prints of values ranging between 128 and 2048 01:04
if i revert 631 and put the fprintf in i get3 prints of 4096, 4096, and 3590 01:05
samcv .tell lizmat yes, doing that should not be broken. Only adding keys changes the iteration order, deleting retains order. it's just adding things that is a pretty bad idea 02:49
nwc10 good *, #moarvm 09:33
ASAN excited by t/spec/S29-conversions/ord_and_chr.rakudo.moar -- paste.scsys.co.uk/579511
this might be a few days old
and I might be AFK for a few days from now
or might not
"nous sommes en vacances"
timotimo is "nous" the plural of "no u"? 10:59
masak I wish that was how French worked... 11:15
MasterDuke anybody have a suggestion for debugging github.com/rakudo/rakudo/issues/2158 ? i'm not thinking very quickly right now and nothing is jumping out at me 12:07
Zoffix \o\ 14:53
xelak \o/ 14:54
timotimo /o/
xelak MasterDuke: I'm not sure if that is related to github.com/rakudo/rakudo/issues/2158 , but github.com/MoarVM/MoarVM/blob/mast...ops.c#L556 looks wrong to me - shouldn't it be "if (*permit > 0) ..." instead of "if (permit > 0) ..." 14:57
MasterDuke xelak: huh, i have no idea what that variable is, but yeah, that does look suspicious 14:59
timotimo: ^^^, with jnthn out, you probably know this area the best
MasterDuke fg 15:09
Zoffix bash: fg: current: no such job
Atlas1 With our IRC ad service you can reach a global audience of entrepreneurs and fentanyl addicts with extraordinary engagement rates! williampitcock.com/ 15:39
I thought you guys might be interested in this blog by freenode staff member Bryan 'kloeri' Ostergaard bryanostergaard.com/ 15:40
Read what IRC investigative journalists have uncovered on the freenode pedophilia scandal encyclopediadramatica.rs/Freenodegate
timotimo that could very well be the cause of the problem 15:41
MasterDuke timotimo, xelak: that didn't fix the problem in the issue. it passed a spectest, but no passing todos 16:37
xelak :-( 16:41
MasterDuke i think that change is correct though 16:53
xelak: feel like making a PR? 16:54
Zoffix New blog post: "Introducing: Newcomer Guide to Contributing to Core Perl 6": rakudo.party/post/Introducing-Newc...ore-Perl-6 17:03
xelak MasterDuke: I will try my best :-) 17:34
lizmat Zoffix++ 21:26
Geth MoarVM: xelak6++ created pull request #925:
Use the pointer dereference instead of the pointer.
22:05
Geth MoarVM: bf13bb3741 | Mint++ | src/io/procops.c
Use the pointer dereference instead of the pointer.

This fixes an obvious typo, since using a pointer comparison in the condition does not make sense.
23:24
MoarVM: 2c74b672c8 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/io/procops.c
Merge pull request #925 from xelak6/master

Use the pointer dereference instead of the pointer.