aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-07-26 11:04:33 -04:00
Code

patch 8.2.3957: error messages are spread out

Browse Source 
Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.
This commit is contained in:
Bram Moolenaar 2021-12-31 17:25:48 +00:00
parent 4b1478093e
commit 1a9922243a
15 changed files with 112 additions and 51 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/arglist.c
View File

@ -673,11 +673,11 @@ do_argfile(exarg_T *eap, int argn)
if (argn < 0 || argn >= ARGCOUNT) if (argn < 0 || argn >= ARGCOUNT)
{ {
if (ARGCOUNT <= 1) if (ARGCOUNT <= 1)
emsg(_("E163: There is only one file to edit")); emsg(_(e_there_is_only_one_file_to_edit));
else if (argn < 0) else if (argn < 0)
emsg(_("E164: Cannot go before first file")); emsg(_(e_cannot_go_before_first_file));
else else
emsg(_("E165: Cannot go beyond last file")); emsg(_(e_cannot_go_beyond_last_file));
} }
else else
{ {

3
src/bufwrite.c
View File

@ -1821,7 +1821,8 @@ buf_write(
|| (mch_lstat((char *)fname, &st) == 0 || (mch_lstat((char *)fname, &st) == 0
&& (st.st_dev != st_old.st_dev && (st.st_dev != st_old.st_dev
|| st.st_ino != st_old.st_ino))) || st.st_ino != st_old.st_ino)))
errmsg = (char_u *)_("E166: Can't open linked file for writing"); errmsg =
(char_u *)_(e_cant_open_linked_file_for_writing);
else else
#endif #endif
{ {

67
src/errors.h
View File

@ -266,7 +266,7 @@ EXTERN char e_undefined_variable_str[]
INIT(= N_("E121: Undefined variable: %s")); INIT(= N_("E121: Undefined variable: %s"));
EXTERN char e_undefined_variable_char_str[] EXTERN char e_undefined_variable_char_str[]
INIT(= N_("E121: Undefined variable: %c:%s")); INIT(= N_("E121: Undefined variable: %c:%s"));
EXTERN char e_function_str_already_exists_add_excl_to_replace[] EXTERN char e_function_str_already_exists_add_bang_to_replace[]
INIT(= N_("E122: Function %s already exists, add ! to replace it")); INIT(= N_("E122: Function %s already exists, add ! to replace it"));
EXTERN char e_undefined_function_str[] EXTERN char e_undefined_function_str[]
INIT(= N_("E123: Undefined function: %s")); INIT(= N_("E123: Undefined function: %s"));
@ -328,7 +328,7 @@ EXTERN char e_not_a_directory_str[]
INIT(= N_("E150: Not a directory: %s")); INIT(= N_("E150: Not a directory: %s"));
EXTERN char e_no_match_str[] EXTERN char e_no_match_str[]
INIT(= N_("E151: No match: %s")); INIT(= N_("E151: No match: %s"));
EXTERN char e_cannot_open_str_for_writing[] EXTERN char e_cannot_open_str_for_writing_1[]
INIT(= N_("E152: Cannot open %s for writing")); INIT(= N_("E152: Cannot open %s for writing"));
EXTERN char e_unable_to_open_str_for_reading[] EXTERN char e_unable_to_open_str_for_reading[]
INIT(= N_("E153: Unable to open %s for reading")); INIT(= N_("E153: Unable to open %s for reading"));
@ -350,9 +350,70 @@ EXTERN char e_unknown_sign_command_str[]
EXTERN char e_breakpoint_not_found_str[] EXTERN char e_breakpoint_not_found_str[]
INIT(= N_("E161: Breakpoint not found: %s")); INIT(= N_("E161: Breakpoint not found: %s"));
#endif #endif
EXTERN char e_no_write_since_last_change_for_buffer_str[]
INIT(= N_("E162: No write since last change for buffer \"%s\""));
EXTERN char e_there_is_only_one_file_to_edit[]
INIT(= N_("E163: There is only one file to edit"));
EXTERN char e_cannot_go_before_first_file[]
INIT(= N_("E164: Cannot go before first file"));
EXTERN char e_cannot_go_beyond_last_file[]
INIT(= N_("E165: Cannot go beyond last file"));
EXTERN char e_cant_open_linked_file_for_writing[]
INIT(= N_("E166: Can't open linked file for writing"));
EXTERN char e_scriptencoding_used_outside_of_sourced_file[]
INIT(= N_("E167: :scriptencoding used outside of a sourced file"));
EXTERN char e_finish_used_outside_of_sourced_file[]
INIT(= N_("E168: :finish used outside of a sourced file"));
EXTERN char e_command_too_recursive[]
INIT(= N_("E169: Command too recursive"));
EXTERN char e_missing_endwhile[]
INIT(= N_("E170: Missing :endwhile"));
EXTERN char e_missing_endfor[]
INIT(= N_("E170: Missing :endfor"));
EXTERN char e_missing_endif[]
INIT(= N_("E171: Missing :endif"));
EXTERN char e_missing_marker[]
INIT(= N_("E172: Missing marker"));
EXTERN char e_nr_more_file_to_edit[]
INIT(= N_("E173: %d more file to edit"));
EXTERN char e_nr_more_files_to_edit[]
INIT(= N_("E173: %d more files to edit"));
EXTERN char e_command_already_exists_add_bang_to_replace_it_str[]
INIT(= N_("E174: Command already exists: add ! to replace it: %s"));
EXTERN char e_no_attribute_specified[]
INIT(= N_("E175: No attribute specified"));
EXTERN char e_invalid_number_of_arguments[]
INIT(= N_("E176: Invalid number of arguments"));
EXTERN char e_count_cannot_be_specified_twice[]
INIT(= N_("E177: Count cannot be specified twice"));
EXTERN char e_invalid_default_value_for_count[]
INIT(= N_("E178: Invalid default value for count"));
EXTERN char e_argument_required_for_str[]
INIT(= N_("E179: argument required for %s"));
EXTERN char e_invalid_complete_value_str[]
INIT(= N_("E180: Invalid complete value: %s"));
EXTERN char e_invalid_attribute_str[]
INIT(= N_("E181: Invalid attribute: %s"));
EXTERN char e_invalid_command_name[]
INIT(= N_("E182: Invalid command name"));
EXTERN char e_user_defined_commands_must_start_with_an_uppercase_letter[]
INIT(= N_("E183: User defined commands must start with an uppercase letter"));
EXTERN char e_no_such_user_defined_command_str[] EXTERN char e_no_such_user_defined_command_str[]
INIT(= N_("E184: No such user-defined command: %s")); INIT(= N_("E184: No such user-defined command: %s"));
EXTERN char e_cannot_find_color_scheme_str[]
INIT(= N_("E185: Cannot find color scheme '%s'"));
EXTERN char e_no_previous_directory[]
INIT(= N_("E186: No previous directory"));
EXTERN char e_directory_unknown[]
INIT(= N_("E187: Directory unknown"));
EXTERN char e_obtaining_window_position_not_implemented_for_this_platform[]
INIT(= N_("E188: Obtaining window position not implemented for this platform"));
EXTERN char e_str_exists_add_bang_to_override[]
INIT(= N_("E189: \"%s\" exists (add ! to override)"));
EXTERN char e_cannot_open_str_for_writing_2[]
INIT(= N_("E190: Cannot open \"%s\" for writing"));
#ifndef FEAT_DIGRAPHS #ifndef FEAT_DIGRAPHS
EXTERN char e_no_digraphs_version[] EXTERN char e_no_digraphs_version[]
INIT(= N_("E196: No digraphs in this version")); INIT(= N_("E196: No digraphs in this version"));

2
src/evalvars.c
View File

@ -638,7 +638,7 @@ heredoc_get(exarg_T *eap, char_u *cmd, int script_get)
marker = dot; marker = dot;
else else
{ {
emsg(_("E172: Missing marker")); emsg(_(e_missing_marker));
return NULL; return NULL;
} }
} }

2
src/ex_cmds2.c
View File

@ -367,7 +367,7 @@ check_changed_any(
buf->b_fname) buf->b_fname)
: :
#endif #endif
semsg(_("E162: No write since last change for buffer \"%s\""), semsg(_(e_no_write_since_last_change_for_buffer_str),
buf_spname(buf) != NULL ? buf_spname(buf) : buf->b_fname)) buf_spname(buf) != NULL ? buf_spname(buf) : buf->b_fname))
{ {
save = no_wait_return; save = no_wait_return;

24
src/ex_docmd.c
View File

@ -677,7 +677,7 @@ do_cmdline(
#endif #endif
) )
{ {
emsg(_("E169: Command too recursive")); emsg(_(e_command_too_recursive));
#ifdef FEAT_EVAL #ifdef FEAT_EVAL
// When converting to an exception, we do not include the command name // When converting to an exception, we do not include the command name
// since this is not an error of the specific command. // since this is not an error of the specific command.
@ -1214,11 +1214,11 @@ do_cmdline(
if (cstack.cs_flags[cstack.cs_idx] & CSF_TRY) if (cstack.cs_flags[cstack.cs_idx] & CSF_TRY)
emsg(_(e_endtry)); emsg(_(e_endtry));
else if (cstack.cs_flags[cstack.cs_idx] & CSF_WHILE) else if (cstack.cs_flags[cstack.cs_idx] & CSF_WHILE)
emsg(_(e_endwhile)); emsg(_(e_missing_endwhile));
else if (cstack.cs_flags[cstack.cs_idx] & CSF_FOR) else if (cstack.cs_flags[cstack.cs_idx] & CSF_FOR)
emsg(_(e_endfor)); emsg(_(e_missing_endfor));
else else
emsg(_(e_endif)); emsg(_(e_missing_endif));
} }
/* /*
@ -5561,8 +5561,8 @@ check_more(
return FAIL; return FAIL;
} }
#endif #endif
semsg(NGETTEXT("E173: %d more file to edit", semsg(NGETTEXT(e_nr_more_file_to_edit,
"E173: %d more files to edit", n), n); e_nr_more_files_to_edit , n), n);
quitmore = 2; // next try to quit is allowed quitmore = 2; // next try to quit is allowed
} }
return FAIL; return FAIL;
@ -5609,7 +5609,7 @@ ex_colorscheme(exarg_T *eap)
#endif #endif
} }
else if (load_colors(eap->arg) == FAIL) else if (load_colors(eap->arg) == FAIL)
semsg(_("E185: Cannot find color scheme '%s'"), eap->arg); semsg(_(e_cannot_find_color_scheme_str), eap->arg);
#ifdef FEAT_VTP #ifdef FEAT_VTP
else if (has_vtp_working()) else if (has_vtp_working())
@ -7377,7 +7377,7 @@ changedir_func(
pdir = get_prevdir(scope); pdir = get_prevdir(scope);
if (pdir == NULL) if (pdir == NULL)
{ {
emsg(_("E186: No previous directory")); emsg(_(e_no_previous_directory));
return FALSE; return FALSE;
} }
new_dir = pdir; new_dir = pdir;
@ -7510,7 +7510,7 @@ ex_pwd(exarg_T *eap UNUSED)
msg((char *)NameBuff); msg((char *)NameBuff);
} }
else else
emsg(_("E187: Unknown")); emsg(_(e_directory_unknown));
} }
/* /*
@ -7716,7 +7716,7 @@ ex_winpos(exarg_T *eap)
} }
else else
# endif # endif
emsg(_("E188: Obtaining window position not implemented for this platform")); emsg(_(e_obtaining_window_position_not_implemented_for_this_platform));
} }
else else
{ {
@ -8303,12 +8303,12 @@ open_exfile(
#endif #endif
if (!forceit && *mode != 'a' && vim_fexists(fname)) if (!forceit && *mode != 'a' && vim_fexists(fname))
{ {
semsg(_("E189: \"%s\" exists (add ! to override)"), fname); semsg(_(e_str_exists_add_bang_to_override), fname);
return NULL; return NULL;
} }
if ((fd = mch_fopen((char *)fname, mode)) == NULL) if ((fd = mch_fopen((char *)fname, mode)) == NULL)
semsg(_("E190: Cannot open \"%s\" for writing"), fname); semsg(_(e_cannot_open_str_for_writing_2), fname);
return fd; return fd;
} }

8
src/ex_eval.c
View File

@ -1413,7 +1413,7 @@ ex_endwhile(exarg_T *eap)
if (!(fl & (CSF_WHILE | CSF_FOR))) if (!(fl & (CSF_WHILE | CSF_FOR)))
{ {
if (!(fl & CSF_TRY)) if (!(fl & CSF_TRY))
eap->errmsg = _(e_endif); eap->errmsg = _(e_missing_endif);
else if (fl & CSF_FINALLY) else if (fl & CSF_FINALLY)
eap->errmsg = _(e_endtry); eap->errmsg = _(e_endtry);
// Try to find the matching ":while" and report what's missing. // Try to find the matching ":while" and report what's missing.
@ -2484,10 +2484,10 @@ cleanup_conditionals(
get_end_emsg(cstack_T *cstack) get_end_emsg(cstack_T *cstack)
{ {
if (cstack->cs_flags[cstack->cs_idx] & CSF_WHILE) if (cstack->cs_flags[cstack->cs_idx] & CSF_WHILE)
return _(e_endwhile); return _(e_missing_endwhile);
if (cstack->cs_flags[cstack->cs_idx] & CSF_FOR) if (cstack->cs_flags[cstack->cs_idx] & CSF_FOR)
return _(e_endfor); return _(e_missing_endfor);
return _(e_endif); return _(e_missing_endif);
} }

3
src/globals.h
View File

@ -1606,14 +1606,11 @@ EXTERN int netbeansSuppressNoLines INIT(= 0); // skip "No lines in buffer"
EXTERN char e_abort[] INIT(= N_("E470: Command aborted")); EXTERN char e_abort[] INIT(= N_("E470: Command aborted"));
EXTERN char e_argreq[] INIT(= N_("E471: Argument required")); EXTERN char e_argreq[] INIT(= N_("E471: Argument required"));
#ifdef FEAT_EVAL #ifdef FEAT_EVAL
EXTERN char e_endif[] INIT(= N_("E171: Missing :endif"));
EXTERN char e_catch[] INIT(= N_("E603: :catch without :try")); EXTERN char e_catch[] INIT(= N_("E603: :catch without :try"));
EXTERN char e_finally[] INIT(= N_("E606: :finally without :try")); EXTERN char e_finally[] INIT(= N_("E606: :finally without :try"));
EXTERN char e_finally_dup[] INIT(= N_("E607: multiple :finally")); EXTERN char e_finally_dup[] INIT(= N_("E607: multiple :finally"));
EXTERN char e_endtry[] INIT(= N_("E600: Missing :endtry")); EXTERN char e_endtry[] INIT(= N_("E600: Missing :endtry"));
EXTERN char e_no_endtry[] INIT(= N_("E602: :endtry without :try")); EXTERN char e_no_endtry[] INIT(= N_("E602: :endtry without :try"));
EXTERN char e_endwhile[] INIT(= N_("E170: Missing :endwhile"));
EXTERN char e_endfor[] INIT(= N_("E170: Missing :endfor"));
EXTERN char e_while[] INIT(= N_("E588: :endwhile without :while")); EXTERN char e_while[] INIT(= N_("E588: :endwhile without :while"));
EXTERN char e_for[] INIT(= N_("E588: :endfor without :for")); EXTERN char e_for[] INIT(= N_("E588: :endfor without :for"));
#endif #endif

2
src/help.c
View File

@ -995,7 +995,7 @@ helptags_one(
if (fd_tags == NULL) if (fd_tags == NULL)
{ {
if (!ignore_writeerr) if (!ignore_writeerr)
semsg(_(e_cannot_open_str_for_writing), NameBuff); semsg(_(e_cannot_open_str_for_writing_1), NameBuff);
FreeWild(filecount, files); FreeWild(filecount, files);
return; return;
} }

4
src/scriptfile.c
View File

@ -1919,7 +1919,7 @@ ex_scriptencoding(exarg_T *eap)
if (!getline_equal(eap->getline, eap->cookie, getsourceline)) if (!getline_equal(eap->getline, eap->cookie, getsourceline))
{ {
emsg(_("E167: :scriptencoding used outside of a sourced file")); emsg(_(e_scriptencoding_used_outside_of_sourced_file));
return; return;
} }
@ -1983,7 +1983,7 @@ ex_finish(exarg_T *eap)
if (getline_equal(eap->getline, eap->cookie, getsourceline)) if (getline_equal(eap->getline, eap->cookie, getsourceline))
do_finish(eap, FALSE); do_finish(eap, FALSE);
else else
emsg(_("E168: :finish used outside of a sourced file")); emsg(_(e_finish_used_outside_of_sourced_file));
} }
/* /*

22
src/usercmd.c
View File

@ -678,7 +678,7 @@ parse_compl_arg(
if (command_complete[i].expand == 0) if (command_complete[i].expand == 0)
{ {
semsg(_("E180: Invalid complete value: %s"), value); semsg(_(e_invalid_complete_value_str), value);
return FAIL; return FAIL;
} }
@ -726,7 +726,7 @@ uc_scan_attr(
if (len == 0) if (len == 0)
{ {
emsg(_("E175: No attribute specified")); emsg(_(e_no_attribute_specified));
return FAIL; return FAIL;
} }
@ -781,7 +781,7 @@ uc_scan_attr(
else else
{ {
wrong_nargs: wrong_nargs:
emsg(_("E176: Invalid number of arguments")); emsg(_(e_invalid_number_of_arguments));
return FAIL; return FAIL;
} }
} }
@ -796,7 +796,7 @@ wrong_nargs:
if (*def >= 0) if (*def >= 0)
{ {
two_count: two_count:
emsg(_("E177: Count cannot be specified twice")); emsg(_(e_count_cannot_be_specified_twice));
return FAIL; return FAIL;
} }
@ -806,7 +806,7 @@ two_count:
if (p != val + vallen || vallen == 0) if (p != val + vallen || vallen == 0)
{ {
invalid_count: invalid_count:
emsg(_("E178: Invalid default value for count")); emsg(_(e_invalid_default_value_for_count));
return FAIL; return FAIL;
} }
} }
@ -840,7 +840,7 @@ invalid_count:
{ {
if (val == NULL) if (val == NULL)
{ {
emsg(_("E179: argument required for -complete")); semsg(_(e_argument_required_for_str), "-complete");
return FAIL; return FAIL;
} }
@ -853,7 +853,7 @@ invalid_count:
*argt |= EX_RANGE; *argt |= EX_RANGE;
if (val == NULL) if (val == NULL)
{ {
emsg(_("E179: argument required for -addr")); semsg(_(e_argument_required_for_str), "-addr");
return FAIL; return FAIL;
} }
if (parse_addr_type_arg(val, (int)vallen, addr_type_arg) == FAIL) if (parse_addr_type_arg(val, (int)vallen, addr_type_arg) == FAIL)
@ -865,7 +865,7 @@ invalid_count:
{ {
char_u ch = attr[len]; char_u ch = attr[len];
attr[len] = '\0'; attr[len] = '\0';
semsg(_("E181: Invalid attribute: %s"), attr); semsg(_(e_invalid_attribute_str), attr);
attr[len] = ch; attr[len] = ch;
return FAIL; return FAIL;
} }
@ -945,7 +945,7 @@ uc_add_command(
#endif #endif
) )
{ {
semsg(_("E174: Command already exists: add ! to replace it: %s"), semsg(_(e_command_already_exists_add_bang_to_replace_it_str),
name); name);
goto fail; goto fail;
} }
@ -1085,7 +1085,7 @@ ex_command(exarg_T *eap)
++p; ++p;
if (!ends_excmd2(eap->arg, p) && !VIM_ISWHITE(*p)) if (!ends_excmd2(eap->arg, p) && !VIM_ISWHITE(*p))
{ {
emsg(_("E182: Invalid command name")); emsg(_(e_invalid_command_name));
return; return;
} }
end = p; end = p;
@ -1097,7 +1097,7 @@ ex_command(exarg_T *eap)
if (!has_attr && ends_excmd2(eap->arg, p)) if (!has_attr && ends_excmd2(eap->arg, p))
uc_list(name, end - name); uc_list(name, end - name);
else if (!ASCII_ISUPPER(*name)) else if (!ASCII_ISUPPER(*name))
emsg(_("E183: User defined commands must start with an uppercase letter")); emsg(_(e_user_defined_commands_must_start_with_an_uppercase_letter));
else if ((name_len == 1 && *name == 'X') else if ((name_len == 1 && *name == 'X')
|| (name_len <= 4 || (name_len <= 4
&& STRNCMP(name, "Next", name_len > 4 ? 4 : name_len) == 0)) && STRNCMP(name, "Next", name_len > 4 ? 4 : name_len) == 0))

6
src/userfunc.c
View File

@ -2360,7 +2360,7 @@ copy_func(char_u *lambda, char_u *global, ectx_T *ectx)
if (fp != NULL) if (fp != NULL)
{ {
// TODO: handle ! to overwrite // TODO: handle ! to overwrite
semsg(_(e_function_str_already_exists_add_excl_to_replace), global); semsg(_(e_function_str_already_exists_add_bang_to_replace), global);
return FAIL; return FAIL;
} }
@ -4380,7 +4380,7 @@ define_function(exarg_T *eap, char_u *name_arg, char_u **line_to_free)
if (fudi.fd_dict != NULL && fudi.fd_newkey == NULL) if (fudi.fd_dict != NULL && fudi.fd_newkey == NULL)
emsg(_(e_funcdict)); emsg(_(e_funcdict));
else if (name != NULL && find_func(name, is_global, NULL) != NULL) else if (name != NULL && find_func(name, is_global, NULL) != NULL)
emsg_funcname(e_function_str_already_exists_add_excl_to_replace, name); emsg_funcname(e_function_str_already_exists_add_bang_to_replace, name);
} }
if (!eap->skip && did_emsg) if (!eap->skip && did_emsg)
@ -4438,7 +4438,7 @@ define_function(exarg_T *eap, char_u *name_arg, char_u **line_to_free)
if (vim9script) if (vim9script)
emsg_funcname(e_name_already_defined_str, name); emsg_funcname(e_name_already_defined_str, name);
else else
emsg_funcname(e_function_str_already_exists_add_excl_to_replace, name); emsg_funcname(e_function_str_already_exists_add_bang_to_replace, name);
goto erret; goto erret;
} }
if (fp->uf_calls > 0) if (fp->uf_calls > 0)

2
src/version.c
View File

@ -749,6 +749,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
3957,
/**/ /**/
3956, 3956,
/**/ /**/

6
src/vim9cmds.c
View File

@ -1520,11 +1520,11 @@ compile_endtry(char_u *arg, cctx_T *cctx)
if (scope == NULL) if (scope == NULL)
emsg(_(e_no_endtry)); emsg(_(e_no_endtry));
else if (scope->se_type == WHILE_SCOPE) else if (scope->se_type == WHILE_SCOPE)
emsg(_(e_endwhile)); emsg(_(e_missing_endwhile));
else if (scope->se_type == FOR_SCOPE) else if (scope->se_type == FOR_SCOPE)
emsg(_(e_endfor)); emsg(_(e_missing_endfor));
else else
emsg(_(e_endif)); emsg(_(e_missing_endif));
return NULL; return NULL;
} }

6
src/vim9compile.c
View File

@ -3109,11 +3109,11 @@ nextline:
if (cctx.ctx_scope != NULL) if (cctx.ctx_scope != NULL)
{ {
if (cctx.ctx_scope->se_type == IF_SCOPE) if (cctx.ctx_scope->se_type == IF_SCOPE)
emsg(_(e_endif)); emsg(_(e_missing_endif));
else if (cctx.ctx_scope->se_type == WHILE_SCOPE) else if (cctx.ctx_scope->se_type == WHILE_SCOPE)
emsg(_(e_endwhile)); emsg(_(e_missing_endwhile));
else if (cctx.ctx_scope->se_type == FOR_SCOPE) else if (cctx.ctx_scope->se_type == FOR_SCOPE)
emsg(_(e_endfor)); emsg(_(e_missing_endfor));
else else
emsg(_(e_missing_rcurly)); emsg(_(e_missing_rcurly));
goto erret; goto erret;