Skip to content

Add location info to MacroVar nodes#15947

Merged
straight-shoota merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:macro-var-locations
Jul 2, 2025
Merged

Add location info to MacroVar nodes#15947
straight-shoota merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:macro-var-locations

Conversation

@Blacksmoke16
Copy link
Member

This manifested itself as:

macro test(val)
  {% if val == 1 %}
    %a = {{val.id}}
  {% else %}
    %a = {{val.id}}
  {% end %}
end

test 1
test 2

having reported coverage of:

{
  "coverage": {
    "test.cr": {
      "2": 1,
      "3": 1,
      "4": 0,
      "5": 1
    }
  }
}

@straight-shoota straight-shoota added this to the 1.17.0 milestone Jul 2, 2025
@straight-shoota straight-shoota merged commit 7498b3c into crystal-lang:master Jul 2, 2025
40 checks passed
@Blacksmoke16 Blacksmoke16 deleted the macro-var-locations branch July 2, 2025 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants