don't return iterate_memory_map's value

It returns void, so the current code is legitimate C++.  But it's better
to just not have the return in the first place.
make read_segment_filter_t return bool instead of int
give memory_map_iterator_t an enum return value

It should only ever return an enum, so make that explicit in the types.