PayloadParser refactor

Need to refactor a few more tests. Otherwise, this is done.
Following

```
if (!out.description.empty())
    out.description << "\n\n";

mdp::MapBytesRangeSet()
```

seems more readable to me and avoids "appending nothing"