This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v2 3/7] Documentation for gdb.thread_from_thread_handle
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Simon Marchi <simon dot marchi at polymtl dot ca>
- Cc: kevinb at redhat dot com, gdb-patches at sourceware dot org
- Date: Fri, 05 May 2017 09:17:49 +0300
- Subject: Re: [PATCH v2 3/7] Documentation for gdb.thread_from_thread_handle
- Authentication-results: sourceware.org; auth=none
- References: <20170408224959.67164a27@pinnacle.lan> <20170408230700.3754771e@pinnacle.lan> <d0448ba9b954ef3cba07228d3b6c71ed@polymtl.ca>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Fri, 05 May 2017 00:01:31 -0400
> From: Simon Marchi <simon.marchi@polymtl.ca>
> Cc: gdb-patches@sourceware.org, eliz@gnu.org
>
> > +@defun gdb.thread_from_thread_handle (thread_handle)
> > +Return the thread object corresponding to the thread handle,
> > +@var{thread_handle}, a thread library specific data structure such as
>
> The comma before @var{thread_handle} feels unnecessary. And I think you
> could shorten it to "... corresponding to @var{thread_handle}, ..."
> without loss of clarity.
Both variants are OK.
> Eli, do you have something to say about this patch?
Sorry, I missed it at the time. It's fine with me.
Thanks.