Skip to content

make check syntax investigate more identifiers#770

Merged
rfindler merged 1 commit intomasterfrom
origin-of-origin
Sep 28, 2025
Merged

make check syntax investigate more identifiers#770
rfindler merged 1 commit intomasterfrom
origin-of-origin

Conversation

@rfindler
Copy link
Member

Specifically identifiers that it finds inside origin fields are treated as if they themselves were in the original program, so their origin fields (etc) are all checked.

This may produce some arrows that don't make sense, possibly. I'm not sure.

Specifically identifiers that it finds inside origin fields are
treated as if they themselves were in the original program, so their
orgin fields (etc) are all checked
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resyntax analyzed 2 files in this pull request and found no issues.

@mflatt
Copy link
Member

mflatt commented Sep 28, 2025

I've confirmed that this works for a Rhombus improvement so that an arrow goes from the Posn use to its binding in

#lang rhombus
class Posn(x, y)
#false :: Posn.of(Int, Int)

@rfindler
Copy link
Member Author

Yay! Thanks.

@rfindler rfindler merged commit 109f7c3 into master Sep 28, 2025
3 checks passed
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.

2 participants