<div>Hi Vegard,</div><div><br></div><div>Thanks for applying my patch.</div><div><br></div><div>While I was merging the master branch and my local branch I noticed that your changes to "C-o" doesn't seem to be correct. "C-o" in Vim is supposed to escape to command mode for only ONE command and then comes back to insert mode. It is similar to 'r' command which enters inert mode to replace the current char and returns back to command mode immediately. In our case here, the existing 'viper-escape-to-vi' function escapes to command mode but never comes back to insert mode again, thus, I don't think it can be binded directly to 'C-o', and this is why I added the 'viper-escape-to-vi-state' function which switches back to insert mode after one command. Plus, depending on the setting of 'viper-ESC-moves-cursor-back', the cursor position may need to be adjusted by '(viper-forward-char 1)' (even though the 'vipre-ESC-moves-cursor-back' option itself doesn't work which is another bug).</div>
<div><br></div><div>Thanks,</div><div><br></div><div>York</div><div><br></div><div>On Sat, Apr 30, 2011 at 11:07 AM, Vegard �ye <<a href="mailto:vegard_oye@hotmail.com">vegard_oye@hotmail.com</a>> wrote:</div><div>On 2011-04-26 18:30 +0200, York Zhao wrote:</div>
<div><br></div><div>> Please find attached my patches for three added features. Note that</div><div>> I have included Christoph Lange's "g; and g," as well.</div><div><br></div><div>Applied in commits c07634c, 32d845f and 1cd84be, with some small</div>
<div>adjustments. It turns out that Viper has an unbound command for</div><div>temporarily exiting to vi state, `viper-escape-to-vi', so I bound</div><div>that to "C-o" in Insert state.</div><div><br></div><div>
--</div><div>Vegard</div><div><br></div><div>_______________________________________________</div><div>implementations-list mailing list</div><div><a href="mailto:implementations-list@lists.ourproject.org">implementations-list@lists.ourproject.org</a></div>
<div><a href="https://lists.ourproject.org/cgi-bin/mailman/listinfo/implementations-list">https://lists.ourproject.org/cgi-bin/mailman/listinfo/implementations-list</a></div><div><br></div>