public inbox for lvm2-cvs@sourceware.org help / color / mirror / Atom feed
From: wysochanski@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2/test/api test.c Date: Tue, 28 Jul 2009 14:12:00 -0000 [thread overview] Message-ID: <20090728141230.18208.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: wysochanski@sourceware.org 2009-07-28 14:12:30 Modified files: test/api : test.c Log message: Update interactive unit test - fix silly vg_close error. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/test/api/test.c.diff?cvsroot=lvm2&r1=1.22&r2=1.23 --- LVM2/test/api/test.c 2009/07/28 13:49:28 1.22 +++ LVM2/test/api/test.c 2009/07/28 14:12:29 1.23 @@ -329,11 +329,11 @@ { vg_t *vg=NULL; - while((vg = dm_hash_lookup(_vgname_hash, vgname))) { + if ((vg = dm_hash_lookup(_vgname_hash, vgname))) { dm_hash_remove(_vgid_hash, lvm_vg_get_uuid(vg)); dm_hash_remove(_vgname_hash, lvm_vg_get_name(vg)); } - while(vg && (vg = dm_hash_lookup(_vgid_hash, vgname))) { + if (!vg && (vg = dm_hash_lookup(_vgid_hash, vgname))) { dm_hash_remove(_vgid_hash, lvm_vg_get_uuid(vg)); dm_hash_remove(_vgname_hash, lvm_vg_get_name(vg)); }
next reply other threads:[~2009-07-28 14:12 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-07-28 14:12 wysochanski [this message] -- strict thread matches above, loose matches on Subject: below -- 2011-03-01 23:29 zkabelac 2011-03-01 23:18 zkabelac 2010-11-25 14:35 mornfall 2010-11-17 20:13 mornfall 2010-11-17 19:17 mornfall 2010-10-25 14:09 wysochanski 2010-05-19 11:53 wysochanski 2010-02-24 18:16 wysochanski 2009-08-07 21:22 wysochanski 2009-07-28 13:49 wysochanski 2009-07-27 21:02 wysochanski 2009-07-27 17:45 wysochanski 2009-07-26 20:59 wysochanski 2009-07-26 20:58 wysochanski 2009-07-26 14:37 wysochanski 2009-07-26 13:08 wysochanski 2009-07-26 13:07 wysochanski 2009-07-26 2:35 wysochanski 2009-07-26 2:35 wysochanski 2009-07-24 12:51 wysochanski 2009-07-24 4:15 wysochanski 2009-07-23 23:40 wysochanski
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20090728141230.18208.qmail@sourceware.org \ --to=wysochanski@sourceware.org \ --cc=lvm-devel@redhat.com \ --cc=lvm2-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).