Patchwork obvious typo and grammar fixes to invoke.texi

login
register
mail settings
Submitter Jonathan Wakely
Date Jan. 25, 2011, 10:38 p.m.
Message ID <AANLkTinGBRL=mq3=8Vfao-Uc48ASAPADp80gNmXPoWMj@mail.gmail.com>
Download mbox | patch
Permalink /patch/80420/
State New
Headers show

Comments

Jonathan Wakely - Jan. 25, 2011, 10:38 p.m.
2011-01-25  Jonathan Wakely  <jwakely.gcc@gmail.com>

        * doc/invoke.texi (Warning Options): Add missing hyphen.
        (-fprofile-dir): Minor grammatical fixes.
        (-fbranch-probabilities): Likewise.

Committed to trunk as obvious.
Gerald Pfeifer - Jan. 31, 2011, 1:36 a.m.
On Tue, 25 Jan 2011, Jonathan Wakely wrote:
> 2011-01-25  Jonathan Wakely  <jwakely.gcc@gmail.com>
> 
>         * doc/invoke.texi (Warning Options): Add missing hyphen.
>         (-fprofile-dir): Minor grammatical fixes.
>         (-fbranch-probabilities): Likewise.

-Set the directory to search the profile data files in to @var{path}.
+Set the directory to search for the profile data files in to @var{path}.

Is this really a lot better now?  Perhaps "directory where to search
for profile data files" or something like that?

Gerald
Jonathan Wakely - Jan. 31, 2011, 9:15 a.m.
On 31 January 2011 01:36, Gerald Pfeifer wrote:
> On Tue, 25 Jan 2011, Jonathan Wakely wrote:
>> 2011-01-25  Jonathan Wakely  <jwakely.gcc@gmail.com>
>>
>>         * doc/invoke.texi (Warning Options): Add missing hyphen.
>>         (-fprofile-dir): Minor grammatical fixes.
>>         (-fbranch-probabilities): Likewise.
>
> -Set the directory to search the profile data files in to @var{path}.
> +Set the directory to search for the profile data files in to @var{path}.
>
> Is this really a lot better now?  Perhaps "directory where to search
> for profile data files" or something like that?

I agree it's still not very good, but I think it is a slight
improvement. I considered "directory in which to search for profile
data files" but thought it was a bit long-winded.  I'm happy to go
with whatever you prefer.

Patch

Index: invoke.texi
===================================================================
--- invoke.texi	(revision 169259)
+++ invoke.texi	(working copy)
@@ -229,7 +229,7 @@  Objective-C and Objective-C++ Dialects}.
 
 @item Warning Options
 @xref{Warning Options,,Options to Request or Suppress Warnings}.
-@gccoptlist{-fsyntax-only  fmax-errors=@var{n}  -pedantic @gol
+@gccoptlist{-fsyntax-only  -fmax-errors=@var{n}  -pedantic @gol
 -pedantic-errors @gol
 -w  -Wextra  -Wall  -Waddress  -Waggregate-return  -Warray-bounds @gol
 -Wno-attributes -Wno-builtin-macro-redefined @gol
@@ -7720,13 +7720,13 @@  default, GCC will emit an error message 
 @item -fprofile-dir=@var{path}
 @opindex fprofile-dir
 
-Set the directory to search the profile data files in to @var{path}.
+Set the directory to search for the profile data files in to @var{path}.
 This option affects only the profile data generated by
 @option{-fprofile-generate}, @option{-ftest-coverage}, @option{-fprofile-arcs}
 and used by @option{-fprofile-use} and @option{-fbranch-probabilities}
 and its related options.
-By default, GCC will use the current directory as @var{path}
-thus the profile data file will appear in the same directory as the object file.
+By default, GCC will use the current directory as @var{path}, thus the
+profile data file will appear in the same directory as the object file.
 
 @item -fprofile-generate
 @itemx -fprofile-generate=@var{path}
@@ -8011,7 +8011,7 @@  With @option{-fbranch-probabilities}, GC
 @samp{REG_BR_PROB} note on each @samp{JUMP_INSN} and @samp{CALL_INSN}.
 These can be used to improve optimization.  Currently, they are only
 used in one place: in @file{reorg.c}, instead of guessing which path a
-branch is mostly to take, the @samp{REG_BR_PROB} values are used to
+branch is most likely to take, the @samp{REG_BR_PROB} values are used to
 exactly determine which path is taken more often.
 
 @item -fprofile-values