[00:04] <Geth> ¦ rakudo: ugexe++ created pull request #6442: RakuAST: drop the vestigial precedence plumbing in add-categorical

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

[01:17] <Geth> ¦ rakudo: ugexe++ created pull request #6443: RakuAST: give all resolvers name suggestion capability

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

[02:14] <Geth> ¦ rakudo/main: 15 commits pushed by (Nick Logan)++

[02:14] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/compare/b837303ce328...ec5adc32356b

[02:31] <Geth> ¦ rakudo/main: 8a23d2b06a | (Nick Logan)++ (committed using GitHub Web editor) | 4 files

[02:31] <Geth> ¦ rakudo/main: RakuAST: interpret a curried argument of a BEGIN-time call (#6418)

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

[02:31] <Geth> ¦ rakudo/main: Previously a BEGIN-time call whose arguments contained a WhateverCode

[02:31] <Geth> ¦ rakudo/main: could not interpret its argument list, so the whole call was compiled

[02:31] <Geth> ¦ rakudo/main: dynamically and run as a throwaway thunk. The WhateverCode built inside

[02:31] <Geth> ¦ rakudo/main: that thunk carried the thunk's frame as its outer, which no longer

[02:31] <Geth> ¦ rakudo/main: matches its static frame once the closure is serialized into a

[02:31] <Geth> ¦ rakudo/main: <…commit message has 24 more lines…>

[02:32] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/8a23d2b06a

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

[02:32] <Geth> ¦ rakudo/main: a3356e1423 | RakuAST: iterate a sunk for loop without calling .map

[02:32] <Geth> ¦ rakudo/main: 21954f9b42 | RakuAST: lower a for loop over an integer range to a counting loop

[02:32] <Geth> ¦ rakudo/main: 675d9868a1 | RakuAST: resolve an operator in scope when approving a lowering

[02:32] <Geth> ¦ rakudo/main: 7f81c1d85f | Merge pull request #6417 from ugexe/ugexe/rakuast-for-statement-iterator

[02:32] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/compare/8a23d2b06ad4...7f81c1d85f77

[02:36] <Geth> ¦ rakudo/main: 911780ace6 | (Nick Logan)++ (committed using GitHub Web editor) | src/Raku/ast/compunit.rakumod

[02:36] <Geth> ¦ rakudo/main: RakuAST: intern Rat and Complex literals (#6421)

[02:36] <Geth> ¦ rakudo/main: 

[02:36] <Geth> ¦ rakudo/main: Previously intern-decimal, intern-rat, and intern-complex on

[02:36] <Geth> ¦ rakudo/main: RakuAST::LiteralBuilder were marked TODO and simply built a fresh

[02:36] <Geth> ¦ rakudo/main: object for every literal. This interns them the same way Int, Num,

[02:36] <Geth> ¦ rakudo/main: and Str literals already are, so repeated occurrences of the same

[02:36] <Geth> ¦ rakudo/main: rational or complex literal in a compilation unit share one object

[02:36] <Geth> ¦ rakudo/main: <…commit message has 7 more lines…>

[02:36] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/911780ace6

[02:37] <Geth> ¦ rakudo/main: c8de6ef062 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files

[02:37] <Geth> ¦ rakudo/main: RakuAST: look the GLOBAL package up at run time (#6419)

[02:37] <Geth> ¦ rakudo/main: 

[02:37] <Geth> ¦ rakudo/main: Previously a GLOBAL-rooted package variable compiled to a reference to

[02:37] <Geth> ¦ rakudo/main: the compilation unit's own GLOBAL package object. Each unit has its own

[02:37] <Geth> ¦ rakudo/main: GLOBAL, and they merge into the process-wide one as units load, so a

[02:37] <Geth> ¦ rakudo/main: precompiled module that wrote %GLOBAL::CONFIG at run time wrote into

[02:37] <Geth> ¦ rakudo/main: its unit's copy, whose stash nothing else consults. The write was

[02:37] <Geth> ¦ rakudo/main: <…commit message has 7 more lines…>

[02:37] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/c8de6ef062

[02:37] <Geth> ¦ rakudo/main: 3b3c0252be | (Nick Logan)++ | 3 files

[02:37] <Geth> ¦ rakudo/main: RakuAST: defer the SameNameAsEnclosing worry to check time

[02:37] <Geth> ¦ rakudo/main: 

[02:37] <Geth> ¦ rakudo/main: Previously IMPL-INSTALL-PACKAGE reported this worry immediately. Symbol

[02:37] <Geth> ¦ rakudo/main: installation runs at BEGIN time, before the parser has attached the

[02:37] <Geth> ¦ rakudo/main: node's origin, so the worry could never be tied to the declaration's

[02:37] <Geth> ¦ rakudo/main: file and line. This collects it on the PackageInstaller and reports it

[02:37] <Geth> ¦ rakudo/main: from PERFORM-CHECK instead, alongside the deferred trait sorries.

[02:37] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/3b3c0252be

[02:37] <Geth> ¦ rakudo/main: 718f289c15 | (Nick Logan)++ | 8 files

[02:37] <Geth> ¦ rakudo/main: RakuAST: report compiler worries on the node instead of the resolver

[02:37] <Geth> ¦ rakudo/main: 

[02:37] <Geth> ¦ rakudo/main: Previously several worries were added directly to the resolver, which

[02:37] <Geth> ¦ rakudo/main: bypasses the node-side check-time handling. This resolves the XXX

[02:37] <Geth> ¦ rakudo/main: comments in Methodish.PERFORM-BEGIN and converts the same pattern in

[02:37] <Geth> ¦ rakudo/main: CompUnit.check, TraitTarget.apply-traits (through its existing deferred

[02:37] <Geth> ¦ rakudo/main: worries list), VarDeclaration::Simple.PERFORM-CHECK, and

[02:37] <Geth> ¦ rakudo/main: <…commit message has 6 more lines…>

[02:37] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/718f289c15

[02:37] <Geth> ¦ rakudo/main: b7675a9671 | (Nick Logan)++ (committed using GitHub Web editor) | 8 files

[02:37] <Geth> ¦ rakudo/main: Merge pull request #6420 from ugexe/ugexe/rakuast-useless-method-worry

[02:37] <Geth> ¦ rakudo/main: 

[02:38] <Geth> ¦ rakudo/main: RakuAST: report compiler worries on the node instead of the resolver

[02:38] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/b7675a9671

[02:38] <Geth> ¦ rakudo/main: 7ae0ddce83 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files

[02:38] <Geth> ¦ rakudo/main: RakuAST: apply val to quoted atoms in quote words literals (#6422)

[02:38] <Geth> ¦ rakudo/main: 

[02:38] <Geth> ¦ rakudo/main: Previously QuotedString.literal-value treated the quotewords processor

[02:38] <Geth> ¦ rakudo/main: exactly like words, and pushed a quoted atom's value into the result

[02:38] <Geth> ¦ rakudo/main: untouched. That left the atom out of val processing, so «1 "2" 3.5»

[02:38] <Geth> ¦ rakudo/main: compiled to a plain Str "2" where the runtime path and the legacy

[02:38] <Geth> ¦ rakudo/main: frontend both produce an IntStr allomorph.

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

[02:38] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/7ae0ddce83

[02:38] <Geth> ¦ rakudo/main: a336359563 | (Nick Logan)++ (committed using GitHub Web editor) | src/Raku/ast/variable-declaration.rakumod

[02:38] <Geth> ¦ rakudo/main: RakuAST: use Scalar as the container type of $ sigil variables (#6424)

[02:38] <Geth> ¦ rakudo/main: 

[02:38] <Geth> ¦ rakudo/main: Previously ContainerCreator set the container type of an ordinary

[02:38] <Geth> ¦ rakudo/main: scalar variable to its value type, with a FIXME noting it should be

[02:38] <Geth> ¦ rakudo/main: Scalar like the container base type. The legacy frontend sets its

[02:38] <Geth> ¦ rakudo/main: container_type to Scalar for the $ sigil as well.

[02:38] <Geth> ¦ rakudo/main: 

[02:38] <Geth> ¦ rakudo/main: <…commit message has 7 more lines…>

[02:38] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/a336359563

[02:38] <Geth> ¦ rakudo/main: 153aacf058 | (Nick Logan)++ (committed using GitHub Web editor) | 3 files

[02:38] <Geth> ¦ rakudo/main: RakuAST: resolve the 'maybe also check?' questions on synthesized nodes (#6425)

[02:38] <Geth> ¦ rakudo/main: 

[02:38] <Geth> ¦ rakudo/main: Four synthesized nodes were sent to begin time with a TODO asking

[02:38] <Geth> ¦ rakudo/main: whether they should also get check time. The answer is no in each

[02:38] <Geth> ¦ rakudo/main: case, for two different reasons.

[02:38] <Geth> ¦ rakudo/main: 

[02:38] <Geth> ¦ rakudo/main: The subset where block and the non-attribute accessor are bound to

[02:39] <Geth> ¦ rakudo/main: <…commit message has 11 more lines…>

[02:39] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/153aacf058

[02:47] <Geth> ¦ rakudo/main: 176b5ca56b | (Nick Logan)++ (committed using GitHub Web editor) | 7 files

[02:47] <Geth> ¦ rakudo/main: RakuAST: support the :dba regex modifier and pass it to FAILGOAL (#6434)

[02:47] <Geth> ¦ rakudo/main: 

[02:47] <Geth> ¦ rakudo/main: Previously :dba("...") in a regex died with an unrecognized modifier

[02:47] <Geth> ¦ rakudo/main: error: the mod metachar's parenthesized string argument never parsed

[02:47] <Geth> ¦ rakudo/main: because HLL::Grammar's quote_EXPR does not match inside the Raku

[02:47] <Geth> ¦ rakudo/main: grammar, and no node existed to carry the name. The grammar now parses

[02:47] <Geth> ¦ rakudo/main: the quoted string itself, a Regex::InternalModifier::Dba node records

[02:47] <Geth> ¦ rakudo/main: <…commit message has 8 more lines…>

[02:47] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/176b5ca56b

[03:24] <ugexe> i went through about 60 TODO/XXX and similar type comments in rakuast code and either 1) deleted it altogether if it was no longer relevant 2) replaced it with a comment that clarifies why something is the way it is 3) implemented whatever it referenced

[04:14] *** ShimmerFairy left
[04:29] <Geth> ¦ rakudo: ugexe++ created pull request #6444: RakuAST: prune the special variables from onlystar routines

[04:29] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6444