> 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