Re: [PATCH 2/2] Documentation hyperlinks for [-Wname-of-option] (PR 87488)

["Manuel López-Ibáñez" <lopezibanez@gmail.com>]

Hi David,

While I agree that this is quite cool to have, the following:

> +    /* DOCUMENTATION_ROOT_URL should be supplied via -D by the Makefile
> +       (see --with-documentation-root-url).
> +
> +       Expect an anchor of the form "index-Wfoo" e.g.
> +       <a name="index-Wformat"></a>, and thus an id within
> +       the URL of "#index-Wformat".  */
> +    return concat (DOCUMENTATION_ROOT_URL,
> +		   "Warning-Options.html",
> +		   "#index", cl_options[option_index].opt_text,
> +		   NULL);

will not work for many -W options:

https://gcc.gnu.org/onlinedocs/gcc/C_002b_002b-Dialect-Options.html#C_002b_002b-Dialect-Options
(scroll to the bottom)

https://gcc.gnu.org/onlinedocs/gcc/Objective-C-and-Objective-C_002b_002b-Dialect-Options.html#Objective-C-and-Objective-C_002b_002b-Dialect-Options
(scroll to the bottom)

https://gcc.gnu.org/onlinedocs/gcc-9.2.0/cpp/Invocation.html#index-Wcomment

https://gcc.gnu.org/onlinedocs/gcc-9.2.0/gfortran/Error-and-Warning-Options.html#Error-and-Warning-Options

I would argue that some options are documented in the wrong place (I think all 
C/C++ -W options should just go to Warning-Options.html), but I also believe 
the HTML page should be language dependent.

Cheers,

Manuel.