Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Map toString brackets #9260

Merged
merged 8 commits into from Mar 31, 2023
Merged

Map toString brackets #9260

merged 8 commits into from Mar 31, 2023

Conversation

RblSb
Copy link
Member

@RblSb RblSb commented Mar 21, 2020

Sorry for copy, i just recreated Haxe repo last week trying to trigger CI.
Closes #8030

@RblSb RblSb mentioned this pull request Mar 21, 2020
tests/runci/targets/Cpp.hx Outdated Show resolved Hide resolved
@RealyUniqueName
Copy link
Member

I'm not sure If we should treat this as a breaking change. I'd merge it for 4.1.

This reverts commit 3e1d9a0.
@RealyUniqueName RealyUniqueName added this to the Backlog milestone Mar 30, 2020
@RealyUniqueName
Copy link
Member

CI disaster is not related.

@Simn are you ok with this for 4.2?

@Simn Simn modified the milestones: Backlog, Release 4.3 Mar 24, 2023
@Simn Simn self-assigned this Mar 25, 2023
@Simn
Copy link
Member

Simn commented Mar 26, 2023

There are some failing tests on the C++ target here.

@RblSb
Copy link
Member Author

RblSb commented Mar 26, 2023

This depends on HaxeFoundation/hxcpp#892
Not sure if hxcpp changes are good tho

@Simn Simn modified the milestones: Release 4.3, Later Mar 30, 2023
@RblSb
Copy link
Member Author

RblSb commented Mar 31, 2023

Jvm fail is not related. I can revert latest commit now and this probably good to merge (after HaxeFoundation/hxcpp#892).
I'm not sure if all map cases are handled in hxcpp PR, with all map types and from dynamic cast cases, but this can be fixed with hxcpp releases if someone will find some toString() bug.

@Simn Simn merged commit 21d8f98 into HaxeFoundation:development Mar 31, 2023
47 of 49 checks passed
@RblSb RblSb deleted the map_tostring branch March 31, 2023 14:02
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.

[std] use map literal syntax in Map's toString()
4 participants