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)
Fri, May 15, 3:30 PM
Unknown Object (File)
Fri, May 15, 12:50 AM
Unknown Object (File)
Thu, May 14, 11:35 PM
Unknown Object (File)
Thu, May 14, 11:35 PM
Unknown Object (File)
Thu, May 14, 8:21 PM
Unknown Object (File)
Mon, May 11, 11:38 PM
Unknown Object (File)
Sun, May 10, 1:13 PM
Unknown Object (File)
Sat, May 9, 12:28 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.