remote-utils.c: Add missing space

remote-utils.c: Add missing space

[Maciej W. Rozycki]

Hello,

 This change should be obvious, but it modifies run-time behaviour, so I'm 
asking just in case it hits somebody doing something strange.

2007-03-30  Chris Dearman  <chris@mips.com>

	* remote-utils.c (gr_files_info): Add missing space when printing
	baud rate.

 OK to apply?

  Maciej

12226-1.diff
Index: binutils-quilt/src/gdb/remote-utils.c
===================================================================
--- binutils-quilt.orig/src/gdb/remote-utils.c	2007-03-30 12:23:08.000000000 +0100
+++ binutils-quilt/src/gdb/remote-utils.c	2007-03-30 12:23:38.000000000 +0100
@@ -437,7 +437,7 @@
 #else
   printf_filtered ("\tAttached to %s", sr_get_device ());
   if (baud_rate != -1)
-    printf_filtered ("at %d baud", baud_rate);
+    printf_filtered (" at %d baud", baud_rate);
   printf_filtered ("\n");
 #endif
 

Re: remote-utils.c: Add missing space

[Daniel Jacobowitz]

On Fri, Mar 30, 2007 at 12:28:29PM +0100, Maciej W. Rozycki wrote:
> Hello,
> 
>  This change should be obvious, but it modifies run-time behaviour, so I'm 
> asking just in case it hits somebody doing something strange.
> 
> 2007-03-30  Chris Dearman  <chris@mips.com>
> 
> 	* remote-utils.c (gr_files_info): Add missing space when printing
> 	baud rate.
> 
>  OK to apply?

Interesting timing...

Have you encountered this code at runtime?  I ask because this file
was on my list to delete for this week.  There are no callers in-tree.

-- 
Daniel Jacobowitz
CodeSourcery

Re: remote-utils.c: Add missing space

[Maciej W. Rozycki]

On Fri, 30 Mar 2007, Daniel Jacobowitz wrote:

> Have you encountered this code at runtime?  I ask because this file
> was on my list to delete for this week.  There are no callers in-tree.

 I have not -- the patch comes from our internal tree and I am a mere 
messenger. ;-)  It might just have been a casual fix for a typo spotted.  
Chris?

  Maciej

Re: remote-utils.c: Add missing space

[Chris Dearman]

Maciej W. Rozycki wrote:

>  I have not -- the patch comes from our internal tree and I am a mere 
> messenger. ;-)  It might just have been a casual fix for a typo spotted.  
> Chris?

   I think that's probably the case. If there are no callers to the 
function and it's slated for removal I think the case is closed :)

Chris

-- 
Chris Dearman          7200 Cambridge Research Park     +44 1223 203108
MIPS Technologies (UK) Waterbeach, Cambs CB25 9TL  fax  +44 1223 203181