Update iptables.py

Spaces were not handled correctly when parsing arguments, both flags and log prefix can contain spaces.
use comprehension in aliases.list_aliases
use list comprehension in CkMinions._check_pcre_minions
modules.npm: docstring in middle of function -> regular comment
zpool_status: str.splitlines() already returns a list
rm unused `with_env` arg from cmdmod._run()
fix overly loose signature in freebsdservice.status to match other service.status-es
rm unused arg `packages` from states.pip.removed
rm unused args from salt.modules.daemontools.{start,stop}

still states.service-compatible
honor the `bits` argument to modules.tls.create_ca()
honor `path` argument to modules.s3.head()
honor service_url parameters in modules.s3
rm semicolon from Python in states.tomcat.war_deployed
states.stateconf: use conventional param name: kws -> kwargs
Revert to os.walk, the tests are failing with the new safe_walk
rm unused arg 'unique' from states.user._changes
Fix safe_walk test
Merge pull request #4773 from cvrebert/patch-1

modules.npm: docstring in middle of function -> regular comment
Merge pull request #4775 from cvrebert/patch-2

zpool_status: str.splitlines() already returns a list
Merge pull request #4776 from cvrebert/patch-3

rm unused `with_env` arg from cmdmod._run()