From: Brad King Date: Mon, 02 Sep 2013 17:19:44 GMT Subject: Re: [PATCH v2 4/8] refs: factor delete_ref loose ref step into a helper Message-ID: <5224C8B0.20206@kitware.com> In-Reply-To: <52221A15.20804@alum.mit.edu> On 08/31/2013 12:30 PM, Michael Haggerty wrote: > Given that ret is only returned, you could restore the filename before > the if statement and replace the ret variable with an immediate return > statement: Good idea. Fixed in next revision. Thanks, -Brad