On Fri, 3 Feb 2023 22:56:58 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:
> Please review a trivial update to `DocCommentParser` and corresponding > updates to tests, to fix the caret position in the two forms of the "no tag > name found" error message. > > The fix is simply to use the overload of `erroneous` that allows the > preferred caret position to be specified. This overload was not available at > the time the affected lines were originally written. > > The "natural" place to test this is in the `doctree` tests (that is, > `TagTest.java`) but that test is not set up for easy monitoring of the > end-user visible caret, so a new doclet test is created for that. > (`TestNoTagName.java`). > > BadPackageCommentTest.java is also affected. It is updated in a manner > similar to that in a parallel PR for handling at-escapes. However, the > changed caret position will eventually probably cause a minor change conflict > that will need to be resolved. Marked as reviewed by iris (Reviewer). ------------- PR: https://git.openjdk.org/jdk/pull/12421
