Make length() disregard trailing spaces in char(n) values, per discussion

some time ago and recent patch from Gavin Sherry.  Update documentation
to point out that trailing spaces are insignificant in char(n).

diff --git a/doc/src/sgml/datatype.sgml b/doc/src/sgml/datatype.sgml
index 7bbe676a9a12945b857e7160af5c21bbf8715896..bcfe16905883be1edf95c991fc291e28e20f54e1 100644 (file)
--- a/doc/src/sgml/datatype.sgml
+++ b/doc/src/sgml/datatype.sgml
@@ -1,5 +1,5 @@
 
 
  
@@ -901,6 +901,18 @@ CREATE TABLE tablename (
     management systems have it as well.
    
 
+   
+    Values of type character are physically padded
+    with spaces to the specified width n, and are
+    stored and displayed that way.  However, the padding spaces are
+    treated as semantically insignificant.  Trailing spaces are
+    disregarded when comparing two values of type character,
+    and they will be removed when converting a character value
+    to one of the other string types.  Note that trailing spaces
+    are semantically significant in
+    character varying and text values.
+   
+
    
     The storage requirement for data of these types is 4 bytes plus the
     actual string, and in case of character plus the
@@ -922,7 +934,11 @@ CREATE TABLE tablename (
     
      There are no performance differences between these three types,
      apart from the increased storage size when using the blank-padded
-     type.
+     type.  While character(n) has performance
+     advantages in some other database systems, it has no such advantages in
+     PostgreSQL.  In most situations
+     text or character varying should be used
+     instead.
     
    
 

diff --git a/src/backend/utils/adt/varchar.c b/src/backend/utils/adt/varchar.c
index 2c10ca1485a5b1ba9cb56bdc3661f3c30a1c6af2..6b26a91192cd2e69491373eae928d8856f1d480d 100644 (file)
--- a/src/backend/utils/adt/varchar.c
+++ b/src/backend/utils/adt/varchar.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/adt/varchar.c,v 1.103 2003/11/29 19:51:59 pgsql Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/adt/varchar.c,v 1.104 2004/02/01 06:27:48 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -510,14 +510,16 @@ Datum
 bpcharlen(PG_FUNCTION_ARGS)
 {
    BpChar     *arg = PG_GETARG_BPCHAR_P(0);
+   int         len;
 
-   /* optimization for single byte encoding */
-   if (pg_database_encoding_max_length() <= 1)
-       PG_RETURN_INT32(VARSIZE(arg) - VARHDRSZ);
+   /* get number of bytes, ignoring trailing spaces */
+   len = bcTruelen(arg);
+   
+   /* in multibyte encoding, convert to number of characters */
+   if (pg_database_encoding_max_length() != 1)
+       len = pg_mbstrlen_with_len(VARDATA(arg), len);
 
-   PG_RETURN_INT32(
-             pg_mbstrlen_with_len(VARDATA(arg), VARSIZE(arg) - VARHDRSZ)
-       );
+   PG_RETURN_INT32(len);
 }
 
 Datum