Skip to content

Add location info to some MacroIf nodes#14885

Merged
straight-shoota merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:macro-if-locations
Aug 14, 2024
Merged

Add location info to some MacroIf nodes#14885
straight-shoota merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:macro-if-locations

Conversation

@Blacksmoke16
Copy link
Member

Resolves #14884.

Not sure if this covers every context, but at least is a start as it handles the most common.

@Blacksmoke16 Blacksmoke16 added kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:compiler:parser topic:lang:macros labels Aug 9, 2024
@straight-shoota straight-shoota added this to the 1.14.0 milestone Aug 13, 2024
@straight-shoota straight-shoota merged commit 4f31010 into crystal-lang:master Aug 14, 2024
@Blacksmoke16 Blacksmoke16 deleted the macro-if-locations branch September 30, 2024 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:compiler:parser topic:lang:macros

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MacroIf then/else missing location information

2 participants