Serhiy Storchaka added the comment:
Here is updated patch. It doesn't include any additions to locale alias table
(including devanagari). Added several tests cases (many other test cases for
removed aliases already exist). optimize() is called only once, looks as second
run has no effect
Marc-Andre Lemburg added the comment:
On 26.12.2013 21:19, Serhiy Storchaka wrote:
Here is updated patch. It doesn't include any additions to locale alias table
(including devanagari). Added several tests cases (many other test cases for
removed aliases already exist). optimize() is
Marc-Andre Lemburg added the comment:
On 26.12.2013 22:43, Serhiy Storchaka wrote:
Serhiy Storchaka added the comment:
Could you add a test for the optimization function ?
I have no ideas. The optimization function is a part of the
makelocalealias.py
which ran manually very rarely
Marc-Andre Lemburg added the comment:
On 26.12.2013 23:19, Serhiy Storchaka wrote:
Serhiy Storchaka added the comment:
I probably wasn't clear: I meant some tests that show that the
alias definitions (on the left) from the X11 file are actually mapped to the
correct alias locales (on the
Roundup Robot added the comment:
New changeset 63bc68d7f449 by Serhiy Storchaka in branch 'default':
Issue #20046: Locale alias table no longer contains entities which can be
http://hg.python.org/cpython/rev/63bc68d7f449
--
nosy: +python-dev
___
Serhiy Storchaka added the comment:
Thank you Marc-Andre for your reviews.
--
assignee: - serhiy.storchaka
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Marc-Andre Lemburg added the comment:
I thought some more about this approach. I'm +1 on it. The locale
lookup is not time critical, so the table optimization makes sense.
Nice idea, Serhiy !
On 22.12.2013 00:38, Serhiy Storchaka wrote:
Serhiy Storchaka added the comment:
* the patch
New submission from Serhiy Storchaka:
Proposed patch removes over 400 entities from locale alias tables. They are
redundant because they can be calculated on fly.
Also it enables utf8 aliases. Now this adds not hundreds of redundant aliases,
but only 8 new locales:
+'be_bg.utf8':
Changes by Serhiy Storchaka storch...@gmail.com:
--
dependencies: +Fix makelocalealias.py for Python 3, Fixed support for Indian
locales
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20046
Serhiy Storchaka added the comment:
Example.
'br_fr':'br_FR.ISO8859-1',
-'br_fr.iso88591': 'br_FR.ISO8859-1',
-'br_fr.iso885914': 'br_FR.ISO8859-14',
-'br_fr.iso885915':
Marc-Andre Lemburg added the comment:
On 21.12.2013 20:33, Serhiy Storchaka wrote:
Serhiy Storchaka added the comment:
Example.
'br_fr':'br_FR.ISO8859-1',
-'br_fr.iso88591': 'br_FR.ISO8859-1',
-'br_fr.iso885914':
Serhiy Storchaka added the comment:
* the patch seems to include some unrelated changes, e.g. the
devanagari fixes and a few new mappings
May be. In any case I have added issue20027 as dependency. New mappings were
added when enable UTF-8 locales in makelocalealias.py (I can split this in
12 matches
Mail list logo