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

P2498 Forward compatibility of text_encoding with additional encoding registries #1157

Closed
wg21bot opened this issue Dec 18, 2021 · 4 comments
Labels
B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e C++23 Targeted at C++23 IS Ship vehicle: IS LEWG Library Evolution size - small paper size estimate
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Dec 18, 2021

P2498R0 Forward compatibility of text_encoding with additional encoding registries (Peter Brett)

@wg21bot wg21bot added LEWG Library Evolution SG16 Text processing labels Dec 18, 2021
@wg21bot wg21bot added this to the 2021-telecon milestone Dec 18, 2021
@brycelelbach brycelelbach added C++23 Targeted at C++23 B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e IS Ship vehicle: IS size - small paper size estimate ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list labels Dec 22, 2021
@jensmaurer jensmaurer modified the milestones: 2021-telecon, 2022-telecon Jan 1, 2022
@cor3ntin cor3ntin removed LEWG Library Evolution ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list labels Jan 16, 2022
@cor3ntin
Copy link

@tahonermann @peter-b I'm removing the LEWG tag, please add it back if and when SG-16 forward this. Thanks

@wg21bot
Copy link
Collaborator Author

wg21bot commented Jan 22, 2022

P2498R1 Forward compatibility of text_encoding with additional encoding registries (Peter Brett)

@tahonermann
Copy link
Collaborator

SG16 reviewed P2498R1 during its 2022-02-09 telecon. The following polls were taken.

  • Poll 1A: It should be more explicit in the identifiers used by the text_encoding
    interface that the facility is tied to the IANA character sets database.

    • Attendance: 8

      SF F N A SA
      2 3 1 2 0
    • Weak consensus in favor.

  • Poll 1B: The text_encoding class design should be modified to facilitate potential
    future association with additional encoding registries without retaining a bias towards
    the IANA registry.

    • Attendance: 8 (one abstention)

      SF F N A SA
      1 3 2 0 1
    • No consensus.

    • SA: I think IANA is a reasonable default and others can be added; we shouldn't slow
      down progress.

  • Poll 1C: The text_encoding class should be renamed to iana_text_encoding.

    • Attendance: 8

      SF F N A SA
      1 2 2 3 0
    • No consensus.

  • Poll 1D: Address feedback on wording in P2498R1 "Forward compatibility of text_encoding
    with additional encoding registries", and forward the paper as revised to LEWG as a bug
    fix for P1885 "Naming text encodings to demystify them" with a recommended ship vehicle
    of C++23.

    • Attendance: 8 (one abstention)

      SF F N A SA
      1 4 0 2 0
    • Weak consensus in favor.

Removing the SG16 label and adding LEWG. @brycelelbach, please ensure this paper is promptly scheduled for review as it proposes a change to a new C++23 feature.

@tahonermann tahonermann added LEWG Library Evolution and removed SG16 Text processing labels Feb 13, 2022
@peter-b
Copy link
Collaborator

peter-b commented Feb 23, 2022

Based on recent feedback on P1885, I won't be progressing this proposal any further.

@peter-b peter-b closed this as completed Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e C++23 Targeted at C++23 IS Ship vehicle: IS LEWG Library Evolution size - small paper size estimate
Projects
None yet
Development

No branches or pull requests

6 participants