Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2968)

#10665: Expand unicodedata module documentation

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 2 months ago by alexander.belopolsky
Modified:
2 years, 1 month ago
Reviewers:
ezio.melotti
CC:
mal_egenix.com, loewis, Georg, sasha, haypo, ezio.melotti, docs_python.org
Visibility:
Public.

Patch Set 1 #

Total comments: 16
Unified diffs Side-by-side diffs Delta from patch set Stats Patch
Doc/library/unicodedata.rst View 3 chunks +169 lines, -18 lines 16 comments Download

Messages

Total messages: 3
ezio.melotti
http://bugs.python.org/review/10665/diff/1830/3423 File Doc/library/unicodedata.rst (left): http://bugs.python.org/review/10665/diff/1830/3423#oldcode66 Doc/library/unicodedata.rst:66: You can also use the simplified syntax for the ...
2 years, 2 months ago #1
belopolsky_users.sourceforge.net
http://bugs.python.org/review/10665/diff/1830/3423 File Doc/library/unicodedata.rst (right): http://bugs.python.org/review/10665/diff/1830/3423#newcode64 Doc/library/unicodedata.rst:64: -1 On 2011/03/26 14:31:35, ezio.melotti wrote: > This example ...
2 years, 1 month ago #2
ezio.melotti
2 years, 1 month ago #3
http://bugs.python.org/review/10665/diff/1830/3423
File Doc/library/unicodedata.rst (right):

http://bugs.python.org/review/10665/diff/1830/3423#newcode64
Doc/library/unicodedata.rst:64: -1
The confusing parts are:
 1) the char *seems* to have a decimal value (9), but it doesn't;
 2) the default value is a decimal, but it's "wrong".
Maybe something like unicodedata.decimal('a', None) might be better.

http://bugs.python.org/review/10665/diff/1830/3423#newcode77
Doc/library/unicodedata.rst:77: -1
Using None or 'no digit' or something similar is OK.
FWIW I've seen and used '(undefined)' in these cases, I'm not sure if it's
something commonly used though.

http://bugs.python.org/review/10665/diff/1830/3423#newcode103
Doc/library/unicodedata.rst:103: | **General Categories**                       
                           |
This badly:
Markup is unsupported in LaTeX:
library/unicodedata:: column or row spanning cells are not yet implemented.
make: *** [build] Error 1

i.e. it doesn't even pass the `make latex` step.
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld cbc36f91f3f7