On Mon, 5 Feb 2024 16:46:40 GMT, Martin Fox <m...@openjdk.org> wrote:
>> Turning on warnings-as-errors for the macOS glass native code. Deprecated >> declarations are excluded and still appear as warnings. >> >> In the code that tries to locate the application's dock icon there were >> three instances where `NO` was being passed into a method that required a >> pointer to a `BOOL`, not a `BOOL`. I suspect the intent was to check that >> the path pointed to an existing file but not a directory. Since JavaFX has >> gone this long without screening out directories correctly I decided not to >> fix that behavior except at the very end. >> >> The only other changes of note are sending some NSNotification objects to >> delegate API's that require them even though we know they're ignored on the >> other side. It was the easiest way to get rid of the warning. > > Martin Fox has updated the pull request incrementally with two additional > commits since the last revision: > > - Work-around for using a call not present in the Xcode 13.3 SDK. > - Need correctly formed NSNotification when running tests This is now compiling on 13.3 and I was able to do a Robot test run on macOS 14 with only one failure that's also present in the master branch. I still don't know why the compiler doesn't flag the `activate` call, in macOS 13 `NSApp` does not respond to that selector and will throw an exception if it's sent. ------------- PR Comment: https://git.openjdk.org/jfx/pull/687#issuecomment-1932546143