RFR: 8232438: Remove ?is-external=true from external links

[Pavel Rappo]

Hi Hannes,

What is the background of this "?is-external=true" feature? How was it used in
the first place? Is there anything I can read on that?

I'm surprised to see the tests' @bug tags are updated to include this bug number.

I thought that an @bug tag needs to be updated when the test has been meaningfully
changed. In other words, when the relation between the test and the bug is
stronger than mere "updated because otherwise fails".

I appreciate that the semantics of @bug might depend on the area, still:

  https://openjdk.java.net/jtreg/faq.html#when-should-i-update-the-bug-entry-in-a-test-description

-Pavel

> On 20 Feb 2020, at 13:09, Hannes Wallnöfer <hannes.wallnoefer at oracle.com> wrote:
> 
> Please review:
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8232438
> Webrev: http://cr.openjdk.java.net/~hannesw/8232438/webrev.00/
> 
> The is-external=true query was the only use of query functionality in DocLink, so I removed support for query strings in DocLink. The rest of the patch consists of removing the query string from test fixtures.
> 
> Thanks,
> Hannes