ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

block: Use a GString in bdrv_perm_names() 

This is a bit more efficient than having to allocate and free memory
for each new permission.

The default size (30) is enough for "consistent read, write, resize".

Signed-off-by: Alberto Garcia <berto@igalia.com>
Message-id: 20200110171518.22168-1-berto@igalia.com
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
This commit is contained in:
Alberto Garcia 2020-01-10 18:15:18 +01:00 committed by Max Reitz
parent 3a75a870ef
commit e2a7423a11
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
block.c
View File

@ -1998,18 +1998,19 @@ char *bdrv_perm_names(uint64_t perm)
{ 0, NULL } { 0, NULL }
}; };
char *result = g_strdup(""); GString *result = g_string_sized_new(30);
struct perm_name *p; struct perm_name *p;
for (p = permissions; p->name; p++) { for (p = permissions; p->name; p++) {
if (perm & p->perm) { if (perm & p->perm) {
char *old = result; if (result->len > 0) {
result = g_strdup_printf("%s%s%s", old, *old ? ", " : "", p->name); g_string_append(result, ", ");
g_free(old); }
g_string_append(result, p->name);
} }
} }
return result; return g_string_free(result, FALSE);
} }
/* /*