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

Use libc++ #9558

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

Use libc++ #9558

wants to merge 25 commits into from

Conversation

grendello
Copy link
Contributor

@grendello grendello commented Nov 26, 2024

Always statically link libc++ into our runtime and enable
C++ exceptions, needed for future work.

Additionally, transition from traditional C printf style logging to
C++ std::format. The advantage is that the format string is more
expressive (doesn't require knowing the argument type, allows for positional
placeholders as well as extra formatting specifiers, if needed), safer (all
arguments are type-checked at build time) and faster (format strings are parsed and
verified at build time, runtime involves only string rendering)

@grendello grendello force-pushed the dev/grendel/use-libc++ branch 3 times, most recently from 0e11dc4 to cdf9a44 Compare December 3, 2024 12:40
@grendello grendello force-pushed the dev/grendel/use-libc++ branch from cdf9a44 to 5a4d188 Compare December 3, 2024 14:11
* main:
  [native] More C++ tweaks and changes (#9478)
  Add android-platform-support to .gitignore (#9590)
  [ci] Skip JDK install if near match is found in $(JI_JAVA_HOME) (#9585)
  Localized file check-in by OneLocBuild Task: Build definition ID 17928: Build ID 10653360 (#9587)
  Revert "Try dependabot max_length param (#9529)"
  Try dependabot max_length param (#9529)
  LEGO: Pull request (#9575)
  Bump to DevDiv/android-platform-support@52dd010a (#9553)
  Localized file check-in by OneLocBuild Task (#9574)
@grendello grendello marked this pull request as ready for review December 5, 2024 16:17
@grendello
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@grendello grendello mentioned this pull request Dec 5, 2024
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