bookmark: remove useless line in `validdest`
This case is already covered by the `if old == new:` conditional at the start of
the function.
bookmark: issue a single call to `allsuccessors` per loop
Update to this code was minimalist when `allsuccessors` argument were changed
from a list to a set. As this code is getting my attention again I realised we
can drastically simplify this part of the code by issue a single call to
`allsuccessors`.
bookmark: simplify mutability check in `validdest`
There is no need to check phase by hand. We already have a ctx method for that.
bundle: add revset expression to show bundle contents (
issue3487)
bundle() revset expression returns all changes that are present
in the bundle file (no matter whether they are in the repo or not).
Bundle file should be specified via -R option.