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

Re: [Xen-devel] [PATCH v2 for-4.7 08/14] libxl: fix shutdown_reason type in list_domains

To: Roger Pau Monne <roger.pau@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
From: Doug Goldstein <cardoe@xxxxxxxxxx>
Date: Tue, 26 Apr 2016 10:19:54 -0500
Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>, Wei Liu <wei.liu2@xxxxxxxxxx>
Delivery-date: Tue, 26 Apr 2016 15:20:06 +0000
List-id: Xen developer discussion <xen-devel.lists.xen.org>

On 4/26/16 9:52 AM, Roger Pau Monne wrote:
> It should be an enum, not an unsigned.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

Reviewed-by: Doug Goldstein <cardoe@xxxxxxxxxx>

I had to drop the shutdown_reason >= 0 check a few lines below in my
version of this patch. But maybe its not necessary after all.

-- 
Doug Goldstein

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

References:
- [Xen-devel] [PATCH v2 for-4.7 00/14] Fixes for compiling with clang
  - From: Roger Pau Monne
- [Xen-devel] [PATCH v2 for-4.7 08/14] libxl: fix shutdown_reason type in list_domains
  - From: Roger Pau Monne

Prev by Date: Re: [Xen-devel] [PATCH v2 for-4.7 08/14] libxl: fix shutdown_reason type in list_domains
Next by Date: Re: [Xen-devel] [PATCH v9 12/27] xsplice: Implement support for applying/reverting/replacing patches.
Previous by thread: Re: [Xen-devel] [PATCH v2 for-4.7 08/14] libxl: fix shutdown_reason type in list_domains
Next by thread: [Xen-devel] [PATCH v2 for-4.7 10/14] libxl: add the printf-like attributes to a couple of functions
Index(es):
- Date
- Thread

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