Skip to content

Overload display fixes - #13012

Open
RblSb wants to merge 1 commit into
HaxeFoundation:developmentfrom
RblSb:overload-display
Open

Overload display fixes#13012
RblSb wants to merge 1 commit into
HaxeFoundation:developmentfrom
RblSb:overload-display

Conversation

@RblSb

@RblSb RblSb commented Aug 11, 2026

Copy link
Copy Markdown
Member
  • typerDisplay change inverts signature overloads list, currently it's displayed incorrectly in IDE
  • typer change fixes foo(S| completion by re-raising Unknown_ident error, so there is still Some/None reported fields in completion while typing names
  • callUnification change fixes foo(<completion>) case, when foo signature has second argument like (a:Option<Bool>, cb:() -> Void) and foo(<inserted null display value>) gets Not_enough_arguments for second cb argument. Now completion items for first overload is preserved, since extract_delayed_display should be called before loop candidates, that processes second Int overload, also gets Not_enough_arguments error and incorrectly overwrites ctx.g.delayed_display with Int overload)

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.

1 participant