Hi, * Thomas Petazzoni <[email protected]> [2010-02-11 23:25:17]:
> class i18n_template_code_CODE(i18n): > def __init__(self, language, locale_path): > """Install the _() function for the chosen locale other > @@ -65,6 +68,9 @@ class i18n_template_code_CODE(i18n): > e.g. É and E are equals in French map index""" > return a == b > > + def isrtl(self): > + return False > + Side question: where's this class used? I don't see it anywhere in OCitySMap's code. I have other comments to the i18n.py file, but it's not relevant here in this review. The rest LGTM, although again I'd ask for some comments, especially when you have this kind of funky computations -- some explanations would help understanding (and troubleshooting if needed) the code. If you can't write the comments because you don't remember what the code does, it'll only prove I just made my point :-) - Maxime -- Maxime Petazzoni <http://www.bulix.org> ``One by one, the penguins took away my sanity.'' Linux kernel and software developer at MontaVista Software
signature.asc
Description: Digital signature
