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

Fix handling of more than 15 arguments in the fmt branch #437

Closed
vitaut opened this issue Dec 17, 2016 · 0 comments
Closed

Fix handling of more than 15 arguments in the fmt branch #437

vitaut opened this issue Dec 17, 2016 · 0 comments

Comments

@vitaut
Copy link
Contributor

vitaut commented Dec 17, 2016

This works in master but not in std branch:

fmt::format("{}{}{}{}{}{}{}{}{}{}{}{}{}{}{}{}",
                        0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 'a', 'b', 'c', 'd', 'e', 'f');
vitaut added a commit that referenced this issue Dec 23, 2016
@vitaut vitaut closed this as completed Dec 23, 2016
gsjaardema added a commit to gsjaardema/fmt that referenced this issue Jan 14, 2020
The intel-17 and intel-18 compilers seem to require that `u` be `const`:
```
/src/fmt/format.h(226): warning fmtlib#437: reference to local variable of enclosing function is not allowed
        char data[sizeof(u)];
```
If `u` is declared as `const auto u =1u` instead of just `auto u=1u`, the file compiles with no warnings.
vitaut pushed a commit that referenced this issue Jan 15, 2020
The intel-17 and intel-18 compilers seem to require that `u` be `const`:
```
/src/fmt/format.h(226): warning #437: reference to local variable of enclosing function is not allowed
        char data[sizeof(u)];
```
If `u` is declared as `const auto u =1u` instead of just `auto u=1u`, the file compiles with no warnings.
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

No branches or pull requests

1 participant