[PATCH, fortran] Interoperability with C int128_t types
FX
fxcoudert@gmail.com
Wed Apr 16 12:13:00 GMT 2008
> And also the ChangeLogs and texi changes.
Oh, I only saw that mail after I sent my review. The doc changes are
fine, but I'd change the wording to make it even more obvious: as
128-bit ISO_C_BINDING integer kinds are not a currently existing
extension, we want to avoid surprising people. Maybe saying it in a
different paragraph:
"In addition to the integer named constants required by the Fortran
2003 standard, GNU Fortran provides as an extension named constants
for the 128-bit integer types supported by the C compiler: C_INT128_T,
... [write here the full list]".
Then, I'd add a new column to the table so that extensions are more
clearly labeled: like, using "Extension" as a column header and
putting an "Ext." for the new symbols.
Regards,
FX
--
FX Coudert
http://www.homepages.ucl.ac.uk/~uccafco/
More information about the Fortran
mailing list