[00:27] *** hankache_ joined
[00:31] *** hankache left
[01:49] *** rakkable left
[02:02] *** lizmat left
[02:05] *** rakkable joined
[02:23] *** lizmat joined
[05:08] *** vrurg joined
[05:09] *** vrurg_ left
[06:39] *** hankache_ left
[08:01] *** nine_ left
[08:01] *** nine joined
[10:31] <Geth> ¦ rakudo/lizmat-82: 2bad874309 | (Elizabeth Mattijsen)++ | src/Raku/ast/literals.rakumod

[10:31] <Geth> ¦ rakudo/lizmat-82: RakuAST: if "&val" is not resolved, don't bother doing a call

[10:31] <Geth> ¦ rakudo/lizmat-82: 

[10:31] <Geth> ¦ rakudo/lizmat-82: Before this commit a dummy Callable was being inserted to mimic

[10:31] <Geth> ¦ rakudo/lizmat-82: val() processing.  There's no need for that, as that Callable just

[10:31] <Geth> ¦ rakudo/lizmat-82: returned the value as is.

[10:31] <Geth> ¦ rakudo/lizmat-82: 

[10:31] <Geth> ¦ rakudo/lizmat-82: Instead just call the val() processor if it is available.

[10:31] <Geth> ¦ rakudo/lizmat-82: 

[10:31] <Geth> ¦ rakudo/lizmat-82: Practically this will only affect core setting compilation before

[10:31] <Geth> ¦ rakudo/lizmat-82: val() processing as becomes active.  Still felt this was needed

[10:31] <Geth> ¦ rakudo/lizmat-82: for code clarity.

[10:31] <Geth> ¦ rakudo/lizmat-82: review: https://github.com/rakudo/rakudo/commit/2bad874309

[10:31] <Geth> ¦ rakudo: lizmat++ created pull request #6861: RakuAST: if "&val" is not resolved, don't bother doing a call

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

[11:08] <Geth> ¦ rakudo/lizmat-83: 1f5d3c77df | (Elizabeth Mattijsen)++ | src/core.c/allomorphs.rakumod

[11:08] <Geth> ¦ rakudo/lizmat-83: Remove return statement from "parse_win" in val processing

[11:08] <Geth> ¦ rakudo/lizmat-83: 

[11:08] <Geth> ¦ rakudo/lizmat-83: Since all of the choices in "parse_win" contained either a return

[11:08] <Geth> ¦ rakudo/lizmat-83: or a die statement, and "parse_win" was in one case called at the

[11:08] <Geth> ¦ rakudo/lizmat-83: end of the val() sub, it felt like a good idea to remove the return

[11:08] <Geth> ¦ rakudo/lizmat-83: statements from the "parse_win" Callable, and add a return

[11:08] <Geth> ¦ rakudo/lizmat-83: statement to the one case where "parse_win" needed it.

[11:08] <Geth> ¦ rakudo/lizmat-83: review: https://github.com/rakudo/rakudo/commit/1f5d3c77df

[11:08] <Geth> ¦ rakudo: lizmat++ created pull request #6862: Remove return statement from "parse_win" in val processing

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

[11:12] *** samebchase joined
[11:12] *** samebchase left
[11:19] <Geth> ¦ rakudo/lizmat-84: 1be1db5141 | (Elizabeth Mattijsen)++ | src/core.c/allomorphs.rakumod

[11:19] <Geth> ¦ rakudo/lizmat-84: Remove GUARDED-NUMERIC implementation-detail sub

[11:19] <Geth> ¦ rakudo/lizmat-84: 

[11:19] <Geth> ¦ rakudo/lizmat-84: It was only called in one place, and the code was minimal, so

[11:19] <Geth> ¦ rakudo/lizmat-84: just move the code to where it was being called

[11:19] <Geth> ¦ rakudo/lizmat-84: review: https://github.com/rakudo/rakudo/commit/1be1db5141

[11:33] *** samebchase joined
[12:04] <Geth> ¦ rakudo/lizmat-85: 7310615851 | (Elizabeth Mattijsen)++ | 20 files

[12:04] <Geth> ¦ rakudo/lizmat-85: RakuAST; make .get-implicit-lookups return an nqp::list

[12:04] <Geth> ¦ rakudo/lizmat-85: 

[12:04] <Geth> ¦ rakudo/lizmat-85: Rather than a HLL list.  Since all calls to .get-implicit-lookups

[12:04] <Geth> ¦ rakudo/lizmat-85: were wrapped in a self.IMPL-UNWRAP-LIST, and .get-implicit-lookups

[12:04] <Geth> ¦ rakudo/lizmat-85: always wrapped any NQP list in a self.IMPL-WRAP-LIST, it doesn't

[12:04] <Geth> ¦ rakudo/lizmat-85: make any sense to do this.  So remove the IMPL-WRAP-LIST from

[12:04] <Geth> ¦ rakudo/lizmat-85: .get-implicit-lookups, and remove all IMPL-UNWRAP-LIST from calls

[12:04] <Geth> ¦ rakudo/lizmat-85: to .get-implicit-lookups.

[12:04] <Geth> ¦ rakudo/lizmat-85: 

[12:04] <Geth> ¦ rakudo/lizmat-85: This method is getting called more than 100K times during setting

[12:04] <Geth> ¦ rakudo/lizmat-85: compilation.

[12:04] <Geth> ¦ rakudo/lizmat-85: review: https://github.com/rakudo/rakudo/commit/7310615851

[12:05] <Geth> ¦ rakudo: lizmat++ created pull request #6863: Remove GUARDED-NUMERIC implementation-detail sub

[12:05] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6863

[12:05] <Geth> ¦ rakudo: lizmat++ created pull request #6864: RakuAST; make .get-implicit-lookups return an nqp::list

[12:05] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6864

[12:35] *** lizmat left
[13:05] *** ShimmerFairy left
[13:22] *** lizmat joined
[13:35] *** ShimmerFairy joined
[13:42] *** Pixi left
[13:42] *** coleman left
[13:42] *** kjp left
[13:42] *** djinni` left
[13:49] *** Pixi joined
[13:49] *** coleman joined
[13:49] *** kjp joined
[13:49] *** djinni` joined
[14:15] <Geth> ¦ rakudo/lizmat-86: 2773abb581 | (Elizabeth Mattijsen)++ | src/Raku/ast/literals.rakumod

[14:15] <Geth> ¦ rakudo/lizmat-86: RakuAST: inline ::QuotedString.IMPL-CHECK-PROCESSORS

[14:15] <Geth> ¦ rakudo/lizmat-86: 

[14:15] <Geth> ¦ rakudo/lizmat-86: It was only called in one place.  Also only do processor checking

[14:15] <Geth> ¦ rakudo/lizmat-86: only if there are any processors to check

[14:15] <Geth> ¦ rakudo/lizmat-86: review: https://github.com/rakudo/rakudo/commit/2773abb581

[14:16] <Geth> ¦ rakudo: lizmat++ created pull request #6865: RakuAST: inline ::QuotedString.IMPL-CHECK-PROCESSORS

[14:16] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6865

[14:37] *** ShimmerFairy left
[14:39] *** ShimmerFairy joined
[14:48] <Geth> ¦ rakudo/lizmat-87: 46e8a0d99b | (Elizabeth Mattijsen)++ | src/Raku/ast/literals.rakumod

[14:48] <Geth> ¦ rakudo/lizmat-87: RakuAST: simplify ::QuotedString.IMPL-NATIVE-LITERAL-KIND

[14:48] <Geth> ¦ rakudo/lizmat-87: 

[14:48] <Geth> ¦ rakudo/lizmat-87: Checking for processors can be done in a single if.  Use the

[14:48] <Geth> ¦ rakudo/lizmat-87: same logic as the "ast-type" method which follows

[14:48] <Geth> ¦ rakudo/lizmat-87: review: https://github.com/rakudo/rakudo/commit/46e8a0d99b

[14:48] <Geth> ¦ rakudo: lizmat++ created pull request #6866: RakuAST: simplify ::QuotedString.IMPL-NATIVE-LITERAL-KIND

[14:48] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6866

[14:49] *** finanalyst joined
[14:59] * [Coke] returns (off line yesterday for volunteer reasons, then network issues, then exhaustion. :)

[14:59] <lizmat> [Coke]: welcome back

[15:03] <[Coke]> j

[15:13] <Geth> ¦ rakudo/lizmat-88: e62cc8e44e | (Elizabeth Mattijsen)++ | 2 files

[15:13] <Geth> ¦ rakudo/lizmat-88: RakuAST: access val() processing in an easier way

[15:13] <Geth> ¦ rakudo/lizmat-88: 

[15:13] <Geth> ¦ rakudo/lizmat-88: By adding a val() candidate taking a native string, and then

[15:13] <Geth> ¦ rakudo/lizmat-88: not bothering HLLization in ::QuoteString.IMPL-PROCESS-PART

[15:13] <Geth> ¦ rakudo/lizmat-88: review: https://github.com/rakudo/rakudo/commit/e62cc8e44e

[15:14] <Geth> ¦ rakudo: lizmat++ created pull request #6867: RakuAST: access val() processing in an easier way

[15:14] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6867

[15:40] *** ilogger2 left
[15:48] *** ShimmerFairy left
[15:50] *** ilogger2 joined
[15:50] *** ShimmerFairy joined
[16:16] *** vrurg_ joined
[16:18] *** vrurg left
[17:00] <Geth> ¦ rakudo/lizmat-89: 5a655d5bf3 | (Elizabeth Mattijsen)++ | src/Raku/ast/literals.rakumod

[17:00] <Geth> ¦ rakudo/lizmat-89: RakuAST: re-image ::QuotedString.IMPL-EXPR-QAST

[17:00] <Geth> ¦ rakudo/lizmat-89: 

[17:00] <Geth> ¦ rakudo/lizmat-89: Instigated by https://github.com/rakudo/rakudo/issues/6860 the

[17:00] <Geth> ¦ rakudo/lizmat-89: problem was that it was assumed that concatenation of interpolated

[17:00] <Geth> ¦ rakudo/lizmat-89: strings could be done with nqp::ops such as concat and join.

[17:00] <Geth> ¦ rakudo/lizmat-89: 

[17:00] <Geth> ¦ rakudo/lizmat-89: However, that means that any additional candidates to infix:<~>

[17:00] <Geth> ¦ rakudo/lizmat-89: <…commit message has 6 more lines…>

[17:00] <Geth> ¦ rakudo/lizmat-89: review: https://github.com/rakudo/rakudo/commit/5a655d5bf3

[17:00] <Geth> ¦ rakudo: lizmat++ created pull request #6868: RakuAST: re-image ::QuotedString.IMPL-EXPR-QAST

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

[18:31] <Geth> ¦ rakudo/main: 0138d1dd6f | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 12 files

[18:31] <Geth> ¦ rakudo/main: RakuAST: native int variables don't need := 0 initialization (#6835)

[18:31] <Geth> ¦ rakudo/main: 

[18:31] <Geth> ¦ rakudo/main: * RakuAST: native int variables don't need := 0 initialization

[18:31] <Geth> ¦ rakudo/main: 

[18:31] <Geth> ¦ rakudo/main: Native int variables are "born" with the value 0.

[18:31] <Geth> ¦ rakudo/main: 

[18:31] <Geth> ¦ rakudo/main: This removes this unnecessary initialization from many lexical

[18:31] <Geth> ¦ rakudo/main: variables, and also changes some variables to native ints as they're

[18:31] <Geth> ¦ rakudo/main: clearly intended to be used as flags.

[18:31] <Geth> ¦ rakudo/main: 

[18:31] <Geth> ¦ rakudo/main: * Oops, forgot one "int"

[18:31] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/0138d1dd6f

[19:35] <Geth> ¦ rakudo/lizmat-90: 41e8b3ccf8 | (Elizabeth Mattijsen)++ | src/Raku/ast/var-lowering.rakumod

[19:35] <Geth> ¦ rakudo/lizmat-90: RakuAST: streamline ::IMPL::VarLoweringFrame.IMPL-WALK

[19:35] <Geth> ¦ rakudo/lizmat-90: 

[19:35] <Geth> ¦ rakudo/lizmat-90: - remove unnecessary variable

[19:35] <Geth> ¦ rakudo/lizmat-90: - merge 2 if structures

[19:35] <Geth> ¦ rakudo/lizmat-90: - rewrite without needing return

[19:35] <Geth> ¦ rakudo/lizmat-90: review: https://github.com/rakudo/rakudo/commit/41e8b3ccf8

[19:36] <Geth> ¦ rakudo: lizmat++ created pull request #6870: RakuAST: streamline ::IMPL::VarLoweringFrame.IMPL-WALK

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

[20:05] *** lizmat left
[20:05] *** lizmat_ joined
[20:36] *** hankache_ joined
[21:30] *** hankache_ left
[21:31] *** hankache joined
[21:51] *** lizmat_ left
[21:52] *** lizmat joined
[21:57] *** hankache left
[21:57] *** finanalyst left