Loading ...
Sorry, an error occurred while loading the content.

[Patch] fix minor bug in handling shellpipe and shellredir

Expand Messages
  • LCD 47
    Expansion of shellpipe and shellredir doesn t handle %%s as advertised. Probably nobody does that, but anyway, patch attached. /lcd diff -r 74301ef4297a
    Message 1 of 2 , Aug 2, 2013
    • 0 Attachment
      Expansion of shellpipe and shellredir doesn't handle "%%s" as
      advertised. Probably nobody does that, but anyway, patch attached.

      /lcd


      diff -r 74301ef4297a src/ex_cmds.c
      --- a/src/ex_cmds.c Thu Aug 01 18:38:26 2013 +0200
      +++ b/src/ex_cmds.c Fri Aug 02 12:34:43 2013 +0300
      @@ -1626,6 +1626,11 @@
      for (p = opt; (p = vim_strchr(p, '%')) != NULL; ++p)
      if (p[1] == 's')
      break;
      + else
      + {
      + if (p[1] == '%')
      + p++;
      + }
      if (p != NULL)
      {
      *end = ' '; /* not really needed? Not with sh, ksh or bash */

      --
      --
      You received this message from the "vim_dev" maillist.
      Do not top-post! Type your reply below the text you are replying to.
      For more information, visit http://www.vim.org/maillist.php

      ---
      You received this message because you are subscribed to the Google Groups "vim_dev" group.
      To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe@....
      For more options, visit https://groups.google.com/groups/opt_out.
    • Bram Moolenaar
      ... Thanks, I ll include it. -- A day without sunshine is like, well, night. /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net ///
      Message 2 of 2 , Aug 2, 2013
      • 0 Attachment
        LCD wrote:

        > Expansion of shellpipe and shellredir doesn't handle "%%s" as
        > advertised. Probably nobody does that, but anyway, patch attached.
        >
        > /lcd
        >
        >
        > diff -r 74301ef4297a src/ex_cmds.c
        > --- a/src/ex_cmds.c Thu Aug 01 18:38:26 2013 +0200
        > +++ b/src/ex_cmds.c Fri Aug 02 12:34:43 2013 +0300
        > @@ -1626,6 +1626,11 @@
        > for (p = opt; (p = vim_strchr(p, '%')) != NULL; ++p)
        > if (p[1] == 's')
        > break;
        > + else
        > + {
        > + if (p[1] == '%')
        > + p++;
        > + }
        > if (p != NULL)
        > {
        > *end = ' '; /* not really needed? Not with sh, ksh or bash */

        Thanks, I'll include it.

        --
        A day without sunshine is like, well, night.

        /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net \\\
        /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
        \\\ an exciting new programming language -- http://www.Zimbu.org ///
        \\\ help me help AIDS victims -- http://ICCF-Holland.org ///

        --
        --
        You received this message from the "vim_dev" maillist.
        Do not top-post! Type your reply below the text you are replying to.
        For more information, visit http://www.vim.org/maillist.php

        ---
        You received this message because you are subscribed to the Google Groups "vim_dev" group.
        To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe@....
        For more options, visit https://groups.google.com/groups/opt_out.
      Your message has been successfully submitted and would be delivered to recipients shortly.