Refactor delete code

Makes logic more linear. Drops zpool28fix support. Restores r and R
flags, but not to be passed directly to zfs_cmd. Instead, we mimic the
functionality.

Also, $rm_snapshot_pattern was dropped. Currently, this is less
efficient, but it does make it more clear, and all deletion goes
through the same path now. Caching (perhaps through an eval array)
could help performance if needed.
Drop '-r' and zpool28fix support from RmZfsSnapshot
Replace sed call with built-in string parsing.