[Patches] [ python-Patches-633547 ] Plural forms support for gettext

noreply@sourceforge.net noreply@sourceforge.net
Tue, 05 Nov 2002 01:36:32 -0800


Patches item #633547, was opened at 2002-11-05 00:43
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=633547&group_id=5470

Category: Library (Lib)
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: Juan David Ibez Palomar (jdavid)
Assigned to: Nobody/Anonymous (nobody)
Summary: Plural forms support for gettext

Initial Comment:
Adds support for plural forms to the gettext module.
The test script has been rewritten to use unittest.

----------------------------------------------------------------------

>Comment By: Martin v. Lwis (loewis)
Date: 2002-11-05 10:36

Message:
Logged In: YES 
user_id=21627

The patch looks quite good, overall. However, I don't like
the use of eval to generate the plural form function: it is,
in general, a security issue to evaluate a string that you
read from some file.

I would prefer if it parses the string, or uses other
mechanisms to establish "safety": for example, if the only
identifier occurring in the string is 'n', then this would
be a good test. You might want to use
tokenize.generate_tokens for that.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=633547&group_id=5470