Patch 7.3.235

67 views
Skip to first unread message

Bram Moolenaar

unread,
Jun 25, 2011, 11:36:57 PM6/25/11
to vim...@googlegroups.com

Patch 7.3.235
Problem: ";" gets stuck on a "t" command, it's not useful.
Solution: Add the ';' flag in 'cpo'. (Christian Brabandt)
Files: runtime/doc/motion.txt, runtime/doc/options.txt, src/option.h,
src/search.c src/testdir/test81.in, src/testdir/test81.ok,
src/testdir/Makefile, src/testdir/Make_amiga.mak,
src/testdir/Make_dos.mak, src/testdir/Make_ming.mak,
src/testdir/Make_os2.mak, src/testdir/Make_vms.mms


*** ../vim-7.3.234/runtime/doc/motion.txt 2010-08-15 21:57:17.000000000 +0200
--- runtime/doc/motion.txt 2011-06-26 05:15:58.000000000 +0200
***************
*** 269,279 ****
{char} can be entered like with the |f| command.

*;*
! ; Repeat latest f, t, F or T [count] times.

*,*
, Repeat latest f, t, F or T in opposite direction
! [count] times.

==============================================================================
3. Up-down motions *up-down-motions*
--- 269,279 ----
{char} can be entered like with the |f| command.

*;*
! ; Repeat latest f, t, F or T [count] times. See |cpo-;|

*,*
, Repeat latest f, t, F or T in opposite direction
! [count] times. See also |cpo-;|

==============================================================================
3. Up-down motions *up-down-motions*
*** ../vim-7.3.234/runtime/doc/options.txt 2011-06-12 20:42:17.000000000 +0200
--- runtime/doc/options.txt 2011-06-26 05:15:58.000000000 +0200
***************
*** 2090,2095 ****
--- 2117,2128 ----
*cpo->*
> When appending to a register, put a line break before
the appended text.
+ *cpo-;*
+ ; When using |,| or |;| to repeat the last |t| search
+ and the cursor is right in front of the searched
+ character, the cursor won't move. When not included,
+ the cursor would skip over it and jump to the
+ following occurence.

POSIX flags. These are not included in the Vi default value, except
when $VIM_POSIX was set on startup. |posix|
*** ../vim-7.3.234/src/option.h 2011-06-12 22:13:37.000000000 +0200
--- src/option.h 2011-06-26 05:17:58.000000000 +0200
***************
*** 169,178 ****
#define CPO_SUBPERCENT '/' /* % in :s string uses previous one */
#define CPO_BACKSL '\\' /* \ is not special in [] */
#define CPO_CHDIR '.' /* don't chdir if buffer is modified */
/* default values for Vim, Vi and POSIX */
#define CPO_VIM "aABceFs"
! #define CPO_VI "aAbBcCdDeEfFgHiIjJkKlLmMnoOpPqrRsStuvwWxXyZ$!%*-+<>"
! #define CPO_ALL "aAbBcCdDeEfFgHiIjJkKlLmMnoOpPqrRsStuvwWxXyZ$!%*-+<>#{|&/\\."

/* characters for p_ww option: */
#define WW_ALL "bshl<>[],~"
--- 169,180 ----
#define CPO_SUBPERCENT '/' /* % in :s string uses previous one */
#define CPO_BACKSL '\\' /* \ is not special in [] */
#define CPO_CHDIR '.' /* don't chdir if buffer is modified */
+ #define CPO_SCOLON ';' /* using "," and ";" will skip over char if
+ * cursor would not move */
/* default values for Vim, Vi and POSIX */
#define CPO_VIM "aABceFs"
! #define CPO_VI "aAbBcCdDeEfFgHiIjJkKlLmMnoOpPqrRsStuvwWxXyZ$!%*-+<>;"
! #define CPO_ALL "aAbBcCdDeEfFgHiIjJkKlLmMnoOpPqrRsStuvwWxXyZ$!%*-+<>#{|&/\\.;"

/* characters for p_ww option: */
#define WW_ALL "bshl<>[],~"
*** ../vim-7.3.234/src/search.c 2011-05-10 16:41:13.000000000 +0200
--- src/search.c 2011-06-26 05:20:45.000000000 +0200
***************
*** 1546,1551 ****
--- 1546,1552 ----
int col;
char_u *p;
int len;
+ int stop = TRUE;
#ifdef FEAT_MBYTE
static char_u bytes[MB_MAXBYTES];
static int bytelen = 1; /* >1 for multi-byte char */
***************
*** 1580,1585 ****
--- 1581,1592 ----
t_cmd = last_t_cmd;
c = lastc;
/* For multi-byte re-use last bytes[] and bytelen. */
+
+ /* Force a move of at least one char, so ";" and "," will move the
+ * cursor, even if the cursor is right in front of char we are looking
+ * at. */
+ if (vim_strchr(p_cpo, CPO_SCOLON) == NULL && count == 1)
+ stop = FALSE;
}

if (dir == BACKWARD)
***************
*** 1612,1625 ****
}
if (bytelen == 1)
{
! if (p[col] == c)
break;
}
else
{
! if (vim_memcmp(p + col, bytes, bytelen) == 0)
break;
}
}
}
else
--- 1619,1633 ----
}
if (bytelen == 1)
{
! if (p[col] == c && stop)
break;
}
else
{
! if (vim_memcmp(p + col, bytes, bytelen) == 0 && stop)
break;
}
+ stop = TRUE;
}
}
else
***************
*** 1629,1636 ****
{
if ((col += dir) < 0 || col >= len)
return FAIL;
! if (p[col] == c)
break;
}
}
}
--- 1637,1645 ----
{
if ((col += dir) < 0 || col >= len)
return FAIL;
! if (p[col] == c && stop)
break;
+ stop = TRUE;
}
}
}
*** ../vim-7.3.234/src/testdir/test81.in 2011-06-26 05:34:33.000000000 +0200
--- src/testdir/test81.in 2011-06-26 05:30:31.000000000 +0200
***************
*** 0 ****
--- 1,18 ----
+ Test for t movement command and 'cpo-;' setting
+
+ STARTTEST
+ :set nocompatible
+ :set cpo-=;
+ /firstline/
+ j0tt;D
+ $Ty;D:set cpo+=;
+ j0tt;;D
+ $Ty;;D:?firstline?+1,$w! test.out
+ :qa!
+ ENDTEST
+
+ firstline
+ aaa two three four
+ bbb yee yoo four
+ ccc two three four
+ ddd yee yoo four
*** ../vim-7.3.234/src/testdir/test81.ok 2011-06-26 05:34:33.000000000 +0200
--- src/testdir/test81.ok 2011-06-26 05:31:33.000000000 +0200
***************
*** 0 ****
--- 1,4 ----
+ aaa two
+ bbb y
+ ccc
+ ddd yee y
*** ../vim-7.3.234/src/testdir/Makefile 2011-06-19 04:31:54.000000000 +0200
--- src/testdir/Makefile 2011-06-26 05:09:56.000000000 +0200
***************
*** 26,32 ****
test64.out test65.out test66.out test67.out test68.out \
test69.out test70.out test71.out test72.out test73.out \
test74.out test75.out test76.out test77.out test78.out \
! test79.out test80.out

SCRIPTS_GUI = test16.out

--- 26,32 ----
test64.out test65.out test66.out test67.out test68.out \
test69.out test70.out test71.out test72.out test73.out \
test74.out test75.out test76.out test77.out test78.out \
! test79.out test80.out test81.out

SCRIPTS_GUI = test16.out

*** ../vim-7.3.234/src/testdir/Make_amiga.mak 2011-06-19 04:31:54.000000000 +0200
--- src/testdir/Make_amiga.mak 2011-06-26 05:09:07.000000000 +0200
***************
*** 28,34 ****
test61.out test62.out test63.out test64.out test65.out \
test66.out test67.out test68.out test69.out test70.out \
test71.out test72.out test73.out test74.out test75.out \
! test76.out test77.out test78.out test79.out test80.out

.SUFFIXES: .in .out

--- 28,35 ----
test61.out test62.out test63.out test64.out test65.out \
test66.out test67.out test68.out test69.out test70.out \
test71.out test72.out test73.out test74.out test75.out \
! test76.out test77.out test78.out test79.out test80.out \
! test81.out

.SUFFIXES: .in .out

***************
*** 128,130 ****
--- 129,132 ----
test78.out: test78.in
test79.out: test79.in
test80.out: test80.in
+ test81.out: test81.in
*** ../vim-7.3.234/src/testdir/Make_dos.mak 2011-06-19 04:31:54.000000000 +0200
--- src/testdir/Make_dos.mak 2011-06-26 05:09:16.000000000 +0200
***************
*** 29,35 ****
test42.out test52.out test65.out test66.out test67.out \
test68.out test69.out test71.out test72.out test73.out \
test74.out test75.out test76.out test77.out test78.out \
! test79.out test80.out

SCRIPTS32 = test50.out test70.out

--- 29,35 ----
test42.out test52.out test65.out test66.out test67.out \
test68.out test69.out test71.out test72.out test73.out \
test74.out test75.out test76.out test77.out test78.out \
! test79.out test80.out test81.out

SCRIPTS32 = test50.out test70.out

*** ../vim-7.3.234/src/testdir/Make_ming.mak 2011-06-19 04:31:54.000000000 +0200
--- src/testdir/Make_ming.mak 2011-06-26 05:09:24.000000000 +0200
***************
*** 49,55 ****
test42.out test52.out test65.out test66.out test67.out \
test68.out test69.out test71.out test72.out test73.out \
test74.out test75.out test76.out test77.out test78.out \
! test79.out test80.out

SCRIPTS32 = test50.out test70.out

--- 49,55 ----
test42.out test52.out test65.out test66.out test67.out \
test68.out test69.out test71.out test72.out test73.out \
test74.out test75.out test76.out test77.out test78.out \
! test79.out test80.out test81.out

SCRIPTS32 = test50.out test70.out

*** ../vim-7.3.234/src/testdir/Make_os2.mak 2011-06-19 04:31:54.000000000 +0200
--- src/testdir/Make_os2.mak 2011-06-26 05:09:33.000000000 +0200
***************
*** 28,34 ****
test61.out test62.out test63.out test64.out test65.out \
test66.out test67.out test68.out test69.out test70.out \
test71.out test72.out test73.out test74.out test75.out \
! test76.out test77.out test78.out test79.out test80.out

.SUFFIXES: .in .out

--- 28,35 ----
test61.out test62.out test63.out test64.out test65.out \
test66.out test67.out test68.out test69.out test70.out \
test71.out test72.out test73.out test74.out test75.out \
! test76.out test77.out test78.out test79.out test80.out \
! test81.out

.SUFFIXES: .in .out

*** ../vim-7.3.234/src/testdir/Make_vms.mms 2011-06-19 04:31:54.000000000 +0200
--- src/testdir/Make_vms.mms 2011-06-26 05:09:42.000000000 +0200
***************
*** 4,10 ****
# Authors: Zoltan Arpadffy, <arpa...@polarhome.com>
# Sandor Kopanyi, <sandor....@mailbox.hu>
#
! # Last change: 2011 Jun 19
#
# This has been tested on VMS 6.2 to 8.3 on DEC Alpha, VAX and IA64.
# Edit the lines in the Configuration section below to select.
--- 4,10 ----
# Authors: Zoltan Arpadffy, <arpa...@polarhome.com>
# Sandor Kopanyi, <sandor....@mailbox.hu>
#
! # Last change: 2011 Jun 26
#
# This has been tested on VMS 6.2 to 8.3 on DEC Alpha, VAX and IA64.
# Edit the lines in the Configuration section below to select.
***************
*** 75,81 ****
test61.out test62.out test63.out test64.out test65.out \
test66.out test67.out test68.out test69.out \
test71.out test72.out test74.out test75.out test76.out \
! test77.out test78.out test79.out test80.out

# Known problems:
# Test 30: a problem around mac format - unknown reason
--- 75,81 ----
test61.out test62.out test63.out test64.out test65.out \
test66.out test67.out test68.out test69.out \
test71.out test72.out test74.out test75.out test76.out \
! test77.out test78.out test79.out test80.out test81.out

# Known problems:
# Test 30: a problem around mac format - unknown reason
*** ../vim-7.3.234/src/version.c 2011-06-26 04:48:56.000000000 +0200
--- src/version.c 2011-06-26 05:33:53.000000000 +0200
***************
*** 711,712 ****
--- 711,714 ----
{ /* Add new patch number below this line */
+ /**/
+ 235,
/**/

--
hundred-and-one symptoms of being an internet addict:
226. You sit down at the computer right after dinner and your spouse
says "See you in the morning."

/// Bram Moolenaar -- Br...@Moolenaar.net -- 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 ///

Reply all
Reply to author
Forward
0 new messages