Skip to content

Conversation

@topolarity
Copy link
Member

Adding notes as I go through this code in detail.

@topolarity topolarity requested a review from vtjnash May 2, 2025 13:01
@topolarity topolarity changed the title gf.c: Add some comments to ml_matches gf.c: Add some comments to ml_matches (NFC) May 2, 2025
Mostly an exercise for me to go through this code in detail, but these
seemed like useful breadcrumbs for the next reader.
@vtjnash vtjnash force-pushed the ct/ml_matches-comments branch from bba0f22 to b1fcfdc Compare September 17, 2025 20:25
@vtjnash vtjnash added merge me PR is reviewed. Merge when all tests are passing backport 1.12 Change should be backported to release-1.12 labels Sep 17, 2025
@oscardssmith oscardssmith merged commit f2cc6b0 into JuliaLang:master Sep 24, 2025
7 of 9 checks passed
@oscardssmith oscardssmith removed the merge me PR is reviewed. Merge when all tests are passing label Sep 24, 2025
KristofferC pushed a commit that referenced this pull request Sep 24, 2025
Adding notes as I go through this code in detail.

Co-authored-by: Jameson Nash <[email protected]>
(cherry picked from commit f2cc6b0)
@KristofferC KristofferC mentioned this pull request Sep 24, 2025
24 tasks
KristofferC pushed a commit that referenced this pull request Sep 24, 2025
Adding notes as I go through this code in detail.

Co-authored-by: Jameson Nash <[email protected]>
(cherry picked from commit f2cc6b0)
@KristofferC KristofferC mentioned this pull request Sep 30, 2025
47 tasks
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Sep 30, 2025
xal-0 pushed a commit to xal-0/julia that referenced this pull request Sep 30, 2025
Adding notes as I go through this code in detail.

Co-authored-by: Jameson Nash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants