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 06dc4fe..ed37332 100644 (file)
--- a/src/backend/utils/adt/cash.c
+++ b/src/backend/utils/adt/cash.c
@@ -13,7 +13,7 @@
  * this version handles 64 bit numbers and so can hold values up to
  * $92,233,720,368,547,758.07.
  *
- * $PostgreSQL: pgsql/src/backend/utils/adt/cash.c,v 1.70 2007/02/27 23:48:07 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/adt/cash.c,v 1.71 2007/07/12 23:51:10 tgl Exp $
  */
 
 #include "postgres.h"
@@ -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++;