aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-08-29 09:39:04 +0200
committerMax Kellermann <max@duempel.org>2008-08-29 09:39:04 +0200
commita208d654b32a17ce60fbb2d48deb1d45b100cb06 (patch)
tree9efabae7d5a4358402287b1b9840ace067be8693
parent9352fc9e6670721f696dda5a15fa1d2419c2e6f8 (diff)
downloadmpd-a208d654b32a17ce60fbb2d48deb1d45b100cb06.tar.gz
mpd-a208d654b32a17ce60fbb2d48deb1d45b100cb06.tar.xz
mpd-a208d654b32a17ce60fbb2d48deb1d45b100cb06.zip
tag: try not to duplicate the input string
Try to detect if the string needs Latin1-UTF8 conversion, or whitespace cleanup. If not, we don't need to allocate temporary memory, leading to decreased heap fragmentation.
-rw-r--r--src/tag.c31
1 files changed, 18 insertions, 13 deletions
diff --git a/src/tag.c b/src/tag.c
index ff8b1dbd1..a27417554 100644
--- a/src/tag.c
+++ b/src/tag.c
@@ -347,8 +347,8 @@ int tag_equal(struct tag *tag1, struct tag *tag2)
return 1;
}
-static inline char *fix_utf8(char *str, size_t *length_r) {
- char *temp;
+static inline const char *fix_utf8(const char *str, size_t *length_r) {
+ const char *temp;
assert(str != NULL);
@@ -357,7 +357,6 @@ static inline char *fix_utf8(char *str, size_t *length_r) {
DEBUG("not valid utf8 in tag: %s\n",str);
temp = latin1StrToUtf8Dup(str);
- free(str);
*length_r = strlen(temp);
return temp;
}
@@ -366,22 +365,28 @@ static void appendToTagItems(struct tag *tag, enum tag_type type,
const char *value, size_t len)
{
unsigned int i = tag->numOfItems;
- char *duplicated = xmalloc(len + 1);
-
- memcpy(duplicated, value, len);
- duplicated[len] = '\0';
-
- duplicated = fix_utf8(duplicated, &len);
- stripReturnChar(duplicated);
+ const char *p;
+
+ p = fix_utf8(value, &len);
+ if (memchr(p, '\n', len) != NULL) {
+ char *duplicated = xmalloc(len + 1);
+ memcpy(duplicated, p, len);
+ duplicated[len] = '\0';
+ if (p != value)
+ xfree(p);
+
+ stripReturnChar(duplicated);
+ p = duplicated;
+ }
tag->numOfItems++;
tag->items = xrealloc(tag->items,
tag->numOfItems * sizeof(*tag->items));
- len = strlen(duplicated);
- tag->items[i] = tag_pool_get_item(type, duplicated, len);
+ tag->items[i] = tag_pool_get_item(type, p, len);
- free(duplicated);
+ if (p != value)
+ xfree(p);
}
void tag_add_item_n(struct tag *tag, enum tag_type itemType,