On Tue, 11 Feb 2025 17:55:25 GMT, Oliver Schmidtmer <oschmidt...@openjdk.org> 
wrote:

>> Webkit JavaFx upgrade from 619.1 to 620.1 <br>
>> Build is verified in mac , windows and linux. Sanity testing looks fine. No 
>> issues seen, except one observed CanvasPattern test where it works only on 
>> Linux but failed on Mac/Windows (WIP).
>> Windows build is using clang-cl compiler because Webkit has dropped MSVC 
>> compiler support.
>
> modules/javafx.web/src/main/native/Source/WTF/wtf/java/FileSystemJava.cpp 
> line 1:
> 
>> 1: /*
> 
> I would recommend to add "fprintf(stderr, XXX NOT IMPLEMENTED\n");" outputs 
> to the the four changed / new functions stubs, as for the older unimplemented 
> methods. 
> This would make issues as we had in 
> [JDK-8337481](https://bugs.openjdk.org/browse/JDK-8337481) easier to track.

That's a good idea.

@jaybhaskar Can you either make this change or else file a follow-up issue?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1703#discussion_r1951370805

Reply via email to