diff --git a/contrib/ltree/lquery_op.c b/contrib/ltree/lquery_op.c index d580339283..a6466f575f 100644 --- a/contrib/ltree/lquery_op.c +++ b/contrib/ltree/lquery_op.c @@ -25,17 +25,16 @@ static char * getlexeme(char *start, char *end, int *len) { char *ptr; - int charlen; - while (start < end && (charlen = pg_mblen(start)) == 1 && t_iseq(start, '_')) - start += charlen; + while (start < end && t_iseq(start, '_')) + start += pg_mblen(start); ptr = start; if (ptr >= end) return NULL; - while (ptr < end && !((charlen = pg_mblen(ptr)) == 1 && t_iseq(ptr, '_'))) - ptr += charlen; + while (ptr < end && !t_iseq(ptr, '_')) + ptr += pg_mblen(ptr); *len = ptr - start; return start; diff --git a/contrib/ltree/ltree.h b/contrib/ltree/ltree.h index 6939d7349a..2a80a02495 100644 --- a/contrib/ltree/ltree.h +++ b/contrib/ltree/ltree.h @@ -126,7 +126,7 @@ typedef struct #define LQUERY_HASNOT 0x01 -#define ISALNUM(x) ( t_isalnum(x) || ( pg_mblen(x) == 1 && t_iseq((x), '_') ) ) +#define ISALNUM(x) ( t_isalnum(x) || t_iseq(x, '_') ) /* full text query */ diff --git a/contrib/ltree/ltxtquery_io.c b/contrib/ltree/ltxtquery_io.c index 3eca5cb8ff..8ab0ce8e52 100644 --- a/contrib/ltree/ltxtquery_io.c +++ b/contrib/ltree/ltxtquery_io.c @@ -64,13 +64,13 @@ gettoken_query(QPRS_STATE *state, int32 *val, int32 *lenval, char **strval, uint switch (state->state) { case WAITOPERAND: - if (charlen == 1 && t_iseq(state->buf, '!')) + if (t_iseq(state->buf, '!')) { (state->buf)++; *val = (int32) '!'; return OPR; } - else if (charlen == 1 && t_iseq(state->buf, '(')) + else if (t_iseq(state->buf, '(')) { state->count++; (state->buf)++; @@ -97,11 +97,11 @@ gettoken_query(QPRS_STATE *state, int32 *val, int32 *lenval, char **strval, uint errmsg("modifiers syntax error"))); *lenval += charlen; } - else if (charlen == 1 && t_iseq(state->buf, '%')) + else if (t_iseq(state->buf, '%')) *flag |= LVAR_SUBLEXEME; - else if (charlen == 1 && t_iseq(state->buf, '@')) + else if (t_iseq(state->buf, '@')) *flag |= LVAR_INCASE; - else if (charlen == 1 && t_iseq(state->buf, '*')) + else if (t_iseq(state->buf, '*')) *flag |= LVAR_ANYEND; else { @@ -110,14 +110,14 @@ gettoken_query(QPRS_STATE *state, int32 *val, int32 *lenval, char **strval, uint } break; case WAITOPERATOR: - if (charlen == 1 && (t_iseq(state->buf, '&') || t_iseq(state->buf, '|'))) + if (t_iseq(state->buf, '&') || t_iseq(state->buf, '|')) { state->state = WAITOPERAND; *val = (int32) *(state->buf); (state->buf)++; return OPR; } - else if (charlen == 1 && t_iseq(state->buf, ')')) + else if (t_iseq(state->buf, ')')) { (state->buf)++; state->count--; @@ -125,7 +125,7 @@ gettoken_query(QPRS_STATE *state, int32 *val, int32 *lenval, char **strval, uint } else if (*(state->buf) == '\0') return (state->count) ? ERR : END; - else if (charlen == 1 && !t_iseq(state->buf, ' ')) + else if (!t_iseq(state->buf, ' ')) return ERR; break; default: