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.