On Sun, Jun 11, 2017 at 03:03:56PM +0100, Raymond wrote: > Transform the following functions (which never return anything other than 0, > and whose return value is never used) to void: > > * pfctl_clear_stats, pfctl_clear_interface_flags, pfctl_clear_rules, > pfctl_clear_src_nodes, pfctl_clear_states > * pfctl_kill_src_nodes, pfctl_net_kill_states, pfctl_label_kill_states, > pfctl_id_kill_states, pfctl_key_kill_states > > inside main: merge two identical if conditions next to each other into one. > > credit to > - awolk@ for the code reading > - mikeb for pointing out we can void all _clear_ functions > - ghostyy for pointing out all _kill_ functions can be voided >
OK awolk@, would like to have an OK from a pf person before committing.