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 std::format from c++20 #1015

Open
2 of 5 tasks
nekiro opened this issue Jan 4, 2025 · 1 comment
Open
2 of 5 tasks

Use std::format from c++20 #1015

nekiro opened this issue Jan 4, 2025 · 1 comment
Labels
enhancement New feature or request Status: Pending Test This PR or Issue requires more testing Type: Enhancement New feature or request

Comments

@nekiro
Copy link
Collaborator

nekiro commented Jan 4, 2025

Priority

Enhancement

Area

  • Data
  • Source
  • Docker
  • Other

What is missing?

I noticed the project still uses custom code to format prints (custom snprintf).
I believe it would be better to use what c++20 provides, so std::format directly.
It's easier to use, has auto casting via {} and probably way more features, not to mention it's in the standard and we can remove some custom code for free. Custom formatters for classes are also very nice.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@nekiro nekiro added the enhancement New feature or request label Jan 4, 2025
@github-actions github-actions bot added Status: Pending Test This PR or Issue requires more testing Type: Enhancement New feature or request labels Jan 4, 2025
@Zbizu
Copy link
Contributor

Zbizu commented Jan 19, 2025

looks like entire stdext namespace is no longer necessary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Status: Pending Test This PR or Issue requires more testing Type: Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants