- Patch #1400181, fix unicode string formatting to not use the locale.
This is how string objects work. u'%f' could use , instead of . for the decimal point. Now both strings and unicode always use periods. This is the code that would break: import locale locale.setlocale(locale.LC_NUMERIC, 'de_DE') u'%.1f' % 1.0 assert '1.0' == u'%.1f' % 1.0 I couldn't create a test case which fails, but this fixes the problem. Will backport.
This commit is contained in:
parent
ab92afd100
commit
fc76d633e8
|
@ -12,6 +12,10 @@ What's New in Python 2.5 alpha 1?
|
||||||
Core and builtins
|
Core and builtins
|
||||||
-----------------
|
-----------------
|
||||||
|
|
||||||
|
- Patch #1400181, fix unicode string formatting to not use the locale.
|
||||||
|
This is how string objects work. u'%f' could use , instead of .
|
||||||
|
for the decimal point. Now both strings and unicode always use periods.
|
||||||
|
|
||||||
- Bug #1244610, #1392915, fix build problem on OpenBSD 3.7 and 3.8.
|
- Bug #1244610, #1392915, fix build problem on OpenBSD 3.7 and 3.8.
|
||||||
configure would break checking curses.h.
|
configure would break checking curses.h.
|
||||||
|
|
||||||
|
|
|
@ -6579,26 +6579,31 @@ getnextarg(PyObject *args, int arglen, int *p_argidx)
|
||||||
#define F_ALT (1<<3)
|
#define F_ALT (1<<3)
|
||||||
#define F_ZERO (1<<4)
|
#define F_ZERO (1<<4)
|
||||||
|
|
||||||
static
|
static int
|
||||||
int usprintf(register Py_UNICODE *buffer, char *format, ...)
|
strtounicode(Py_UNICODE *buffer, const char *charbuffer)
|
||||||
{
|
{
|
||||||
register int i;
|
register long i;
|
||||||
int len;
|
long len = strlen(charbuffer);
|
||||||
va_list va;
|
|
||||||
char *charbuffer;
|
|
||||||
va_start(va, format);
|
|
||||||
|
|
||||||
/* First, format the string as char array, then expand to Py_UNICODE
|
|
||||||
array. */
|
|
||||||
charbuffer = (char *)buffer;
|
|
||||||
len = vsprintf(charbuffer, format, va);
|
|
||||||
for (i = len - 1; i >= 0; i--)
|
for (i = len - 1; i >= 0; i--)
|
||||||
buffer[i] = (Py_UNICODE) charbuffer[i];
|
buffer[i] = (Py_UNICODE) charbuffer[i];
|
||||||
|
|
||||||
va_end(va);
|
|
||||||
return len;
|
return len;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int
|
||||||
|
doubletounicode(Py_UNICODE *buffer, size_t len, const char *format, double x)
|
||||||
|
{
|
||||||
|
PyOS_ascii_formatd((char *)buffer, len, format, x);
|
||||||
|
return strtounicode(buffer, (char *)buffer);
|
||||||
|
}
|
||||||
|
|
||||||
|
static int
|
||||||
|
longtounicode(Py_UNICODE *buffer, size_t len, const char *format, long x)
|
||||||
|
{
|
||||||
|
PyOS_snprintf((char *)buffer, len, format, x);
|
||||||
|
return strtounicode(buffer, (char *)buffer);
|
||||||
|
}
|
||||||
|
|
||||||
/* XXX To save some code duplication, formatfloat/long/int could have been
|
/* XXX To save some code duplication, formatfloat/long/int could have been
|
||||||
shared with stringobject.c, converting from 8-bit to Unicode after the
|
shared with stringobject.c, converting from 8-bit to Unicode after the
|
||||||
formatting is done. */
|
formatting is done. */
|
||||||
|
@ -6648,7 +6653,7 @@ formatfloat(Py_UNICODE *buf,
|
||||||
PyOS_snprintf(fmt, sizeof(fmt), "%%%s.%d%c",
|
PyOS_snprintf(fmt, sizeof(fmt), "%%%s.%d%c",
|
||||||
(flags&F_ALT) ? "#" : "",
|
(flags&F_ALT) ? "#" : "",
|
||||||
prec, type);
|
prec, type);
|
||||||
return usprintf(buf, fmt, x);
|
return doubletounicode(buf, buflen, fmt, x);
|
||||||
}
|
}
|
||||||
|
|
||||||
static PyObject*
|
static PyObject*
|
||||||
|
@ -6740,9 +6745,9 @@ formatint(Py_UNICODE *buf,
|
||||||
prec, type);
|
prec, type);
|
||||||
}
|
}
|
||||||
if (sign[0])
|
if (sign[0])
|
||||||
return usprintf(buf, fmt, -x);
|
return longtounicode(buf, buflen, fmt, -x);
|
||||||
else
|
else
|
||||||
return usprintf(buf, fmt, x);
|
return longtounicode(buf, buflen, fmt, x);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int
|
static int
|
||||||
|
|
Loading…
Reference in New Issue