From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shriram Rajagopalan Subject: Re: [PATCH 3 of 3] remus: handle exceptions while installing/unstalling net buffer Date: Tue, 21 Jun 2011 14:43:39 -0400 Message-ID: References: <0af6803eeb9f278abd17.1307361112@athos.nss.cs.ubc.ca> <19968.53138.564453.916057@mariner.uk.xensource.com> Reply-To: rshriram@cs.ubc.ca Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1433120927==" Return-path: In-Reply-To: <19968.53138.564453.916057@mariner.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Jackson Cc: Brendan Cully , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org --===============1433120927== Content-Type: multipart/alternative; boundary=001636d34af957e29d04a63d3dbd --001636d34af957e29d04a63d3dbd Content-Type: text/plain; charset=ISO-8859-1 On Tue, Jun 21, 2011 at 1:06 PM, Ian Jackson wrote: > Brendan Cully writes ("Re: [Xen-devel] [PATCH 3 of 3] remus: handle > exceptions while installing/unstalling net buffer"): > > Bare except clauses are bad style. Please only handle the exact > > exceptions you are expecting. > > > > Nacked-by: Brendan Cully > > I agree. Shriram ? > > Ian. > > Yes, I ve been meaning to send out the revised version for a while. Will send it out today. shriram --001636d34af957e29d04a63d3dbd Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
On Tue, Jun 21, 2011 at 1:06 PM, Ian Jackson <Ian.Jackson= @eu.citrix.com> wrote:
Brendan Cully writes ("Re: [Xen-devel] [PATCH 3 of 3] remus: handle ex= ceptions while installing/unstalling net buffer"):
> Bare except clauses are bad style. Please only handl= e the exact
> exceptions you are expecting.
>
> Nacked-by: Brendan Cully <bren= dan@cs.ubc.ca>

I agree. =A0Shriram ?

Ian.

Yes, I ve been meaning to send out the revised ve= rsion for a while. Will
send it out today.

shriram
--001636d34af957e29d04a63d3dbd-- --===============1433120927== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel --===============1433120927==--