X Tutup
Skip to content

Fix 2 bugs with Language Standard Feature#1317

Open
MAJigsaw77 wants to merge 2 commits intoHaxeFoundation:masterfrom
FunkinCrew:bugfix/cxx-standard-fixes
Open

Fix 2 bugs with Language Standard Feature#1317
MAJigsaw77 wants to merge 2 commits intoHaxeFoundation:masterfrom
FunkinCrew:bugfix/cxx-standard-fixes

Conversation

@MAJigsaw77
Copy link
Contributor

  • Before it was implemented, we were able to set HXCPP_CPP17 inside a Build.xml but in the latest commit we cannot unless we define it inside the command, this behaviour is now fixed by moving the check inside createFileGroup, it allows the xml to be parsed first and then the set for the HXCPP_CPP17 would work just fine.
  • Setting the language standard on files element doesnt work as expected, if theres a section element inside the files element it, in that section the language standard that has been set my the parent will be forgotten, this behaviour is now fixed and allows sections to have the language standard work properly.

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

X Tutup