[Bug fortran/78545] Possible correction to online LTIME documentation

[janus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

janus at gcc dot gnu.org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
Version|unknown |7.0
 Resolution|--- |FIXED
   Target Milestone|--- |7.0

--- Comment #8 from janus at gcc dot gnu.org ---
Fixed with the above two commits. Closing.

[Bug fortran/78545] Possible correction to online LTIME documentation

[janus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

--- Comment #7 from janus at gcc dot gnu.org ---
Author: janus
Date: Mon Dec 19 10:26:04 2016
New Revision: 243794

URL: https://gcc.gnu.org/viewcvs?rev=243794=gcc=rev
Log:
2016-12-19  Janus Weil  

PR fortran/78545
* intrinsic.texi: Minor documentation fixes for non-standard
time-related intrinsics. Add references to standard intrinsics and
linkify some existing references.

Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/intrinsic.texi

[Bug fortran/78545] Possible correction to online LTIME documentation

[dominiq at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

--- Comment #6 from dominiq at gcc dot gnu.org ---
Author: dominiq
Date: Sun Dec 18 18:03:36 2016
New Revision: 243785

URL: https://gcc.gnu.org/viewcvs?rev=243785=gcc=rev
Log:
2016-12-18  Dominique d'Humieres  

PR fortran/78545
* intrinsic.texi: Fix documentation for GMTIME and LTIME.

* intrinsics/date_and_time.c: Fix comments for GMTIME and LTIME.


Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/intrinsic.texi
trunk/libgfortran/ChangeLog
trunk/libgfortran/intrinsics/date_and_time.c

[Bug fortran/78545] Possible correction to online LTIME documentation

[janus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

--- Comment #5 from janus at gcc dot gnu.org ---
Patch at: https://gcc.gnu.org/ml/fortran/2016-11/msg00244.html

Dominique, I had already approved this patch back in November, but apparently
you have never committed it?

[Bug fortran/78545] Possible correction to online LTIME documentation

[janus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

--- Comment #4 from janus at gcc dot gnu.org ---
(In reply to Dominique d'Humieres from comment #3)
> > Furthermore the documentation for LTIME should link to the related
> > functions IDATE, ITIME and in particular the F95 standard intrinsic
> > DATE_AND_TIME.
> 
> Is the following addition to GMTIME and LTIME OK?
> 
> This intrinsic routine is provided for backwards compatibility with 
> GNU Fortran 77.  In new code, programmers should consider the use of 
> the @ref{DATE_AND_TIME} intrinsic defined by the Fortran 90(95??) and later
> standard(s??).

Yes, sounds good to me. The documentation says DATE_AND_TIME is from F95, so:
"... defined by Fortran 95 and later standards."

Or just: "... defined by the Fortran 95 standard".


> I am not fond to add more links to the F77 extensions: if done, it has to be
> consistent for all of then. Not sure it worths the pain!

Agreed. But it would be nice if the non-standard extensions would at least
refer to a standard alternative. So, all of LTIME, ITIME, IDATE should have a
"See also" section that lists DATE_AND_TIME.

[Bug fortran/78545] Possible correction to online LTIME documentation

[dominiq at lps dot ens.fr]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

--- Comment #3 from Dominique d'Humieres  ---
> Furthermore the documentation for LTIME should link to the related
> functions IDATE, ITIME and in particular the F95 standard intrinsic
> DATE_AND_TIME.

Is the following addition to GMTIME and LTIME OK?

This intrinsic routine is provided for backwards compatibility with 
GNU Fortran 77.  In new code, programmers should consider the use of 
the @ref{DATE_AND_TIME} intrinsic defined by the Fortran 90(95??) and later
standard(s??).

I am not fond to add more links to the F77 extensions: if done, it has to be
consistent for all of then. Not sure it worths the pain!

[Bug fortran/78545] Possible correction to online LTIME documentation

[dominiq at lps dot ens.fr]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

Dominique d'Humieres  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |dominiq at lps dot 
ens.fr

--- Comment #2 from Dominique d'Humieres  ---
I have a patch.

[Bug fortran/78545] Possible correction to online LTIME documentation

[janus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78545

janus at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2016-11-26
 CC||janus at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from janus at gcc dot gnu.org ---
It seems that the docu for GMTIME has the same issue - not only on the website,
but also in libgfortran/intrinsics/date_and_time.c.

Furthermore the doumentation for LTIME should link to the related functions
IDATE, ITIME and in particular the F95 standard intrinsic DATE_AND_TIME.