RFR: 8301813: Bad caret position in error message
Jonathan Gibbons
jjg at openjdk.org
Fri Feb 3 23:04:39 UTC 2023
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.
-------------
Commit messages:
- 8301813: Bad caret position in error message
Changes: https://git.openjdk.org/jdk/pull/12421/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=12421&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8301813
Stats: 148 lines in 5 files changed: 134 ins; 6 del; 8 mod
Patch: https://git.openjdk.org/jdk/pull/12421.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/12421/head:pull/12421
PR: https://git.openjdk.org/jdk/pull/12421
More information about the compiler-dev
mailing list