From 6c1719a68a9066b9a6f9f403fb411b5a6a067bb9 Mon Sep 17 00:00:00 2001
From: Daniel Hahler <git@thequod.de>
Date: Tue, 12 May 2015 21:12:30 +0200
Subject: [PATCH 1/2] Fix rename behaviour with empty cword; honor
 b:changedtick

Ref: https://github.com/davidhalter/jedi-vim/issues/416
---
 jedi_vim.py | 39 ++++++++++++++++++++++++---------------
 1 file changed, 24 insertions(+), 15 deletions(-)

diff --git a/jedi_vim.py b/jedi_vim.py
index 3deb89c..0857a52 100644
--- a/jedi_vim.py
+++ b/jedi_vim.py
@@ -197,7 +197,7 @@ def completions():
 
 @_check_jedi_availability(show_error=True)
 @catch_and_print_exceptions
-def goto(mode = "goto", no_output=False):
+def goto(mode="goto", no_output=False):
     """
     :param str mode: "related_name", "definition", "assignment", "auto"
     :return: list of definitions/assignments
@@ -433,30 +433,39 @@ def cmdline_call_signatures(signatures):
 @catch_and_print_exceptions
 def rename():
     if not int(vim.eval('a:0')):
-        _rename_cursor = vim.current.window.cursor
-
-        vim_command('normal A ')  # otherwise startinsert doesn't work well
-        vim.current.window.cursor = _rename_cursor
-
         vim_command('augroup jedi_rename')
         vim_command('autocmd InsertLeave <buffer> call jedi#rename(1)')
         vim_command('augroup END')
 
         vim_command("let s:jedi_replace_orig = expand('<cword>')")
         vim_command('normal! diw')
-        vim_command(':startinsert')
+        vim_command("let s:jedi_changedtick = b:changedtick")
+        vim_command('startinsert')
+
     else:
-        # reset autocommand
+        # Remove autocommand.
         vim_command('autocmd! jedi_rename InsertLeave')
 
-        replace = vim_eval("expand('<cword>')")
-        vim_command('normal! u')  # undo new word
+        # Get replacement, if there is something on the cursor.
+        # This won't be the case when the user ends insert mode right away,
+        # and `<cword>` would pick up the nearest word instead.
+        if vim_eval('getline(".")[getpos(".")[2]-1]') != ' ':
+            replace = vim_eval("expand('<cword>')")
+        else:
+            replace = None
+
         cursor = vim.current.window.cursor
-        vim_command('normal! u')  # undo the space at the end
+
+        # Undo new word, but only if something was changed, which is not the
+        # case when ending insert mode right away.
+        if vim_eval('b:changedtick != s:jedi_changedtick') == '1':
+            vim_command('normal! u')  # Undo new word.
+        vim_command('normal! u')  # Undo diw.
+
         vim.current.window.cursor = cursor
 
-        return do_rename(replace)
-
+        if replace:
+            return do_rename(replace)
 
 def rename_visual():
     replace = vim.eval('input("Rename to: ")')
@@ -465,8 +474,8 @@ def rename_visual():
 
 
 def do_rename(replace, orig = None):
-    if replace is None:
-        echo_highlight('No rename possible, if no name is given.')
+    if not len(replace):
+        echo_highlight('No rename possible without name.')
         return
 
     if orig is None:

From 0dd1b9fd44868f6fea9ebf443c5a9dec0fcb8409 Mon Sep 17 00:00:00 2001
From: Daniel Hahler <git@thequod.de>
Date: Tue, 12 May 2015 21:59:49 +0200
Subject: [PATCH 2/2] Fix PEP8 style

---
 jedi_vim.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/jedi_vim.py b/jedi_vim.py
index 0857a52..266ee35 100644
--- a/jedi_vim.py
+++ b/jedi_vim.py
@@ -73,7 +73,9 @@ def vim_eval(string):
 
 
 def no_jedi_warning():
-    vim.command('echohl WarningMsg | echom "Please install Jedi if you want to use jedi-vim." | echohl None')
+    vim.command('echohl WarningMsg'
+                '| echom "Please install Jedi if you want to use jedi-vim."'
+                '| echohl None')
 
 
 def echo_highlight(msg):
@@ -467,13 +469,14 @@ def rename():
         if replace:
             return do_rename(replace)
 
+
 def rename_visual():
     replace = vim.eval('input("Rename to: ")')
     orig = vim.eval('getline(".")[(getpos("\'<")[2]-1):getpos("\'>")[2]]')
     do_rename(replace, orig)
 
 
-def do_rename(replace, orig = None):
+def do_rename(replace, orig=None):
     if not len(replace):
         echo_highlight('No rename possible without name.')
         return