Geth rakudo: ugexe++ created pull request #6652:
Measure a case-folded interpolation match against the target		 00:52
rakudo: ugexe++ created pull request #6653:
Throw for a hash used as a regex assertion		 00:53
01:54 kurahaupo left 02:11 coleman left, summerisle left, ugexe left 02:27 coleman joined, summerisle joined, ugexe joined
Geth rakudo: ugexe++ created pull request #6654:
Match the lookahead greedily in the LTM before-assertion test		 03:31
rakudo/main: f4f55b2775 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files
Match the initializer symbol before rejecting a signature declaration (#6651)

Previously initializer:sym<=> checked for a signature declaration before matching its symbol, so that group led the candidate and set its declarative prefix. Once a sequential alternation ends the prefix, that candidate has an empty prefix and gets tried on any input. A my :(...) declaration with no initializer at all then died with the assignment error instead of the missing initializer one.
This matches the symbol first, so the candidate is only tried when an assignment is actually there.		 03:34
nqp/main: 8172e76567 | (Nick Logan)++ | 2 files
Stop the declarative prefix at a sequential alternation

Previously the NFA for a sequential alternation covered its first branch and an empty path, then kept going with whatever followed the group. That made the declarative prefix of `[ x || y ] z` behave like
  `x? z`, so a branch of a longest token alternation containing that
group was never tried on input such as `yz` even though `y` matches ... (6 more lines)		 04:01
nqp/main: 26181555c8 | (Nick Logan)++ | src/QRegex/NFA.nqp
Stop building a repetition after the declarative prefix has ended

Previously the counted repetition loop kept calling the builder with whatever state the previous repetition returned, and a repetition that ended the declarative prefix returns no state. The loop then pushed edges onto the fate list, and a sequential alternation under a counted quantifier is a common way to end the prefix.
This leaves the loop once a repetition ends the prefix. Only the paths that skipped the remaining repetitions carry on.		 04:02
nqp/main: f639323a13 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files
Merge pull request #871 from ugexe/ugexe/altseq-terminates-ltm

Stop the declarative prefix at a sequential alternation
rakudo: ugexe++ created pull request #6655:
Bump NQP to stop the declarative prefix at a sequential alternation		 04:04
rakudo/main: ab802d1eef | (Nick Logan)++ (committed using GitHub Web editor) | t/02-rakudo/ltm-before-assertion.t
Match the lookahead greedily in the LTM before-assertion test (#6654)

Previously the test grammar looked ahead with `.+?`, which only contributed to the declarative prefix because the NFA treated a frugal quantifier like a greedy one. S05 ends the longest token at a frugal quantifier, so the test would then assert nothing about the lookahead being inlined, which is what it was written for.
This looks ahead with `.+` instead. Inside a lookahead nothing is consumed, so the assertion succeeds on the same input either way.		 04:46
rakudo/main: 6b984fe73f | (Nick Logan)++ (committed using GitHub Web editor) | tools/templates/NQP_REVISION
Bump NQP to stop the declarative prefix at a sequential alternation (#6655)
 05:16
05:20 kurahaupo joined
Geth rakudo/main: 2d14ad281b | (Nick Logan)++ (committed using GitHub Web editor) | 2 files
RakuAST: split long decimal numbers before converting them to Int (#6647)

Previously the literal builder handed a whole decimal digit string to the VM bigint parser, which is quadratic in the digit count. A capture variable, integer literal, or regex quantifier bound with a million digits took around a minute to compile, while the legacy frontend parsed the same capture index into a native int in under a second. ... (7 more lines)		 05:23
rakudo/main: d6ada9bd14 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files
Backtrack into interpolated regex alternatives (#6650)

Previously an array (or list, or any junction) interpolated into a regex was tried once: INTERPOLATE walked every candidate the NFA passed and returned the longest match as a plain cursor. When the rest of the regex failed after it there was nothing to backtrack into, so
  `'abc' ~~ / @(< a ab >) b /` was Nil where the literal `< a ab >` form
... (17 more lines)		 05:26
06:17 kurahaupo left 07:26 lizmat joined 08:13 lizmat left 08:31 kurahaupo joined 11:27 patrickb_ joined 11:34 patrickb left, rakkable left, patrickb_ is now known as patrickb
[Coke] pretty sure both of these were already covered: 14:43
github.com/coke/raku-blin-release-...ailures.md
up through d6ada9b
er... not quite. the followup was through d6ada9b. the initial run was 13e3f3a 14:44
14:55 kurahaupo left
ugexe not sure what is up with Text::Markdown::Discount really 15:15
the test output looks the same between previous and the failing case other than zef says FAIL at the end 15:16
so maybe a test exiting non-zero... but with no errors for whatever reason
"Root cause is in the module, not rakudo. It declares mkd_document(MMIOT, CArray[Str]) and reads the HTML back from slot 0. A managed CArray[Str] frees each element string with MoarVM's allocator when it is collected, so the char* owned by the discount document gets handed to mimalloc. Since MoarVM moved to mimalloc v3 in August that free segfaults instead of being ignored. Changing the 15:29
declaration to CArray[Pointer] and returning nativecast(Str, $buf[0]) fixes it." 15:30
Geth nqp: ugexe++ created pull request #872:
End the declarative prefix at a frugal quantifier		 15:40
[Coke] anyone know where the git repo for Text::Markdown::Discount is? it's on REA. 17:02
tempted to add it to the skip pile if we can't get the maintainer
ugexe its at github.com/raku-community-modules/...n-Discount but hasnt been touched in forever and has two open PRs that make it seem like there are other problems with it 17:12
i'd probably just throw it in the skip pile
Geth rakudo/main: 3044d0d479 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files
Look ahead greedily for a bare rational or complex literal (#6656)

Previously `bare_rat_number` and `bare_complex_number`, and their RakuAST counterparts, looked ahead with `+?` to tell `<1/2>` and
  `<1+2i>` apart from quote words. Inside a lookahead nothing is
consumed, and neither `/` nor `i` is in the character class being scanned, so a frugal scan and a greedy scan stop at the same place and ... (6 more lines)		 17:20
rakudo/main: b61226d8bc | (Nick Logan)++ (committed using GitHub Web editor) | 4 files
Throw for a hash used as a regex assertion (#6653)

Previously a hash interpolated as a regex assertion, `<$h>`, `<%h>`, or a hash element of `<@a>`, matched nothing and said so to nobody, while the plain `$h` form threw X::Syntax::Reserved. Roast marks the hash assertion as reserved too.
... (6 more lines)
roast: ugexe++ created pull request #909:
Keep the ss:i:m test free of samemark		 17:58
Blin/main: 2271cf3cae | (Will Coleda)++ | resources/skips.json
Skip Text::Markdown::Discount tests
 18:02
18:05 camelia left, nine left, sjn left 18:06 sjn joined 18:07 nine joined 18:08 camelia joined 19:27 finanalyst joined
[Coke] Did a bunch of cleanup on Text-Markdown-Discount 19:44
Trying to get it ready so someone can cut a release as needed. 19:45
20:57 lizmat joined
[Coke] two failures off github.com/rakudo/rakudo/commit/b6...adfd3a0f6f 21:49
github.com/coke/raku-blin-release-...ailures.md
... weird, both of those say that it's not found. 22:05
22:06 finanalyst left
[Coke] re-redoing... 22:12
Looks like it's just the Needle::Compile issue remaining, sorry for the noise.
github.com/coke/raku-blin-release-...ailures.md updated 22:18
Geth roast: 85a87909a5 | (Nick Logan)++ (committed using GitHub Web editor) | S05-substitution/subst.t
Keep the ss:i:m test free of samemark (#909)

The test substitutes marked, mixed case input with ss:i:m and checks that the whitespace of the matched text survives in the replacement. The :i and :m adverbs are only there so the pattern matches that input at all, and the marks in the input only gave :m something to ignore.
... (13 more lines)		 22:34
linkable6 ROAST#909 [closed]: github.com/Raku/roast/pull/909 Keep the ss:i:m test free of samemark
roast: ugexe++ created pull request #910:
Keep the ss:i:m test free of samemark		 22:39
roast: ugexe++ created pull request #911:
Sync more recent roast adjustments to 6.d-errata		 22:41
rakudo: ugexe++ created pull request #6657:
Stop :s substitution copying marks into the replacement		 22:43
22:47 lizmat left 23:06 japhb left 23:15 japhb joined
Geth nqp: ugexe++ created pull request #873:
Give character class methods a declarative prefix		 23:44
roast: ugexe++ created pull request #912:
Test that character class methods take part in longest token matching		 23:46

Powered by Cro and the Raku® Programming Language.

Please report any issues / comments / feature requests as an issue on App::Raku::Log.

Thank you!