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

nested replacement fields may omit arg_id #10

Closed
wants to merge 1 commit into from

Conversation

puetzk
Copy link

@puetzk puetzk commented May 13, 2020

syntax.html already has examples like fmt::format("{:.{}f}", 3.14, 1) using this,
and https://en.cppreference.com/w/cpp/utility/format/formatter#width_and_precision shows that this is the case for the C++20 std::format

The Format Specification Mini-Language grammar seems to be the only one not showing this; update it to match.

syntax.html already has examples like `fmt::format("{:.{}f}", 3.14, 1)` using this, 
and https://en.cppreference.com/w/cpp/utility/format/formatter#width_and_precision shows that this is the case for the C++20 std::format

The Format Specification Mini-Language grammar seems to be the only one not showing this; update it to match.
@vitaut
Copy link
Contributor

vitaut commented May 13, 2020

Thanks for the PR but this file is auto-generated and will be rewritten by the next update. Please submit a PR to change the source here: https://github.com/fmtlib/fmt/blob/master/doc/syntax.rst

@puetzk
Copy link
Author

puetzk commented May 13, 2020

Thanks. I looked for a .rst or other source file, but didn't look in the right repository...

@puetzk puetzk deleted the patch-1 branch May 13, 2020 23:22
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.

2 participants