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

[PATCH 0/5] Some cleanup of libxl JSON handling



Another batch which has been hanging around for a while and never
submitted.  Here the observation is most of the data passed to the JSON
libraries aren't modified.  Marking the arguments to the functions as
constant is correct and makes the API rather similar to the one used
for "SXP".  Once that is done some more merging is possible.

There seems to be a bit of a difference in how the interface for
libjson-c is formatted versus the libyajl variant.  I've left it similar
to how it appeared to be done, but that looks a bit odd.  If there is a
desire for linebreaks and indentation feel free.

This has been rebased for the new libjson-c library.  Only difference was
needing to add some duplicate hunks for the new library.

Elliott Mitchell (5):
  tools/libxl: Mark pointer args of many functions constant
  tools/xl: Mark libxl_domain_config * arg of printf_info_*() const
  tools/libxl: Tiny optimization of libxl__mac_is_default()
  tools/xl: Rename printf_info()/list_domains_details() to dump_by_...()
  tools/xl: Merge down debug/dry-run section of create_domain()

 tools/include/libxl_json.h        | 38 ++++++++++++++++---------------
 tools/libs/light/gentypes.py      | 12 +++++-----
 tools/libs/light/libxl_cpuid.c    |  7 +++---
 tools/libs/light/libxl_internal.c |  8 +++----
 tools/libs/light/libxl_internal.h | 18 +++++++--------
 tools/libs/light/libxl_json.c     | 33 ++++++++++++++-------------
 tools/libs/light/libxl_nocpuid.c  |  4 ++--
 tools/xl/xl.h                     | 13 +++++++----
 tools/xl/xl_info.c                | 33 +++++++++++++++------------
 tools/xl/xl_misc.c                |  2 +-
 tools/xl/xl_sxp.c                 |  6 ++---
 tools/xl/xl_vmcontrol.c           | 14 +-----------
 12 files changed, 95 insertions(+), 93 deletions(-)

-- 
2.47.3




 


Rackspace

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