[Python-checkins] r42954 - in python/trunk: Doc/lib/libunicodedata.tex Include/ucnhash.h Lib/encodings/idna.py Lib/stringprep.py Modules/unicodedata.c

["Martin v. Löwis"]

M.-A. Lemburg wrote:
> ich habe für diesen UCDB 4.1 Patch gar keinen SF-Eintrag gesehen.

Nein, ich hatte auch keinen gemacht.

> Ich verstehe zwar, weswegen Du UCBD 3.2 kompatibel bleiben willst,
> allerdings denke ich, daß der eingeschlagene Weg der falsche ist:
> es wäre wesentlich einfacher gewesen, das bisherige Modul unicodedata
> (zusammen mit den zugehörigen .c und .h Dateien) umzubenennen in
> z.B. unicodedata32 und dann unicodedata auf 4.1 umzustellen.

Einfacher vielleicht: warum aber falsch? Es sind so deutlich weniger
Daten.

> Mit Deinem Patch müssen jetzt Anwender von unicodedata
> stets zwei Versionen der kompletten Datenbank laden.

Das ist ja genau der Trick: sie müssen das nicht. Die Unterstützung
von Unicode 3.2 kostet nur 18kB.

> Ein neues Objekt für den Lookup wäre mit dem einfacheren
> Ansatz auch nicht notwendig gewesen, genausowenig wie
> die API-Änderung im C Objekt für ucnhash.

Richtig - das war allerdings insgesamt nicht so viel Aufwand.

Ciao,
Martin