On Tue, Apr 26, 2016 at 04:52:20PM +0200, Roger Pau Monne wrote: > Or else clang complains with: > > implicit conversion from enumeration type 'yajl_gen_status' to different > enumeration type 'yajl_status' [-Werror,-Wenum-conversion] > > Signed-off-by: Roger Pau Monné <roger....@citrix.com>
Acked-by: Wei Liu <wei.l...@citrix.com> > --- > Cc: Ian Jackson <ian.jack...@eu.citrix.com> > Cc: Wei Liu <wei.l...@citrix.com> > --- > tools/libxl/libxl_json.c | 29 ++++++++++++++++++----------- > 1 file changed, 18 insertions(+), 11 deletions(-) > > diff --git a/tools/libxl/libxl_json.c b/tools/libxl/libxl_json.c > index 3b695dd..6bb0695 100644 > --- a/tools/libxl/libxl_json.c > +++ b/tools/libxl/libxl_json.c > @@ -617,42 +617,48 @@ yajl_status libxl__json_object_to_yajl_gen(libxl__gc > *gc, > int idx = 0; > yajl_status rc; > > +#define CONVERT_YAJL_GEN_TO_STATUS(gen) \ > + ((gen) == yajl_gen_status_ok ? yajl_status_ok : yajl_status_error); > + Extraneous ";" at the end. I can fix that up while committing. Wei. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel