On Sun, Sep 22, 2024 at 04:17:11PM +0200, Mikael Morin wrote:
> -@opindex @code{std=}@var{std} option
> +@opindex std=@var{std} option
IMHO this one should be just
@opindex std=@var{std}
The option part is superfluous.
> -@opindex @code{idirafter @var{dir}}
> +@opindex idirafter @var{dir}
> -@opindex @code{imultilib @var{dir}}
> +@opindex imultilib @var{dir}
> -@opindex @code{iprefix @var{prefix}}
> +@opindex iprefix @var{prefix}
> -@opindex @code{isysroot @var{dir}}
> +@opindex isysroot @var{dir}
> -@opindex @code{iquote @var{dir}}
> +@opindex iquote @var{dir}
> -@opindex @code{isystem @var{dir}}
> +@opindex isystem @var{dir}
> -@opindex @code{fintrinsic-modules-path} @var{dir}
> +@opindex fintrinsic-modules-path @var{dir}
For consistency with the other parts of the documentation, I'd
leave the " @var{dir}" part from all of these (sure, keep it as
is in @item/@itemx), seems nothing in the non-Fortran manual
uses arguments after space in the index:
grep @opindex..*' ' */*.texi
doc/invoke.texi:@opindex mnopm
fortran/invoke.texi:@opindex @code{std=}@var{std} option
fortran/invoke.texi:@opindex @code{idirafter @var{dir}}
fortran/invoke.texi:@opindex @code{imultilib @var{dir}}
fortran/invoke.texi:@opindex @code{iprefix @var{prefix}}
fortran/invoke.texi:@opindex @code{isysroot @var{dir}}
fortran/invoke.texi:@opindex @code{iquote @var{dir}}
fortran/invoke.texi:@opindex @code{isystem @var{dir}}
fortran/invoke.texi:@opindex @code{fintrinsic-modules-path} @var{dir}
(grep before your patch; the mnopm case is just superfluous space
before the option name, should be fixed one day if it isn't ignored).
Otherwise LGTM.
Jakub
