On Thu, 4 May 2023 08:06:11 GMT, Maxim Kartashev <mkartas...@openjdk.org> wrote:
>> Getting bounds of a `GraphicsDevice` and insets of a screen are fairly >> frequent operations. When working over the network (remote X session), those >> can take several *seconds* to complete, introducing repeated freezes even >> when typing in an editor feels only slightly sluggish. >> >> On the other hand, neither bounds nor insets change very often - if at all - >> during the lifetime of an application. So caching their values seems like a >> natural solution to the problem. The caches need to be reset only when >> there's a possibility of change in the screens configuration. >> >> A similar patch has been living in [JetBrains >> Runtime](https://github.com/JetBrains/JetBrainsRuntime/) for well over a >> year. > > Maxim Kartashev has updated the pull request incrementally with one > additional commit since the last revision: > > Double-checked locking for insets and less synchronization for bounds The latest version looks fine. ------------- Marked as reviewed by serb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/13346#pullrequestreview-1440858185
