# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 2018-02-05 11:40:49
# Node ID fd54846e1f8edaef01bf4ed4f2675d8ba06c5bca
# Parent  218b77c4c87a040e1f2995a8252ca3fae9dcf0ba

ui: write prompt text in ui.prompt(), not in ui._readline()

self.label() is replaced by label= option, which should make it clearer why
we can't pass the text to raw_input(prompt).

diff --git a/mercurial/ui.py b/mercurial/ui.py
--- a/mercurial/ui.py
+++ b/mercurial/ui.py
@@ -1246,7 +1246,7 @@ class ui(object):
 
         return i
 
-    def _readline(self, prompt=''):
+    def _readline(self):
         if self._isatty(self.fin):
             try:
                 # magically add command line editing support, where
@@ -1258,11 +1258,6 @@ class ui(object):
             except Exception:
                 pass
 
-        # call write() so output goes through subclassed implementation
-        # e.g. color extension on Windows
-        self.write(prompt, prompt=True)
-        self.flush()
-
         # prompt ' ' must exist; otherwise readline may delete entire line
         # - http://bugs.python.org/issue12833
         with self.timeblockedsection('stdio'):
@@ -1281,8 +1276,10 @@ class ui(object):
         if not self.interactive():
             self.write(msg, ' ', default or '', "\n")
             return default
+        self.write(msg, label='ui.prompt', prompt=True)
+        self.flush()
         try:
-            r = self._readline(self.label(msg, 'ui.prompt'))
+            r = self._readline()
             if not r:
                 r = default
             if self.configbool('ui', 'promptecho'):