This is the mail archive of the gdb-patches@sourceware.cygnus.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

hexnumnstr; Was: RFA: patch to remote.c for larger download packet support (part 1)


"Frank Ch. Eigler" wrote:

I was was wondering what the hexnumstr was about.  It fixes a bug that
has nothing to do with your proposed change - v-good :-)

Can you please separate that change out and check it in as a separate
patch.

	enjoy,
		Andrew

> @@ -3090,7 +3114,7 @@ remote_write_bytes (memaddr, myaddr, len
>    while (len > 0)
>      {
>        unsigned char *p, *plen;
> -      int todo;
> +      int todo, plenlen;
>        int i;
>  

BTW.  It's good patch style to put new variable declarations on separate
lines - makes identifying changes easier.  Enjoy.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]