aboutsummaryrefslogtreecommitdiff
path: root/src/backend/utils/adt/formatting.c
diff options
context:
space:
mode:
authorPeter Eisentraut <peter_e@gmx.net>2011-12-27 21:19:09 +0200
committerPeter Eisentraut <peter_e@gmx.net>2011-12-27 21:19:09 +0200
commit037a82704ce644e2b1c3946345b54444caddb1a5 (patch)
treed89abc55ac119b86d69c01f819ab8fb22a1382c6 /src/backend/utils/adt/formatting.c
parentd383c23f6fbc4a79dae66483cf4f7051121008ad (diff)
downloadpostgresql-037a82704ce644e2b1c3946345b54444caddb1a5.tar.gz
postgresql-037a82704ce644e2b1c3946345b54444caddb1a5.zip
Standardize treatment of strcmp() return value
Always compare the return value to 0, don't use cute tricks like if (!strcmp(...)).
Diffstat (limited to 'src/backend/utils/adt/formatting.c')
-rw-r--r--src/backend/utils/adt/formatting.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/utils/adt/formatting.c b/src/backend/utils/adt/formatting.c
index be0966ca90e..96c89931b50 100644
--- a/src/backend/utils/adt/formatting.c
+++ b/src/backend/utils/adt/formatting.c
@@ -1012,7 +1012,7 @@ index_seq_search(char *str, const KeyWord *kw, const int *index)
do
{
- if (!strncmp(str, k->name, k->len))
+ if (strncmp(str, k->name, k->len) == 0)
return k;
k++;
if (!k->name)
@@ -1032,7 +1032,7 @@ suff_search(char *str, KeySuffix *suf, int type)
if (s->type != type)
continue;
- if (!strncmp(str, s->name, s->len))
+ if (strncmp(str, s->name, s->len) == 0)
return s;
}
return NULL;