[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    RFR: 8235151: Inexistent notifyQuit method referred from iOS GlassHelper.m
From:       Jose Pereda <jpereda () openjdk ! java ! net>
Date:       2019-11-30 22:31:29
Message-ID: bw7I8YqqwjHyRSlNZ_Bi7B6MOMR6MbG9b4xwAiHitQI=.2f6f62b0-25e5-4d65-ab54-d86cc23ebda8 () github ! com
[Download RAW message or body]

`GlassHelper.m` for iOS contains the public method `ApplicationNotifyQuitMethod()`, \
that tries to find via reflection the `notifyQuit()` method in the \
`com.sun.glass.ui.Application` class. 

But the method `notifyQuit` doesn't exist in that class. However `notifyWillQuit` \
exists, and it is used in the Mac platform, for instance, when \
`ApplicationNotifyWillQuitMethod()` is called.

This PR renames `notifyQuit` to `notifyWillQuit`, and therefore \
`ApplicationNotifyQuitMethod` is also renamed to `ApplicationNotifyWillQuitMethod`.

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

Commits:
 - 0fe426ab: Rename notifyQuit and ApplicationNotifyQuitMethod to notifyWillQuit and \
ApplicationNotifyWillQuitMethod

Changes: https://git.openjdk.java.net/jfx/pull/58/files
 Webrev: https://webrevs.openjdk.java.net/jfx/58/webrev.00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8235151
  Stats: 9 lines in 3 files changed: 0 ins; 0 del; 9 mod
  Patch: https://git.openjdk.java.net/jfx/pull/58.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/58/head:pull/58

PR: https://git.openjdk.java.net/jfx/pull/58


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic