|SUCCESS| net/iavf: remove redundant assignment in FDIR parser

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/169576

_AI Code Review_

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

# DPDK Patch Review

## Patch Summary
This patch removes a redundant assignment to `next_type` in the IPv4 case of the iavf FDIR parser. The variable is assigned but never subsequently read before the case block ends.

---

## Review Findings

### Correctness

**No correctness issues found.**

The patch correctly identifies and removes a dead store. The variable `next_type` is assigned the value `(item + 1)->type` but this value is never used in the remainder of the case block or after it. This is indeed redundant code.

I verified the context:
1. The assignment occurs within the `case RTE_FLOW_ITEM_TYPE_IPV4:` block
2. No code between this assignment and the end of the case reads `next_type`
3. Similar patterns in other case blocks (where `next_type` IS used) suggest this was copy-paste leftover

### Code Style

**No style violations found.**

The removal is a pure deletion with no new code introduced, so there are no style concerns.

### API and Documentation

**No API or documentation changes required.**

This is an internal driver implementation fix with no externally visible effects:
- No public API changes
- No ABI changes  
- No behavior changes (the dead store had no effect)
- Release notes are not required for internal dead code removal

### Process Compliance

**Info:** The patch is tagged with `Cc: stable at dpdk.org`, which is appropriate if this is considered a cleanup that could benefit stable branches. However, since this is purely cosmetic (dead code removal with no functional impact), the stable tag is optional rather than required. The maintainers may choose whether to backport this cleanup.

---

## Summary

**No errors or warnings.** This is a clean patch that correctly removes dead code identified by static analysis. The change improves code clarity by eliminating a misleading assignment that could confuse readers about the variable's purpose.

The patch is ready for merge as-is.