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] Fix usage of inferior_ptid in two thread_alive implementations

From: Simon Marchi <simon dot marchi at polymtl dot ca>
To: Pedro Alves <palves at redhat dot com>
Cc: Simon Marchi <simon dot marchi at ericsson dot com>, gdb-patches at sourceware dot org
Date: Thu, 09 Feb 2017 11:46:53 -0500
Subject: Re: [PATCH] Fix usage of inferior_ptid in two thread_alive implementations
Authentication-results: sourceware.org; auth=none
References: <20170207212450.2232-1-simon.marchi@ericsson.com> <0322ea0c-ce96-8b7c-5d2d-0efa87ab71ed@redhat.com>

On 2017-02-08 07:40, Pedro Alves wrote:

On 02/07/2017 09:24 PM, Simon Marchi wrote:

While inspecting some target code, I noticed that in these two
implementations of thread_alive, inferior_ptid is referenced directly
instead of using the ptid passed as parameters.  I guess that it is
wrong, although I can't really test it in both cases.


I can't test either, but it looks right to me.


Soooo.. is this an approval :) ?

Follow-Ups:
- Re: [PATCH] Fix usage of inferior_ptid in two thread_alive implementations
  - From: Pedro Alves

References:
- [PATCH] Fix usage of inferior_ptid in two thread_alive implementations
  - From: Simon Marchi
- Re: [PATCH] Fix usage of inferior_ptid in two thread_alive implementations
  - From: Pedro Alves

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