> I found that adding the `final` modifier does not add too much visual noise 
> to the documentation. I didn't find any occurrence where there were more than 
> two modifiers on a member summary (usually either `static final` or 
> `protected final`). 
> 
> On the other hand, I do think the `final` modifier adds important information 
> about the member. Although this is more true for fields than for other 
> members, for reasons of consistency I decided to show it for all kinds of 
> members except for those where it is not allowed (such as for enums).
> 
> To my surprise I also noticed that there weren't any test cases for field 
> summaries, so I added a new test.

Hannes Wallnöfer has refreshed the contents of this pull request, and previous 
commits have been removed. The incremental views will show differences compared 
to the previous content of the PR. The pull request contains one new commit 
since the last revision:

  JDK-8258602: JavaDoc field summary does not indicate final modifier

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3716/files
  - new: https://git.openjdk.java.net/jdk/pull/3716/files/074b7326..a6a9d202

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3716&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3716&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3716.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3716/head:pull/3716

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

Reply via email to