Fix some trivial clang/gcc warnings in bsnmptc.c
authorngie <ngie@FreeBSD.org>
Sat, 14 May 2016 04:13:28 +0000 (04:13 +0000)
committerngie <ngie@FreeBSD.org>
Sat, 14 May 2016 04:13:28 +0000 (04:13 +0000)
commit9a3d27c1e5ae4f1f4584801d043c7ea2cefc1191
tree76c55a63184a1887e1f2a8c55838110ee9893c27
parent905d53ad9a6a69f6e4b51bd5613b39a9747bba45
Fix some trivial clang/gcc warnings in bsnmptc.c

- By definition, `enum snmp_tc` can't be false (the implied starting sequence
  index for the enum is 0). Don't test for it being < 0.
- Staticize `struct snmp_text_conv` to mute a -Wmissing-variable-declarations
  warning from clang.
- Remove set but unused variable, ptr, in parse_bridge_id(..) and
  parse_bport_id(..) to mute warning from gcc 4.9+.
- Mark value and string unused in snmp_inetaddr2oct(..) and parse_inetaddr(..)
  as they're just stub functions.

MFC after: 1 week
Sponsored by: EMC / Isilon Storage Division
usr.sbin/bsnmpd/tools/libbsnmptools/bsnmptc.c