home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 7.4 / 7.4.327 < prev    next >
Encoding:
Internet Message Format  |  2014-06-16  |  4.2 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.4.327
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=UTF-8
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 7.4.327
  11. Problem:    When 'verbose' is set to display the return value of a function,
  12.         may get E724 repeatedly.
  13. Solution:   Do not give an error for verbose messages. Abort conversion to
  14.         string after an error.
  15. Files:        src/eval.c
  16.  
  17.  
  18. *** ../vim-7.4.326/src/eval.c    2014-06-12 18:39:16.828400409 +0200
  19. --- src/eval.c    2014-06-17 12:48:12.083946675 +0200
  20. ***************
  21. *** 134,139 ****
  22. --- 134,142 ----
  23.   #define COPYID_INC 2
  24.   #define COPYID_MASK (~0x1)
  25.   
  26. + /* Abort conversion to string after a recursion error. */
  27. + static int  did_echo_string_emsg = FALSE;
  28. + 
  29.   /*
  30.    * Array to hold the hashtab with variables local to each sourced script.
  31.    * Each item holds a variable (nameless) that points to the dict_T.
  32. ***************
  33. *** 6686,6691 ****
  34. --- 6689,6696 ----
  35.       }
  36.   
  37.       line_breakcheck();
  38. +     if (did_echo_string_emsg)  /* recursion error, bail out */
  39. +         break;
  40.       }
  41.   
  42.       /* Allocate result buffer with its total size, avoid re-allocation and
  43. ***************
  44. *** 7460,7467 ****
  45.           if (s != NULL)
  46.           ga_concat(&ga, s);
  47.           vim_free(tofree);
  48. !         if (s == NULL)
  49.           break;
  50.       }
  51.       }
  52.       if (todo > 0)
  53. --- 7465,7474 ----
  54.           if (s != NULL)
  55.           ga_concat(&ga, s);
  56.           vim_free(tofree);
  57. !         if (s == NULL || did_echo_string_emsg)
  58.           break;
  59. +         line_breakcheck();
  60. + 
  61.       }
  62.       }
  63.       if (todo > 0)
  64. ***************
  65. *** 7619,7627 ****
  66.   
  67.       if (recurse >= DICT_MAXNEST)
  68.       {
  69. !     EMSG(_("E724: variable nested too deep for displaying"));
  70.       *tofree = NULL;
  71. !     return NULL;
  72.       }
  73.       ++recurse;
  74.   
  75. --- 7626,7641 ----
  76.   
  77.       if (recurse >= DICT_MAXNEST)
  78.       {
  79. !     if (!did_echo_string_emsg)
  80. !     {
  81. !         /* Only give this message once for a recursive call to avoid
  82. !          * flooding the user with errors.  And stop iterating over lists
  83. !          * and dicts. */
  84. !         did_echo_string_emsg = TRUE;
  85. !         EMSG(_("E724: variable nested too deep for displaying"));
  86. !     }
  87.       *tofree = NULL;
  88. !     return (char_u *)"{E724}";
  89.       }
  90.       ++recurse;
  91.   
  92. ***************
  93. *** 7689,7695 ****
  94.           *tofree = NULL;
  95.       }
  96.   
  97. !     --recurse;
  98.       return r;
  99.   }
  100.   
  101. --- 7703,7710 ----
  102.           *tofree = NULL;
  103.       }
  104.   
  105. !     if (--recurse == 0)
  106. !     did_echo_string_emsg = FALSE;
  107.       return r;
  108.   }
  109.   
  110. ***************
  111. *** 23303,23309 ****
  112. --- 23318,23327 ----
  113.               msg_outnum((long)argvars[i].vval.v_number);
  114.               else
  115.               {
  116. +             /* Do not want errors such as E724 here. */
  117. +             ++emsg_off;
  118.               s = tv2string(&argvars[i], &tofree, numbuf2, 0);
  119. +             --emsg_off;
  120.               if (s != NULL)
  121.               {
  122.                   if (vim_strsize(s) > MSG_BUF_CLEN)
  123. ***************
  124. *** 23395,23402 ****
  125.   
  126.           /* The value may be very long.  Skip the middle part, so that we
  127.            * have some idea how it starts and ends. smsg() would always
  128. !          * truncate it at the end. */
  129.           s = tv2string(fc->rettv, &tofree, numbuf2, 0);
  130.           if (s != NULL)
  131.           {
  132.           if (vim_strsize(s) > MSG_BUF_CLEN)
  133. --- 23413,23422 ----
  134.   
  135.           /* The value may be very long.  Skip the middle part, so that we
  136.            * have some idea how it starts and ends. smsg() would always
  137. !          * truncate it at the end. Don't want errors such as E724 here. */
  138. !         ++emsg_off;
  139.           s = tv2string(fc->rettv, &tofree, numbuf2, 0);
  140. +         --emsg_off;
  141.           if (s != NULL)
  142.           {
  143.           if (vim_strsize(s) > MSG_BUF_CLEN)
  144. *** ../vim-7.4.326/src/version.c    2014-06-14 12:53:27.394152699 +0200
  145. --- src/version.c    2014-06-17 12:41:45.019932032 +0200
  146. ***************
  147. *** 736,737 ****
  148. --- 736,739 ----
  149.   {   /* Add new patch number below this line */
  150. + /**/
  151. +     327,
  152.   /**/
  153.  
  154. -- 
  155. TALL KNIGHT: We shall say Ni! again to you if you do not appease us.
  156. ARTHUR:      All right!  What do you want?
  157. TALL KNIGHT: We want ... a shrubbery!
  158.                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  159.  
  160.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  161. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  162. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  163.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  164.