> nodes = None > - if opts['bookmark'] or revs: > + if opts.get('bookmark') or revs: There's one more `opts['bookmark']`. That's why `opts.get('bookmark')` was rewritten as `opts[...]`.