Skip to content

Prevent ambiguity in name lookup - #2287

Merged
vitaut merged 1 commit into
fmtlib:masterfrom
DanielaE:fix/prevent-ambiguity-in-name-lookup
May 14, 2021
Merged

vitaut merged 1 commit into
fmtlib:masterfrom
DanielaE:fix/prevent-ambiguity-in-name-lookup

Conversation

@DanielaE

@DanielaE DanielaE commented May 14, 2021

Copy link
Copy Markdown
Contributor

Unqualified calls to make_format_args may find the same name by ADL in <format> if this C++20 header happens to be directly or indirectly included in a translation unit. Do a qualified lookup instead.

Unqualified calls to 'make_format_args' may find the same name by ADL in <format> if this C++20 header happens to be directly or indirectly included in a translation unit. Do a qualified lookup instead.
@DanielaE

Copy link
Copy Markdown
Contributor Author

This addresses issue #2284

@DanielaE
DanielaE marked this pull request as draft May 14, 2021 11:19
@DanielaE
DanielaE marked this pull request as ready for review May 14, 2021 12:37
@vitaut
vitaut merged commit ea94d6d into fmtlib:master May 14, 2021
@vitaut

vitaut commented May 14, 2021

Copy link
Copy Markdown
Contributor

Thank you!

@DanielaE
DanielaE deleted the fix/prevent-ambiguity-in-name-lookup branch May 14, 2021 14:12
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