Clean up rte_option
Status from this message: Applied, thanks
Get series with b4
b4 am [email protected]Versions newest first
Patches in v1 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | option: use bare option string as namelib/ lib/ | +6 | 0 replies | |
| 2/8 | option: do not use static iteratorlib/ | +6 | 0 replies | |
| 3/8 | option: add usage stringlib/ lib/ lib/ +2 more | +27 | 0 replies | |
| 4/8 | option: rename name fieldlib/ lib/ lib/ | +6 | 0 replies | |
| 5/8 | option: check against common option on registerlib/ | +14 | 0 replies | |
| 6/8 | option: fix log level of error in registerlib/ | +1 | 0 replies | |
| 7/8 | option: improve library documentationlib/ | +6 | 0 replies | |
| 8/8 | option: return an error when register faillib/ lib/ | +10 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH v1 0/8] Clean up rte_option
rte_option lib might benefit from a few improvements.
Overview:
- list registered options on /rte/app --help
- check duplicates within base EAL options
- rename some fields to be closer to standard struct option and just cleaner overall.
- fix some strange code (static iterator?)
- fix doc, improve API a little...
Gaetan Rivet (8):
option: use bare option string as name
option: do not use static iterator
option: add usage string
option: rename name field
option: check against common option on register
option: fix log level of error in register
option: improve library documentation
option: return an error when register fail
lib/librte_eal/common/eal_common_options.c | 1 + lib/librte_eal/common/eal_private.h | 7 +++ lib/librte_eal/common/include/rte_option.h | 17 +++++-- lib/librte_eal/common/rte_option.c | 57 +++++++++++++++++++--- lib/librte_telemetry/rte_telemetry.c | 3 +- 5 files changed, 72 insertions(+), 13 deletions(-)
Signature · 1 line
2.19.1