Skip to content

[exec.get.compl.sched] Rename completion-tag to completion-fn-tag and reorder text.#8818

Merged
tkoeppe merged 1 commit intocplusplus:mainfrom
tkoeppe:compltag
Mar 27, 2026
Merged

[exec.get.compl.sched] Rename completion-tag to completion-fn-tag and reorder text.#8818
tkoeppe merged 1 commit intocplusplus:mainfrom
tkoeppe:compltag

Conversation

@tkoeppe
Copy link
Copy Markdown
Contributor

@tkoeppe tkoeppe commented Mar 27, 2026

The renaming avoids a clash with the concept "completion-tag" defined in [exec.snd.expos]. The reordering makes it so that names are defined before they are referenced.

… reorder text.

The renaming avoids a clash with the concept "completion-tag" defined
in [exec.snd.expos]. The reordering makes it so that names are defined
before they are referenced.
@tkoeppe tkoeppe requested a review from dietmarkuehl March 27, 2026 12:00
@tkoeppe
Copy link
Copy Markdown
Contributor Author

tkoeppe commented Mar 27, 2026

image

@tkoeppe
Copy link
Copy Markdown
Contributor Author

tkoeppe commented Mar 27, 2026

@RobertLeahy: I'd welcome your take on this.

@jwakely
Copy link
Copy Markdown
Member

jwakely commented Mar 27, 2026

This does not conflict with the renaming in P4154

@RobertLeahy
Copy link
Copy Markdown
Contributor

@RobertLeahy: I'd welcome your take on this.

Looks good to me.

@tkoeppe tkoeppe merged commit 6fc1650 into cplusplus:main Mar 27, 2026
2 checks passed
@tkoeppe tkoeppe deleted the compltag branch March 27, 2026 21:53
@timsong-cpp
Copy link
Copy Markdown
Contributor

belated - this looks fine to me

@tkoeppe
Copy link
Copy Markdown
Contributor Author

tkoeppe commented Mar 28, 2026

Thanks, Tim, appreciated!

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.

4 participants