Skip to content

Commit 23e7b70

Browse files
committed
diagnostics: fix overload of emit_diagnostic [PR117109]
I accidentally broke "make gcc.pot" in r15-4081 by adding a member function diagnostic_context::emit_diagnostic with a gmsgid in a different position to the existing emit_diagnostic functions, which exgettext's parser can't handle. Fixed thusly. gcc/ChangeLog: PR bootstrap/117109 * diagnostic-format-sarif.cc (diagnostic_output_format_init_sarif_file): Rename diagnostic_context::emit_diagnostic to diagnostic_context::emit_diagnostic_with_group. * diagnostic.cc (diagnostic_context::emit_diagnostic): Rename to... (diagnostic_context::emit_diagnostic_with_group): ...this. (diagnostic_context::emit_diagnostic_va): Rename to... (diagnostic_context::emit_diagnostic_with_group_va): ...this. * diagnostic.h (diagnostic_context::emit_diagnostic): Rename to... (diagnostic_context::emit_diagnostic_with_group): ...this. (diagnostic_context::emit_diagnostic_va): Rename to... (diagnostic_context::emit_diagnostic_with_group_va): ...this. Signed-off-by: David Malcolm <dmalcolm@redhat.com>
1 parent 2c08ddd commit 23e7b70

File tree

3 files changed

+31
-27
lines changed

3 files changed

+31
-27
lines changed

gcc/diagnostic-format-sarif.cc

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3435,8 +3435,9 @@ diagnostic_output_format_init_sarif_file (diagnostic_context &context,
34353435
if (!base_file_name)
34363436
{
34373437
rich_location richloc (line_maps, UNKNOWN_LOCATION);
3438-
context.emit_diagnostic (DK_ERROR, richloc, nullptr, 0,
3439-
"unable to determine filename for SARIF output");
3438+
context.emit_diagnostic_with_group
3439+
(DK_ERROR, richloc, nullptr, 0,
3440+
"unable to determine filename for SARIF output");
34403441
return;
34413442
}
34423443

@@ -3447,9 +3448,10 @@ diagnostic_output_format_init_sarif_file (diagnostic_context &context,
34473448
if (!outf)
34483449
{
34493450
rich_location richloc (line_maps, UNKNOWN_LOCATION);
3450-
context.emit_diagnostic (DK_ERROR, richloc, nullptr, 0,
3451-
"unable to open %qs for SARIF output: %m",
3452-
filename.get ());
3451+
context.emit_diagnostic_with_group
3452+
(DK_ERROR, richloc, nullptr, 0,
3453+
"unable to open %qs for SARIF output: %m",
3454+
filename.get ());
34533455
return;
34543456
}
34553457
diagnostic_output_file output_file (outf, true, std::move (filename));

gcc/diagnostic.cc

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1160,18 +1160,19 @@ diagnostic_context::warning_enabled_at (location_t loc,
11601160
/* Emit a diagnostic within a diagnostic group on this context. */
11611161

11621162
bool
1163-
diagnostic_context::emit_diagnostic (diagnostic_t kind,
1164-
rich_location &richloc,
1165-
const diagnostic_metadata *metadata,
1166-
diagnostic_option_id option_id,
1167-
const char *gmsgid, ...)
1163+
diagnostic_context::
1164+
emit_diagnostic_with_group (diagnostic_t kind,
1165+
rich_location &richloc,
1166+
const diagnostic_metadata *metadata,
1167+
diagnostic_option_id option_id,
1168+
const char *gmsgid, ...)
11681169
{
11691170
begin_group ();
11701171

11711172
va_list ap;
11721173
va_start (ap, gmsgid);
1173-
bool ret = emit_diagnostic_va (kind, richloc, metadata, option_id,
1174-
gmsgid, &ap);
1174+
bool ret = emit_diagnostic_with_group_va (kind, richloc, metadata, option_id,
1175+
gmsgid, &ap);
11751176
va_end (ap);
11761177

11771178
end_group ();
@@ -1182,11 +1183,12 @@ diagnostic_context::emit_diagnostic (diagnostic_t kind,
11821183
/* As above, but taking a va_list *. */
11831184

11841185
bool
1185-
diagnostic_context::emit_diagnostic_va (diagnostic_t kind,
1186-
rich_location &richloc,
1187-
const diagnostic_metadata *metadata,
1188-
diagnostic_option_id option_id,
1189-
const char *gmsgid, va_list *ap)
1186+
diagnostic_context::
1187+
emit_diagnostic_with_group_va (diagnostic_t kind,
1188+
rich_location &richloc,
1189+
const diagnostic_metadata *metadata,
1190+
diagnostic_option_id option_id,
1191+
const char *gmsgid, va_list *ap)
11901192
{
11911193
begin_group ();
11921194

gcc/diagnostic.h

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -505,17 +505,17 @@ class diagnostic_context
505505
return m_option_classifier.option_unspecified_p (option_id);
506506
}
507507

508-
bool emit_diagnostic (diagnostic_t kind,
509-
rich_location &richloc,
510-
const diagnostic_metadata *metadata,
511-
diagnostic_option_id option_id,
512-
const char *gmsgid, ...)
508+
bool emit_diagnostic_with_group (diagnostic_t kind,
509+
rich_location &richloc,
510+
const diagnostic_metadata *metadata,
511+
diagnostic_option_id option_id,
512+
const char *gmsgid, ...)
513513
ATTRIBUTE_GCC_DIAG(6,7);
514-
bool emit_diagnostic_va (diagnostic_t kind,
515-
rich_location &richloc,
516-
const diagnostic_metadata *metadata,
517-
diagnostic_option_id option_id,
518-
const char *gmsgid, va_list *ap)
514+
bool emit_diagnostic_with_group_va (diagnostic_t kind,
515+
rich_location &richloc,
516+
const diagnostic_metadata *metadata,
517+
diagnostic_option_id option_id,
518+
const char *gmsgid, va_list *ap)
519519
ATTRIBUTE_GCC_DIAG(6,0);
520520

521521
bool report_diagnostic (diagnostic_info *);

0 commit comments

Comments
 (0)