[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] Consolidate printing of block driver option
Re: [Qemu-devel] [PATCH 1/3] Consolidate printing of block driver options
Mon, 06 Dec 2010 11:47:27 +0100
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:18.104.22.168) Gecko/20101103 Fedora/1.0-0.33.b2pre.fc14 Lightning/1.0b3pre Thunderbird/3.1.6
On 12/06/10 11:37, Stefan Hajnoczi wrote:
> On Mon, Dec 6, 2010 at 10:20 AM, Jes Sorensen <address@hidden> wrote:
>> On 12/06/10 10:32, Stefan Hajnoczi wrote:
>>> On Mon, Dec 6, 2010 at 8:17 AM, <address@hidden> wrote:
>>> Why goto out2 and not just return like the bs > 1 && out_baseimg check?
>> It is cleaner, I'd rather convert the bs_n test to do it too.
> "out2" tells me nothing and is just indirection for a return. At this
> point no resources have been acquired and it is simplest to bail out
A consistent out path is more likely to catch issues if the code is
modified later. I am not a big fan of the random mix of return vs goto
out that we spray over the code.... or having help() call exit() for