summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2021-07-17cppu : use std::mutex in DisposedCallerAdminArnaud Versini2-5/+4
Change-Id: I3db567a9aed42167ea24eebcf673f19106595f83 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117948 Tested-by: Jenkins Reviewed-by: Arnaud Versini <arnaud.versini@libreoffice.org>
2021-07-17drop sal_uInt16* constructor in SfxItemSetNoel Grandin4-22/+5
Change-Id: Ifb283a49b01c9c6421e385f697e749439db6a53f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119008 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17use WhichRangesContainer in swNoel Grandin29-190/+147
Change-Id: I8662b2e03b0dbe3a7206d8b59ae3556e3b2e75a3 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119007 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17Revert "Notebookbar: skip early init in all apps"Szymon Kłos1-1/+2
This reverts commit 6af0a488a059982a8b5d72fe399c7e8841a2e9b4. Delayed notebookbar init in Calc causes "number format" combobox to not work. (it works after some number is typed into document only). Original problem fixed by reverted change doesn't exist anymore because new shortcuts for paste special and paste unformatted text were introduced Change-Id: Ie112ae0df7b6d3782916b98c96f1a17b15396115 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118771 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com> Reviewed-by: Pranam Lashkari <lpranam@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119101 Tested-by: Jenkins Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
2021-07-17Update git submodulesSteve Fanning1-0/+0
* Update helpcontent2 from branch 'master' to b31f5bed5155ffcda336c3e7a1e18e60a7382228 - Add example to help description of Calc's ROT13 function. Change-Id: I32c654dd8f86f508fce8555f033faa34b81b3e6c Reviewed-on: https://gerrit.libreoffice.org/c/help/+/119038 Tested-by: Jenkins Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2021-07-17jsdialog: move list of enabled dialogs to separate fileSzymon Kłos4-49/+97
Change-Id: I98c92b336fbf76f3dc94265a408c9b3b46fcea08 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115453 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com> Reviewed-by: Andras Timar <andras.timar@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119014 Tested-by: Jenkins Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
2021-07-17Update git submodulesOlivier Hallot1-0/+0
* Update helpcontent2 from branch 'master' to 2271ad00c7e0312071b78706da40a778171bfb9a - Refactor database files Change-Id: I10909197c08cec40819658d2d0951800133409da Reviewed-on: https://gerrit.libreoffice.org/c/help/+/119099 Tested-by: Jenkins Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2021-07-17move BuildWhichTable and simplifyNoel Grandin4-65/+11
move it to the only place using it, and simplify Change-Id: I4a46b5d01f042e7b2e99b7dd1d28f67feda037d0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119086 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17osl::Mutex->std::mutex in SharedStringPoolNoel Grandin1-6/+6
std::mutex is slightly faster Change-Id: I0741cd1ed0a011d5e8d6099c189c85f50060a326 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119087 Tested-by: Noel Grandin <noel.grandin@collabora.co.uk> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17sdremote: introduce early return to improve handleAcceptedConnectionAndrzej Hunt1-49/+51
This should make it easier to understand the handshake sequence. Change-Id: If06e98cdfe7295ed00efae61815a8696a90e9533 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119085 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17sdremote: clean up now-unneeded "found" flagAndrzej Hunt1-8/+3
Since ???, this code now lives in a standalone method, allowing us to return as soon as know that the handshake is complete. This lets us remove aFound and simplify the code a little. Change-Id: I51d5281492d2e4280a101e67d606073f4d064c29 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119084 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17sdremote: avoid infinite loop if client disconnects during handshakeAndrzej Hunt2-54/+64
The following loop can turn into an infinite loop if the client disconnects at the wrong point in time, because pSocket->readLine() returns 0 (indicating disconnection), aLine remains empty (no data was read), and we just keep looping because we never bothered to check the return value: do { pSocket->readLine( aLine ); } while ( aLine.getLength() > 0 ); Aside from spinning unnecessarily, this infinite loop also stops the server from being able to accept any new incoming connections - in effect this causes a DOS of the server. Hence we need to start checking readLine's return value, and break out of this loop - and in fact break of the surrounding outer loop too (because we discard this connection and want to wait for another connection to arrive). Because of the nested looping it's hard to come up with a clean solution that only involves changing this loop - we'd probably need to introduce a temporary to remember that the client has disconnected, and check that temporary after the do...while - letting us 'continue' the outer loop. Therefore we first extract the code that handles newly accepted clients into a separate method, which then lets us simplify the implementation by returning at those points that we know a client has disappeared. That unfortunately makes this bug-fix patch a little more noisy than expected, but it is a refactoring that we'd want to make anyway. (There are further improvement we can make here, but I'll put those in a separate commit to simplify cherry-picking of just this fix. Examples include moving to smart pointers instead of new/delete, introducing an early return instead of the really long if statement, etc.) Change-Id: I13c6efa622a1b1de10b72757ea07e5f4660749fb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119083 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17sdremote: remove commented out experimental checkAndrzej Hunt1-7/+0
The entire if statement was introduced only to check for experimental mode in the following commit - the xContext check is seemingly only needed for null safety: 79c1b16a96a6 (sdremote: make it possible to have only bluetooth enabled, 2012-11-27) Later, the epxerimental mode check was disabled in: b9d2671ae11d (merge WiFi support into remote control feature toggle, 2013-08-01) Given that the remote is clearly no longer experimental, it's time to remove the commented code entirely - and I think it's also safe to remove the xContext check too. (Note: the remote IS still controlled by the EnableSdRemote option, and IS disabled by default.) Change-Id: Id118924021d5029b4f15df9cbb3eba28f3b902c0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119091 Tested-by: Jenkins Reviewed-by: Andrzej Hunt <andrzej@ahunt.org>
2021-07-17[API CHANGE] reduce cost of numeric conversionNoel Grandin5-3/+78
on a hot path, since we already know the length of these strings. Which requires adding some new variants of our string conversion functions Change-Id: I1877f8f3c72934c07f14eec7e73bbe8d7b0f1808 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119065 Tested-by: Noel Grandin <noel.grandin@collabora.co.uk> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17osl::Mutex->std::mutex in OWeakConnectionPointNoel Grandin1-24/+23
it is at least a little bit cheaper than osl::Mutex (since it's not recursive) Change-Id: I4ecbbf33045888b9f5111c4224346341e71d4b05 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119066 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-17tdf#143400: fix query with most functions could not be created in GUIJulien Nabet1-4/+2
Regression from: https://cgit.freedesktop.org/libreoffice/core/commit/?id=04aafba860f613c20e7078d038cc83eb02de0b54 loplugin:stringadd simplify some *StringBuffer operations pulled from a larger patch which I created with a more permissive variant of this plugin Specifically here: -------------- dbaccess/source/ui/querydesign/QueryDesignView.cxx -------------- index 22408bbc58aa..63c5d07998ec 100644 @@ -658,11 +658,10 @@ namespace if ( field->isAggregateFunction() ) { OSL_ENSURE(!field->GetFunction().isEmpty(),"Function name must not be empty! ;-("); - OUStringBuffer aTmpStr2( field->GetFunction()); - aTmpStr2.append("("); - aTmpStr2.append(aTmpStr.makeStringAndClear()); - aTmpStr2.append(")"); - aTmpStr = aTmpStr2; + aTmpStr = field->GetFunction() + + "(" + + aTmpStr + + ")"; } if (!rFieldAlias.isEmpty() && Change-Id: Ib0b57e6c44a58a6a5f9c98aebc6a1213b35108d1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119088 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2021-07-17Bin useless commentTor Lillqvist1-3/+0
Change-Id: Ifc7e36aa1df56cf532b698c8671cabbb35061bac Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119076 Tested-by: Tor Lillqvist <tml@collabora.com> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2021-07-16update creditsChristian Lohmaier1-2314/+2367
Change-Id: I1a09193e888318084a7d9b3c76c049e373db545b
2021-07-16tdf#142923 Deleting comment in Writer leaves visual artifact behindNoel Grandin1-6/+6
Change-Id: Ia2e55886513e81847dd0ac6c697e7cea34e01bdc Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119075 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-16Removed executable bits on source fileAndrea Gelmini1-0/+0
Change-Id: I540ce9b69fcae15f12ac11ca6f66bb8fc7459985 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119073 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Jenkins
2021-07-16fix svl::detail::validRanges2(m_pWhichRanges) assert seen in dbaccessCaolán McNamara1-5/+5
Change-Id: I4328d0eae0030d6455b80d8506496fe44cf02195 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119080 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-16Update git submodulesAndrea Gelmini1-0/+0
* Update helpcontent2 from branch 'master' to ac68c2a4c5efff2c13ae474b37d12fb08be691ff - Fix typo Change-Id: I687dab780da7bcb3379cd2e2e52c1a5b47e98923 Reviewed-on: https://gerrit.libreoffice.org/c/help/+/119071 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2021-07-16Removed executable bits on source fileAndrea Gelmini1-0/+0
Change-Id: I37cf25d37b8b3e5a7c7c1dd2f54bbcc2c9d77a65 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119072 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2021-07-16Fix typosAndrea Gelmini5-5/+5
Change-Id: Iae3eec214849676be51ded133c1ffd9cf2e56ef0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119074 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2021-07-16tdf#92796 ODF import: remove unused bitmap fillsMichael Stahl17-114/+353
With CWS impress64 a partial fix for this was implemented to drop unreferenced named items including all non-color fills after ODF import, but this is only done in sd so move the code that does that to svx and call it from sc and sw as well. Implement some UNO interface for this, it's at least better than a magic string, and not obvious how a better solution would look like since it's known only at the end of the import if a bitmap is used or not. Another problem: when the Area tab is used to change to a different kind of fill, the items with the details for the previous fill aren't cleared, and so they are written to ODF files. Hence bitmaps in the file can be referenced even if they aren't actually used, and bloat up the files. Fix this by dropping all unused draw:fill-image-name attributes in ODF import. Also do the same for Gradient and Hatch fills; Transparency gradients can be combined with anything so leave them as they are. Change-Id: I0b591fd9f963d974d0c3e7208b99621ad61dd93c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118950 Tested-by: Jenkins Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2021-07-16avoid a map lookup on a hot pathNoel Grandin1-4/+4
Change-Id: I4e514c7212c0b349796f3791b4c7f3f06d3fa22a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119064 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-16Related: tdf#143357 explicitly set the default of focus-on-clickCaolán McNamara3-0/+3
this should already be the default according to https://developer.gnome.org/gtk3/stable/GtkWidget.html but it's apparently False nevertheless, so make it explicit here for these widgets where there is a focus watcher to commit their changes and the absence of this is non-cosmetic. Just these initially for a small backportable change. Change-Id: I96442b4e844ce0446f56276c1b648ca5ce57b740 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119077 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2021-07-16crashtesting: UaF on layout of fdo53985-1.docxCaolán McNamara2-1/+58
Change-Id: Id8ca0d277f485347e21bd8d6d68de2a7de13de48 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119060 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2021-07-16Only change SwLayAction::m_bAgain via SetAgainCaolán McNamara4-8/+12
no logic change intended Change-Id: Ib0174f8040faa3efde7b9c5ba9b062bac5a35da3 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118983 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2021-07-16vcl: check mpWindowImpl for nullptr.Michael Meeks1-1/+2
Crash in calc's mouse capture / end tracking. Change-Id: If5b4cded8ebfc04d1fcf645a7b74184532ab2338 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119034 Tested-by: Jenkins Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
2021-07-16Output proper language tag instead of Language+Country concatenationEike Rathke1-1/+2
Change-Id: I83d7f94cf304a58f2ff68b2a667ba66de262f4f3 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119070 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Jenkins
2021-07-16Output proper language tag instead of Language+Country concatenationEike Rathke1-2/+2
That way also for more complex tags with 'qlt' language you get a meaningful output. It's only stderr, but.. Change-Id: I3c6e2451ace5ac9f192a11c97e95376c17046f17 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119068 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Jenkins
2021-07-16Porting C++/Java DocumentLoader example to PythonHossein2-0/+56
* Using UNO for Python * Opening the file in LibreOffice if it is listeing on port 2083 Change-Id: I14b1a38da5d66dd2ee8c859071c148ce08a080d1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118142 Reviewed-by: Thorsten Behrens <thorsten.behrens@allotropia.de> Tested-by: Jenkins
2021-07-16-Werror=class-memaccessNoel Grandin1-2/+4
gcc doesn't like memcpy here error: ‘void* memcpy(void*, const void*, size_t)’ writing to an object of type ‘struct std::pair<short unsigned int, short unsigned int>’ with no trivial copy-assignment; use copy-assignment or copy-initialization instead [-Werror=class-memaccess] 1459 | memcpy(p, other.m_pairs, m_size * sizeof(WhichPair)); Change-Id: I44055d0d4dec589af7f98d62f106b701f1f5a4cd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119063 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-16Clarify that SvtLanguageTable::GetLanguageType() expects the UI nameEike Rathke1-0/+6
Apparently several places get that wrong and pass a lang::Locale.Language instead.. to be investigated. Change-Id: I7f20ae2b66de6527b70f9936ef8216ddc5dac835 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119062 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Jenkins
2021-07-16extensions: bibliography database window: add UI to set a page number on a URLMiklos Vajna3-18/+214
Again, to be consistent with the Define Bibliography Entry dialog which already had this. Change-Id: If43506a3c2cb105551df7ec8198a749315458409 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119021 Reviewed-by: Miklos Vajna <vmiklos@collabora.com> Tested-by: Jenkins
2021-07-16Fix lang::Locale.Language usage, tdf#128191 follow-upEike Rathke1-1/+1
Using plain lang::Locale.Language is always wrong, it may even be 'qlt' for a more complex language tag. As the InfoBar message is "Please install the hyphenation package for locale “%1”." actually use the BCP 47 language tag of that character/paragraph attribution. Spotted in https://ask.libreoffice.org/en/question/280102/install-the-hyphenation-package-for-locale-ka/ https://ask.libreoffice.org/en/question/238764/error-message-reads-install-the-hyphenation-package-for-locale-zh/ Change-Id: I5805d4d711989a9d0260940666d3eb33eae830af Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119020 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Jenkins
2021-07-16Use WhichRangesContainer in editengMike Kaganski5-354/+231
Change-Id: Ia6516df33341181990c2b8b8affa859395831007 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119013 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2021-07-16Resolves: tdf#143398 mouse event processed twiceCaolán McNamara1-4/+0
this is a problem since: commit 67b157e994d9bef01f6117b53fc29e1fee538715 Author: Caolán McNamara <caolanm@redhat.com> Date: Tue Jun 29 13:00:45 2021 +0100 tdf#143088 multiple Application::EventListeners is expensive turns out that AddChildEventListener is sufficient to get events for the widget and its children so the widget-only AddEventListener isn't needed or wanted Change-Id: I635419df10fce1d4441b2d85627a6cbe99adb6aa Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119019 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2021-07-16use common unit converter in docx import of wrap contourRegina Henschel1-8/+19
This is a follow up to commit af99c01570adc0c9205e1650d10866f80bb8118a to address the review comment. Change-Id: Iac0a6cf9cb29273229833465cd2d84cdbcc4a3d0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119004 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2021-07-16Drop unused ctorMike Kaganski2-27/+0
Change-Id: I5144d23d4e6f8e01035ef88a222f609184043c6a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119005 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2021-07-16tdf#139840 - Use utl::TextSearch to implement the InStr functionAndreas Heinisch3-15/+34
In addtion, fixed a crash if the start position is greater than the length of the string being searched. Change-Id: I9bcda1131324bdfac6957018e91b3a36dd2dc3d6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118996 Tested-by: Jenkins Reviewed-by: Andreas Heinisch <andreas.heinisch@yahoo.de>
2021-07-16use WhichRangesContainer in sdNoel Grandin2-14/+4
Change-Id: I528905c14f40bee85689f24ebaef8e8fde1b107b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118999 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-16restore compatibility with older popplersKevin Suo1-3/+8
with poppler 20.09: /home/rene/LibreOffice/git/libreoffice-7-2/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx: In member function 'int pdfi::PDFOutDev::parseFont(long long int, GfxFont*, GfxState*) const': /home/rene/LibreOffice/git/libreoffice-7-2/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx:438:39: error: 'class GfxFont' has no member named 'getNameWithoutSubsetTag' https://www.google.com/search?q=getNameWithoutSubsetTag&oq=getNameWithoutSubsetTag&aqs=chrome..69i57.784j0j7&sourceid=chrome&ie=UTF-8 suggests it was added in 20.12 Change-Id: I4eacd2d740cb689ff9b3c6cab59376e01b1ba162 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118977 Tested-by: René Engelhard <rene@debian.org> Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-16tdf#139654 Font feature sensativityDeb Barkley-Yeung6-11/+88
Disable font preview's "Features..." button if there are no features Change-Id: Ibcf6b34abe55818a3b938a65f4fd720ed724c8ee Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118965 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-07-16tdf#82111 rtf import: ensure CR before section breakJustin Luth3-0/+9
In other places, there were caveats for applying a CR when inserting instead of importing, or when in a header, but none of those seem to apply in this case. A section cannot occur in a table, and anytime a section occurs, it ought to finish off the paragraph. So a simple clause seems good enough here. There were basically two different existing unit tests that match this condition, and neither of them were currently mis-handled. I safely adapted one of them to imitate the error condition I was addressing. Change-Id: Ie8ccd3978276bf10321e0bf80141572f40102874 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118819 Tested-by: Jenkins Reviewed-by: Justin Luth <justin_luth@sil.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2021-07-16sw: handle mbProtectForm when replacing compat optionsMiklos Vajna1-1/+1
This is off by default and it may be in for Word documents. Change-Id: I5f694bb505c2b43bea67712a2076dc5f370d03b0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118994 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2021-07-16use WhichRangesContainer in scNoel Grandin2-7/+5
Change-Id: Ie14250245aa2a8618b3ad6b7a355942621b40833 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118998 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2021-07-16use WhichRangesContainer in reportdesignNoel Grandin2-15/+9
Change-Id: I8ae7e48a1fd0bd5504d68b04ee0af3b3d51d58cd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118997 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2021-07-16sdremote: Transmitter: s/mQueuesNotEmpty/mProcessingRequired/Andrzej Hunt2-8/+8
Make mProcessingRequired's name a bit more self-explanatory to make it clear what we're actually using it for. See also a 8e6cdb0 which fixed a race condition caused by incorrect use of this Condition. Change-Id: I6ad63dbd5ae8ed767f42ea22e568ac47a4d08642 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118752 Tested-by: Jenkins Reviewed-by: Andrzej Hunt <andrzej@ahunt.org>