|WARNING| [7/7] doc: add missing headers to doxy-api-index.md
dpdklab at iol.unh.edu
dpdklab at iol.unh.edu
Thu Jul 2 22:27:39 CEST 2026
Test-Label: ai-code-review
Test-Status: WARNING
https://dpdk.org/patch/166498
_AI Code Review_
=== Patch Review: series.patch (via Claude) ===
Review date: 2026-07-02
# DPDK Patch Review
## Patch 1/7: doc: detect ignored public headers
### Errors
**1. Variable name typo in lib/meson.build**
Line 224 uses undefined variable `doc_dir` instead of the correct variable from the loop (`d`):
```c
warning('public header directory @0@ is not listed in @1@'.format(doc_dir,
doc_build_path))
```
Should be:
```c
warning('public header directory @0@ is not listed in @1@'.format(d,
doc_build_path))
```
This will cause a build failure when triggered.
---
## Patch 2/7: doc: document rte_os.h
No issues found.
---
## Patch 3/7: doc: fix typos in rte_bus_pci.h
No issues found.
---
## Patch 4/7: doc: fix typos in rte_bus_vmbus.h
No issues found.
---
## Patch 5/7: doc: add missing globs to doxy-api.conf.in
No issues found.
---
## Patch 6/7: doc: add missing drivers to doxy-api.conf.in
No issues found.
---
## Patch 7/7: doc: add missing headers to doxy-api-index.md
No issues found.
---
## Summary
The series correctly addresses documentation gaps for public headers.
The meson build verification logic is a good addition to prevent future documentation drift.
**Patches 2-7**: Clean, no issues.
**Patch 1**: Contains one error (variable name typo) that must be fixed.
More information about the test-report
mailing list