From: Yao Qi <yao@codesourcery.com>
Date: Sat, 22 Sep 2012 13:04:55 +0000 (+0000)
Subject: gdb/testsuite/
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=84cebc4a97b19238286cf7d4d254085f4b08e838;p=binutils-gdb.git

gdb/testsuite/

	* remote.c (remote_get_trace_status): Remove setting default
	values of fields of 'ts'.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 97d1d40b4af..0b1d7718e10 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2012-09-22  Yao Qi  <yao@codesourcery.com>
+
+	* remote.c (remote_get_trace_status): Remove setting default
+	values of fields of 'ts'.
+
 2012-09-21  Jan Kratochvil  <jan.kratochvil@redhat.com>
 
 	Fix internal error on canonicalization of clang types.
diff --git a/gdb/remote.c b/gdb/remote.c
index a3309c7aacf..1750bee07eb 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -10588,15 +10588,11 @@ remote_get_trace_status (struct trace_status *ts)
   /* We're working with a live target.  */
   ts->from_file = 0;
 
-  /* Set some defaults.  */
-  ts->running_known = 0;
-  ts->stop_reason = trace_stop_reason_unknown;
-  ts->traceframe_count = -1;
-  ts->buffer_free = 0;
-
   if (*p++ != 'T')
     error (_("Bogus trace status reply from target: %s"), target_buf);
 
+  /* Function 'parse_trace_status' sets default value of each field of
+     'ts' at first, so we don't have to do it here.  */
   parse_trace_status (p, ts);
 
   return ts->running;