Fix small colorization omission

Submitted by Jakub Jelinek on April 12, 2013, 9:37 a.m.

Details

Message ID 20130412093704.GB16463@tucnak.redhat.com
State New
Headers show

Commit Message

Jakub Jelinek April 12, 2013, 9:37 a.m.
Hi!

While writing the HTML sample for changes.html, I've noticed one
omission, while when we start lines with test.C:1 or test.C:1:28
or similar (i.e. diagnostic_build_prefix created prefix), it is colorized
with locus color, if we start it just with test.C (filename only),
we don't.  The following patch fixes that, and attached is also
corresponding changes.html adjustment.

Ok for trunk?

2013-04-12  Jakub Jelinek  <jakub@redhat.com>

	* diagnostic.h (file_name_as_prefix): Add context argument.
	* diagnostic.c (file_name_as_prefix): Likewise.  Colorize
	the string as locus.
	* langhooks.c (lhd_print_error_function): Adjust caller.
cp/
	* error.c (cp_print_error_function): Adjust file_name_as_prefix
	caller.


	Jakub
--- htdocs/gcc-4.9/changes.html	2013-04-12 09:10:29.655357028 +0200
+++ htdocs/gcc-4.9/changes.html	2013-04-12 11:33:43.336660425 +0200
@@ -42,7 +42,7 @@
     Sample diagnostics output:<br/>
     <pre>
     $ g++ -fdiagnostics-color=always -S -Wall test.C
-    test.C: In function &lsquo;<b>int foo()</b>&rsquo;:
+    <b>test.C:</b> In function &lsquo;<b>int foo()</b>&rsquo;:
     <b>test.C:1:14:</b> <b style='color:magenta'>warning:</b> no return statement in function returning non-void [-Wreturn-type]
      int foo () { }
                   <b style='color:lime'>^</b>

Patch hide | download patch | download mbox

--- gcc/diagnostic.h.jj	2013-04-08 15:38:20.000000000 +0200
+++ gcc/diagnostic.h	2013-04-12 11:25:53.031235252 +0200
@@ -293,6 +293,6 @@  void diagnostic_set_caret_max_width (dia
 
 
 /* Pure text formatting support functions.  */
-extern char *file_name_as_prefix (const char *);
+extern char *file_name_as_prefix (diagnostic_context *, const char *);
 
 #endif /* ! GCC_DIAGNOSTIC_H */
--- gcc/diagnostic.c.jj	2013-04-12 08:13:22.000000000 +0200
+++ gcc/diagnostic.c	2013-04-12 11:27:54.086563485 +0200
@@ -72,9 +72,12 @@  build_message_string (const char *msg, .
 
 /* Same as diagnostic_build_prefix, but only the source FILE is given.  */
 char *
-file_name_as_prefix (const char *f)
+file_name_as_prefix (diagnostic_context *context, const char *f)
 {
-  return build_message_string ("%s: ", f);
+  const char *locus_cs, *locus_ce;
+  locus_cs = colorize_start (pp_show_color (context->printer), "locus");
+  locus_ce = colorize_stop (pp_show_color (context->printer));
+  return build_message_string ("%s%s:%s ", locus_cs, f, locus_ce);
 }
 
 
--- gcc/langhooks.c.jj	2013-04-12 08:13:22.000000000 +0200
+++ gcc/langhooks.c	2013-04-12 11:28:32.068365378 +0200
@@ -372,7 +372,7 @@  lhd_print_error_function (diagnostic_con
       const char *old_prefix = context->printer->prefix;
       tree abstract_origin = diagnostic_abstract_origin (diagnostic);
       char *new_prefix = (file && abstract_origin == NULL)
-			 ? file_name_as_prefix (file) : NULL;
+			 ? file_name_as_prefix (context, file) : NULL;
 
       pp_set_prefix (context->printer, new_prefix);
 
--- gcc/cp/error.c.jj	2013-04-12 08:13:22.000000000 +0200
+++ gcc/cp/error.c	2013-04-12 11:28:46.053286955 +0200
@@ -2948,7 +2948,7 @@  cp_print_error_function (diagnostic_cont
       const char *file = LOCATION_FILE (diagnostic->location);
       tree abstract_origin = diagnostic_abstract_origin (diagnostic);
       char *new_prefix = (file && abstract_origin == NULL)
-			 ? file_name_as_prefix (file) : NULL;
+			 ? file_name_as_prefix (context, file) : NULL;
 
       pp_base_set_prefix (context->printer, new_prefix);