public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* as error output not -j64 safe
@ 2014-05-15 22:48 Mike Stump
  2014-05-15 22:52 ` Andrew Pinski
  0 siblings, 1 reply; 11+ messages in thread
From: Mike Stump @ 2014-05-15 22:48 UTC (permalink / raw)
  To: binutils

So when dealing with error messages, it is important that they be one per line and -j safe.  This fixes intermingled output when -j64 is used on linux:

diff --git a/binutils/gas/messages.c b/binutils/gas/messages.c
index e1734f2..c3a551b 100644
--- a/binutils/gas/messages.c
+++ b/binutils/gas/messages.c
@@ -219,13 +219,10 @@ as_bad_internal (char *file, unsigned int line, char *buffer)
   if (file)
     {
       if (line != 0)
-       fprintf (stderr, "%s:%u: ", file, line);
+       fprintf (stderr, "%s:%u: %s %s\n", file, line, _("Error:"), buffer);
       else
-       fprintf (stderr, "%s: ", file);
+       fprintf (stderr, "%s: %s %s\n", file, _("Error:"), buffer);
     }
-  fprintf (stderr, _("Error: "));
-  fputs (buffer, stderr);
-  (void) putc ('\n', stderr);
 #ifndef NO_LISTING
   listing_error (buffer);
 #endif

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2014-05-22  8:40 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-15 22:48 as error output not -j64 safe Mike Stump
2014-05-15 22:52 ` Andrew Pinski
2014-05-15 23:13   ` Mike Stump
2014-05-16  8:51     ` Alan Modra
2014-05-20 20:28       ` Mike Stump
2014-05-20 22:01         ` Matthew Fortune
2014-05-20 22:49           ` Matthew Fortune
2014-05-20 22:57             ` Mike Stump
2014-05-22  7:22               ` Alan Modra
2014-05-22  8:10                 ` Mike Stump
2014-05-22  8:40                   ` Alan Modra

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).