[00:05] <[Coke]> is https://github.com/croservices/cro-webapp the canon repo?

[00:23] <Geth> ¦ rakudo/main: 83baac0d6a | (Elizabeth Mattijsen)++ | src/Raku/ast/name.rakumod

[00:23] <Geth> ¦ rakudo/main: Use a lookup table for ::Name::Part::Simple.is-pseudo-package

[00:23] <Geth> ¦ rakudo/main: 

[00:23] <Geth> ¦ rakudo/main: Instead of repeated comparisons

[00:23] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/83baac0d6a

[00:23] <Geth> ¦ rakudo/main: 4fa2be82f8 | (Elizabeth Mattijsen)++ | src/Raku/ast/name.rakumod

[00:23] <Geth> ¦ rakudo/main: Make sure ::Name::Part::Simple returns a Bool

[00:23] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/4fa2be82f8

[00:23] <Geth> ¦ rakudo/main: 66c194ccb1 | (Will Coleda)++ (committed using GitHub Web editor) | src/Raku/ast/name.rakumod

[00:23] <Geth> ¦ rakudo/main: Merge pull request #6770 from rakudo/lizmat-40

[00:23] <Geth> ¦ rakudo/main: 

[00:23] <Geth> ¦ rakudo/main: Use a lookup table for ::Name::Part::Simple.is-pseudo-package

[00:23] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/66c194ccb1

[00:29] <Geth> ¦ rakudo/main: 5 commits pushed by (Nick Logan)++, (Will Coleda)++

[00:29] <Geth> ¦ rakudo/main: 1ed9dc4878 | RakuAST: keep a lexical that a nested use reaches by name

[00:29] <Geth> ¦ rakudo/main: de5aa23786 | RakuAST: parse a heredoc body in the scopes where it starts

[00:29] <Geth> ¦ rakudo/main: a6b94c838e | RakuAST: die on a heredoc consumed before its body is parsed

[00:29] <Geth> ¦ rakudo/main: fb79d40eac | Leave a closing brace after a heredoc statement to the enclosing block

[00:29] <Geth> ¦ rakudo/main: 9e97041efa | Merge pull request #6774 from ugexe/ugexe/rakuast-heredoc-scoping

[00:29] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/compare/66c194ccb199...9e97041efaba

[00:35] <Geth> ¦ rakudo/main: 2202fa2992 | (Nick Logan)++ | 2 files

[00:35] <Geth> ¦ rakudo/main: RakuAST: only take a listop invocant colon before any comma

[00:35] <Geth> ¦ rakudo/main: 

[00:35] <Geth> ¦ rakudo/main: Previously `comb sum 1000,2000: 2` failed to compile with "Only identical

[00:35] <Geth> ¦ rakudo/main: operators may be list associative" because the colon was parsed as part

[00:35] <Geth> ¦ rakudo/main: of the arguments to `sum`, even though `sum` had already seen a comma.

[00:35] <Geth> ¦ rakudo/main: The legacy grammar clears `$*INVOCANT_OK` when it parses a comma

[00:35] <Geth> ¦ rakudo/main: (4969e671ef), so the colon instead becomes the invocant colon of `comb`,

[00:35] <Geth> ¦ rakudo/main: <…commit message has 16 more lines…>

[00:35] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/2202fa2992

[00:35] <Geth> ¦ rakudo/main: 30b42b2460 | (Nick Logan)++ | 3 files

[00:35] <Geth> ¦ rakudo/main: RakuAST: keep the invocant of a semicolon separated argument list

[00:35] <Geth> ¦ rakudo/main: 

[00:35] <Geth> ¦ rakudo/main: Previously `f(A: 1, 2; 3)` called the sub `f` with `\(((1, 2),), (3,))`

[00:35] <Geth> ¦ rakudo/main: instead of calling the method `f` on `A` with `\((1, 2), (3,))`. The

[00:35] <Geth> ¦ rakudo/main: semiarglist action rebuilds the argument list from the args of each

[00:35] <Geth> ¦ rakudo/main: section, which lost the invocant of the first one. Those args were also

[00:35] <Geth> ¦ rakudo/main: a single comma list, because `ArgList.from-invocant-list` left unpacking

[00:35] <Geth> ¦ rakudo/main: <…commit message has 5 more lines…>

[00:35] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/30b42b2460

[00:35] <Geth> ¦ rakudo/main: 8de7894122 | (Will Coleda)++ (committed using GitHub Web editor) | 4 files

[00:35] <Geth> ¦ rakudo/main: Merge pull request #6773 from ugexe/ugexe/rakuast-invocant-colon-after-comma

[00:35] <Geth> ¦ rakudo/main: 

[00:35] <Geth> ¦ rakudo/main: RakuAST: give an invocant colon to the listop it belongs to

[00:35] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/8de7894122

[00:36] <Geth> ¦ rakudo: ugexe++ created pull request #6776: Optimize $x ** 2 and $x²

[00:36] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6776

[01:40] <ugexe> https://github.com/rakudo/rakudo/pull/6775/changes regresses ::("") to return False for .not-empty. also i suspect the code might be faster without the changes (adding nqp::hllbool) and just adding --> Bool to the return constraint instead

[01:41] <ugexe> nqp::hllboolfor doesnt get inlined

[02:02] <ugexe> s/not-empty/is-empty/

[02:15] <Geth> ¦ rakudo/revert-6775-lizmat-42: 1e88c7a0f7 | (Nick Logan)++ (committed using GitHub Web editor) | src/Raku/ast/name.rakumod

[02:15] <Geth> ¦ rakudo/revert-6775-lizmat-42: Revert "Make sure ::Name::Part.is-empty always returns a Bool (#6775)"

[02:15] <Geth> ¦ rakudo/revert-6775-lizmat-42: 

[02:15] <Geth> ¦ rakudo/revert-6775-lizmat-42: This reverts commit b22a0daf20825694b2e4c20e036b3fedee743c91.

[02:15] <Geth> ¦ rakudo/revert-6775-lizmat-42: review: https://github.com/rakudo/rakudo/commit/1e88c7a0f7

[02:20] <Geth> ¦ rakudo: ugexe++ created pull request #6778: Revert "Make sure ::Name::Part.is-empty always returns a Bool"

[02:20] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6778

[02:50] <Geth> ¦ rakudo: ugexe++ created pull request #6779: RakuAST: restore is-empty for an empty string name part and inline Bool return checks

[02:50] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6779

[02:52] <ugexe> so --> Bool wasn't faster, but with that ^ PR it is. i fixed the is-empty regression and switched the hllboolfor stuff out for the --> Bool stuff and it does benchmark faster

[03:40] <Geth> ¦ rakudo/main: 4 commits pushed by (Nick Logan)++

[03:40] <Geth> ¦ rakudo/main: e62f971f2b | RakuAST: restore is-empty for an empty string name part

[03:40] <Geth> ¦ rakudo/main: 2a9aedcc1a | RakuAST: make the Bool return check small enough to inline

[03:40] <Geth> ¦ rakudo/main: eb5ebefd37 | RakuAST: declare Bool returns on name predicates

[03:40] <Geth> ¦ rakudo/main: 6baa50125b | Merge pull request #6779 from ugexe/ugexe/rakuast-inline-bool-return

[03:40] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/compare/8de7894122c3...6baa50125bcf

[04:50] *** japhb left
[05:05] *** japhb joined
[08:46] <lizmat> ok, so if I understand this correctly, specifying --> Bool (in the RakuAST bootstrap) is enough to turn an int into a proper Bool on return ?

[09:01] <lizmat> do we consider resolvers as implementation-detail or not?

[09:01] <lizmat> I would tend to think they are implementation detail?

[10:22] <Geth> ¦ rakudo/lizmat-43: 613f7bf2b1 | (Elizabeth Mattijsen)++ | 5 files

[10:22] <Geth> ¦ rakudo/lizmat-43: RakuAST: use more lookup tables and --> Bool

[10:22] <Geth> ¦ rakudo/lizmat-43: 

[10:22] <Geth> ¦ rakudo/lizmat-43: - affects some "is-xxx" public methods

[10:22] <Geth> ¦ rakudo/lizmat-43: - add lookup table in IMPL-THUNK-ARGUMENTS because it makes sense

[10:22] <Geth> ¦ rakudo/lizmat-43: review: https://github.com/rakudo/rakudo/commit/613f7bf2b1

[10:22] <Geth> ¦ rakudo: lizmat++ created pull request #6780: RakuAST: use more lookup tables and --> Bool

[10:22] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6780

[10:23] <Geth> ¦ rakudo/main: 4 commits pushed by (Nick Logan)++, (Elizabeth Mattijsen)++

[10:23] <Geth> ¦ rakudo/main: 13e65aec25 | Square a number by multiplying it with itself

[10:23] <Geth> ¦ rakudo/main: f336b137aa | RakuAST: give a literal postfix a static callee lookup

[10:23] <Geth> ¦ rakudo/main: 76d4625d79 | RakuAST: leave a square to the power routine

[10:23] <Geth> ¦ rakudo/main: 357a090822 | Merge pull request #6776 from ugexe/ugexe/square-fast-path

[10:23] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/compare/6baa50125bcf...357a09082261

[10:26] *** hankache joined
[11:10] <Geth> ¦ rakudo/lizmat-44: c88ce28f66 | (Elizabeth Mattijsen)++ | 10 files

[11:10] <Geth> ¦ rakudo/lizmat-44: RakuAST: some more --> Bool for "is-" accessors

[11:10] <Geth> ¦ rakudo/lizmat-44: 

[11:10] <Geth> ¦ rakudo/lizmat-44: So that they properly return a Bool rather than a 0 or 1

[11:10] <Geth> ¦ rakudo/lizmat-44: review: https://github.com/rakudo/rakudo/commit/c88ce28f66

[11:11] <Geth> ¦ rakudo: lizmat++ created pull request #6781: RakuAST: some more --> Bool for "is-" accessors

[11:11] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6781

[11:17] *** hankache left
[11:18] *** ShimmerFairy left