From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Date: Sat, 07 May 2011 20:24:24 +0000 Subject: Re: [PATCH] net/bonding: adjust codingstyle for bond_3ad files Message-Id: <1304799864.1738.10.camel@Joe-Laptop> List-Id: References: <20110507012717.GA15854@x61.tchesoft.com> <1304733100.11874.33.camel@Joe-Laptop> <20110507173141.GA4204@x61.tchesoft.com> <1304791360.1738.6.camel@Joe-Laptop> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: matt mooney Cc: David Miller , Jay Vosburgh , Andy Gospodarek , shemminger@vyatta.com, netdev@vger.kernel.org, Nicolas Kaiser , kernel-janitors@vger.kernel.org, aquini@linux.com On Sat, 2011-05-07 at 12:35 -0700, matt mooney wrote: > On Sat, May 7, 2011 at 11:02 AM, Joe Perches wrote: > > I'd write this not using a retval variable at all as: [] > But isn't the preferred style to have a single exit point? I don't think so. It's not pascal.