diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2009-04-23 07:19:09 +0000 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2009-04-23 07:19:09 +0000 |
| commit | 283939a321bc492045d00c80146d52672cfc5378 (patch) | |
| tree | c88e5f4c1d3716077413c3614daf5fcc66ec0235 /src/backend/utils/adt/varlena.c | |
| parent | 8d4f2ecd41312e57422901952cbad234d293060b (diff) | |
| download | postgresql-283939a321bc492045d00c80146d52672cfc5378.tar.gz postgresql-283939a321bc492045d00c80146d52672cfc5378.zip | |
varstr_cmp and any comparison function that piggybacks on it can return
any negative or positive number, not just -1 or 1. Fix comment on
varstr_cmp and citext test case accordingly.
As pointed out by Zdenek Kotala, and buildfarm member gothic moth.
Diffstat (limited to 'src/backend/utils/adt/varlena.c')
| -rw-r--r-- | src/backend/utils/adt/varlena.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/backend/utils/adt/varlena.c b/src/backend/utils/adt/varlena.c index 5df6f05c337..052d0578ac7 100644 --- a/src/backend/utils/adt/varlena.c +++ b/src/backend/utils/adt/varlena.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/adt/varlena.c,v 1.169 2009/01/01 17:23:50 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/utils/adt/varlena.c,v 1.170 2009/04/23 07:19:09 heikki Exp $ * *------------------------------------------------------------------------- */ @@ -1138,7 +1138,8 @@ text_position_cleanup(TextPositionState *state) * Comparison function for text strings with given lengths. * Includes locale support, but must copy strings to temporary memory * to allow null-termination for inputs to strcoll(). - * Returns -1, 0 or 1 + * Returns an integer less than, equal to, or greater than zero, indicating + * whether arg1 is less than, equal to, or greater than arg2. */ int varstr_cmp(char *arg1, int len1, char *arg2, int len2) |