This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Use std::vector for displaced_step_inferior_states

From: Simon Marchi <simon dot marchi at ericsson dot com>
To: Pedro Alves <palves at redhat dot com>, Simon Marchi <simon dot marchi at polymtl dot ca>
Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, "dblaikie at gmail dot com" <dblaikie at gmail dot com>
Date: Fri, 23 Nov 2018 18:26:10 +0000
Subject: Re: [PATCH] Use std::vector for displaced_step_inferior_states
References: <20181122031229.15621-1-simon.marchi@ericsson.com> <a0ecf64e-55e8-10fa-4dcc-faa63ecb2369@redhat.com> <68730078c8a3a37d65ad3046348ccbc6@polymtl.ca> <1f966d70-b48d-35a9-9841-8d580371c710@redhat.com>

On 2018-11-22 12:17 p.m., Pedro Alves wrote:
>> I did a quick test of having a pointer to displaced_step_inferior_state in the inferior structure (the implementation of displaced_step_inferior_state stays in infrun.c), it seems to work well.  Would you prefer that?
> 
> I think that would be better, yeah.  Either pointer or object (and moving the
> struct to some header), both are fine with me.

Dropping this patch in favor of:

https://sourceware.org/ml/gdb-patches/2018-11/msg00390.html

Simon

References:
- [PATCH] Use std::vector for displaced_step_inferior_states
  - From: Simon Marchi
- Re: [PATCH] Use std::vector for displaced_step_inferior_states
  - From: Pedro Alves
- Re: [PATCH] Use std::vector for displaced_step_inferior_states
  - From: Simon Marchi
- Re: [PATCH] Use std::vector for displaced_step_inferior_states
  - From: Pedro Alves

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]