From: Muraoka Taro <koron.kaoriya@gmail.com>
Date: Wed, 21 Jan 2026 19:41:58 +0000 (+0000)
Subject: check.vim: Use silent command modifier
X-Git-Tag: v9.1.2102~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3456303430a3fd2e628a850f90845cc4cbaa7994;p=thirdparty%2Fvim.git

check.vim: Use silent command modifier

Problem:  When checking a .po file (make -C src/po check), errors are not
          displayed.

Solution: Adding "silent" to some normal commands in check.vim
          suppresses unnecessary output (Muraoka Taro)

This is because the output of check.vim is redirected to /dev/null.
However, if you stop the redirection, check.vim generates a lot of
output and becomes very slow.

When these commands are run in ex mode, they output the contents of the
line the cursor is pointing to.  This caused a lot of output.

closes: #19227

Signed-off-by: Muraoka Taro <koron.kaoriya@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
---

diff --git a/src/po/Makefile b/src/po/Makefile
index ebfb4c1d33..82b6a0922a 100644
--- a/src/po/Makefile
+++ b/src/po/Makefile
@@ -42,7 +42,7 @@ converted: $(MOCONVERTED)
 .po.ck:
 	$(VIMPROG) -u NONE --noplugins -e -s -X --cmd "set enc=utf-8" \
 		-S check.vim -c "if error == 0 | q | else | num 2 | cq | endif" \
-		$< >/dev/null
+		$<
 	touch $@
 
 check: $(CHECKFILES)
diff --git a/src/po/check.vim b/src/po/check.vim
index 8ecf05a560..f1c60a9341 100644
--- a/src/po/check.vim
+++ b/src/po/check.vim
@@ -22,7 +22,7 @@ set nowrapscan
 func! GetMline()
   let idline = substitute(getline('.'), '"\(.*\)"$', '\1', '')
   while line('.') < line('$')
-    +
+    silent +
     let line = getline('.')
     if line[0] != '"'
       break
@@ -60,8 +60,8 @@ endfunc
 
 " Start at the first "msgid" line.
 let wsv = winsaveview()
-1
-keeppatterns /^msgid\>
+silent 1
+silent keeppatterns /^msgid\>
 
 " When an error is detected this is set to the line number.
 " Note: this is used in the Makefile.
@@ -154,7 +154,7 @@ endwhile
 " - msgstr "E123 ..."     missing colon
 " - msgstr "..."          missing error code
 "
-1
+silent 1
 if search('msgid "\("\n"\)\?\([EW][0-9]\+:\).*\nmsgstr "\("\n"\)\?[^"]\@=\2\@!') > 0
   echomsg 'Mismatching error/warning code in line ' . line('.')
   if error == 0
@@ -164,8 +164,8 @@ endif
 
 " Check that the \n at the end of the msgid line is also present in the msgstr
 " line.  Skip over the header.
-1
-keeppatterns /^"MIME-Version:
+silent 1
+silent keeppatterns /^"MIME-Version:
 while 1
   let lnum = search('^msgid\>')
   if lnum <= 0
@@ -191,8 +191,8 @@ endwhile
 " final '""', '\n"', ' "' '/"' '."' '-"' are OK
 " Beware, it can give false positives if the message is split
 " in the middle of a word
-1
-keeppatterns /^"MIME-Version:
+silent 1
+silent keeppatterns /^"MIME-Version:
 while 1
   let lnum = search('^msgid\>')
   if lnum <= 0
@@ -259,7 +259,7 @@ if executable("msgfmt")
 endif
 
 " Check that the plural form is properly initialized
-1
+silent 1
 let plural = search('^msgid_plural ', 'n')
 if (plural && search('^"Plural-Forms: ', 'n') == 0) || (plural && search('^msgstr\[0\] ".\+"', 'n') != plural + 1)
   if search('^"Plural-Forms: ', 'n') == 0