Skip to content

Conversation

Tsche
Copy link
Member

@Tsche Tsche commented Mar 20, 2025

Fixes #7751

@jensmaurer
Copy link
Member

@Tsche , if you want your real name to appear in the git commit history, please configure it in the github profile settings.

@jwakely, does this need an LWG issue?

@jwakely
Copy link
Member

jwakely commented Mar 20, 2025

I'll do some archaeology and see if it's been discussed or just forgotten (probably the latter)

@Tsche
Copy link
Member Author

Tsche commented Mar 20, 2025

@Tsche , if you want your real name to appear in the git commit history, please configure it in the github profile settings.

Thanks. I'd prefer committing it under this pseudonym instead if that's okay with you.

@jwakely jwakely self-assigned this May 14, 2025
@Tsche
Copy link
Member Author

Tsche commented Aug 25, 2025

@jwakely I've done some digging but could not find any discussion about this in old meeting minutes. P0088r3 and earlier versions thereof already miss the throws clause on that ctor.

It looks like this defect is an artifact of a change between P0088R0 and P0088R1. Note how in R0 neither one of the emplaced_type_t/emplaced_index_t (as they were then called) + initializer_list constructors have a throws clause. In R1 only one of them gained it.

This seems like a simple oversight to me. How would you like to proceed?

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.

[variant.ctor] missing Throws clause on the last ctor
3 participants