|WARNING| cmdline: add cmdline_ prefix to dprintf

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: WARNING
https://dpdk.org/patch/169203

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-09-08

# DPDK Patch Review

## Summary
This patch renames the `dprintf` macro in the cmdline library to `cmdline_dprintf` to avoid namespace collision with the glibc function of the same name. The patch is straightforward and mostly mechanical.

---

## Errors

None found.

---

## Warnings

None found.

---

## Info

### Macro Parenthesization (Info)
In `cmdline_cirbuf.h`, the disabled debug macro could use stronger parenthesization:

**Current:**
```c
#define cmdline_dprintf(...) ((void)0)
```

**Suggested (slightly more defensive):**
```c
#define cmdline_dprintf(...) do { (void)0; } while (0)
```

This ensures the macro can be safely used in all contexts (e.g., as a single statement in an `if` without braces), though the current `((void)0)` form is also acceptable and commonly used.

---

## Overall Assessment

**PASS**

The patch correctly addresses a namespace collision issue. All uses of the `dprintf` macro are consistently renamed to `cmdline_dprintf`. The change is purely mechanical and does not introduce correctness issues.

- No resource leaks, use-after-free, or race conditions introduced
- No forbidden tokens or API violations
- Code style is consistent with existing code
- No documentation changes needed (internal debug macro)
- The `cmdline_` prefix follows DPDK naming conventions for library-specific symbols

The patch is ready for integration.