From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1263 invoked by alias); 10 Apr 2007 19:22:18 -0000 Received: (qmail 1241 invoked by uid 22791); 10 Apr 2007 19:22:17 -0000 X-Spam-Check-By: sourceware.org Received: from heller.inter.net.il (HELO heller.inter.net.il) (213.8.233.23) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 10 Apr 2007 20:22:06 +0100 Received: from HOME-C4E4A596F7 (IGLD-84-229-253-80.inter.net.il [84.229.253.80]) by heller.inter.net.il (MOS 3.7.3a-GA) with ESMTP id CHV50497 (AUTH halo1); Tue, 10 Apr 2007 22:21:54 +0300 (IDT) Date: Tue, 10 Apr 2007 19:22:00 -0000 Message-Id: From: Eli Zaretskii To: Daniel Jacobowitz CC: bwilson@tensilica.com, gdb-patches@sources.redhat.com In-reply-to: <20070410154030.GF10890@caradoc.them.org> (message from Daniel Jacobowitz on Tue, 10 Apr 2007 11:40:30 -0400) Subject: Re: [PATCH] fix hyphenation and matters of style in docs Reply-to: Eli Zaretskii References: <4607FE10.8000402@tensilica.com> <20070410154030.GF10890@caradoc.them.org> X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2007-04/txt/msg00101.txt.bz2 > > > > 2007-03-26 Bob Wilson > > > > * gdb.texinfo (Contributors, Continuing and Stepping) > > (Fortran Defaults, HPPA, TUI, TUI Commands, Configure Options) > > (General Query Packets, File-I/O remote protocol extension) > > (Protocol basics, The F reply packet, write) > > (Protocol-specific representation of datatypes, Memory transfer): > > Fix hyphenation, punctuation and grammar problems. > > (Cygwin Native): Likewise. Also fix misuse of @pxref and use > > 'section' instead of 'subsection' in the text. > > (Non-debug DLL symbols): Avoid 'subsubsection' in the text. > > (i386): Remove period from section name. > > Eli, did you see this patch? Either I didn't see or somehow managed to delete it inadvertently, since I don't have it in my INBOX. Sorry for that. Reviewing it now, I agree with the suggested changes, but have a few questions/requests: . Please replace "@code{configure}" with "@file{configure}" or "@command{configure}". . I don't understand this change: -@item F@var{retcode},@var{errno},@var{Ctrl-C flag};@var{call specific attachment} +@item F@var{retcode},@var{errno},@var{Ctrl-C flag};@var{call specific +attachment} Maybe you wanted to say "call-specific attachment"? if so, I agree. . Since this patch was posted, the section names were rewritten to use a consistent capitalization, but this one doesn't: +@subsection Protocol-specific representation of datatypes With these gotchas taken care of, Bob, please commit the patch, and thanks. And thanks to Daniel for the heads-up.