Hi Nguyen. I just had a need for isprint() myself, and then I found your code here.
I had a look at the POSIX locale as describe here: http://sourceware.org/git/?p=glibc.git;a=blob;f=localedata/locales/POSIX Some remarks below. Am 14.10.2012 16:26, schrieb Nguyen Thai Ngoc Duy: > -- 8< -- > diff --git a/ctype.c b/ctype.c > index faeaf34..0bfebb4 100644 > --- a/ctype.c > +++ b/ctype.c > @@ -11,18 +11,21 @@ enum { > D = GIT_DIGIT, > G = GIT_GLOB_SPECIAL, /* *, ?, [, \\ */ > R = GIT_REGEX_SPECIAL, /* $, (, ), +, ., ^, {, | */ > - P = GIT_PATHSPEC_MAGIC /* other non-alnum, except for ] and } */ > + P = GIT_PATHSPEC_MAGIC, /* other non-alnum, except for ] and } */ > + X = GIT_CNTRL, > + U = GIT_PUNCT, > + Z = GIT_CNTRL | GIT_SPACE > }; > > const unsigned char sane_ctype[256] = { > - 0, 0, 0, 0, 0, 0, 0, 0, 0, S, S, 0, 0, S, 0, 0, /* 0.. 15 */ > - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, /* 16.. 31 */ > + X, X, X, X, X, X, X, X, X, Z, Z, X, X, Z, X, X, /* 0.. 15 */ > + X, X, X, X, X, X, X, X, X, X, X, X, X, X, X, X, /* 16.. 31 */ "Normal" isspace() also includes vertical tab (11) and form-feed (12) as white-space characters. Is there a reason, why they are not included here? > S, P, P, P, R, P, P, P, R, R, G, R, P, P, R, P, /* 32.. 47 */ > D, D, D, D, D, D, D, D, D, D, P, P, P, P, P, G, /* 48.. 63 */ > P, A, A, A, A, A, A, A, A, A, A, A, A, A, A, A, /* 64.. 79 */ > - A, A, A, A, A, A, A, A, A, A, A, G, G, 0, R, P, /* 80.. 95 */ > + A, A, A, A, A, A, A, A, A, A, A, G, G, U, R, P, /* 80.. 95 */ > P, A, A, A, A, A, A, A, A, A, A, A, A, A, A, A, /* 96..111 */ > - A, A, A, A, A, A, A, A, A, A, A, R, R, 0, P, 0, /* 112..127 */ > + A, A, A, A, A, A, A, A, A, A, A, R, R, U, P, X, /* 112..127 */ > /* Nothing in the 128.. range */ > }; > > diff --git a/git-compat-util.h b/git-compat-util.h > index f8b859c..db77f3e 100644 > --- a/git-compat-util.h > +++ b/git-compat-util.h [...] > @@ -527,6 +533,13 @@ extern const unsigned char sane_ctype[256]; > #define isupper(x) sane_iscase(x, 0) > #define is_glob_special(x) sane_istest(x,GIT_GLOB_SPECIAL) > #define is_regex_special(x) sane_istest(x,GIT_GLOB_SPECIAL | > GIT_REGEX_SPECIAL) > +#define iscntrl(x) (sane_istest(x,GIT_CNTRL)) > +#define ispunct(x) sane_istest(x, GIT_PUNCT | GIT_REGEX_SPECIAL | \ > + GIT_GLOB_SPECIAL | GIT_PATHSPEC_MAGIC) > +#define isxdigit(x) (hexval_table[x] != -1) > +#define isprint(x) (sane_istest(x, GIT_ALPHA | GIT_DIGIT | GIT_SPACE | \ > + GIT_PUNCT | GIT_REGEX_SPECIAL | GIT_GLOB_SPECIAL | \ > + GIT_PATHSPEC_MAGIC)) "Normal" isprint() only includes space (32) from the white-space characters. The other white-space characters are not considered printable. Do we want to stay close to the "original", or not? Regards Jan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
