From: Martin Wilck <mwilck@xxxxxxxx> Hi Christophe, hi Ben, this series attempts to improve the robustness of the code by making the lookup tables used for pretty printing and wildcard hanling static const arrays. This requires getting rid of the variable "width" field in these arrays. I could have simply split of "width" into a separate global array variable, but as the field width are designed to change depending on the data being printed, it makes more sense to use local variables for it and pass them to the actual table-formatting code. A couple of minor cleanups are done along the way. As always, feedback welcome Martin Martin Wilck (11): libmultipath: make multipath_data etc. static libmultipath: move path_data etc. to print.c libmultipath: make pd_lookup() etc. return an index libmultipath: use ARRAY_SIZE for iterating over wildcard arrays libmultipath: drop padding code in _snprint_pathgroup() libmultipath: snprint_foreign_topology(): split out lockless variant multipathd: drop unnecessary path layout calls libmultipath: add a cleanup function for unsigned char * libmultipath: make sprint_path_marginal() static libmultipath: introduce width argument for pretty-printing functions libmultipath: change wildcard handler tables to static const libmultipath/foreign.c | 72 ++++-- libmultipath/foreign.h | 25 ++- libmultipath/libmultipath.version | 5 +- libmultipath/print.c | 360 +++++++++++++++++------------- libmultipath/print.h | 60 ++--- libmultipath/util.c | 5 + libmultipath/util.h | 1 + multipath/main.c | 7 +- multipathd/cli_handlers.c | 79 ++++--- 9 files changed, 360 insertions(+), 254 deletions(-) -- 2.33.1 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/dm-devel
