patch 8.1.0743: giving error messages is not flexible
Problem: Giving error messages is not flexible.
Solution: Add semsg(). Change argument from "char_u *" to "char *", also
for msg() and get rid of most MSG macros. (Ozaki Kiichi, closes
#3302) Also make emsg() accept a "char *" argument. Get rid of
an enormous number of type casts.
diff --git a/src/undo.c b/src/undo.c
index 0cb07a3..5dafee8 100644
--- a/src/undo.c
+++ b/src/undo.c
@@ -158,30 +158,30 @@
++header_count;
if (uhp == curbuf->b_u_curhead && ++seen_b_u_curhead > 1)
{
- EMSG("b_u_curhead found twice (looping?)");
+ emsg("b_u_curhead found twice (looping?)");
return;
}
if (uhp == curbuf->b_u_newhead && ++seen_b_u_newhead > 1)
{
- EMSG("b_u_newhead found twice (looping?)");
+ emsg("b_u_newhead found twice (looping?)");
return;
}
if (uhp->uh_magic != UH_MAGIC)
- EMSG("uh_magic wrong (may be using freed memory)");
+ emsg("uh_magic wrong (may be using freed memory)");
else
{
/* Check pointers back are correct. */
if (uhp->uh_next.ptr != exp_uh_next)
{
- EMSG("uh_next wrong");
- smsg((char_u *)"expected: 0x%x, actual: 0x%x",
+ emsg("uh_next wrong");
+ smsg("expected: 0x%x, actual: 0x%x",
exp_uh_next, uhp->uh_next.ptr);
}
if (uhp->uh_alt_prev.ptr != exp_uh_alt_prev)
{
- EMSG("uh_alt_prev wrong");
- smsg((char_u *)"expected: 0x%x, actual: 0x%x",
+ emsg("uh_alt_prev wrong");
+ smsg("expected: 0x%x, actual: 0x%x",
exp_uh_alt_prev, uhp->uh_alt_prev.ptr);
}
@@ -190,7 +190,7 @@
{
if (uep->ue_magic != UE_MAGIC)
{
- EMSG("ue_magic wrong (may be using freed memory)");
+ emsg("ue_magic wrong (may be using freed memory)");
break;
}
}
@@ -214,13 +214,13 @@
if (seen_b_u_newhead == 0 && curbuf->b_u_oldhead != NULL
&& !(newhead_may_be_NULL && curbuf->b_u_newhead == NULL))
- EMSGN("b_u_newhead invalid: 0x%x", curbuf->b_u_newhead);
+ semsg("b_u_newhead invalid: 0x%x", curbuf->b_u_newhead);
if (curbuf->b_u_curhead != NULL && seen_b_u_curhead == 0)
- EMSGN("b_u_curhead invalid: 0x%x", curbuf->b_u_curhead);
+ semsg("b_u_curhead invalid: 0x%x", curbuf->b_u_curhead);
if (header_count != curbuf->b_u_numhead)
{
- EMSG("b_u_numhead invalid");
- smsg((char_u *)"expected: %ld, actual: %ld",
+ emsg("b_u_numhead invalid");
+ smsg("expected: %ld, actual: %ld",
(long)header_count, (long)curbuf->b_u_numhead);
}
}
@@ -316,7 +316,7 @@
/* Don't allow changes when 'modifiable' is off. */
if (!curbuf->b_p_ma)
{
- EMSG(_(e_modifiable));
+ emsg(_(e_modifiable));
return FALSE;
}
@@ -324,7 +324,7 @@
/* In the sandbox it's not allowed to change the text. */
if (sandbox != 0)
{
- EMSG(_(e_sandbox));
+ emsg(_(e_sandbox));
return FALSE;
}
#endif
@@ -333,7 +333,7 @@
* caller of getcmdline() may get confused. */
if (textlock != 0)
{
- EMSG(_(e_secure));
+ emsg(_(e_secure));
return FALSE;
}
@@ -413,12 +413,12 @@
{
if (netbeans_is_guarded(top, bot))
{
- EMSG(_(e_guarded));
+ emsg(_(e_guarded));
return FAIL;
}
if (curbuf->b_p_ro)
{
- EMSG(_(e_nbreadonly));
+ emsg(_(e_nbreadonly));
return FAIL;
}
}
@@ -439,7 +439,7 @@
{
/* This happens when the FileChangedRO autocommand changes the
* file in a way it becomes shorter. */
- EMSG(_("E881: Line count changed unexpectedly"));
+ emsg(_("E881: Line count changed unexpectedly"));
return FAIL;
}
}
@@ -861,7 +861,7 @@
static void
corruption_error(char *mesg, char_u *file_name)
{
- EMSG3(_("E825: Corrupted undo file (%s): %s"), mesg, file_name);
+ semsg(_("E825: Corrupted undo file (%s): %s"), mesg, file_name);
}
static void
@@ -1553,7 +1553,7 @@
if (p_verbose > 0)
{
verbose_enter();
- smsg((char_u *)
+ smsg(
_("Cannot write undo file in any directory in 'undodir'"));
verbose_leave();
}
@@ -1601,7 +1601,7 @@
{
if (name == NULL)
verbose_enter();
- smsg((char_u *)
+ smsg(
_("Will not overwrite with undo file, cannot read: %s"),
file_name);
if (name == NULL)
@@ -1623,7 +1623,7 @@
{
if (name == NULL)
verbose_enter();
- smsg((char_u *)
+ smsg(
_("Will not overwrite, this is not an undo file: %s"),
file_name);
if (name == NULL)
@@ -1649,14 +1649,14 @@
O_CREAT|O_EXTRA|O_WRONLY|O_EXCL|O_NOFOLLOW, perm);
if (fd < 0)
{
- EMSG2(_(e_not_open), file_name);
+ semsg(_(e_not_open), file_name);
goto theend;
}
(void)mch_setperm(file_name, perm);
if (p_verbose > 0)
{
verbose_enter();
- smsg((char_u *)_("Writing undo file: %s"), file_name);
+ smsg(_("Writing undo file: %s"), file_name);
verbose_leave();
}
@@ -1688,7 +1688,7 @@
fp = fdopen(fd, "w");
if (fp == NULL)
{
- EMSG2(_(e_not_open), file_name);
+ semsg(_(e_not_open), file_name);
close(fd);
mch_remove(file_name);
goto theend;
@@ -1743,8 +1743,8 @@
#ifdef U_DEBUG
if (headers_written != buf->b_u_numhead)
{
- EMSGN("Written %ld headers, ...", headers_written);
- EMSGN("... but numhead is %ld", buf->b_u_numhead);
+ semsg("Written %ld headers, ...", headers_written);
+ semsg("... but numhead is %ld", buf->b_u_numhead);
}
#endif
@@ -1756,7 +1756,7 @@
write_error:
fclose(fp);
if (!write_ok)
- EMSG2(_("E829: write error in undo file: %s"), file_name);
+ semsg(_("E829: write error in undo file: %s"), file_name);
#if defined(WIN3264)
/* Copy file attributes; for systems where this can only be done after
@@ -1846,7 +1846,7 @@
if (p_verbose > 0)
{
verbose_enter();
- smsg((char_u *)_("Not reading undo file, owner differs: %s"),
+ smsg(_("Not reading undo file, owner differs: %s"),
file_name);
verbose_leave();
}
@@ -1860,7 +1860,7 @@
if (p_verbose > 0)
{
verbose_enter();
- smsg((char_u *)_("Reading undo file: %s"), file_name);
+ smsg(_("Reading undo file: %s"), file_name);
verbose_leave();
}
@@ -1868,7 +1868,7 @@
if (fp == NULL)
{
if (name != NULL || p_verbose > 0)
- EMSG2(_("E822: Cannot open undo file for reading: %s"), file_name);
+ semsg(_("E822: Cannot open undo file for reading: %s"), file_name);
goto error;
}
bi.bi_buf = curbuf;
@@ -1880,7 +1880,7 @@
if (fread(magic_buf, UF_START_MAGIC_LEN, 1, fp) != 1
|| memcmp(magic_buf, UF_START_MAGIC, UF_START_MAGIC_LEN) != 0)
{
- EMSG2(_("E823: Not an undo file: %s"), file_name);
+ semsg(_("E823: Not an undo file: %s"), file_name);
goto error;
}
version = get2c(fp);
@@ -1889,14 +1889,14 @@
#ifdef FEAT_CRYPT
if (*curbuf->b_p_key == NUL)
{
- EMSG2(_("E832: Non-encrypted file has encrypted undo file: %s"),
+ semsg(_("E832: Non-encrypted file has encrypted undo file: %s"),
file_name);
goto error;
}
bi.bi_state = crypt_create_from_file(fp, curbuf->b_p_key);
if (bi.bi_state == NULL)
{
- EMSG2(_("E826: Undo file decryption failed: %s"), file_name);
+ semsg(_("E826: Undo file decryption failed: %s"), file_name);
goto error;
}
if (crypt_whole_undofile(bi.bi_state->method_nr))
@@ -1912,13 +1912,13 @@
bi.bi_used = 0;
}
#else
- EMSG2(_("E827: Undo file is encrypted: %s"), file_name);
+ semsg(_("E827: Undo file is encrypted: %s"), file_name);
goto error;
#endif
}
else if (version != UF_VERSION)
{
- EMSG2(_("E824: Incompatible undo file: %s"), file_name);
+ semsg(_("E824: Incompatible undo file: %s"), file_name);
goto error;
}
@@ -2122,13 +2122,13 @@
#ifdef U_DEBUG
for (i = 0; i < num_head; ++i)
if (uhp_table_used[i] == 0)
- EMSGN("uhp_table entry %ld not used, leaking memory", i);
+ semsg("uhp_table entry %ld not used, leaking memory", i);
vim_free(uhp_table_used);
u_check(TRUE);
#endif
if (name != NULL)
- smsg((char_u *)_("Finished reading undo file %s"), file_name);
+ smsg(_("Finished reading undo file %s"), file_name);
goto theend;
error:
@@ -2488,7 +2488,7 @@
if (absolute)
{
- EMSGN(_("E830: Undo number %ld not found"), step);
+ semsg(_("E830: Undo number %ld not found"), step);
return;
}
@@ -2678,7 +2678,7 @@
|| bot > curbuf->b_ml.ml_line_count + 1)
{
unblock_autocmds();
- IEMSG(_("E438: u_undo: line numbers wrong"));
+ iemsg(_("E438: u_undo: line numbers wrong"));
changed(); /* don't want UNCHANGED now */
return;
}
@@ -2988,7 +2988,7 @@
}
#endif
- smsg_attr_keep(0, (char_u *)_("%ld %s; %s #%ld %s"),
+ smsg_attr_keep(0, _("%ld %s; %s #%ld %s"),
u_oldcount < 0 ? -u_oldcount : u_oldcount,
_(msgstr),
did_undo ? _("before") : _("after"),
@@ -3165,7 +3165,7 @@
return; /* nothing changed before */
if (curbuf->b_u_curhead != NULL)
{
- EMSG(_("E790: undojoin is not allowed after undo"));
+ emsg(_("E790: undojoin is not allowed after undo"));
return;
}
if (!curbuf->b_u_synced)
@@ -3270,7 +3270,7 @@
{
if (curbuf->b_u_newhead == NULL || curbuf->b_u_newhead->uh_entry == NULL)
{
- IEMSG(_("E439: undo list corrupt"));
+ iemsg(_("E439: undo list corrupt"));
return NULL;
}
return curbuf->b_u_newhead->uh_entry;
@@ -3302,7 +3302,7 @@
uep->ue_bot = uep->ue_top + uep->ue_size + 1 + extra;
if (uep->ue_bot < 1 || uep->ue_bot > curbuf->b_ml.ml_line_count)
{
- IEMSG(_("E440: undo line missing"));
+ iemsg(_("E440: undo line missing"));
uep->ue_bot = uep->ue_top + 1; /* assume all lines deleted, will
* get all the old lines back
* without deleting the current