diagnostic.c (emit_diagnostic): Move va_end call after user of the va_list.

* diagnostic.c (emit_diagnostic): Move va_end call after user
	of the va_list.
	(warning, warning_at, pedwarn, permerror): Ditto.

From-SVN: r186591
This commit is contained in:
Michael Matz 2012-04-19 12:51:25 +00:00 committed by Michael Matz
parent d0a854af24
commit 80f2435130
2 changed files with 22 additions and 6 deletions

View File

@ -1,3 +1,9 @@
2012-04-19 Michael Matz <matz@suse.de>
* diagnostic.c (emit_diagnostic): Move va_end call after user
of the va_list.
(warning, warning_at, pedwarn, permerror): Ditto.
2012-04-19 Richard Guenther <rguenther@suse.de>
* ira-int.h (ira_allocno_object_iter_cond): Avoid out-of-bound

View File

@ -714,6 +714,7 @@ emit_diagnostic (diagnostic_t kind, location_t location, int opt,
{
diagnostic_info diagnostic;
va_list ap;
bool ret;
va_start (ap, gmsgid);
if (kind == DK_PERMERROR)
@ -727,9 +728,10 @@ emit_diagnostic (diagnostic_t kind, location_t location, int opt,
if (kind == DK_WARNING || kind == DK_PEDWARN)
diagnostic.option_index = opt;
}
va_end (ap);
return report_diagnostic (&diagnostic);
ret = report_diagnostic (&diagnostic);
va_end (ap);
return ret;
}
/* An informative note at LOCATION. Use this for additional details on an error
@ -771,13 +773,15 @@ warning (int opt, const char *gmsgid, ...)
{
diagnostic_info diagnostic;
va_list ap;
bool ret;
va_start (ap, gmsgid);
diagnostic_set_info (&diagnostic, gmsgid, &ap, input_location, DK_WARNING);
diagnostic.option_index = opt;
ret = report_diagnostic (&diagnostic);
va_end (ap);
return report_diagnostic (&diagnostic);
return ret;
}
/* A warning at LOCATION. Use this for code which is correct according to the
@ -789,12 +793,14 @@ warning_at (location_t location, int opt, const char *gmsgid, ...)
{
diagnostic_info diagnostic;
va_list ap;
bool ret;
va_start (ap, gmsgid);
diagnostic_set_info (&diagnostic, gmsgid, &ap, location, DK_WARNING);
diagnostic.option_index = opt;
ret = report_diagnostic (&diagnostic);
va_end (ap);
return report_diagnostic (&diagnostic);
return ret;
}
/* A "pedantic" warning at LOCATION: issues a warning unless
@ -815,12 +821,14 @@ pedwarn (location_t location, int opt, const char *gmsgid, ...)
{
diagnostic_info diagnostic;
va_list ap;
bool ret;
va_start (ap, gmsgid);
diagnostic_set_info (&diagnostic, gmsgid, &ap, location, DK_PEDWARN);
diagnostic.option_index = opt;
ret = report_diagnostic (&diagnostic);
va_end (ap);
return report_diagnostic (&diagnostic);
return ret;
}
/* A "permissive" error at LOCATION: issues an error unless
@ -835,13 +843,15 @@ permerror (location_t location, const char *gmsgid, ...)
{
diagnostic_info diagnostic;
va_list ap;
bool ret;
va_start (ap, gmsgid);
diagnostic_set_info (&diagnostic, gmsgid, &ap, location,
permissive_error_kind (global_dc));
diagnostic.option_index = permissive_error_option (global_dc);
ret = report_diagnostic (&diagnostic);
va_end (ap);
return report_diagnostic (&diagnostic);
return ret;
}
/* A hard error: the code is definitely ill-formed, and an object file