[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/3] xen: Control-flow cleanup

To: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
From: Bastian Blank <waldi@xxxxxxxxxx>
Date: Wed, 24 Mar 2010 23:51:44 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 24 Mar 2010 15:52:51 -0700
List-id: Xen developer discussion <xen-devel.lists.xensource.com>

On Wed, Mar 24, 2010 at 10:53:13AM -0700, Jeremy Fitzhardinge wrote:
> Thanks, I agree, but I tweaked it to use "goto out" to preserve the  
> single return, and fixed some lines which can now be unwrapped with the  
> shallower indent.

What is the reason for this single return except that it makes the code
less readable?

Bastian

-- 
Totally illogical, there was no chance.
                -- Spock, "The Galileo Seven", stardate 2822.3

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

References:
- [Xen-devel] [PATCH 1/3] xen: Control-flow cleanup
  - From: Bastian Blank
- Re: [Xen-devel] [PATCH 1/3] xen: Control-flow cleanup
  - From: Jeremy Fitzhardinge

Prev by Date: RE: [Xen-devel] RE: Hey James, how to analyze this BSoD?
Next by Date: Re: [Xen-devel] [PATCH 3/3] xen: Don't register GSI in pre-setup
Previous by thread: Re: [Xen-devel] [PATCH 1/3] xen: Control-flow cleanup
Next by thread: [Xen-devel] [PATCH 2/3] xen: Map physical interrupt before setting up GSI
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.