|
06:00
[Coke] left
07:08
librasteve_ joined
07:56
[Coke] joined
08:29
finanalyst joined
09:38
librasteve_ left
10:22
finanalyst left
10:25
finanalyst joined
|
|||
| Geth | rakudo/main: eef0e946cf | (Elizabeth Mattijsen)++ | src/Raku/ast/variable-declaration.rakumod RakuAST: the default scope of constants is 'my' Not our. Reported by finanalyst++ in: github.com/Raku/problem-solving/issues/514 |
10:27 | |
|
10:45
finanalyst left
|
|||
| Geth | rakudo/main: 5cf4368e97 | (Elizabeth Mattijsen)++ | src/Raku/ast/variable-declaration.rakumod Revert "RakuAST: the default scope of constants is 'my'" This change was a bit premature, and needs a resolution to problem solving issue github.com/Raku/problem-solving/issues/514 This will then also need revision gating logic |
11:11 | |
| lizmat | TIL that "our" variables also create a MY:: entry | 11:13 | |
|
11:22
librasteve_ joined
|
|||
| librasteve | i'm sure there's a good reason for that | 11:24 | |
| ;-) | |||
| lizmat | yeah, makes sense, quicker lookups and lowering | 11:25 | |
| librasteve | starts work on the weekly (a day late, hopefully not a dollar short) | 11:26 | |
| lizmat | ++librasteve | 11:30 | |
| Geth | rakudo/main: 7fc93d521c | (Elizabeth Mattijsen)++ | 7 files Make all core constants "my" scoped Better to be explicit! |
12:18 | |
| rakudo/lizmat-my-constant: 25f3f36897 | (Elizabeth Mattijsen)++ | 4 files Make default scope for "constant" "my" in 6.e In response to github.com/Raku/problem-solving/issues/514 Handles both the legacy grammar as well as the new Raku grammar. - adds a "default-scope" attribute to VarDeclaration::Constant - make the default for that "my" - adapts the "default-scope" method to return that attribute - checks language revision to apply "my" or "our" as default scope - make sure that "default-scope" is specified in .raku |
12:35 | ||
| rakudo: lizmat++ created pull request #6080: Make default scope for "constant" "my" in 6.e |
|||
| rakudo/main: fffdb91fc6 | (David Schultz)++ (committed using GitHub Web editor) | src/core.c/Cool.rakumod In methods of `Cool` that call methods of `Str`, coerce `Cool` parameters to the adequate types (#6021) Str.contains() requires the second positional to be an Int. So one needs to convert the Cool $pos to Int, just as in the three multis above. This should fix the fact that Rakudo freezes when calling .contains with a Regex for $needle and non-Int Cool for $pos, e.g.: 12345.contains(/5/, "1") ... (9 more lines) |
14:09 | ||