public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] Rename find_type_baseclass_by_name to cp_find_type_baseclass_by_name
@ 2014-12-13 18:50 Doug Evans
  2014-12-17  6:16 ` Doug Evans
  0 siblings, 1 reply; 2+ messages in thread
From: Doug Evans @ 2014-12-13 18:50 UTC (permalink / raw)
  To: gdb-patches

Hi.

There's a convention in cp-namespace.c that exported functions have a
cp_ prefix, so I added one to the only one that doesn't have it.

2014-12-13  Doug Evans  <xdje42@gmail.com>

	* cp-namespace.c (cp_find_type_baseclass_by_name): Renamed from
	find_type_baseclass_by_name.  All callers updated.

diff --git a/gdb/c-exp.y b/gdb/c-exp.y
index 92ff3b6..9d1c4c7 100644
--- a/gdb/c-exp.y
+++ b/gdb/c-exp.y
@@ -3019,7 +3019,7 @@ classify_inner_name (struct parser_state *par_state,
      relative to the `this' pointer.  */
   if (yylval.ssym.sym == NULL)
     {
-      struct type *base_type = find_type_baseclass_by_name (type, copy);
+      struct type *base_type = cp_find_type_baseclass_by_name (type, copy);
 
       if (base_type != NULL)
 	{
@@ -3038,7 +3038,7 @@ classify_inner_name (struct parser_state *par_state,
 	 named COPY when we really wanted a base class of the same name.
 	 Double-check this case by looking for a base class.  */
       {
-	struct type *base_type = find_type_baseclass_by_name (type, copy);
+	struct type *base_type = cp_find_type_baseclass_by_name (type, copy);
 
 	if (base_type != NULL)
 	  {
diff --git a/gdb/cp-namespace.c b/gdb/cp-namespace.c
index c083327..3066737 100644
--- a/gdb/cp-namespace.c
+++ b/gdb/cp-namespace.c
@@ -697,7 +697,7 @@ cp_lookup_symbol_nonlocal (const char *name,
    named NAME and return its type.  If not found, return NULL.  */
 
 struct type *
-find_type_baseclass_by_name (struct type *parent_type, const char *name)
+cp_find_type_baseclass_by_name (struct type *parent_type, const char *name)
 {
   int i;
 
@@ -713,7 +713,7 @@ find_type_baseclass_by_name (struct type *parent_type, const char *name)
       if (streq (base_name, name))
 	return type;
 
-      type = find_type_baseclass_by_name (type, name);
+      type = cp_find_type_baseclass_by_name (type, name);
       if (type != NULL)
 	return type;
     }
diff --git a/gdb/cp-support.h b/gdb/cp-support.h
index efd1e18..2594e34 100644
--- a/gdb/cp-support.h
+++ b/gdb/cp-support.h
@@ -215,8 +215,8 @@ struct type *cp_lookup_transparent_type (const char *name);
 
 /* See description in cp-namespace.c.  */
 
-struct type *find_type_baseclass_by_name (struct type *parent_type,
-					  const char *name);
+struct type *cp_find_type_baseclass_by_name (struct type *parent_type,
+					     const char *name);
 
 /* Functions from cp-name-parser.y.  */
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Rename find_type_baseclass_by_name to cp_find_type_baseclass_by_name
  2014-12-13 18:50 [PATCH] Rename find_type_baseclass_by_name to cp_find_type_baseclass_by_name Doug Evans
@ 2014-12-17  6:16 ` Doug Evans
  0 siblings, 0 replies; 2+ messages in thread
From: Doug Evans @ 2014-12-17  6:16 UTC (permalink / raw)
  To: gdb-patches

On Sat, Dec 13, 2014 at 10:49 AM, Doug Evans <xdje42@gmail.com> wrote:
> Hi.
>
> There's a convention in cp-namespace.c that exported functions have a
> cp_ prefix, so I added one to the only one that doesn't have it.
>
> 2014-12-13  Doug Evans  <xdje42@gmail.com>
>
>         * cp-namespace.c (cp_find_type_baseclass_by_name): Renamed from
>         find_type_baseclass_by_name.  All callers updated.

Committed.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-12-17  6:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-13 18:50 [PATCH] Rename find_type_baseclass_by_name to cp_find_type_baseclass_by_name Doug Evans
2014-12-17  6:16 ` Doug Evans

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).