[patch, fortran] lang.opt: Adjust documentation.

[patch, fortran] lang.opt: Adjust documentation.

[Brooks Moses]

[-- Attachment #1: Type: text/plain, Size: 768 bytes --]

This patch edits a few descriptions in lang.opt.  The reason that it's 
not committed as obvious is that it removes -fpreprocessed from the 
"gfortran --help" output.  (It's not documented in the invoke.texi 
documentation either.)  As best I could tell from how this is used, it 
seems to be a "for internal use only" option, but I'm not sure of that 
and would like to have confirmation before committing this.  Comments, 
anyone?

------------------------------------------------------------------
2007-03-24  Brooks Moses  <brooks.moses@codesourcery.com>

	* lang.opt: Minor edits to descriptions.

------------------------------------------------------------------

Tested with "make check-fortran" on i686-pc-linux-gnu.  Ok for mainline?

Thanks much!
- Brooks


[-- Attachment #2: lang-opt-descriptions.diff --]
[-- Type: text/x-patch, Size: 578 bytes --]

Index: lang.opt
===================================================================
--- lang.opt	(revision 123170)
+++ lang.opt	(working copy)
@@ -219,11 +219,11 @@
 
 fpack-derived
 Fortran
-Try to layout derived types as compact as possible
+Try to lay out derived types as compactly as possible
 
 fpreprocessed
 Fortran
-Treat the input file as preprocessed
+; For internal use only: Treat the input file as preprocessed
 
 frange-check
 Fortran
@@ -263,7 +263,7 @@
 
 std=gnu
 Fortran
-Conform nothing in particular
+Conform to nothing in particular
 
 std=legacy
 Fortran

Re: [patch, fortran] lang.opt: Adjust documentation.

[FX Coudert]

> This patch edits a few descriptions in lang.opt.  The reason that  
> it's not committed as obvious is that it removes -fpreprocessed  
> from the "gfortran --help" output.  (It's not documented in the  
> invoke.texi documentation either.)  As best I could tell from how  
> this is used, it seems to be a "for internal use only" option, but  
> I'm not sure of that and would like to have confirmation before  
> committing this.  Comments, anyone?

How does that relate to your old post about this same option (- 
fpreprocessed)? (see http://gcc.gnu.org/ml/fortran/2006-11/ 
msg00281.html)

I also think it's for internal use only (and Google doesn't mention  
anyone non-internal use), so I am all for your patch. OK after  
waiting a few days, to let people complain.

FX

Re: [patch, fortran] lang.opt: Adjust documentation.

[Mike Stump]

On Mar 24, 2007, at 11:06 AM, Brooks Moses wrote:
> fpreprocessed
>  Fortran
> -Treat the input file as preprocessed
> +; For internal use only: Treat the input file as preprocessed

The Undocumented tag comes to mind, though, I will note that this  
flag is documented for C.

Re: [patch, fortran] lang.opt: Adjust documentation.

[Brooks Moses]

Mike Stump wrote:
> On Mar 24, 2007, at 11:06 AM, Brooks Moses wrote:
>> fpreprocessed
>>  Fortran
>> -Treat the input file as preprocessed
>> +; For internal use only: Treat the input file as preprocessed
> 
> The Undocumented tag comes to mind, though, I will note that this  
> flag is documented for C.

Indeed.  As FX reminded me, when I was looking into this a half-year ago 
I discovered that using the Undocumented tag in fortran/lang.opt didn't 
affect anything since it was documented in C.  However, that was with 
the old --help system, and I'm not sure how the new one handles this.

Currently, something seems to be broken in the gcc -v --help output for 
C and C++ (it's reporting "The following options are specific to the 
language C: No options with the desired characteristics were found"), so 
I can't actually tell what this is doing.  However, the new system does 
seem to be written under the assumption that all *.opt files that list 
an option give it the same documentation string, so I will apply the 
other parts of this patch but leave this bit out.

Thanks!
- Brooks, who just filed PRs 31349 through 31352 about the --help output 
while tracking this down.  :)

Re: [patch, fortran] lang.opt: Adjust documentation.

[Brooks Moses]

[-- Attachment #1: Type: text/plain, Size: 555 bytes --]

Brooks Moses wrote:
> Currently, something seems to be broken in the gcc -v --help output for 
> C and C++ (it's reporting "The following options are specific to the 
> language C: No options with the desired characteristics were found"), so 
> I can't actually tell what this is doing.  However, the new system does 
> seem to be written under the assumption that all *.opt files that list 
> an option give it the same documentation string, so I will apply the 
> other parts of this patch but leave this bit out.

And committed, as attached.

- Brooks

[-- Attachment #2: lang-opt-descriptions_2.diff --]
[-- Type: text/x-patch, Size: 450 bytes --]

Index: lang.opt
===================================================================
--- lang.opt	(revision 123170)
+++ lang.opt	(working copy)
@@ -219,7 +219,7 @@
 
 fpack-derived
 Fortran
-Try to layout derived types as compact as possible
+Try to lay out derived types as compactly as possible
 
 fpreprocessed
 Fortran
@@ -263,7 +263,7 @@
 
 std=gnu
 Fortran
-Conform nothing in particular
+Conform to nothing in particular
 
 std=legacy
 Fortran