On Thu, 4 Aug 2022 06:25:27 GMT, ScientificWare <d...@openjdk.org> wrote:
>> ScientificWare has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains eight additional >> commits since the last revision: >> >> - Merge branch 'openjdk:master' into scientificware-patch-001-DTDBuilder >> - Merge branch 'openjdk:master' into scientificware-patch-001-DTDBuilder >> - Merge branch 'openjdk:master' into scientificware-patch-001-DTDBuilder >> - Merge branch 'openjdk:master' into scientificware-patch-001-DTDBuilder >> - Merge branch 'openjdk:master' into scientificware-patch-001-DTDBuilder >> - Merge branch 'openjdk:master' into scientificware-patch-001-DTDBuilder >> - Merge branch 'openjdk:master' into scientificware-patch-001-DTDBuilder >> - Remove unused imports. >> >> Some imports are no more used. >> >> - java.io.FileNotFoundException; >> - java.io.BufferedInputStream; >> - java.io.OutputStream; >> - java.util.BitSet; >> - java.util.StringTokenizer; >> - java.util.Properties; >> - java.util.zip.DeflaterOutputStream; >> - java.util.zip.Deflater; >> - java.net.URL; > > Just waiting to be reviewed. Hello @scientificware, what you propose here, I think looks fine. Additionally can you also update the copyright years on this file from `Copyright (c) 1998, 2020,` to `Copyright (c) 1998, 2022,`? I see that the changed file resides at `make/jdk/src/classes/build/tools/dtdbuilder/DTDBuilder.java` and I think this needs to be reviewed by the build team. @magicus, is it OK if I re-add the `build` label for this PR? ------------- PR: https://git.openjdk.org/jdk/pull/9360
