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.262 < prev    next >
Encoding:
Internet Message Format  |  2014-04-22  |  9.5 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.4.262
  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.262
  11. Problem:    Duplicate code in regexec().
  12. Solution:   Add line_lbr flag to regexec_nl().
  13. Files:        src/regexp.c, src/regexp_nfa.c, src/regexp.h
  14.  
  15.  
  16. *** ../vim-7.4.261/src/regexp.c    2014-04-23 18:48:43.546854558 +0200
  17. --- src/regexp.c    2014-04-23 18:59:38.606838773 +0200
  18. ***************
  19. *** 3709,3733 ****
  20.   /* TRUE if using multi-line regexp. */
  21.   #define REG_MULTI    (reg_match == NULL)
  22.   
  23. ! static int  bt_regexec __ARGS((regmatch_T *rmp, char_u *line, colnr_T col));
  24.   
  25.   /*
  26.    * Match a regexp against a string.
  27.    * "rmp->regprog" is a compiled regexp as returned by vim_regcomp().
  28.    * Uses curbuf for line count and 'iskeyword'.
  29.    *
  30.    * Return TRUE if there is a match, FALSE if not.
  31.    */
  32.       static int
  33. ! bt_regexec(rmp, line, col)
  34.       regmatch_T    *rmp;
  35.       char_u    *line;    /* string to match against */
  36.       colnr_T    col;    /* column to start looking for match */
  37.   {
  38.       reg_match = rmp;
  39.       reg_mmatch = NULL;
  40.       reg_maxline = 0;
  41. !     reg_line_lbr = FALSE;
  42.       reg_buf = curbuf;
  43.       reg_win = NULL;
  44.       ireg_ic = rmp->rm_ic;
  45. --- 3709,3736 ----
  46.   /* TRUE if using multi-line regexp. */
  47.   #define REG_MULTI    (reg_match == NULL)
  48.   
  49. ! static int  bt_regexec_nl __ARGS((regmatch_T *rmp, char_u *line, colnr_T col, int line_lbr));
  50. ! 
  51.   
  52.   /*
  53.    * Match a regexp against a string.
  54.    * "rmp->regprog" is a compiled regexp as returned by vim_regcomp().
  55.    * Uses curbuf for line count and 'iskeyword'.
  56. +  * if "line_lbr" is TRUE  consider a "\n" in "line" to be a line break.
  57.    *
  58.    * Return TRUE if there is a match, FALSE if not.
  59.    */
  60.       static int
  61. ! bt_regexec_nl(rmp, line, col, line_lbr)
  62.       regmatch_T    *rmp;
  63.       char_u    *line;    /* string to match against */
  64.       colnr_T    col;    /* column to start looking for match */
  65. +     int        line_lbr;
  66.   {
  67.       reg_match = rmp;
  68.       reg_mmatch = NULL;
  69.       reg_maxline = 0;
  70. !     reg_line_lbr = line_lbr;
  71.       reg_buf = curbuf;
  72.       reg_win = NULL;
  73.       ireg_ic = rmp->rm_ic;
  74. ***************
  75. *** 3738,3772 ****
  76.       return (bt_regexec_both(line, col, NULL) != 0);
  77.   }
  78.   
  79. - #if defined(FEAT_MODIFY_FNAME) || defined(FEAT_EVAL) \
  80. -     || defined(FIND_REPLACE_DIALOG) || defined(PROTO)
  81. - 
  82. - static int  bt_regexec_nl __ARGS((regmatch_T *rmp, char_u *line, colnr_T col));
  83. - 
  84. - /*
  85. -  * Like vim_regexec(), but consider a "\n" in "line" to be a line break.
  86. -  */
  87. -     static int
  88. - bt_regexec_nl(rmp, line, col)
  89. -     regmatch_T    *rmp;
  90. -     char_u    *line;    /* string to match against */
  91. -     colnr_T    col;    /* column to start looking for match */
  92. - {
  93. -     reg_match = rmp;
  94. -     reg_mmatch = NULL;
  95. -     reg_maxline = 0;
  96. -     reg_line_lbr = TRUE;
  97. -     reg_buf = curbuf;
  98. -     reg_win = NULL;
  99. -     ireg_ic = rmp->rm_ic;
  100. - #ifdef FEAT_MBYTE
  101. -     ireg_icombine = FALSE;
  102. - #endif
  103. -     ireg_maxcol = 0;
  104. -     return (bt_regexec_both(line, col, NULL) != 0);
  105. - }
  106. - #endif
  107. - 
  108.   static long bt_regexec_multi __ARGS((regmmatch_T *rmp, win_T *win, buf_T *buf, linenr_T lnum, colnr_T col, proftime_T *tm));
  109.   
  110.   /*
  111. --- 3741,3746 ----
  112. ***************
  113. *** 7985,7995 ****
  114.   {
  115.       bt_regcomp,
  116.       bt_regfree,
  117. -     bt_regexec,
  118. - #if defined(FEAT_MODIFY_FNAME) || defined(FEAT_EVAL) \
  119. -     || defined(FIND_REPLACE_DIALOG) || defined(PROTO)
  120.       bt_regexec_nl,
  121. - #endif
  122.       bt_regexec_multi
  123.   #ifdef DEBUG
  124.       ,(char_u *)""
  125. --- 7959,7965 ----
  126. ***************
  127. *** 8003,8013 ****
  128.   {
  129.       nfa_regcomp,
  130.       nfa_regfree,
  131. -     nfa_regexec,
  132. - #if defined(FEAT_MODIFY_FNAME) || defined(FEAT_EVAL) \
  133. -     || defined(FIND_REPLACE_DIALOG) || defined(PROTO)
  134.       nfa_regexec_nl,
  135. - #endif
  136.       nfa_regexec_multi
  137.   #ifdef DEBUG
  138.       ,(char_u *)""
  139. --- 7973,7979 ----
  140. ***************
  141. *** 8131,8137 ****
  142.       char_u      *line;  /* string to match against */
  143.       colnr_T     col;    /* column to start looking for match */
  144.   {
  145. !     return rmp->regprog->engine->regexec(rmp, line, col);
  146.   }
  147.   
  148.   #if defined(FEAT_MODIFY_FNAME) || defined(FEAT_EVAL) \
  149. --- 8097,8103 ----
  150.       char_u      *line;  /* string to match against */
  151.       colnr_T     col;    /* column to start looking for match */
  152.   {
  153. !     return rmp->regprog->engine->regexec_nl(rmp, line, col, FALSE);
  154.   }
  155.   
  156.   #if defined(FEAT_MODIFY_FNAME) || defined(FEAT_EVAL) \
  157. ***************
  158. *** 8145,8151 ****
  159.       char_u *line;
  160.       colnr_T col;
  161.   {
  162. !     return rmp->regprog->engine->regexec_nl(rmp, line, col);
  163.   }
  164.   #endif
  165.   
  166. --- 8111,8117 ----
  167.       char_u *line;
  168.       colnr_T col;
  169.   {
  170. !     return rmp->regprog->engine->regexec_nl(rmp, line, col, TRUE);
  171.   }
  172.   #endif
  173.   
  174. *** ../vim-7.4.261/src/regexp_nfa.c    2014-04-06 21:33:39.675363743 +0200
  175. --- src/regexp_nfa.c    2014-04-23 19:00:44.354837189 +0200
  176. ***************
  177. *** 311,317 ****
  178.   static long nfa_regexec_both __ARGS((char_u *line, colnr_T col));
  179.   static regprog_T *nfa_regcomp __ARGS((char_u *expr, int re_flags));
  180.   static void nfa_regfree __ARGS((regprog_T *prog));
  181. ! static int nfa_regexec __ARGS((regmatch_T *rmp, char_u *line, colnr_T col));
  182.   static long nfa_regexec_multi __ARGS((regmmatch_T *rmp, win_T *win, buf_T *buf, linenr_T lnum, colnr_T col, proftime_T *tm));
  183.   static int match_follows __ARGS((nfa_state_T *startstate, int depth));
  184.   static int failure_chance __ARGS((nfa_state_T *state, int depth));
  185. --- 311,317 ----
  186.   static long nfa_regexec_both __ARGS((char_u *line, colnr_T col));
  187.   static regprog_T *nfa_regcomp __ARGS((char_u *expr, int re_flags));
  188.   static void nfa_regfree __ARGS((regprog_T *prog));
  189. ! static int  nfa_regexec_nl __ARGS((regmatch_T *rmp, char_u *line, colnr_T col, int line_lbr));
  190.   static long nfa_regexec_multi __ARGS((regmmatch_T *rmp, win_T *win, buf_T *buf, linenr_T lnum, colnr_T col, proftime_T *tm));
  191.   static int match_follows __ARGS((nfa_state_T *startstate, int depth));
  192.   static int failure_chance __ARGS((nfa_state_T *state, int depth));
  193. ***************
  194. *** 7060,7078 ****
  195.    * Match a regexp against a string.
  196.    * "rmp->regprog" is a compiled regexp as returned by nfa_regcomp().
  197.    * Uses curbuf for line count and 'iskeyword'.
  198.    *
  199.    * Return TRUE if there is a match, FALSE if not.
  200.    */
  201.       static int
  202. ! nfa_regexec(rmp, line, col)
  203.       regmatch_T    *rmp;
  204.       char_u    *line;    /* string to match against */
  205.       colnr_T    col;    /* column to start looking for match */
  206.   {
  207.       reg_match = rmp;
  208.       reg_mmatch = NULL;
  209.       reg_maxline = 0;
  210. !     reg_line_lbr = FALSE;
  211.       reg_buf = curbuf;
  212.       reg_win = NULL;
  213.       ireg_ic = rmp->rm_ic;
  214. --- 7060,7080 ----
  215.    * Match a regexp against a string.
  216.    * "rmp->regprog" is a compiled regexp as returned by nfa_regcomp().
  217.    * Uses curbuf for line count and 'iskeyword'.
  218. +  * If "line_lbr" is TRUE consider a "\n" in "line" to be a line break.
  219.    *
  220.    * Return TRUE if there is a match, FALSE if not.
  221.    */
  222.       static int
  223. ! nfa_regexec_nl(rmp, line, col, line_lbr)
  224.       regmatch_T    *rmp;
  225.       char_u    *line;    /* string to match against */
  226.       colnr_T    col;    /* column to start looking for match */
  227. +     int        line_lbr;
  228.   {
  229.       reg_match = rmp;
  230.       reg_mmatch = NULL;
  231.       reg_maxline = 0;
  232. !     reg_line_lbr = line_lbr;
  233.       reg_buf = curbuf;
  234.       reg_win = NULL;
  235.       ireg_ic = rmp->rm_ic;
  236. ***************
  237. *** 7083,7117 ****
  238.       return (nfa_regexec_both(line, col) != 0);
  239.   }
  240.   
  241. - #if defined(FEAT_MODIFY_FNAME) || defined(FEAT_EVAL) \
  242. -     || defined(FIND_REPLACE_DIALOG) || defined(PROTO)
  243. - 
  244. - static int  nfa_regexec_nl __ARGS((regmatch_T *rmp, char_u *line, colnr_T col));
  245. - 
  246. - /*
  247. -  * Like nfa_regexec(), but consider a "\n" in "line" to be a line break.
  248. -  */
  249. -     static int
  250. - nfa_regexec_nl(rmp, line, col)
  251. -     regmatch_T    *rmp;
  252. -     char_u    *line;    /* string to match against */
  253. -     colnr_T    col;    /* column to start looking for match */
  254. - {
  255. -     reg_match = rmp;
  256. -     reg_mmatch = NULL;
  257. -     reg_maxline = 0;
  258. -     reg_line_lbr = TRUE;
  259. -     reg_buf = curbuf;
  260. -     reg_win = NULL;
  261. -     ireg_ic = rmp->rm_ic;
  262. - #ifdef FEAT_MBYTE
  263. -     ireg_icombine = FALSE;
  264. - #endif
  265. -     ireg_maxcol = 0;
  266. -     return (nfa_regexec_both(line, col) != 0);
  267. - }
  268. - #endif
  269. - 
  270.   
  271.   /*
  272.    * Match a regexp against multiple lines.
  273. --- 7085,7090 ----
  274. *** ../vim-7.4.261/src/regexp.h    2013-06-11 10:53:14.000000000 +0200
  275. --- src/regexp.h    2014-04-23 18:58:18.614840701 +0200
  276. ***************
  277. *** 149,159 ****
  278.   {
  279.       regprog_T    *(*regcomp)(char_u*, int);
  280.       void    (*regfree)(regprog_T *);
  281. !     int        (*regexec)(regmatch_T*, char_u*, colnr_T);
  282. ! #if defined(FEAT_MODIFY_FNAME) || defined(FEAT_EVAL) \
  283. !     || defined(FIND_REPLACE_DIALOG) || defined(PROTO)
  284. !     int        (*regexec_nl)(regmatch_T*, char_u*, colnr_T);
  285. ! #endif
  286.       long    (*regexec_multi)(regmmatch_T*, win_T*, buf_T*, linenr_T, colnr_T, proftime_T*);
  287.   #ifdef DEBUG
  288.       char_u    *expr;
  289. --- 149,155 ----
  290.   {
  291.       regprog_T    *(*regcomp)(char_u*, int);
  292.       void    (*regfree)(regprog_T *);
  293. !     int        (*regexec_nl)(regmatch_T*, char_u*, colnr_T, int);
  294.       long    (*regexec_multi)(regmmatch_T*, win_T*, buf_T*, linenr_T, colnr_T, proftime_T*);
  295.   #ifdef DEBUG
  296.       char_u    *expr;
  297. *** ../vim-7.4.261/src/version.c    2014-04-23 18:48:43.546854558 +0200
  298. --- src/version.c    2014-04-23 18:52:20.102849340 +0200
  299. ***************
  300. *** 736,737 ****
  301. --- 736,739 ----
  302.   {   /* Add new patch number below this line */
  303. + /**/
  304. +     262,
  305.   /**/
  306.  
  307. -- 
  308. From "know your smileys":
  309.  ~#:-(    I just washed my hair, and I can't do nuthin' with it.
  310.  
  311.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  312. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  313. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  314.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  315.