Looks fine to me, too.
+1
On 7/29/2019 4:56 PM, Philip Race wrote:
OK +1 assuming you ran the small set of PNG SplashScreen tests we have ..
-phil.
On 7/29/19, 3:51 PM, Sergey Bylokhov wrote:
The fix is updated:
http://cr.openjdk.java.net/~serb/8217676/webrev.01
On 29.07.2019 13:10, Phil Race wrote:
Yes, I believe we should keep it for this case.
-phil.
On 7/29/19 12:54 PM, Sergey Bylokhov wrote:
Hi, Phil.
I can move it back, but do we need this introduction? It seems we
do not have such for other libraries like libjpeg.
On 29.07.2019 11:48, Phil Race wrote:
We seem to have lost all these legal notice lines :
*
- * This file is available under and governed by the GNU General
Public
- * License version 2 only, as published by the Free Software
Foundation.
- * However, the following notice accompanied the original version
of this
- * file and, per its terms, should not be removed:
- * -phil.
On 7/25/19 5:55 PM, Sergey Bylokhov wrote:
Hello.
Please review the fix for JDK 13.
Bug: https://bugs.openjdk.java.net/browse/JDK-8217676
Fix: http://cr.openjdk.java.net/~serb/8217676/webrev.00
This is an upgrade of the libpng library to the latest version.
Most changes are just "drop and replace".