[1/8] Don't forward-declare struct objfile in frame.h

Message ID 20200208152758.29385-2-tom@tromey.com
State New
Headers show
Series
  • Share DWARF frame information across inferiors
Related show

Commit Message

Tom Tromey Feb. 8, 2020, 3:27 p.m.
dwarf2/frame.h forward-declares struct objfile, but there's no need
for this.

gdb/ChangeLog
2020-02-08  Tom Tromey  <tom@tromey.com>

	* dwarf2/frame.h (struct objfile): Don't forward declare.

Change-Id: I4d54d46ac9422eeb64dc5f0b934792e77a875aa5
---
 gdb/ChangeLog      | 4 ++++
 gdb/dwarf2/frame.h | 1 -
 2 files changed, 4 insertions(+), 1 deletion(-)

-- 
2.17.2

Comments

Simon Marchi Feb. 12, 2020, 12:51 a.m. | #1
On 2020-02-08 10:27 a.m., Tom Tromey wrote:
> dwarf2/frame.h forward-declares struct objfile, but there's no need

> for this.

> 

> gdb/ChangeLog

> 2020-02-08  Tom Tromey  <tom@tromey.com>

> 

> 	* dwarf2/frame.h (struct objfile): Don't forward declare.

> 

> Change-Id: I4d54d46ac9422eeb64dc5f0b934792e77a875aa5

> ---

>  gdb/ChangeLog      | 4 ++++

>  gdb/dwarf2/frame.h | 1 -

>  2 files changed, 4 insertions(+), 1 deletion(-)

> 

> diff --git a/gdb/dwarf2/frame.h b/gdb/dwarf2/frame.h

> index a0ec3615a45..dd7315c8515 100644

> --- a/gdb/dwarf2/frame.h

> +++ b/gdb/dwarf2/frame.h

> @@ -23,7 +23,6 @@

>  #define DWARF2_FRAME_H 1

>  

>  struct gdbarch;

> -struct objfile;

>  struct frame_info;

>  struct dwarf2_per_cu_data;

>  struct agent_expr;

> -- 

> 2.17.2

> 


I'd say "dwarf2/frame.h" in the subject, since there's also a frame.h in
the gdb directory.  But other than that, I think you could push patches like
this one right away.

Simon
Tom Tromey Feb. 12, 2020, 1 a.m. | #2
>>>>> "Simon" == Simon Marchi <simark@simark.ca> writes:


Simon> I'd say "dwarf2/frame.h" in the subject, since there's also a frame.h in
Simon> the gdb directory.

Good idea, I did that.

Simon> But other than that, I think you could push patches like
Simon> this one right away.

Yeah, I will try to do that, since there's little point in anybody
having to review these.

Tom

Patch

diff --git a/gdb/dwarf2/frame.h b/gdb/dwarf2/frame.h
index a0ec3615a45..dd7315c8515 100644
--- a/gdb/dwarf2/frame.h
+++ b/gdb/dwarf2/frame.h
@@ -23,7 +23,6 @@ 
 #define DWARF2_FRAME_H 1
 
 struct gdbarch;
-struct objfile;
 struct frame_info;
 struct dwarf2_per_cu_data;
 struct agent_expr;