Fix a portability bug (ye olde not casting a <ctype.h> argument to

unsigned char).  Fortunately we still have buildfarm machines that
will flag this.  Seems to be new in CVS HEAD, so no backpatch.

diff --git a/src/backend/utils/adt/cash.c b/src/backend/utils/adt/cash.c
index f7110e477c4e45b0c73bd7edc20c542e92b18b30..900ba604ee8313f6166610b6f06db3a0786f35eb 100644 (file)
--- a/src/backend/utils/adt/cash.c
+++ b/src/backend/utils/adt/cash.c
@@ -240,8 +240,9 @@ cash_in(PG_FUNCTION_ARGS)
                }
        }
 
-       /* should only be trailing digits followed by whitespace or closing paren */
-       while (isdigit(*s)) s++;
+       /* should only be trailing digits followed by whitespace or right paren */
+       while (isdigit((unsigned char) *s))
+               s++;
        while (isspace((unsigned char) *s) || *s == ')')
                s++;