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

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by David Malcolm <dmalc...@gcc.gnu.org>:

https://gcc.gnu.org/g:fa29cf0c3f19b648e30b16fd2485c3c17a528a6e

commit r10-7994-gfa29cf0c3f19b648e30b16fd2485c3c17a528a6e
Author: David Malcolm <dmalc...@redhat.com>
Date:   Thu Dec 5 14:47:35 2019 -0500

    Fix warning URLs for Fortran and analyzer [PR 92830]

    PR 92830 reports that we always use "gcc/Warning-Options.html" when we
    emit escaped documentation URLs when printing "[-Wname-of-option]" for
    a warning.

    This page is wrong for most Fortran warnings, and for analyzer warnings.

    I considered various schemes involving adding extra tags to the .opt
    format to capture where options are documented, but for now this patch
    fixes the issue by introducing some special-casing logic.
    It only fixes the URLs for warning options, not for other command-line
    options, but those are the only options for which get_option_url is
    currently called.

    gcc/ChangeLog:
            PR 92830
            * configure.ac (DOCUMENTATION_ROOT_URL): Drop trailing "gcc/" from
            default value, so that it can by supplied by get_option_html_page.
            * configure: Regenerate.
            * opts.c: Include "selftest.h".
            (get_option_html_page): New function.
            (get_option_url): Use it.  Reformat to place comments next to the
            expressions they refer to.
            (selftest::test_get_option_html_page): New.
            (selftest::opts_c_tests): New.
            * selftest-run-tests.c (selftest::run_tests): Call
            selftest::opts_c_tests.
            * selftest.h (selftest::opts_c_tests): New decl.

Reply via email to