On Thu, 10 Sep 2020 10:40:15 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> @kevinrushforth thanks. Done. >> >> Similar issues: >> https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8215014 >> https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8251246 >> https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8223237 >> >> could be joined somehow? > > The code in java.base was updated to use String::isEmpty in JDK 12 > (JDK-8215281). There was follow-up in JDK 13 to do > the same in the java.desktop module (JDK-8223237). Changing the remaining > usages make sense although I see that more > more than half are in tests. It would be good to hear from security-dev on > the changes to the Apache Santuario code > (in java.xml.crypto module) in case it would be better to contribute those > upstream instead. Ditto for the Apache Xalan > code (in the java.xml module) but it may be significantly forked already that > it doesn't matter. I assume you can use > JDK-8215014 rather than creating a new issue. This should be broken up to deal with the files in different functional areas under different bugids and PRs. Otherwise the cross-posting to so many lists is prohibitive. Files in different areas need to be reviewed by different teams. Thank you. ------------- PR: https://git.openjdk.java.net/jdk/pull/29