> This is a conceptually simple change to ensure that all "id"s generated by 
> the standard doclet use `-` to separate the words of the id. The `src/` 
> change is primarily in the recently-new `HtmlIds` class, but there was one 
> oversight in the recent work for `HtmlIds` in `SourceToHTMLConverter`, which 
> is now fixed.
> 
> It's notable now that all "static" ids generated by the doclet contain at 
> least one `-`, and so never conflict with the id for any field name, which 
> cannot contain a `-`.
> 
> The tests are updated; all tests pass, of course, including the implicit link 
> check that is default on in almost all tests. A full doc check run on a full 
> JDK docs build did not reveal any unexpected problems.   It wasn't clear 
> which tests to nominate with a bug number, and so (somewhat unusually) I have 
> added the bug number to all affected tests.

Jonathan Gibbons has updated the pull request with a new target base due to a 
merge or a rebase. The pull request now contains two commits:

 - Merge remote-tracking branch 'upstream/master' into htmlIds
   
   # Conflicts:
   #    
test/langtools/jdk/javadoc/doclet/testPackageDeprecation/TestPackageDeprecation.java
   #    test/langtools/jdk/javadoc/doclet/testUseOption/TestUseOption.java
 - JDK-8261976: Normalize id's used by the standard doclet

-------------

Changes: https://git.openjdk.java.net/jdk/pull/2635/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2635&range=01
  Stats: 307 lines in 33 files changed: 1 ins; 1 del; 305 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2635.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2635/head:pull/2635

PR: https://git.openjdk.java.net/jdk/pull/2635

Reply via email to