]> granicus.if.org Git - postgresql/commitdiff
Minor perf tweak for _SPI_strdup(): if we're going to call strlen()
authorNeil Conway <neilc@samurai.com>
Sat, 12 Jan 2008 10:38:32 +0000 (10:38 +0000)
committerNeil Conway <neilc@samurai.com>
Sat, 12 Jan 2008 10:38:32 +0000 (10:38 +0000)
anyway, it is faster to memcpy() than to strcpy().

src/backend/utils/adt/xml.c

index f61c6669034008ec36dcb663c6da6b71b29c5112..295e826576a7c4b2b93182be3ed17cf7f2a8a960 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/backend/utils/adt/xml.c,v 1.64 2008/01/01 19:45:53 momjian Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/adt/xml.c,v 1.65 2008/01/12 10:38:32 neilc Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1821,9 +1821,10 @@ map_sql_value_to_xml_value(Datum value, Oid type)
 static char *
 _SPI_strdup(const char *s)
 {
-       char       *ret = SPI_palloc(strlen(s) + 1);
+       size_t          len = strlen(s) + 1;
+       char       *ret = SPI_palloc(len);
 
-       strcpy(ret, s);
+       memcpy(ret, s, len);
        return ret;
 }