Skip to content

zebra: remove unused zevpn_lookup() declaration#22513

Open
iurmanj6WIND wants to merge 1 commit into
FRRouting:masterfrom
iurmanj6WIND:remove-dead-code
Open

zebra: remove unused zevpn_lookup() declaration#22513
iurmanj6WIND wants to merge 1 commit into
FRRouting:masterfrom
iurmanj6WIND:remove-dead-code

Conversation

@iurmanj6WIND

Copy link
Copy Markdown
Contributor

Remove unused zevpn_lookup() declaration in zebra_vxlan_private.h.

Remove unused zevpn_lookup() declaration in zebra_vxlan_private.h.

Signed-off-by: Justin Iurman <justin.iurman@6wind.com>
@greptile-apps

greptile-apps Bot commented Jun 29, 2026

Copy link
Copy Markdown

Greptile Summary

This PR removes the orphaned extern struct zebra_evpn *zevpn_lookup(vni_t vni) declaration from zebra/zebra_vxlan_private.h. A grep across the entire repository confirms no definition or call site for this symbol exists anywhere in the tree, making the removal safe.

  • Removes one stale header declaration that had no corresponding definition or caller anywhere in the codebase.

Confidence Score: 5/5

Removing a single dead extern declaration with no definition and no callers anywhere in the tree is a no-risk cleanup.

The deleted line is a forward declaration that has no matching definition and is never referenced. The grep across the full repository found zero occurrences after the removal, so nothing breaks.

No files require special attention.

Important Files Changed

Filename Overview
zebra/zebra_vxlan_private.h Removes the unused zevpn_lookup() extern declaration; no definition or call site exists anywhere in the repo, so the removal is safe.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["zebra_vxlan_private.h\n(header)"] -- "previously declared" --> B["zevpn_lookup() extern\n❌ removed"]
    B -. "no definition exists" .-> C["(nowhere in codebase)"]
    B -. "no call sites exist" .-> C
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["zebra_vxlan_private.h\n(header)"] -- "previously declared" --> B["zevpn_lookup() extern\n❌ removed"]
    B -. "no definition exists" .-> C["(nowhere in codebase)"]
    B -. "no call sites exist" .-> C
Loading

Reviews (1): Last reviewed commit: "zebra: remove unused zevpn_lookup() decl..." | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant