|
[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
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |