| Geth | nqp: 378622e4a2 | (Samantha McVey)++ | tools/build/MOAR_REVISION Bump MoarVM for windows-1252/1 multi char replacement support Changes: 2018.02-33-ge639691..2018.02-35-g1fca17994 1fca17994 Add windows1251/2 support for replacements with > 1 chars 535cc2b6c Create store_write_barrier macro |
07:07 | |
| nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...g1fca17994 a53d1fe941 | (Samantha McVey)++ | t/moar/11-decode.t Test checks that 1 or two successive replacements work. |
|||
| rakudo: 65874b1555 | (Zoffix Znet)++ | 2 files Fix regression on &dd and Junctions Fixes github.com/rakudo/rakudo/issues/1604 R#1604 nqp::can will give true if class has FALLBACK defined. Revert back to shielding the .name call behind the `try`, but check if we're dealing with a uncontainerized Failure first, so we don't mark it handled it accidentally. |
08:15 | ||
| synopsebot | R#1604 [open]: github.com/rakudo/rakudo/issues/1604 [⚠ blocker ⚠] dd(Failure) fix broke dd(Junction) | ||
| AlexDaniel | [Coke]: by the way, who is the grant manager for JJ? | 11:26 | |
| releasable6: status | 11:40 | ||
| releasable6 | AlexDaniel, Next release in 7 days and ≈7 hours. Blockers: github.com/rakudo/rakudo/issues?q=...%9A%A0%22. Changelog for this release was not started yet | ||
| AlexDaniel, Details: gist.github.com/a76150cf8f3a4cf9c4...f6a38c7e31 | |||
| Geth | nqp: a90c01152f | usev6++ | src/vm/jvm/runtime/org/perl6/nqp/io/FileHandle.java [JVM] Support some more modes for opening files Makes same tests from S32-io/open.t pass: * open PATH, :create * open PATH, :truncate, :create |
13:25 | |
| roast: dogbert17++ created pull request #396: Added tests for RT #126563 |
13:39 | ||
| synopsebot | RT#126563 [open]: rt.perl.org/Ticket/Display.html?id=126563 [BUG] 'X' meta-operator fails with RHS input from parenthesized output of another 'X' | ||
| dogbert17 | .seen lizmat | 14:33 | |
| yoleaux | I saw lizmat 8 Mar 2018 11:12Z in #perl6-dev: <lizmat> my talk proposal for OSCON has been rejected, so I won't be going to OSCON | ||
| dogbert17 | Zoffix: is it possible that you fixed RT #130843 during your ninja days ? | 15:15 | |
| synopsebot | RT#130843 [new]: rt.perl.org/Ticket/Display.html?id=130843 [MOAR][IO] .seek(... SeekFromCurrent) seeks to incorrect place if called after .readchars | ||
| MasterDuke | i just updated github.com/rakudo/rakudo/pull/1317 if anyone has thoughts/questions/opinions | 16:31 | |
| timotimo | any ways in which it is slower? | 16:33 | |
| MasterDuke | oh forgot to mention in the comment, `my @a = 1, |@b` is the same speed | 16:40 | |
| none of the stuff i tried was slower | |||
| AlexDaniel | anyone has any ideas for github.com/rakudo/rakudo/issues/1257 ? | 17:54 | |
| it would be really cool to fix that | |||
| (e.g. debian testing currently has rakudo 2017.06, presumably because of that issue) | 17:55 | ||
| it's now bisected to a moarvm commit | |||
| Geth | roast: ronaldxs++ created pull request #397: S16 io eof wsl script fudge |
18:15 | |
| timotimo | i have an idea | 18:17 | |
| Zoffix | dogbert17: that sounds more like jnthn++'s encoding rework that fixed that | ||
| dogbert17 | Zoffix: thx | 18:18 | |
| Geth | roast: 5939a9c3fe | (Ronald Schmidt)++ | packages/Test/Util.pm fudge to skip run-with-tty tests on WSL because of script command problems, issue #395 |
||
| roast: 6c8fe14516 | (Ronald Schmidt)++ | packages/Test/Util.pm improve Test::Util WSL fudge skip message |
|||
| roast: 1154bc6ca7 | (Zoffix Znet)++ (committed using GitHub Web editor) | packages/Test/Util.pm Merge pull request #397 from ronaldxs/S16-io-eof-wsl-script-fudge S16 io eof wsl script fudge |
|||
| synopsebot | SPEC#395 [open]: github.com/perl6/roast/issues/395 S16-io/eof.t broken on WSL from run-with-tty/script command | ||
| MasterDuke | timotimo: is it a good one? | 18:19 | |
| timotimo | the endian flip is inside the validate_operand function, so maybe all we have to do is do that first | 18:20 | |
| AlexDaniel: do you also have a big endian system available to test on? | 18:21 | ||
| AlexDaniel | timotimo: no, unfortunately | ||
| dogbert17 | Zoffix: edited the PR via github. First time. | 18:23 | |
| Geth | roast: 51370205bf | (Jan-Olof Hendig)++ | S03-metaops/cross.t Added tests for RT #126563 |
18:27 | |
| roast: 6cedf6938e | dogbert17++ (committed using GitHub Web editor) | S03-metaops/cross.t Update cross.t |
|||
| roast: b68073aa15 | (Zoffix Znet)++ (committed using GitHub Web editor) | S03-metaops/cross.t Merge pull request #396 from dogbert17/test-rt-126563 Added tests for RT #126563 |
|||
| synopsebot | RT#126563 [open]: rt.perl.org/Ticket/Display.html?id=126563 [BUG] 'X' meta-operator fails with RHS input from parenthesized output of another 'X' | ||
| dogbert17 | Zoffix++ | 18:28 | |
| AlexDaniel | Geth: ver github.com/rakudo/rakudo/commit/25...b0673f26e2 | 19:04 | |
| Geth | AlexDaniel, version bump brought in these changes: github.com/perl6/nqp/compare/2016....7-gb416158 | ||
| AlexDaniel | Geth: ver github.com/perl6/nqp/commit/b41615...b00b8d1b21 | ||
| Geth | AlexDaniel, version bump brought in these changes: github.com/MoarVM/MoarVM/compare/2...4-g31eccd7 | ||
| Zoffix wonders how to "unsubscribe" from all Issues in a repo you're already subscribed to... | 19:24 | ||
| gfldex | Zoffix: this might help developer.github.com/v3/issues/ | 19:27 | |
| Zoffix | Yeah, there's a "filter:subscribed" filter, but don't want to try to figure out how to write a script that hits that API endpoint xkcd.com/1319/ | 19:31 | |
| I just went through still-open issues and unsubbed manually. | |||
| jstuder | hey 6devs, I've got two open PRs and I was wondering if anyone has had the time to review | 19:45 | |
| github.com/rakudo/rakudo/pull/1601 | |||
| github.com/rakudo/rakudo/pull/1542 | |||
| i haven't received a lot of comments, so I don't know if they are no good or not | 19:46 | ||
| both have MoarVM and NQP components | 19:47 | ||
| timotimo | how does that state.rakudo.moar test file fail? | 19:58 | |
| jstuder | not quite sure off hand. It's been a while since I ran it. | 19:59 | |
| I will recompile and rerun and get back to you | |||
| timotimo | i'll be cooking and enjoying dinner in just a bit, so may not be able to help for a bit | 20:01 | |
| jstuder | ok no problem. I'll just post the output here on IRC when i've got it and if you can check it later if you like. Thanks for taking a look. | 20:02 | |
| I'll post it on the github PR thread too | 20:05 | ||
| probably easier to find there then a sea of IRC logs :) | |||
| AlexDaniel | lol what, the cloak works through the matrix netwok O_o | 21:59 | |
| geekosaur | implied by the fact that matrix users otherwise get special matrix cloaks | 22:03 | |
| meaning, freenode knows about matrix and can therefore do things like that | 22:04 | ||
| AlexDaniel | yeah but I expected it to be nonoverridable | 22:06 | |
| mst | group cloaks > gateway cloaks > unaffiliated cloaks | 22:09 | |
| so a plain unaffiliated/alexdaniel would be overriden by the matrix gateway cloak | |||
| but your cloak is Sufficiently Special to take precedence | |||
| AlexDaniel | cool | 22:10 | |
| Geth | roast: jstuder-gh++ created pull request #398: RT#102994: Unfudge state init test and add more |
22:28 | |
| synopsebot | RT#102994 [open]: rt.perl.org/Ticket/Display.html?id=102994 [BUG] State variables are never initialized if the first call of the containing block does not reach them | ||
| samcv | what should i name the option for .decode and encode to specifiy to decode/encode strict instead of loosely? | 22:31 | |
| which in 6.d will then become the default | |||
| :strict ? what do you people think | |||
| pmurias | where is @*comp_line_directives declared? | 22:40 | |
| nqp-m: say(nqp::elems(@*ARG)); # why is this even allowed in NQP when Perl 6 doesn't allow using an undeclared dynamic variable | 22:45 | ||
| camelia | 0 | ||
| geekosaur | presumably because that's one of the things that lets nqp be faster than perl 6 | 22:46 | |
| checking dynvars is more expensive | |||
| Geth | rakudo: b6e5d7fce5 | (Daniel Green)++ | src/core/List.pm6 Add two special cases for infix:<,> Since these are likely to be common cases, give them a fast path. |
22:47 | |
| rakudo: 2a3947816f | (Zoffix Znet)++ (committed using GitHub Web editor) | src/core/List.pm6 Merge pull request #1317 from MasterDuke17/add_some_special_cases_for_infix_comma Add two special cases for infix:<,> |
|||
| MasterDuke | heh, Zoffix++ | 22:48 | |
| geekosaur | because now you have vars that are secretly half lexical and half dynamic. which is both more expensive to keep track of and can introduce bugs when behavior is attached to the wrong aspect (which iirc is one perl 6 has run into) | ||
| MasterDuke | but if someone can figure out a way to make the generic case better, i'd 100% be fine with removing those special cases | ||
| samcv | MasterDuke: do you know how i can pass NULL to a nqp function? | 22:51 | |
| that will be received as NULL by my C function | |||
| I tried nqp::null and that does not seem to be working | 22:52 | ||
| MasterDuke | 0? | ||
| samcv | it's a string argument | ||
| ah maybe it is working. i think it may | 22:53 | ||
| MasterDuke | huh, don't know if you can. maybe have to check for vmnull and if so change to NULL | ||
| samcv | it seems to evaluate the the MVMString * as false so it must be working and the issue elsewhere | ||
| ok before this i need to solve an issue with multi method precedence. i have multi method decode(Blob:D: $encoding, Str:D :$replacement, Bool:D :$strict = False) added but it is calling multi method decode(Blob:D: $encoding) | 22:57 | ||
| it calls the new multi when I use :replacement but if i just use :strict (the boolean) then it does not call it | |||
| should i set it above the other one in the file? | |||
| MasterDuke | .tell pmurias i added that @*comp_line_directives stuff when i was adding support for `#line 2 SETTING::src/core/core_prologue.pm6` (so error message gave the real file instead of gen/moar/CORE.setting). but i barely knew what i was doing, so feel free to do something better | 22:58 | |
| samcv | github.com/samcv/rakudo/commit/0f2...77fb11L118 | 22:59 | |
| MasterDuke | samcv: as i understand it, named args aren't used for MMD, so yeah, might have to do that | 23:00 | |
| samcv | or I could just remove the other one | 23:01 | |
| since it should still work | |||
| geekosaur | and there is an order dependency iirc, because dispatch got too slow otherwise? | ||
| samcv | also would be a good idea for me to remove it and make sure all tests pass in roast | ||
| ah it also may be that i specified Str:D :$encoding | 23:04 | ||
| and if i don't specify :encoding then it won't choose that multi |