Future/bison
1
0
Fork 0
mirror of https://git.savannah.gnu.org/git/bison.git synced 2026-03-17 08:13:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

warnings: refactoring

Browse Source 
The code here was too confusing, this seems more natural.

* src/complain.c (error_message): Move the indentation check and the category
output to complains. Also, no longer take a 'warnings' argument.
(complains): Factor calls to error_message.

Signed-off-by: Akim Demaille <akim@lrde.epita.fr>
This commit is contained in:
Theophile Ranquet
2012-10-01 15:01:02 +00:00
committed by Akim Demaille
parent 46b7d74cb5
commit a686f6cdc0
1 changed files with 11 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
src/complain.c
View File

@@ -84,7 +84,6 @@ void
error_message (const location *loc, warnings flags, const char *prefix, error_message (const location *loc, warnings flags, const char *prefix,
const char *message, va_list args) const char *message, va_list args)
{ {
(void) flags;
unsigned pos = 0; unsigned pos = 0;
if (loc) if (loc)
@@ -95,6 +94,8 @@ error_message (const location *loc, warnings flags, const char *prefix,
if (indent_ptr) if (indent_ptr)
{ {
if (*indent_ptr)
prefix = NULL;
if (!*indent_ptr) if (!*indent_ptr)
*indent_ptr = pos; *indent_ptr = pos;
else if (*indent_ptr > pos) else if (*indent_ptr > pos)
@@ -125,21 +126,16 @@ complains (const location *loc, warnings flags, const char *message,
va_list args) va_list args)
{ {
const char* prefix =
flags & fatal ? _("fatal error")
: flags & (errors_flag | complaint) ? _("error")
: _("warning");
complaint_issued |= flags & (complaint | errors_flag);
if (flags & (warnings_flag | silent | fatal | complaint))
error_message (loc, flags, prefix, message, args);
if (flags & fatal) if (flags & fatal)
{ exit (EXIT_FAILURE);
error_message (loc, fatal, _("fatal error"), message, args);
exit (EXIT_FAILURE);
}
else if (flags & (complaint | warnings_flag | silent))
{
const char* prefix =
flags & (errors_flag | complaint) ? _("error") : _("warning");
if (flags & (complaint | errors_flag))
complaint_issued = true;
error_message (loc, flags,
indent_ptr && *indent_ptr ? NULL : prefix,
message, args);
}
} }
void void