move sizeof_pkt into remote_trace_find

The global sizeof_pkt is only used in remote_trace_find, like so:

  reply = remote_get_noisy_reply (&(rs->buf), &sizeof_pkt);

I think in this situation it is more correct to use the recorded size
of the buffer.  Otherwise it seems that some skew could result.

	* remote.c (sizeof_pkt): Remove.
	(remote_trace_find): Use rs->buf_size, not sizeof_pkt.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1f0f1e7719a8ecfa8a69692a05800983ed1c7ccc..08800de8c1981d48f5d98c621008d8d8e18b13e2 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2013-08-14  Tom Tromey  <tromey@redhat.com>
+
+       * remote.c (sizeof_pkt): Remove.
+       (remote_trace_find): Use rs->buf_size, not sizeof_pkt.
+
 2013-08-14  Tom Tromey  <tromey@redhat.com>
 
        * remote.c (struct remote_state) <use_threadinfo_query,

diff --git a/gdb/remote.c b/gdb/remote.c
index 780a1be06589d4cad40645c028de0a5af59ea09b..0fa5fcdbc7d9fd41f1ace06082fdca1d4e3f400f 100644 (file)
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -490,8 +490,6 @@ struct remote_arch_state
   long remote_packet_size;
 };
 
-long sizeof_pkt = 2000;
-
 /* Utility: generate error from an incoming stub packet.  */
 static void
 trace_error (char *buf)
@@ -10971,7 +10969,7 @@ remote_trace_find (enum trace_find_type type, int num,
     }
 
   putpkt (rs->buf);
-  reply = remote_get_noisy_reply (&(rs->buf), &sizeof_pkt);
+  reply = remote_get_noisy_reply (&(rs->buf), &rs->buf_size);
   if (*reply == '\0')
     error (_("Target does not support this command."));