# HG changeset patch
# User Anton Shestakov <av6@dwimlabs.net>
# Date 1536768257 -28800
# Node ID ef3c9ecb80991118ae2be66d3707aa76d726c70f
# Parent  aabf436c11cb0b87f210bd00acde01bf6a5b309c
cmdrewrite: rename variable and move it closer to where it is used

Renaming to satisfy pep8-naming linter plugin, which complained on this
variable because all-uppercase names are reserved for constants, and constants
need to be module-level. This variable is not a good choice for moving up to
the module level because it's used only in one place.

Additionally, let's have it closer to the place it's actually used to
potentially avoid an extra call to _() function (however cheap it may be).

diff -r aabf436c11cb -r ef3c9ecb8099 hgext3rd/evolve/cmdrewrite.py
--- a/hgext3rd/evolve/cmdrewrite.py	Tue Sep 04 19:30:19 2018 +0800
+++ b/hgext3rd/evolve/cmdrewrite.py	Thu Sep 13 00:04:17 2018 +0800
@@ -186,7 +186,6 @@
         lockmod.release(lock, wlock)
 
 def _editandapply(ui, repo, pats, old, p1, fp, diffopts):
-    RETRYCHOICE = _('try to fix the patch (yn)?$$ &Yes $$ &No')
     newnode = None
     while newnode is None:
         fp.seek(0)
@@ -220,7 +219,8 @@
             defaultchoice = 0 # yes
             if not ui.interactive:
                 defaultchoice = 1 # no
-            if ui.promptchoice(RETRYCHOICE, default=defaultchoice):
+            retrychoice = _('try to fix the patch (yn)?$$ &Yes $$ &No')
+            if ui.promptchoice(retrychoice, default=defaultchoice):
                 raise error.Abort(_("Could not apply amended path"))
             else:
                 # consider a third choice where we restore the original patch