> The `NestedClassWriter` interface seems to have never been used. I delete it 
> and reuse the freed name for the `NestedClassWriterImpl` class. If I don't 
> rename `NestedClassWriterImpl` to `NestedClassWriter`, it might confuse 
> future maintainers. Indeed, the "Impl" suffix on a class' name typically 
> implies that there is an interface which that class implements.

Pavel Rappo 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 branch 'master' into 8284447
 - Initial commit

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

Changes: https://git.openjdk.java.net/jdk/pull/8129/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8129&range=01
  Stats: 39 lines in 5 files changed: 0 ins; 32 del; 7 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8129.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8129/head:pull/8129

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

Reply via email to