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

[format.string.std] Replace "Derived Extracted Property" with simply "property" #6224

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cpplearner
Copy link
Contributor

"Derived Extracted Property" refers to the fact that the data for the East_Asian_Width property are extracted into the file DerivedEastAsianWidth.txt. It is not really a terminology.

@jensmaurer
Copy link
Member

@cor3ntin , @tahonermann , any opinions on the above? The rationale sounds plausible.

@cor3ntin
Copy link
Contributor

Unicode says

When referencing a Unicode character property, it is customary to prepend the word “Unicode” to the name of the property, unless it is clear from context that the Unicode Standard
is the source of the specification.

I think the change make sense but maybe we should do something similar for http://eel.is/c++draft/lex.name#1

@tahonermann
Copy link
Contributor

I agree that the change is reasonable; the context is sufficient for readers to discover details about the property as desired. I also agree with Corentin's suggestion to replace similar wording elsewhere.

@jensmaurer
Copy link
Member

jensmaurer commented Apr 13, 2023

@cor3ntin , what do you want to change in http://eel.is/c++draft/lex.name#1 ? The "Derived Core Properties" mention is in an note, explaining XID_Start and XID_Continue. That seems reasonably explanatory to me.

@cor3ntin
Copy link
Contributor

Maybe something like
The properties XID_Start and XID_Continue are described by UAX #44 of the Unicode Standard.

Ie the fact that they are derived is not really adding any information

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.

None yet

4 participants