> The result of Util.baseName(systemId) can be empty, causing the compiler to 
> set an empty classname. Add a check to make sure it will not set the empty 
> classname.
> 
> Alternatively, it may report an error, but that would be disruptive. As the 
> transform can proceed without the provided classname (by using the default), 
> adding a check is better than reporting an error.
> 
> I've verified the patch with the proposed fix for JDK-8276241. Harold has 
> also confirmed it fixes the tests in his builds.

Joe Wang has updated the pull request incrementally with one additional commit 
since the last revision:

  s/equals/isEmpty

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/6620/files
  - new: https://git.openjdk.java.net/jdk/pull/6620/files/ef0e6b88..24b0152a

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

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

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

Reply via email to