Hi Thomas and David, This patchset is good enhance for argparse library, we hope it could upstream to DPDK 24.11. Please look at it.
Thanks On 2024/7/4 11:07, fengchengwen wrote: > Kindly ping. > > Best regards, > Chengwen Feng > > On 2024/3/18 19:18, Chengwen Feng wrote: >> I found a couple of issues when I revisited the argparse_autotest >> output, so got this patchset. >> >> Chengwen Feng (6): >> argparse: refine error message >> argparse: remove dead code >> argparse: replace flag enum with marco >> argparse: fix argument flags operate as uint32 type >> test/argparse: refine testcases >> argparse: fix doc don't display two hyphens >> >> --- >> v4: address Thomas's comment on 4/6 commit: >> - remove unused macros TEST_ARGPARSE_FLAG_HAS_ARG_BITMASK and >> TEST_ARGPARSE_FLAG_VAL_TYPE_BITMASK. >> v3: >> - address Thomas's comment on 4/6 comit. >> - add commit: fix doc don't display two hyphens. >> v2: address David Marchand's comment: >> - replace flag enum with marco. >> - replace flag's hardcode with macro in test_argparse.c. >> >> app/test/test_argparse.c | 65 +++++++++++--------- >> doc/guides/prog_guide/argparse_lib.rst | 47 +++++++------- >> lib/argparse/rte_argparse.c | 61 +++++++++--------- >> lib/argparse/rte_argparse.h | 85 ++++++++++++-------------- >> 4 files changed, 127 insertions(+), 131 deletions(-) >> > . >
