[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v3 for-2.9 04/11] rbd: Clean up after the pr
From: |
Jeff Cody |
Subject: |
Re: [Qemu-devel] [PATCH RFC v3 for-2.9 04/11] rbd: Clean up after the previous commit |
Date: |
Mon, 27 Mar 2017 22:13:36 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, Mar 27, 2017 at 03:26:28PM +0200, Markus Armbruster wrote:
> This code in qemu_rbd_parse_filename()
>
> found_str = qemu_rbd_next_tok(p, '\0', &p);
> p = found_str;
>
> has no effect. Drop it, and simplify qemu_rbd_next_tok().
>
> Signed-off-by: Markus Armbruster <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Jeff Cody <address@hidden>
> ---
> block/rbd.c | 24 +++++++++---------------
> 1 file changed, 9 insertions(+), 15 deletions(-)
>
> diff --git a/block/rbd.c b/block/rbd.c
> index 0fea348..182a5a3 100644
> --- a/block/rbd.c
> +++ b/block/rbd.c
> @@ -104,19 +104,17 @@ static char *qemu_rbd_next_tok(char *src, char delim,
> char **p)
>
> *p = NULL;
>
> - if (delim != '\0') {
> - for (end = src; *end; ++end) {
> - if (*end == delim) {
> - break;
> - }
> - if (*end == '\\' && end[1] != '\0') {
> - end++;
> - }
> - }
> + for (end = src; *end; ++end) {
> if (*end == delim) {
> - *p = end + 1;
> - *end = '\0';
> + break;
> }
> + if (*end == '\\' && end[1] != '\0') {
> + end++;
> + }
> + }
> + if (*end == delim) {
> + *p = end + 1;
> + *end = '\0';
> }
> return src;
> }
> @@ -177,10 +175,6 @@ static void qemu_rbd_parse_filename(const char
> *filename, QDict *options,
> goto done;
> }
>
> - found_str = qemu_rbd_next_tok(p, '\0', &p);
> -
> - p = found_str;
> -
> /* The following are essentially all key/value pairs, and we treat
> * 'id' and 'conf' a bit special. Key/value pairs may be in any order.
> */
> while (p) {
> --
> 2.7.4
>
- Re: [Qemu-devel] [PATCH RFC v3 for-2.9 06/11] rbd: Clean up runtime_opts, fix -drive to reject filename, (continued)
- [Qemu-devel] [PATCH RFC v3 for-2.9 03/11] rbd: Don't limit length of parameter values, Markus Armbruster, 2017/03/27
- [Qemu-devel] [PATCH RFC v3 for-2.9 10/11] Revert "rbd: add support for getting password from QCryptoSecret object", Markus Armbruster, 2017/03/27
- [Qemu-devel] [PATCH RFC v3 for-2.9 04/11] rbd: Clean up after the previous commit, Markus Armbruster, 2017/03/27
- [Qemu-devel] [PATCH RFC v3 for-2.9 07/11] rbd: Clean up qemu_rbd_create()'s detour through QemuOpts, Markus Armbruster, 2017/03/27
- [Qemu-devel] [PATCH RFC v3 for-2.9 11/11] rbd: Fix bugs around -drive parameter "server", Markus Armbruster, 2017/03/27
- [Qemu-devel] [PATCH RFC v3 for-2.9 09/11] rbd: Revert -blockdev parameter password-secret, Markus Armbruster, 2017/03/27