github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
AlexDaniel Kaiepi: any reason why M#1232 is not merged? 12:24
linkable6 M#1232 [open]: github.com/MoarVM/MoarVM/pull/1232 Revert to treating all addresses like those meant for passive sockets
Kaiepi offhand, no, that should've been merged a while ago 12:27
Geth_ MoarVM: c1b28ca76c | (Ben Davies)++ | src/io/syncsocket.c
Revert to treating all addresses like those meant for passive sockets

Related to github.com/rakudo/rakudo/issues/3457
12:28
MoarVM: 31bdc535a0 | (Aleks-Daniel Jakimenko-Aleksejev)++ (committed using GitHub Web editor) | src/io/syncsocket.c
Merge pull request #1232 from Kaiepi/dns-passivity
AlexDaniel Kaiepi: what about M#1199 ? 12:31
linkable6 M#1199 [open]: github.com/MoarVM/MoarVM/pull/1199 Fix builds on OpenBSD
AlexDaniel Kaiepi: looks like jnthn's comments were taken into account
everything else makes sense to me
Kaiepi: and what's the status of M#1137 ? Should I ping somebody about it? 12:38
linkable6 M#1137 [open]: github.com/MoarVM/MoarVM/pull/1137 Implement NativeCall wide string support
Kaiepi hmm 12:44
it needs to be updated pretty badly at this point AlexDaniel, but it's not something i've totally abandoned since it'd be nice to have editline support in the repl 12:45
AlexDaniel Kaiepi: you mean M#1137? But what about M#1199? 12:48
linkable6 M#1137 [open]: github.com/MoarVM/MoarVM/pull/1137 Implement NativeCall wide string support
M#1199 [open]: github.com/MoarVM/MoarVM/pull/1199 Fix builds on OpenBSD
AlexDaniel timotimo, MasterDuke: half of MoarVM PRs on the last page are yours, can you take a look and close anything that's not relevant anymore? github.com/MoarVM/MoarVM/pulls?pag...dated-desc 12:49
Kaiepi part of #1199's a duplicate of a fix i made recently, but the compiler warning fixes would be nice to have 12:51
i'll clean it up
timotimo d'oh
AlexDaniel Kaiepi: ping me when it's ready
Kaiepi AlexDaniel, the only clang warnings around nowadays are some printf warnings in the profiler, but those are now fixed up in that pr 13:12
Geth_ MoarVM: 233609b52f | (Ben Davies)++ | 2 files
Fix some Clang compiler warnings in the profiler
13:15
MoarVM: c7a4915544 | (Aleks-Daniel Jakimenko-Aleksejev)++ (committed using GitHub Web editor) | 2 files
Merge pull request #1199 from Kaiepi/openbsd-build
AlexDaniel Kaiepi: thanks!
Geth ¦ MoarVM/master: 7 commits pushed by (Wictor Lund)++, (Aleks-Daniel Jakimenko-Aleksejev)++ 13:32
¦ MoarVM/master: e6e36fbfd7 | Adding to assertions of alignment invariant to st->size in P6opaque.c
¦ MoarVM/master: bcf3969d89 | Aligning offsets in P6opaque.c
nine AlexDaniel++ # working through those PRs 13:53
AlexDaniel nine: what's the status of github.com/rakudo/rakudo/pull/1151 ? 14:46
MasterDuke i'll look at 632 and 660 again. 689 and 773 i'd want to make sure the changes are actually desired 14:48
AlexDaniel MasterDuke: also github.com/rakudo/rakudo/pull/1791 and github.com/rakudo/rakudo/pull/1873 if possible 14:51
seems to be about the same change? I'm not sure 14:52
MasterDuke i'll take a look 14:53
nine AlexDaniel: closed it 15:39
AlexDaniel nine: ah, so it was rebased and github didn't pick it up? :) 15:40
maybe we can somehow detect if there's any other PR like this 15:41
nine Yes, looks like it
AlexDaniel nine: github.com/MoarVM/MoarVM/pull/1184 15:46
nine: github.com/MoarVM/MoarVM/commit/c6...5b9db2b189
same?
looks same to me. If you close it, then congratulations! You will no longer have any open PRs: github.com/pulls?q=is%3Apr+author%...is%3Aopen+ 15:51
6c: Nil for ^10_000_000 + 1 15:55
committable6 AlexDaniel, gist.github.com/68220835c61ce9a5f3...5932e34a47 15:58
MasterDuke nine: re github.com/MoarVM/MoarVM/pull/660/...r153041327 will this have to be done with a some asm in emit.dasc? 16:11
nine yes 16:48