aboutsummaryrefslogtreecommitdiff
path: root/tests/check-qstring.c
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2018-07-27 08:22:04 +0200
committerMarkus Armbruster <armbru@redhat.com>2018-07-28 09:09:58 +0200
commitba891d68b4ff17faaea3d3a8bfd82af3eed0a134 (patch)
treea60591188270f2ec5ade8eef780b26da5ddf0efd /tests/check-qstring.c
parentb65ab77b3afadd7bb3051b341a5258ff7fb9d246 (diff)
downloadqemu-ba891d68b4ff17faaea3d3a8bfd82af3eed0a134.zip
qemu-ba891d68b4ff17faaea3d3a8bfd82af3eed0a134.tar.gz
qemu-ba891d68b4ff17faaea3d3a8bfd82af3eed0a134.tar.bz2
qstring: Move qstring_from_substr()'s @end one to the right
qstring_from_substr() takes the index of the substring's first and last character. qstring_from_substr(s, 0, SIZE_MAX) denotes an empty substring. Awkward. Shift the end index one to the right. This simplifies both qstring_from_substr() and its callers. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Message-Id: <20180727062204.10401-3-armbru@redhat.com>
Diffstat (limited to 'tests/check-qstring.c')
-rw-r--r--tests/check-qstring.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/check-qstring.c b/tests/check-qstring.c
index f11a7a8..2d07992 100644
--- a/tests/check-qstring.c
+++ b/tests/check-qstring.c
@@ -66,7 +66,7 @@ static void qstring_from_substr_test(void)
{
QString *qs;
- qs = qstring_from_substr("virtualization", 3, 9);
+ qs = qstring_from_substr("virtualization", 3, 10);
g_assert(qs != NULL);
g_assert(strcmp(qstring_get_str(qs), "tualiza") == 0);