Small refactor in ada-lang.c:scan_discrim_bound

Factor out common arithmetic operations for clarity.

gdb/ChangeLog:

	* ada-lang.c (scan_discrim_bound): Factor out arithmetic
	operations.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index b72031a485a9b3eb7f61066d451ca38d20574643..2404edd662778b72a9903795a9f7b55a1fca3361 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2015-09-10  Simon Marchi  <simon.marchi@ericsson.com>
+
+       * ada-lang.c (scan_discrim_bound): Factor out arithmetic
+       operations.
+
 2015-09-10  Simon Marchi  <simon.marchi@ericsson.com>
 
        * ada-lang.c (ada_search_struct_field): Constify parameters

diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c
index a514f655bdceac0352667e63f3e345f504f9b773..d166d1cba0297f578aa07115fbc51c37db99a138 100644 (file)
--- a/gdb/ada-lang.c
+++ b/gdb/ada-lang.c
@@ -11432,24 +11432,29 @@ scan_discrim_bound (const char *str, int k, struct value *dval, LONGEST * px,
 {
   static char *bound_buffer = NULL;
   static size_t bound_buffer_len = 0;
-  const char *pend, *bound;
+  const char *pstart, *pend, *bound;
   struct value *bound_val;
 
   if (dval == NULL || str == NULL || str[k] == '\0')
     return 0;
 
-  pend = strstr (str + k, "__");
+  pstart = str + k;
+  pend = strstr (pstart, "__");
   if (pend == NULL)
     {
-      bound = str + k;
+      bound = pstart;
       k += strlen (bound);
     }
   else
     {
-      GROW_VECT (bound_buffer, bound_buffer_len, pend - (str + k) + 1);
+      int len = pend - pstart;
+
+      /* Strip __ and beyond.  */
+      GROW_VECT (bound_buffer, bound_buffer_len, len + 1);
+      strncpy (bound_buffer, pstart, len);
+      bound_buffer[len] = '\0';
+
       bound = bound_buffer;
-      strncpy (bound_buffer, str + k, pend - (str + k));
-      bound_buffer[pend - (str + k)] = '\0';
       k = pend - str;
     }