gettext docs improvements (take 1)

[Georg, 2010-11-26 11:47:48]

Looks good, see one inline comment. Don't hesitate to rewrap to 80 chars; if the patch is going to be applied to all maintained branches, merging is not a concern. http://codereview.appspot.com/3333042/diff/1/Doc/library/gettext.rst File Doc/library/gettext.rst (right): http://codereview.appspot.com/3333042/diff/1/Doc/library/gettext.rst#newcode171 Doc/library/gettext.rst:171: Return a :class:`*Translations` instance based on the *domain*, *localedir*, This probably meant what it said; all return values are instances of `Translation`.

[merwok, 2010-11-26 23:41:26]

Thanks for the review. I’ll commit as is next week if there are no further comments (especially from Barry). > Don't hesitate to rewrap to 80 chars; if the patch is > going to be applied to all maintained branches, merging is > not a concern. Yes, it’s intended for all three branches. The rationale for not rewrapping was not merging but avoiding noise in the diff itself. http://codereview.appspot.com/3333042/diff/1/Doc/library/gettext.rst File Doc/library/gettext.rst (right): http://codereview.appspot.com/3333042/diff/1/Doc/library/gettext.rst#newcode171 Doc/library/gettext.rst:171: Return a :class:`*Translations` instance based on the *domain*, *localedir*, There is no Translation or Translations class, but NullTranslations, GNUTranslations and user-defined classes. The asterisk is my attempt to convey that, but I’m not sure it’s clear enough.