summaryrefslogtreecommitdiffstats
path: root/src/plugins/platforms/cocoa/qcocoawindow.mm
diff options
context:
space:
mode:
authorMarc Mutz <marc.mutz@kdab.com>2019-08-23 09:35:20 +0200
committerMarc Mutz <marc.mutz@kdab.com>2019-08-23 10:39:43 +0200
commit96ff6e8ebe80215a0d35055c7382bb1cf58fc660 (patch)
tree9f4bc5af1205d2908c8616bccfc033e8ba14f83a /src/plugins/platforms/cocoa/qcocoawindow.mm
parentb1db1dd655788fb685949959c99c901515bfd8ab (diff)
QGuiApplication: drop mutex before emitting fontChanged()
Emitting a signal executes an unknowable amount of code. We shouldn't hold a mutex while doing so. E.g., if the signal emission causes another call to QGuiApplication::setFont(), the old code would deadlock, since applicationFontMutex is not recursive. Fix by taking a copy of the application font under mutex protection, then dropping the lock for the emission of the signal. Change-Id: Ib2569b3a08af6ef5f38459a19f74cb0db27b7772 Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io> Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Diffstat (limited to 'src/plugins/platforms/cocoa/qcocoawindow.mm')
0 files changed, 0 insertions, 0 deletions