| Geth | rakudo/main: b833018251 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 5/N) - smarter submethod and yada checks - replace for by while loops - atpos/atkey vs [] and {} |
11:04 | |
| rakudo/main: 8753b0d9c7 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 5/N) - multi method collision handling |
11:05 | ||
| lizmat | meh, 6/N | 11:06 | |
| ab5tract | lizmat++ | 11:19 | |
| that module has needed a lot of love :) | |||
| lizmat | yeah, and the effects on make (spec)test are noticeable | 11:20 | |
| Geth | rakudo/main: b1167851cd | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 7/N) - abstract role/method logic in a Candidate class - and use that were appropriate |
11:22 | |
| lizmat | bisectable6: 10.3 mod 3.3 | 11:59 | |
| bisectable6 | lizmat, Will bisect the whole range automagically because no endpoints were provided, hang tight | ||
| lizmat | bisectable6: say 10.3 mod 3.3 | ||
| bisectable6 | lizmat, Output on all releases: gist.github.com/7776812ff93a0f9a12...4bc7fa5382 | ||
| lizmat, More than 3 changes to bisect, please try a narrower range like old=2024.01 new=HEAD | |||
| lizmat, Will bisect the whole range automagically because no endpoints were provided, hang tight | |||
| lizmat, Output on all releases: gist.github.com/a000d3cb45e6029ae1...63fbd071e4 | 12:00 | ||
| lizmat, More than 3 changes to bisect, please try a narrower range like old=2024.01 new=HEAD | |||
| lizmat | bisectable6: old=2024.01 say 10.3 mod 3.3 | 12:01 | |
| bisectable6 | lizmat, Bisecting by exit code (old=2024.01 new=b116785). Old exit code: 1 | ||
| lizmat, bisect log: gist.github.com/3e1d7105c7160fa737...c57e65c260 | |||
| lizmat, (2024-02-04) github.com/rakudo/rakudo/commit/71...7429f88fb7 | |||
| Geth | rakudo/main: 5749812681 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 8/N) Abstract various candidate logic into methods in the Candidate class and adapt code accordingly. This may possibly create superfluous Candidate instances, but only if the exact same method is introduced from multiple roles. Which isn't very likely. |
12:34 | |
|
13:06
nebuchad` left
13:07
nebuchad` joined
|
|||
| Geth | rakudo/main: df5fc0e30f | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/MultiMethodContainer.nqp Streamline Metamodel::MultiMethodContainer a bit - change scope of helper class and uptimize it - streamline a multi method a bit faster |
13:09 | |
| lizmat | bisectable6: dd Pod::Block::Table | 13:18 | |
| bisectable6 | lizmat, Will bisect the whole range automagically because no endpoints were provided, hang tight | ||
| lizmat, Ā¦6c (78 commits): Ā«Pod::Block::Tableā¤Ā» | 13:19 | ||
| lizmat, Nothing to bisect! | |||
| lizmat | bisectable6: old=2023.01 dd Pod::Block::Table | ||
| bisectable6 | lizmat, Cannot find revision ā2023.01ā (did you mean ā2024.01ā?) | ||
| lizmat | bisectable6: old=2023.01 dd Pod::Block::Table | ||
| bisectable6 | lizmat, Cannot find revision ā2023.01ā (did you mean ā2024.01ā?) | ||
| lizmat | bisectable6: old=2023.02 dd Pod::Block::Table | ||
| bisectable6 | lizmat, On both starting points (old=2023.02 new=df5fc0e) the exit code is 0 and the output is identical as well | ||
| lizmat, Output on both points: Ā«Pod::Block::Tableā¤Ā» | |||
| lizmat | hmmm | 13:20 | |
| Geth | rakudo/main: e57d8233d6 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/MultiMethodContainer.nqp Make adding a multi method threadsafe And also add a "add_multi_methods" to add multiple multi methods sharing the same name in one fell swoop |
13:27 | |
| ab5tract | BTW: I think I found a bug in the optimizer -- github.com/rakudo/rakudo/blob/e57d...r.nqp#L701 | 13:39 | |
| Geth | rakudo/main: ece585016a | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 9/N) Use the new "add_multi_methods" method |
||
| lizmat | ab5tract: and what would the bug be? | 13:40 | |
| ab5tract | `while ++$i < $n` isn't valid in NQP, unless I'm mistaken? | 13:41 | |
| lizmat | pretty valid in NQP | ||
| ab5tract | I mean, it works, but it isn't the semantics you are expecting | ||
| lizmat | ah? when $i starts at -1, why wouldn't it ? | ||
| ab5tract | if that's the case, why does every other while loop increment in the loop body? | 13:44 | |
| lizmat | well... I usually wrote them like that in the past | ||
| ab5tract | anyway, I spent a good while tearing my hair out before when using this syntax | ||
| lizmat | feel free to change the loop :-) | ||
| ab5tract | and the loop ony worked as expected when I did a separate increment | ||
| lizmat | in the past, until I realized that this way you'd always do one increment too many | 13:45 | |
| especially in the case where the number of elements is 0 | |||
| so I'm changing them nowadays whenever I run into them | 13:46 | ||
| ab5tract | hm, weird. the failure mode I encountered made me think that there was a deeper issue with such expressions, though I couldn't really make sense of what they would be. | 13:52 | |
| Geth | rakudo/main: d442a5baa8 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 10/N) Basically rework the way attributes are beinng registered into object creation. Also rename some variables for readability |
14:17 | |
| rakudo/main: 4fa4ca5225 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 11/N) Abstract attribute conflict resolution into a method in the attribute registration class. |
16:13 | ||
| rakudo/is-item-redux: 5 commits pushed by ab5tract++ | 16:21 | ||
| nqp/main: 70c1641805 | ab5tract++ | src/vm/jvm/runtime/org/raku/nqp/runtime/Ops.java [JVM] Fix capturenamedshash to actually include objects The condition `(flagged & CallSiteDescriptor.ARG_OBJ) != 0` will never be true because ARG_OBJ is defined as 0. It's not clear when this broke, and it also looks like capturenamedshash is never really called anywhere -- at least my angry debug statements never seemed to be output during the building of CORE.setting. |
|||
| ab5tract | And... `is item` should be done! \o/ | 16:26 | |
| lizmat | great! so, merge time is at hand ? | ||
| ab5tract | could be, could be.. maybe let's let the dust settle a bit | 16:28 | |
| lizmat | ok :-) | 16:34 | |
| Geth | rakudo/main: 5c27242bb2 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 12/N) Simplify conflict check: the name always matches, so there's no need to check it |
17:02 | |
| rakudo/main: f4943c9bf2 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/RoleToRoleApplier.nqp Streamline Metamodel::RoleToRoleApplier (Part 13/13) - add some more internal documentation - fix potential issue with setting array types - one else if -> elsif change |
18:44 | ||
| rakudo/main: c6adc333ca | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/MultiMethodContainer.nqp Streamline Metamodel::MultiMethodContainer (Part 1/N) - make sure only one thread is incorporating at a time - for to while loops - atpos/atkey instead of [] {} |
19:49 | ||
|
21:35
japhb left
21:39
japhb joined
22:52
sena_kun left
|
|||