Remaping leader with :execute cannot run two commands at once

The name of the pictureThe name of the pictureThe name of the pictureClash Royale CLAN TAG#URR8PPP











up vote
2
down vote

favorite
1












I have following command: 



map <Leader>ntd :execute ":tabe! ". g:path ."l_todo.txt"<CR><ESC> :execute ":vsplit! ".g:path ."u_todo.txt"<CR>


Both tabe! and vsplit! commands works fine as separate remapings. I want to join those commands into one remaping.



How to make the above command to work?






key-bindings







share|improve this question



















  • 2




    You should try using only once execute call with a pipe | to chain the commands like nnoremap <leader>a :execute "tabnew | vsplit"<CR>. I didn't test it but I think your command should be map <Leader>ntd :execute "tabe! ". g:path ."l_todo.txt | vsplit! ".g:path ."u_todo.txt"<CR> (Also use nore and a mode in your map command i.e. nnoremap, this is a good practice and can save you a lot of debugging`)
    – statox♦
    8 hours ago















up vote
2
down vote

favorite
1












I have following command: 



map <Leader>ntd :execute ":tabe! ". g:path ."l_todo.txt"<CR><ESC> :execute ":vsplit! ".g:path ."u_todo.txt"<CR>


Both tabe! and vsplit! commands works fine as separate remapings. I want to join those commands into one remaping.



How to make the above command to work?






key-bindings







share|improve this question



















  • 2




    You should try using only once execute call with a pipe | to chain the commands like nnoremap <leader>a :execute "tabnew | vsplit"<CR>. I didn't test it but I think your command should be map <Leader>ntd :execute "tabe! ". g:path ."l_todo.txt | vsplit! ".g:path ."u_todo.txt"<CR> (Also use nore and a mode in your map command i.e. nnoremap, this is a good practice and can save you a lot of debugging`)
    – statox♦
    8 hours ago













up vote
2
down vote

favorite
1









up vote
2
down vote

favorite
1






1





I have following command: 



map <Leader>ntd :execute ":tabe! ". g:path ."l_todo.txt"<CR><ESC> :execute ":vsplit! ".g:path ."u_todo.txt"<CR>


Both tabe! and vsplit! commands works fine as separate remapings. I want to join those commands into one remaping.



How to make the above command to work?






key-bindings







share|improve this question















I have following command: 



map <Leader>ntd :execute ":tabe! ". g:path ."l_todo.txt"<CR><ESC> :execute ":vsplit! ".g:path ."u_todo.txt"<CR>


Both tabe! and vsplit! commands works fine as separate remapings. I want to join those commands into one remaping.



How to make the above command to work?






key-bindings




key-bindings






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited 8 hours ago









statox♦

24.4k660125




24.4k660125










asked 8 hours ago









lluke

434




434







  • 2




    You should try using only once execute call with a pipe | to chain the commands like nnoremap <leader>a :execute "tabnew | vsplit"<CR>. I didn't test it but I think your command should be map <Leader>ntd :execute "tabe! ". g:path ."l_todo.txt | vsplit! ".g:path ."u_todo.txt"<CR> (Also use nore and a mode in your map command i.e. nnoremap, this is a good practice and can save you a lot of debugging`)
    – statox♦
    8 hours ago













  • 2




    You should try using only once execute call with a pipe | to chain the commands like nnoremap <leader>a :execute "tabnew | vsplit"<CR>. I didn't test it but I think your command should be map <Leader>ntd :execute "tabe! ". g:path ."l_todo.txt | vsplit! ".g:path ."u_todo.txt"<CR> (Also use nore and a mode in your map command i.e. nnoremap, this is a good practice and can save you a lot of debugging`)
    – statox♦
    8 hours ago








2




2




You should try using only once execute call with a pipe | to chain the commands like nnoremap <leader>a :execute "tabnew | vsplit"<CR>. I didn't test it but I think your command should be map <Leader>ntd :execute "tabe! ". g:path ."l_todo.txt | vsplit! ".g:path ."u_todo.txt"<CR> (Also use nore and a mode in your map command i.e. nnoremap, this is a good practice and can save you a lot of debugging`)
– statox♦
8 hours ago





You should try using only once execute call with a pipe | to chain the commands like nnoremap <leader>a :execute "tabnew | vsplit"<CR>. I didn't test it but I think your command should be map <Leader>ntd :execute "tabe! ". g:path ."l_todo.txt | vsplit! ".g:path ."u_todo.txt"<CR> (Also use nore and a mode in your map command i.e. nnoremap, this is a good practice and can save you a lot of debugging`)
– statox♦
8 hours ago











1 Answer
1






active

oldest

votes

















up vote
4
down vote













You don't need to use two different execute() calls, you can simply call both commands in execute() like this:



nnoremap <Leader>ntd :execute "tabe! ". g:path . "l_todo.txt | vsplit! " . g:path . "u_todo.txt"<CR>


Note the escaped piped character (|) it is escaped so that vim understands that it is part of the command fed to execute.



Otherwise, when you source your vimrc, Vim would try to execute first nnoremap <Leader>ntd :execute ":tabe! ". g:path . "l_todo.txt followed by vsplit! " . g:path . "u_todo.txt"<CR> which is not what you want.



You could also improve the readability of your code like this:



let firstCommand="tabe! ". getcwd() . "/l_todo.txt"
let secondCommand="vsplit! " . getcwd() . "/u_todo.txt"
nnoremap <Leader>ntd :execute firstCommand . "|" . secondCommand<CR>


Here we separate the different part of the string give to execute() to see the two actions executed.



We also use :h getcwd() to get the current working directory so that we don't need a global variable to get the path.



Several other points about your code:



  • You don't need the : in the commands you feed to execute()

  • Always use the non recurcive version of map (i.e. noremap) unless you know what you're doing

  • Always specify a mode to map (see :h map-modes)





share|improve this answer


















  • 1




    With your refactoring, I believe changing g:path mid-session wont be reflected in the mapping. Id make first and second functions instead.
    – D. Ben Knoble
    4 hours ago






  • 1




    @D.BenKnoble You're perfectly right, I guess one could also use :h getcwd() instead of g:path depending on what the variable is meant to contain.
    – statox♦
    4 hours ago










Your Answer







StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "599"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);

else
createEditor();

);

function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
convertImagesToLinks: false,
noModals: false,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);



);













 

draft saved


draft discarded


















StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fvi.stackexchange.com%2fquestions%2f17477%2fremaping-leader-with-execute-cannot-run-two-commands-at-once%23new-answer', 'question_page');

);

Post as a guest

















1 Answer
1






active

oldest

votes








1 Answer
1






active

oldest

votes









active

oldest

votes






active

oldest

votes








up vote
4
down vote













You don't need to use two different execute() calls, you can simply call both commands in execute() like this:



nnoremap <Leader>ntd :execute "tabe! ". g:path . "l_todo.txt | vsplit! " . g:path . "u_todo.txt"<CR>


Note the escaped piped character (|) it is escaped so that vim understands that it is part of the command fed to execute.



Otherwise, when you source your vimrc, Vim would try to execute first nnoremap <Leader>ntd :execute ":tabe! ". g:path . "l_todo.txt followed by vsplit! " . g:path . "u_todo.txt"<CR> which is not what you want.



You could also improve the readability of your code like this:



let firstCommand="tabe! ". getcwd() . "/l_todo.txt"
let secondCommand="vsplit! " . getcwd() . "/u_todo.txt"
nnoremap <Leader>ntd :execute firstCommand . "|" . secondCommand<CR>


Here we separate the different part of the string give to execute() to see the two actions executed.



We also use :h getcwd() to get the current working directory so that we don't need a global variable to get the path.



Several other points about your code:



  • You don't need the : in the commands you feed to execute()

  • Always use the non recurcive version of map (i.e. noremap) unless you know what you're doing

  • Always specify a mode to map (see :h map-modes)





share|improve this answer


















  • 1




    With your refactoring, I believe changing g:path mid-session wont be reflected in the mapping. Id make first and second functions instead.
    – D. Ben Knoble
    4 hours ago






  • 1




    @D.BenKnoble You're perfectly right, I guess one could also use :h getcwd() instead of g:path depending on what the variable is meant to contain.
    – statox♦
    4 hours ago














up vote
4
down vote













You don't need to use two different execute() calls, you can simply call both commands in execute() like this:



nnoremap <Leader>ntd :execute "tabe! ". g:path . "l_todo.txt | vsplit! " . g:path . "u_todo.txt"<CR>


Note the escaped piped character (|) it is escaped so that vim understands that it is part of the command fed to execute.



Otherwise, when you source your vimrc, Vim would try to execute first nnoremap <Leader>ntd :execute ":tabe! ". g:path . "l_todo.txt followed by vsplit! " . g:path . "u_todo.txt"<CR> which is not what you want.



You could also improve the readability of your code like this:



let firstCommand="tabe! ". getcwd() . "/l_todo.txt"
let secondCommand="vsplit! " . getcwd() . "/u_todo.txt"
nnoremap <Leader>ntd :execute firstCommand . "|" . secondCommand<CR>


Here we separate the different part of the string give to execute() to see the two actions executed.



We also use :h getcwd() to get the current working directory so that we don't need a global variable to get the path.



Several other points about your code:



  • You don't need the : in the commands you feed to execute()

  • Always use the non recurcive version of map (i.e. noremap) unless you know what you're doing

  • Always specify a mode to map (see :h map-modes)





share|improve this answer


















  • 1




    With your refactoring, I believe changing g:path mid-session wont be reflected in the mapping. Id make first and second functions instead.
    – D. Ben Knoble
    4 hours ago






  • 1




    @D.BenKnoble You're perfectly right, I guess one could also use :h getcwd() instead of g:path depending on what the variable is meant to contain.
    – statox♦
    4 hours ago












up vote
4
down vote










up vote
4
down vote









You don't need to use two different execute() calls, you can simply call both commands in execute() like this:



nnoremap <Leader>ntd :execute "tabe! ". g:path . "l_todo.txt | vsplit! " . g:path . "u_todo.txt"<CR>


Note the escaped piped character (|) it is escaped so that vim understands that it is part of the command fed to execute.



Otherwise, when you source your vimrc, Vim would try to execute first nnoremap <Leader>ntd :execute ":tabe! ". g:path . "l_todo.txt followed by vsplit! " . g:path . "u_todo.txt"<CR> which is not what you want.



You could also improve the readability of your code like this:



let firstCommand="tabe! ". getcwd() . "/l_todo.txt"
let secondCommand="vsplit! " . getcwd() . "/u_todo.txt"
nnoremap <Leader>ntd :execute firstCommand . "|" . secondCommand<CR>


Here we separate the different part of the string give to execute() to see the two actions executed.



We also use :h getcwd() to get the current working directory so that we don't need a global variable to get the path.



Several other points about your code:



  • You don't need the : in the commands you feed to execute()

  • Always use the non recurcive version of map (i.e. noremap) unless you know what you're doing

  • Always specify a mode to map (see :h map-modes)





share|improve this answer














You don't need to use two different execute() calls, you can simply call both commands in execute() like this:



nnoremap <Leader>ntd :execute "tabe! ". g:path . "l_todo.txt | vsplit! " . g:path . "u_todo.txt"<CR>


Note the escaped piped character (|) it is escaped so that vim understands that it is part of the command fed to execute.



Otherwise, when you source your vimrc, Vim would try to execute first nnoremap <Leader>ntd :execute ":tabe! ". g:path . "l_todo.txt followed by vsplit! " . g:path . "u_todo.txt"<CR> which is not what you want.



You could also improve the readability of your code like this:



let firstCommand="tabe! ". getcwd() . "/l_todo.txt"
let secondCommand="vsplit! " . getcwd() . "/u_todo.txt"
nnoremap <Leader>ntd :execute firstCommand . "|" . secondCommand<CR>


Here we separate the different part of the string give to execute() to see the two actions executed.



We also use :h getcwd() to get the current working directory so that we don't need a global variable to get the path.



Several other points about your code:



  • You don't need the : in the commands you feed to execute()

  • Always use the non recurcive version of map (i.e. noremap) unless you know what you're doing

  • Always specify a mode to map (see :h map-modes)






share|improve this answer














share|improve this answer



share|improve this answer








edited 3 hours ago

























answered 5 hours ago









statox♦

24.4k660125




24.4k660125







  • 1




    With your refactoring, I believe changing g:path mid-session wont be reflected in the mapping. Id make first and second functions instead.
    – D. Ben Knoble
    4 hours ago






  • 1




    @D.BenKnoble You're perfectly right, I guess one could also use :h getcwd() instead of g:path depending on what the variable is meant to contain.
    – statox♦
    4 hours ago












  • 1




    With your refactoring, I believe changing g:path mid-session wont be reflected in the mapping. Id make first and second functions instead.
    – D. Ben Knoble
    4 hours ago






  • 1




    @D.BenKnoble You're perfectly right, I guess one could also use :h getcwd() instead of g:path depending on what the variable is meant to contain.
    – statox♦
    4 hours ago







1




1




With your refactoring, I believe changing g:path mid-session wont be reflected in the mapping. Id make first and second functions instead.
– D. Ben Knoble
4 hours ago




With your refactoring, I believe changing g:path mid-session wont be reflected in the mapping. Id make first and second functions instead.
– D. Ben Knoble
4 hours ago




1




1




@D.BenKnoble You're perfectly right, I guess one could also use :h getcwd() instead of g:path depending on what the variable is meant to contain.
– statox♦
4 hours ago




@D.BenKnoble You're perfectly right, I guess one could also use :h getcwd() instead of g:path depending on what the variable is meant to contain.
– statox♦
4 hours ago

















 

draft saved


draft discarded















































 


draft saved


draft discarded














StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fvi.stackexchange.com%2fquestions%2f17477%2fremaping-leader-with-execute-cannot-run-two-commands-at-once%23new-answer', 'question_page');

);

Post as a guest
































































Comments

Post a Comment

Popular posts from this blog

What does second last employer means? [closed]

Image
Clash Royale CLAN TAG #URR8PPP .everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty margin-bottom:0; up vote -3 down vote favorite Say, for example I have worked in the following companies, Company A => 2010 - 2011 Company B => 2011 - 2012 Company C => 2012 - Till Date Company D => Have offer Now assume Company D is asking to submit documents from my second last employer. So from the list of companies given, which company would be qualified as second last employer? software-industry employer share | improve this question asked Aug 10 '14 at 5:16 Rajaprabhu Aravindasamy 559 1 6 13 closed as off-topic by Jim G., jmort253 ♦ Aug 11 '14 at 1:58 This question appears to be off-topic. The users who voted to close gave this specific reason: "Questions seeking advice on company-specific regulations, agreements, or policies should be directed to your manager or HR department. Questions that address...
Read more

Installing NextGIS Connect into QGIS 3?

Image
Clash Royale CLAN TAG #URR8PPP .everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty margin-bottom:0; up vote 1 down vote favorite Just acquired free version of NEXTGIS and trying to install the plugin for QGIS. It tells me it was not designed for version 3 of QGIS. How do I set up the QGIS-NEXTGIS interface then? qgis qgis-plugins nextgis share | improve this question edited 3 hours ago PolyGeo ♦ 51.9k 17 77 233 asked 4 hours ago IanGS 16 4 New contributor IanGS is a new contributor to this site. Take care in asking for clarification, commenting, and answering. Check out our Code of Conduct. add a comment  |  up vote 1 down vote favorite Just acquired free version of NEXTGIS and trying to install the plugin for QGIS. It tells me it was not designed for version 3 of QGIS. How do I set up the QGIS-NEXTGIS interface then? qgis qgis-plugin...
Read more

One-line joke

A one-liner is a joke that is delivered in a single line. A good one-liner is said to be pithy - concise and meaningful. [1] Comedians and actors use this comedic method as part of their act, e.g. Jimmy Carr, Tommy Cooper, Rodney Dangerfield, Norm Macdonald, Ken Dodd, Stewart Francis, Zach Galifianakis, Mitch Hedberg, Anthony Jeselnik, Milton Jones, Shappi Khorsandi, Jay London, Mark Linn-Baker, Demetri Martin, Groucho Marx, Dan Mintz, Emo Philips, Tim Vine, Steven Wright and Henny Youngman and Arnold Schwarzenegger. [ citation needed ] Many fictional characters are also known to deliver one-liners, including James Bond, who usually includes pithy and laconic quips after disposing of a villain. [ citation needed ] Examples "A baby seal walks into a club." "A dyslexic man walks into a bra." (Anonymous) "There are three types of people, those who can count and those who can't." "Venison's dear, isn't it?" (Jimmy Carr) "An escala...
Read more