histedit: fix call to _getgoal() by adding a byteskwargs() wrapper

I also added some b-prefixes while I was here because I got confused
and it seems silly to not just add them since it clarifies the whole
change.

Differential Revision: https://phab.mercurial-scm.org/D5585

hgext/histedit.py

@@ -1604,7 +1604,8 b' def histedit(ui, repo, *freeargs, **opts'
     # kludge: _chistedit only works for starting an edit, not aborting
     # or continuing, so fall back to regular _texthistedit for those
     # operations.
-    if ui.interface('histedit') == 'curses' and  _getgoal(opts) == goalnew:
+    if ui.interface('histedit') == 'curses' and  _getgoal(
+            pycompat.byteskwargs(opts)) == goalnew:
         return _chistedit(ui, repo, *freeargs, **opts)
     return _texthistedit(ui, repo, *freeargs, **opts)
 
@@ -1621,11 +1622,11 b" goaleditplan = 'edit-plan'"
 goalnew = 'new'
 
 def _getgoal(opts):
-    if opts.get('continue'):
+    if opts.get(b'continue'):
         return goalcontinue
-    if opts.get('abort'):
+    if opts.get(b'abort'):
         return goalabort
-    if opts.get('edit_plan'):
+    if opts.get(b'edit_plan'):
         return goaleditplan
     return goalnew