Skip to content

bpo-47243: Duplicate entry in 'Objects/unicodetype_db.h' #32376

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Sep 28, 2022
Merged

bpo-47243: Duplicate entry in 'Objects/unicodetype_db.h' #32376

merged 4 commits into from
Sep 28, 2022

Conversation

LiarPrincess
Copy link
Contributor

@LiarPrincess LiarPrincess commented Apr 6, 2022 •
edited by miss-islington
Loading

Fix for duplicate 1st entry in 'Objects/unicodetype_db.h':

/* a list of unique character type descriptors */
const _PyUnicode_TypeRecord _PyUnicode_TypeRecords[] = {
    {0, 0, 0, 0, 0, 0},
    {0, 0, 0, 0, 0, 0}, <--- HERE
    {0, 0, 0, 0, 0, 32},
    {0, 0, 0, 0, 0, 48},
    …

https://bugs.python.org/issue47243

https://bugs.python.org/issue47243

Automerge-Triggered-By: GH:isidentical

@the-knights-who-say-ni

This comment was marked as outdated.

Sign in to view
@LiarPrincess
Copy link
Contributor Author

CLA is signed, but there is this 'at least one business day before our records are updated'.

@ambv ambv removed the CLA signed label Apr 15, 2022
@ambv ambv closed this Apr 15, 2022
@ambv ambv reopened this Apr 15, 2022
@bedevere-bot
Copy link

Every change to Python requires a NEWS entry.

Please, add it using the blurb_it Web app or the blurb command-line tool.

@LiarPrincess
Copy link
Contributor Author

NEWS entry added.

Issue after migration #91399.

@cfbolz
Copy link
Contributor

cfbolz commented Sep 19, 2022

FWIW, this change looks correct to me (the original problem was a small miss-thought, I think).

@isidentical
Copy link
Member

@LiarPrincess can you regenerate the unicodedata (and then I think we can merge it in). Thanks @cfbolz for the review!

@miss-islington
Copy link
Contributor

Status check is done, and it's a success ✅.

@miss-islington miss-islington merged commit 0c1d7a0 into python:main Sep 28, 2022
@isidentical
Copy link
Member

Thanks for the PR @LiarPrincess!

@LiarPrincess
Copy link
Contributor Author

Thanks for review.
Updated with main & regenerated 'unicodetype_db.h' with Unicode 15.0.

@LiarPrincess LiarPrincess deleted the Duplicate-entry-in-Objects-unicodetype_db.h branch September 28, 2022 13:59
@LiarPrincess LiarPrincess mannequin mentioned this pull request Oct 30, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@isidentical isidentical isidentical approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@LiarPrincess @the-knights-who-say-ni @bedevere-bot @cfbolz @isidentical @miss-islington @ambv @kumaraditya303
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy