[PATCH v4 0/6] refine argparse library
fengchengwen
fengchengwen at huawei.com
Fri Oct 18 03:09:17 CEST 2024
Hi Thomas and David,
This patchset was already acked by Stephen.
It missed in DPDK 24.07, hope merge in 24.11.
Thanks
On 2024/10/10 9:35, fengchengwen wrote:
> Hi Thomas and David,
>
> Kindly ping for merge.
>
> On 2024/10/10 3:33, Stephen Hemminger wrote:
>> On Mon, 18 Mar 2024 11:18:32 +0000
>> Chengwen Feng <fengchengwen at huawei.com> 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(-)
>>
>> The verb refine is not usually used for edits to documents or software
>> that probably is a translation issue.
>>
>> Series-acked-by: Stephen Hemminger <stephen at networkplumber.org>
>
> Thank Stephen
>
More information about the dev
mailing list