[issue20046] Optimize locale aliases table

2013-12-26 Thread Serhiy Storchaka
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

[issue20046] Optimize locale aliases table

2013-12-26 Thread Marc-Andre Lemburg
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

[issue20046] Optimize locale aliases table

2013-12-26 Thread Marc-Andre Lemburg
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

[issue20046] Optimize locale aliases table

2013-12-26 Thread Marc-Andre Lemburg
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

[issue20046] Optimize locale aliases table

2013-12-26 Thread Roundup Robot
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 ___

[issue20046] Optimize locale aliases table

2013-12-26 Thread Serhiy Storchaka
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

[issue20046] Optimize locale aliases table

2013-12-25 Thread Marc-Andre Lemburg
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

[issue20046] Optimize locale aliases table

2013-12-21 Thread Serhiy Storchaka
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':

[issue20046] Optimize locale aliases table

2013-12-21 Thread Serhiy Storchaka
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

[issue20046] Optimize locale aliases table

2013-12-21 Thread Serhiy Storchaka
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':

[issue20046] Optimize locale aliases table

2013-12-21 Thread Marc-Andre Lemburg
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':

[issue20046] Optimize locale aliases table

2013-12-21 Thread Serhiy Storchaka
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