Page MenuHomeFreeBSD

netmap: Fix compiler warnings in tools
ClosedPublic

Authored by markj on Feb 23 2023, 11:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 23, 3:29 PM
Unknown Object (File)
Sun, Apr 7, 2:28 PM
Unknown Object (File)
Sun, Apr 7, 6:44 AM
Unknown Object (File)
Sat, Apr 6, 8:19 PM
Unknown Object (File)
Mar 23 2024, 7:36 PM
Unknown Object (File)
Jan 10 2024, 9:00 PM
Unknown Object (File)
Dec 20 2023, 8:19 AM
Unknown Object (File)
Nov 30 2023, 12:04 PM
Subscribers

Details

Summary
  • Remove write-only variables, or hide them in cases where their use is conditional or commented out.
  • Check for errors from cmd_apply() in nmreplay.
  • Use ANSI C definitions.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tools/tools/netmap/lb.c
392

This one gives a compile error with clang 15.

vmaffione added inline comments.
tools/tools/netmap/pkt-gen.c
1524

why not dropping this?

1550–1551

why not dropping this? It looks like leftover code

This revision now requires changes to proceed.Feb 26 2023, 3:41 PM
tools/tools/netmap/pkt-gen.c
1550–1551

There are some commented-out prints that reference rx. Should I remove those as well?

Drop commented-out lines in pkt-gen.

Yeah, if not used since you are here please clean it up.

This revision is now accepted and ready to land.Feb 27 2023, 5:26 PM
This revision was automatically updated to reflect the committed changes.