gdb: Convert la_struct_too_deep_ellipsis to a method

Convert language_data::la_struct_too_deep_ellipsis member variable to
a method in language_defn.

There should be no user visible changes after this commit.

gdb/ChangeLog:

	* ada-lang.c (ada_language_data): Remove
	la_struct_too_deep_ellipsis initializer.
	(ada_language::struct_too_deep_ellipsis): New member function.
	* c-lang.c (c_language_data): Remove la_struct_too_deep_ellipsis
	initializer.
	(cplus_language_data): Likewise.
	(asm_language_data): Likewise.
	(minimal_language_data): Likewise.
	* cp-valprint.c (cp_print_value): Update call to
	struct_too_deep_ellipsis.
	* d-lang.c (d_language_data): Remove la_struct_too_deep_ellipsis
	initializer.
	* f-lang.c (f_language_data): Likewise.
	(f_language::struct_too_deep_ellipsis): New member function.
	* go-lang.c (go_language_data): Remove la_struct_too_deep_ellipsis
	initializer.
	* language.c (unknown_language_data): Likewise.
	(auto_language_data): Likewise.
	* language.h (language_data): Delete la_struct_too_deep_ellipsis
	member variable.
	(language_defn::struct_too_deep_ellipsis): New member function.
	* m2-lang.c (m2_language_data): Remove la_struct_too_deep_ellipsis
	initializer.Q
	* objc-lang.c (objc_language_data): Likewise.
	* opencl-lang.c (opencl_language_data): Likewise.
	* p-lang.c (pascal_language_data): Likewise.
	* rust-lang.c (rust_language_data): Likewise.
	* valprint.c (val_print_check_max_depth): Update call to
	struct_too_deep_ellipsis.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 46dcc68d3ad3e4052c214c872518566eb54b454d..ef97ac180de11a263eb7366be9a6f5657cdab3f9 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,35 @@
+2020-09-16  Andrew Burgess  <andrew.burgess@embecosm.com>
+
+       * ada-lang.c (ada_language_data): Remove
+       la_struct_too_deep_ellipsis initializer.
+       (ada_language::struct_too_deep_ellipsis): New member function.
+       * c-lang.c (c_language_data): Remove la_struct_too_deep_ellipsis
+       initializer.
+       (cplus_language_data): Likewise.
+       (asm_language_data): Likewise.
+       (minimal_language_data): Likewise.
+       * cp-valprint.c (cp_print_value): Update call to
+       struct_too_deep_ellipsis.
+       * d-lang.c (d_language_data): Remove la_struct_too_deep_ellipsis
+       initializer.
+       * f-lang.c (f_language_data): Likewise.
+       (f_language::struct_too_deep_ellipsis): New member function.
+       * go-lang.c (go_language_data): Remove la_struct_too_deep_ellipsis
+       initializer.
+       * language.c (unknown_language_data): Likewise.
+       (auto_language_data): Likewise.
+       * language.h (language_data): Delete la_struct_too_deep_ellipsis
+       member variable.
+       (language_defn::struct_too_deep_ellipsis): New member function.
+       * m2-lang.c (m2_language_data): Remove la_struct_too_deep_ellipsis
+       initializer.Q
+       * objc-lang.c (objc_language_data): Likewise.
+       * opencl-lang.c (opencl_language_data): Likewise.
+       * p-lang.c (pascal_language_data): Likewise.
+       * rust-lang.c (rust_language_data): Likewise.
+       * valprint.c (val_print_check_max_depth): Update call to
+       struct_too_deep_ellipsis.
+
 2020-09-16  Felix Willgerodt  <felix.willgerodt@intel.com>
 
        * MAINTAINERS (Write After Approval): Add myself.

diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c
index 5f0e68ea892cc3e0b0ea88c62c87ae46b81c57fb..3ad693fe25ec4983ad23c9e1760c675b74a64bbc 100644 (file)
--- a/gdb/ada-lang.c
+++ b/gdb/ada-lang.c
@@ -13729,7 +13729,6 @@ extern const struct language_data ada_language_data =
   0,                            /* c-style arrays */
   1,                            /* String lower bound */
   &ada_varobj_ops,
-  "(...)"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the Ada language.  */
@@ -14187,6 +14186,10 @@ public:
     return ada_is_string_type (type);
   }
 
+  /* See language.h.  */
+
+  const char *struct_too_deep_ellipsis () const override
+  { return "(...)"; }
 
 protected:
   /* See language.h.  */

diff --git a/gdb/c-lang.c b/gdb/c-lang.c
index 03628704349428438af2e1d653aff953637ffe3a..8a704701c91cc0c603f864471784f65e87902698 100644 (file)
--- a/gdb/c-lang.c
+++ b/gdb/c-lang.c
@@ -895,7 +895,6 @@ extern const struct language_data c_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &c_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the C language.  */
@@ -996,7 +995,6 @@ extern const struct language_data cplus_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &cplus_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* A class for the C++ language.  */
@@ -1194,7 +1192,6 @@ extern const struct language_data asm_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* A class for the ASM language.  */
@@ -1250,7 +1247,6 @@ extern const struct language_data minimal_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* A class for the minimal language.  */

diff --git a/gdb/cp-valprint.c b/gdb/cp-valprint.c
index a02fee6b5524f7ace301787dff0922e67bd6c6cb..819736dac9ea9412866ea08011f8797ada6df09e 100644 (file)
--- a/gdb/cp-valprint.c
+++ b/gdb/cp-valprint.c
@@ -499,8 +499,8 @@ cp_print_value (struct value *val, struct ui_file *stream,
              && recurse >= options->max_depth)
            {
              const struct language_defn *language = current_language;
-             gdb_assert (language->la_struct_too_deep_ellipsis != NULL);
-             fputs_filtered (language->la_struct_too_deep_ellipsis, stream);
+             gdb_assert (language->struct_too_deep_ellipsis () != NULL);
+             fputs_filtered (language->struct_too_deep_ellipsis (), stream);
            }
          else
            {

diff --git a/gdb/d-lang.c b/gdb/d-lang.c
index abef3b6fa4378101057a80be69d11bc6c020b6b6..79cb1e478c439a4f3e31a8ed5d38811fa902c025 100644 (file)
--- a/gdb/d-lang.c
+++ b/gdb/d-lang.c
@@ -148,7 +148,6 @@ extern const struct language_data d_language_data =
   1,                           /* C-style arrays.  */
   0,                           /* String lower bound.  */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the D language.  */

diff --git a/gdb/f-lang.c b/gdb/f-lang.c
index 49f7b0368e2a55346bf1d70a96ab38e320d68429..5beaf3d772c1d4d77d624fe745e98b89591092c3 100644 (file)
--- a/gdb/f-lang.c
+++ b/gdb/f-lang.c
@@ -508,7 +508,6 @@ extern const struct language_data f_language_data =
   0,                           /* arrays are first-class (not c-style) */
   1,                           /* String lower bound */
   &default_varobj_ops,
-  "(...)"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the Fortran language.  */
@@ -707,6 +706,11 @@ public:
                && TYPE_TARGET_TYPE (type)->code () == TYPE_CODE_CHAR));
   }
 
+  /* See language.h.  */
+
+  const char *struct_too_deep_ellipsis () const override
+  { return "(...)"; }
+
 protected:
 
   /* See language.h.  */

diff --git a/gdb/go-lang.c b/gdb/go-lang.c
index cb42ef1b7cfe926d9d713a4d13283700e484668b..deff33a8abf314ba9faac0fd7f7688e00300d08b 100644 (file)
--- a/gdb/go-lang.c
+++ b/gdb/go-lang.c
@@ -523,7 +523,6 @@ extern const struct language_data go_language_data =
   1,                           /* C-style arrays.  */
   0,                           /* String lower bound.  */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the Go language.  */

diff --git a/gdb/language.c b/gdb/language.c
index c993cfc57a6e1d0e96e74be555ea5d8c41111670..914c75b08e7ca49e100d4f0daef46701d8281fe3 100644 (file)
--- a/gdb/language.c
+++ b/gdb/language.c
@@ -798,7 +798,6 @@ extern const struct language_data unknown_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the unknown language.  */
@@ -924,7 +923,6 @@ extern const struct language_data auto_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the fake "auto" language.  */

diff --git a/gdb/language.h b/gdb/language.h
index d2e5b733077d30c44ec915ad932c5097bb38a5ca..f2195ed4e8d88060be2d78aa1bf36da2a06c8e9e 100644 (file)
--- a/gdb/language.h
+++ b/gdb/language.h
@@ -266,12 +266,6 @@ struct language_data
 
     /* Various operations on varobj.  */
     const struct lang_varobj_ops *la_varobj_ops;
-
-    /* This string is used by the 'set print max-depth' setting.  When GDB
-       replaces a struct or union (during value printing) that is "too
-       deep" this string is displayed instead.  */
-    const char *la_struct_too_deep_ellipsis;
-
   };
 
 /* Base class from which all other language classes derive.  */
@@ -553,6 +547,16 @@ struct language_defn : language_data
   /* Return true if TYPE is a string type.  */
   virtual bool is_string_type_p (struct type *type) const;
 
+  /* Return a string that is used by the 'set print max-depth' setting.
+     When GDB replaces a struct or union (during value printing) that is
+     "too deep" this string is displayed instead.  The default value here
+     suits most languages.  If overriding then the string here should
+     ideally be similar in style to the default; an opener, three '.', and
+     a closer.  */
+
+  virtual const char *struct_too_deep_ellipsis () const
+  { return "{...}"; }
+
 protected:
 
   /* This is the overridable part of the GET_SYMBOL_NAME_MATCHER method.

diff --git a/gdb/m2-lang.c b/gdb/m2-lang.c
index 2c39359d2898e38e1c061cef142eb8c977c2f088..9e2d0d75f17cae135e5df2f56623fd8682184478 100644 (file)
--- a/gdb/m2-lang.c
+++ b/gdb/m2-lang.c
@@ -214,7 +214,6 @@ extern const struct language_data m2_language_data =
   0,                           /* arrays are first-class (not c-style) */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the M2 language.  */

diff --git a/gdb/objc-lang.c b/gdb/objc-lang.c
index 4e1d2cdcf2d8123dcd8409df7d34bfb9ef1d3d36..2c49f8d584060e2ba81c2bd6742eb57ae11b21a6 100644 (file)
--- a/gdb/objc-lang.c
+++ b/gdb/objc-lang.c
@@ -343,7 +343,6 @@ extern const struct language_data objc_language_data =
   1,                           /* C-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the Objective-C language.  */

diff --git a/gdb/opencl-lang.c b/gdb/opencl-lang.c
index 2d5ae17c36d4551eaaed1926fb31823bf8ee98e8..0df58ee5e1801bfece3f81598c2b5ddbd77091c9 100644 (file)
--- a/gdb/opencl-lang.c
+++ b/gdb/opencl-lang.c
@@ -1021,7 +1021,6 @@ extern const struct language_data opencl_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the OpenCL language.  */

diff --git a/gdb/p-lang.c b/gdb/p-lang.c
index 18490843a994b4912be94efcc2ddca71a4f9a47a..cbde8e1aecc8b121280d9d6993c1db8edb572ce5 100644 (file)
--- a/gdb/p-lang.c
+++ b/gdb/p-lang.c
@@ -272,7 +272,6 @@ extern const struct language_data pascal_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the Pascal language.  */

diff --git a/gdb/rust-lang.c b/gdb/rust-lang.c
index fa02b18e6fa1ac010875bfc5fd9105bc64addab6..ff0ac7bd42c5617e654ca473ab6c9e7330fa436b 100644 (file)
--- a/gdb/rust-lang.c
+++ b/gdb/rust-lang.c
@@ -1921,7 +1921,6 @@ extern const struct language_data rust_language_data =
   1,                           /* c-style arrays */
   0,                           /* String lower bound */
   &default_varobj_ops,
-  "{...}"                      /* la_struct_too_deep_ellipsis */
 };
 
 /* Class representing the Rust language.  */

diff --git a/gdb/valprint.c b/gdb/valprint.c
index adfbcc0464b597cb5f6d4f5ae17f7a99d79a424f..0749f38983e39df6a17804e65ab5e5721280598a 100644 (file)
--- a/gdb/valprint.c
+++ b/gdb/valprint.c
@@ -1025,8 +1025,8 @@ val_print_check_max_depth (struct ui_file *stream, int recurse,
 {
   if (options->max_depth > -1 && recurse >= options->max_depth)
     {
-      gdb_assert (language->la_struct_too_deep_ellipsis != NULL);
-      fputs_filtered (language->la_struct_too_deep_ellipsis, stream);
+      gdb_assert (language->struct_too_deep_ellipsis () != NULL);
+      fputs_filtered (language->struct_too_deep_ellipsis (), stream);
       return true;
     }