Skip to content

Conversation

ethanfurman
Copy link
Member

No description provided.

@gnprice
Copy link
Contributor

gnprice commented Aug 27, 2019

Hmm, is this right? I think U+2167 really is ROMAN NUMERAL EIGHT.

Here's a table:

>>> import unicodedata
>>> for i in range(0x2160, 0x2170): print("U+{:X} {}  {}".format(i, chr(i), unicodedata.name(chr(i))))
... 
U+2160 Ⅰ  ROMAN NUMERAL ONE
U+2161 Ⅱ  ROMAN NUMERAL TWO
U+2162 Ⅲ  ROMAN NUMERAL THREE
U+2163 Ⅳ  ROMAN NUMERAL FOUR
U+2164 Ⅴ  ROMAN NUMERAL FIVE
U+2165 Ⅵ  ROMAN NUMERAL SIX
U+2166 Ⅶ  ROMAN NUMERAL SEVEN
U+2167 Ⅷ  ROMAN NUMERAL EIGHT
U+2168 Ⅸ  ROMAN NUMERAL NINE
U+2169 Ⅹ  ROMAN NUMERAL TEN
U+216A Ⅺ  ROMAN NUMERAL ELEVEN
U+216B Ⅻ  ROMAN NUMERAL TWELVE
U+216C Ⅼ  ROMAN NUMERAL FIFTY
U+216D Ⅽ  ROMAN NUMERAL ONE HUNDRED
U+216E Ⅾ  ROMAN NUMERAL FIVE HUNDRED
U+216F Ⅿ  ROMAN NUMERAL ONE THOUSAND

... I think I see the issue. We had the right name, but the wrong real live character 🙂 The character in the file is Ⅶ... aka U+2166 ROMAN NUMERAL SEVEN.

I'll send a quick PR to fix it.

@gnprice
Copy link
Contributor

gnprice commented Aug 27, 2019

Sent as #15541 .

lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants