Chainsaw alert.  icmp_result is not defined in the docs and isn't used.
Some additional TODOs, notes on tests, and a few additional test cases.