This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, gcc-wwdocs] Update to Fortran changes
- From: Tobias Burnus <tobias at codesourcery dot com>
- To: Mark Eggleston <mark dot eggleston at codethink dot co dot uk>, gcc-patches <gcc-patches at gcc dot gnu dot org>, fortran <fortran at gcc dot gnu dot org>, <gerald at pfeifer dot com>
- Date: Tue, 26 Nov 2019 12:12:24 +0100
- Subject: Re: [Patch, gcc-wwdocs] Update to Fortran changes
- Ironport-sdr: zyDN56S16WQIHGa8GCs8saREtVsa+a0/K0rlNF3PIMKixtqAgi/T7uqr/jTdajdLb+XIza3UGx AYMGAZb77BtVPoFFcCLmyX81LypNwgt5Ljj8Pd95Y7kfmbT6evhZxIZKE0VIRZxDuO/qf3aNJW zFx0WFBsQLEV3h70D+9XiX814mraWttm6yWGjcXakni+EBB5+uvuqHJ+6iXvbW+4XQxHGPdNoG COTvOiyXRufWeiGpgqPK6/gq7TqVHKSOwV7nuVcq0FV5TTZQlpRYSNl20NEk+bmRozO2U6S6qE DmE=
- Ironport-sdr: OnCjinmksWBoNzWedUoN2STiLLmeIdYy+CZm/VaSxuhf2ckIu+ZfN0H6HJlyF7HRvvndxVOGbw VD+eFi4/0EovLqgScM9YOgwcJohS2QZG7FiAOmezo0bFcNpi0ICOOKykVyR/vykBIFgxkUPb+/ FpFEZTgu2kmSPSoHaaiiKBJ57EBK9jrBPw0WiEOcYh35bzMw9eKku/k9vk/IP5nQKDz98fmXjT m5pzZfUZPuoNZ5ielAnfGuuBFdIlVe3/2zLdle+dV3m5ssayASdX7+KCw37SuC4odG/TNqBM0U yGE=
- References: <8de6eea3-ec83-dc45-0bfc-8c647e355a6f@codethink.co.uk>
Hi Mark,
On 11/26/19 11:46 AM, Mark Eggleston wrote:
I've check the changes with W3 validator. There are no problems
related to my changes, however, it does object to the lack of
character encoding in the file.
Note that some script modifies the file a bit – the final file does have
a character encoding (content="text/html; charset=utf-8").
This is the first time in changing the wwwdoc, I haven't included a
change log as there there doesn't appear to be any ChangeLog files.
There is only a commit log (see 'git log'); the format is still a bit in
a flux as before there was only 'csv' where it applied to a single file.
[With git it is useful to have a meaningful single line (for "git log
--pretty=oneline") and, if needed, an empty line and something more. As
the main repository is going to move to GIT as well, this also applies
there. – But you already follow this style.]
The patch is attached. OK to commit?
I wonder whether you should use a main bullet point with something like
"Legacy improvements (<code>-fdec</code>)" under which you put the
current list. [Or some other wording.] That makes it easier to read for
users – those, who care about legacy support, can quickly look at the
-fdec changes – while those interested in standard Fortran can skip it.
(Except for the character-diagnostic type-info improvement, all are
items seem to be for -fdec or -fdec-…)
In particular, currently, the following item does not make clear that it
is about legacy support: "Variables with the <code>AUTOMATIC</code>
attribute can be used in <code>EQUIVALENCE</code> statements."
I think it can remain as is with the -fdec umbrella item. But without it
really requires the background knowledge that the AUTOMATIC attribute is
only available with -fdec-static (or -fdec).
Cheers,
Tobias