route: silence Clang set but not used warning under WITHOUT_INET

[emaste]

sbin/route/route.c

896

Note I changed these from 0 to NULL in an earlier local commit, which I'll push as well

-                               getaddr(RTAX_IFA, *++argv, 0, nrflags);
+                               getaddr(RTAX_IFA, *++argv, NULL, nrflags);

1184

IMO the (void) cast is better than annotating the arg with __unused for #ifdef conditional cases, so that the warning will come back if the argument later becomes unused in the other cases.

[melifaro]

sbin/route/route.c

1157

As far as I understand, hpp usage is write-only (we zero *hpp in the beginning of INET block and write gethostname data in the end).
In newroute() (the only consumer of getaddr()), this value (stored as hp) is only used to propagate it to another getaddr() invocation for different RTA_ attribute.
As noted above, hpp is write-only, hence it's maybe better simply removing it alltogether from getaddr() and newroute() ?

[emaste]

sbin/route/route.c

1157

D33717

[melifaro]

(housekeeping) - is this change still needed after D33717 ?