This is the mail archive of the gdb-patches@sources.redhat.com 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: [RFA] Add qGetTLSAddr packet documentation to gdb.texinfo


> Date: Mon, 6 Dec 2004 13:58:19 -0700
> From: Kevin Buettner <kevinb@redhat.com>
> 
> Still okay?

Yes, but please fix these nits:

> +@item @code{qGetTLSAddr}:@var{thread-id},@var{offset},@var{load-module-param-1},@dots,@var{load-module-param-N} --- get thread local storage address

This line is typeset as a single line, so it should be as short as
possible.  I suggest to shorten the names of the packet parameters
(e.g., "param1" instead of "load-module-param-1").

> +module.  For example, a @sc{GNU}/Linux system will pass (as

The argument of @sc should not be in caps: @sc{gnu}/Linux.

> +@item @code{""} (empty)
> +    An empty reply indicates that @code{qGetTLSAddr} is not supported by the stub.

The last line should not be indented.


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