patch 9.1.1915: :breaklist accepts unprocessed arguments.

Problem:  :breaklist accepts unprocessed arguments.
Solution: Remove EX_EXTRA flag from the Ex command definition.
          (Doug Kearns)

The command should emit an "E488: Trailing characters" error rather than
silently accept arguments.

closes: #18746

Signed-off-by: Doug Kearns <dougkearns@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>

diff --git a/src/ex_cmds.h b/src/ex_cmds.h
index f91c37d0b816587d84ed502f443d7f7f0e87423a..d588be3b95f471adbf32b55a5107845af57d3c10 100644 (file)
--- a/src/ex_cmds.h
+++ b/src/ex_cmds.h
@@ -234,7 +234,7 @@ EXCMD(CMD_breakdel, "breakdel",     ex_breakdel,
        EX_EXTRA|EX_TRLBAR|EX_CMDWIN|EX_LOCK_OK,
        ADDR_NONE),
 EXCMD(CMD_breaklist,   "breaklist",    ex_breaklist,
-       EX_EXTRA|EX_TRLBAR|EX_CMDWIN|EX_LOCK_OK,
+       EX_TRLBAR|EX_CMDWIN|EX_LOCK_OK,
        ADDR_NONE),
 EXCMD(CMD_browse,      "browse",       ex_wrongmodifier,
        EX_NEEDARG|EX_EXTRA|EX_NOTRLCOM|EX_CMDWIN|EX_LOCK_OK,

diff --git a/src/testdir/test_cmdline.vim b/src/testdir/test_cmdline.vim
index d0987c40a0603d41f01c91e4bb47bc4afb20c22b..2c77957f9b9b5cd51e942b5ef399573fee6deef9 100644 (file)
--- a/src/testdir/test_cmdline.vim
+++ b/src/testdir/test_cmdline.vim
@@ -5176,4 +5176,9 @@ func Test_update_screen_after_wildtrigger()
   call StopVimInTerminal(buf)
 endfunc
 
+func Test_breaklist_args_fails()
+  call assert_match('No breakpoints defined', execute(':breaklist'))
+  call assert_fails(':breaklist extra', 'E488:')
+endfunc
+
 " vim: shiftwidth=2 sts=2 expandtab

diff --git a/src/version.c b/src/version.c
index 3dfd121c9870c314843b57c6231e07c6f94ea858..f501001bb393818780d11d33954dbd5984ba3401 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -729,6 +729,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1915,
 /**/
     1914,
 /**/