diff options
| author | Jan-Marek Glogowski <glogow@fbihome.de> | 2017-08-28 19:58:32 +0200 |
|---|---|---|
| committer | Jan-Marek Glogowski <glogow@fbihome.de> | 2017-09-26 09:42:11 +0200 |
| commit | 4baec725e0dc0713f0d47003e9b10bc3b62f56ff (patch) | |
| tree | 72f21c28416068e46133964e420ca094af8b7587 /vcl/unx | |
| parent | f633dcdfc0ad7a13d096d97b6753b55e8f8a3f07 (diff) | |
WIN run main thread redirects ignoring SolarMutex
This way we can drop all the special nReleased handling. Instead
we use the same mechanism as on Mac, where we keep the lock, but
disable it for the main thread. As a security measure we assert on
duplicate redirects, which should not happen.
As a result we can't use SendMessage on the main thread itself,
which would normally just call the WinProc directly. This could be
accomplished by converting the redirect bool into a counter, which
should be safe, as no other thread could acquire the SolarMutex,
as we don't release it.
Change-Id: Icd87b3da37a2489f3cad2bc80215bf93fc41d388
Reviewed-on: https://gerrit.libreoffice.org/42583
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
Diffstat (limited to 'vcl/unx')
| -rw-r--r-- | vcl/unx/generic/app/salinst.cxx | 4 | ||||
| -rw-r--r-- | vcl/unx/gtk/gtkinst.cxx | 4 |
2 files changed, 2 insertions, 6 deletions
diff --git a/vcl/unx/generic/app/salinst.cxx b/vcl/unx/generic/app/salinst.cxx index 98d56c6cd86e..e38164ee247b 100644 --- a/vcl/unx/generic/app/salinst.cxx +++ b/vcl/unx/generic/app/salinst.cxx @@ -166,10 +166,8 @@ bool X11SalInstance::AnyInput(VclInputFlags nType) return bRet; } -bool X11SalInstance::DoYield(bool bWait, bool bHandleAllCurrentEvents, sal_uLong const nReleased) +bool X11SalInstance::DoYield(bool bWait, bool bHandleAllCurrentEvents) { - (void) nReleased; - assert(nReleased == 0); // not implemented return mpXLib->Yield( bWait, bHandleAllCurrentEvents ); } diff --git a/vcl/unx/gtk/gtkinst.cxx b/vcl/unx/gtk/gtkinst.cxx index 2cd8b11c44d1..bc6fe03c9c2d 100644 --- a/vcl/unx/gtk/gtkinst.cxx +++ b/vcl/unx/gtk/gtkinst.cxx @@ -411,10 +411,8 @@ void GtkInstance::RemoveTimer () m_pTimer = nullptr; } -bool GtkInstance::DoYield(bool bWait, bool bHandleAllCurrentEvents, sal_uLong const nReleased) +bool GtkInstance::DoYield(bool bWait, bool bHandleAllCurrentEvents) { - (void) nReleased; - assert(nReleased == 0); // not implemented EnsureInit(); return GetGtkSalData()->Yield( bWait, bHandleAllCurrentEvents ); } |