This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: Documentation of LTIME
- From: Janus Weil <janus at gcc dot gnu dot org>
- To: Dominique d'Humières <dominiq at lps dot ens dot fr>
- Cc: Janne Blomqvist <blomqvist dot janne at gmail dot com>, Francisco Pena <fran dot pena at usc dot es>, gfortran <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 19 Dec 2016 11:30:29 +0100
- Subject: Re: Documentation of LTIME
- Authentication-results: sourceware.org; auth=none
- References: <42E6ADCB-E228-42B0-BF2B-E630E56C2048@lps.ens.fr> <CAC2FYYqEPKCNp88Yb7RTJtOZjuU1b-VirROSoCrRQgNwNRgm5Q@mail.gmail.com> <B272C514-F1C7-471E-A650-2E126BF51807@lps.ens.fr> <CAKwh3qjt0FVw7n_RU30Mn1JEPp6sixUVK_OECoE58khe=nCDNQ@mail.gmail.com> <CAKwh3qh8JgA2svLZxF=i6R7EdOZyMGdaikyhgBRTs7LKP3vraQ@mail.gmail.com> <473F4144-7133-4DB3-B7E4-EF7A4EFDF4D6@lps.ens.fr> <CAKwh3qgT9iKVxT=LGbb3Mh2sf0Kc2pZSU45GQ3JSXON3HFQC_Q@mail.gmail.com> <CAKwh3qhf9egrL3V7+5gXZTMcBBVY2yG=bYhOtmnSB5AHfkPGYQ@mail.gmail.com> <CAO9iq9H04KbZJdTdW=1uPBtuNX0dM+QNQah8bq3LgtBTy=HP4g@mail.gmail.com> <CAKwh3qh3EMAQxeRAwzU3HqSo_-VKZ=qJ2rjobowS4+CEG8N+mA@mail.gmail.com> <C774DBDC-54F9-4FF6-9185-24FC09320DFA@lps.ens.fr> <CAKwh3qjvgqFRVB=Z0hqwcErsJxegfqFJffq7VVygE_a==NkPBw@mail.gmail.com>
Hi all,
I have just committed some minor additions
(https://gcc.gnu.org/viewcvs?rev=243794&root=gcc&view=rev) to
Dominique's patch at
https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=243785, so that
these documentation issued are finally resolved now.
Cheers,
Janus
2016-11-26 21:45 GMT+01:00 Janus Weil <janus@gcc.gnu.org>:
>>> If not, we definitely need to fix the documentation of LTIME, since
>>> the current version simply does not work with TIME8(), unless one uses
>>> -fdefault-integer-8 (which is not mentioned in the docu).
>>
>> What about the attached patch
>
> Yes, looks good to me. Ok to commit!
>
> One minor nit (optional):
>
> @@ -9635,10 +9650,15 @@ To stat an open file: @ref{FSTAT}, to st
>
> @table @asis
> @item @emph{Description}:
> -Given a system time value @var{TIME} (as provided by the @code{TIME8}
> +Given a system time value @var{TIME} (as provided by the @code{TIME}
> intrinsic), fills @var{VALUES} with values extracted from it appropriate
> to the local time zone using @code{localtime(3)}.
>
> I would use @ref{TIME} here (there are two places where this occurs).
> Also I would add @ref{DATE_AND_TIME} to the "see also" section.
>
> Thanks,
> Janus