The VCL backend vcl/inc/salframe.hxx has an (apparently almost entirely unused) 'Sync' method. Callers of that could call Flush instead which is almost as good (and a duplicate on windows). We should check if in fact Sync() is used outside of the outdevgrind tool (which can use flush instead). Best to remove the method and see what happens =) // flush output buffer, wait till outstanding operations are done virtual void Sync() = 0; If we need it back we can add a parameter to Flush(... bool bWait = false); or something which is a nicer API anyhow. Thanks !
I sent following patch for this bug: https://gerrit.libreoffice.org/#/c/19027/
I sent following patch for this bug: https://gerrit.libreoffice.org/#/c/19102/
I've started (and others continued) a patch to handle this as well. https://gerrit.libreoffice.org/18453
Ryan McCoskrie committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d77af785f962efa6f783a5106671d608c313f853 tdf#93684 - Remove SalFrame::Sync It will be available in 5.1.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Migrating Whiteboard tags to Keywords: (easyHack, difficultyBeginner, skillCpp, topicCleanup) [NinjaEdit]
I believe we can close this issue. There are no more calls to Sync() in vcl/. My git grep does not find nothing. ;)
Remove LibreOffice Dev List from CC on EasyHacks (curtailing excessive email to list) [NinjaEdit]