Added conditional compilation for '()', which was an allowed sign code in a
previous version of PEP 3101. It's currently not compiled in, but I want to
leave it because it might be useful in the future and it makes
calc_number_widths() clearer. It justifies NumberFieldWidths.rsign
and .n__rsign.
diff --git a/Objects/stringlib/formatter.h b/Objects/stringlib/formatter.h
index 23a031d..5d06df4 100644
--- a/Objects/stringlib/formatter.h
+++ b/Objects/stringlib/formatter.h
@@ -13,6 +13,8 @@
be. These are the only non-static functions defined here.
*/
+#define ALLOW_PARENS_FOR_SIGN 0
+
/*
get_integer consumes 0 or more decimal digit characters from an
input string, updates *result with the corresponding positive
@@ -73,7 +75,8 @@
is_sign_element(STRINGLIB_CHAR c)
{
switch (c) {
- case ' ': case '+': case '-': case '(':
+ case ' ': case '+': case '-':
+ case '(':
return 1;
default:
return 0;
@@ -132,9 +135,11 @@
if (end-ptr >= 1 && is_sign_element(ptr[0])) {
format->sign = ptr[0];
ptr++;
+#if ALLOW_PARENS_FOR_SIGN
if (end-ptr >= 1 && ptr[0] == ')') {
ptr++;
}
+#endif
}
/* The special case for 0-padding (backwards compat) */
@@ -247,6 +252,7 @@
r->n_lsign = 1;
r->lsign = (actual_sign == '-' ? '-' : '+');
}
+#if ALLOW_PARENS_FOR_SIGN
else if (format->sign == '(') {
if (actual_sign == '-') {
r->n_lsign = 1;
@@ -255,6 +261,7 @@
r->rsign = ')';
}
}
+#endif
else if (format->sign == ' ') {
r->n_lsign = 1;
r->lsign = (actual_sign == '-' ? '-' : ' ');