android/Bootstrap/Makefile.shared | 4 chart2/source/controller/main/ChartWindow.cxx | 1 chart2/source/controller/sidebar/ChartColorWrapper.cxx | 5 chart2/source/controller/sidebar/ChartSidebarModifyListener.cxx | 7 chart2/source/controller/sidebar/ChartSidebarSelectionListener.cxx | 7 external/libpng/StaticLibrary_libpng.mk | 2 framework/source/services/ContextChangeEventMultiplexer.cxx | 24 ++- officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu | 11 + sc/source/ui/undo/undoblk3.cxx | 4 sc/source/ui/view/gridwin3.cxx | 25 +++ sc/source/ui/view/spelldialog.cxx | 8 + sc/source/ui/view/tabvwshe.cxx | 21 ++ sd/source/ui/animations/SlideTransitionPane.cxx | 2 sd/source/ui/func/fusel.cxx | 16 +- sd/source/ui/slidesorter/view/SlideSorterView.cxx | 23 +- sd/source/ui/view/Outliner.cxx | 40 ++--- sfx2/inc/SfxRedactionHelper.hxx | 4 store/source/lockbyte.cxx | 1 store/source/storbase.hxx | 1 svx/source/dialog/hdft.cxx | 79 +++++----- svx/source/tbxctrls/tbxcolor.cxx | 7 svx/source/tbxctrls/tbxcolorupdate.cxx | 2 unotools/source/i18n/resmgr.cxx | 8 + 23 files changed, 212 insertions(+), 90 deletions(-)
New commits: commit b94a77b4efeda9802b21a3fb86e03b168f251a66 Author: Tor Lillqvist <t...@collabora.com> AuthorDate: Thu Apr 30 13:05:28 2020 +0300 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:41:16 2020 +0100 Include <cstdlib> for std::malloc() and std::free() Change-Id: Ice6964858f39709dfd7e55ed8d7fe2d71da6acad Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93205 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Tor Lillqvist <t...@collabora.com> diff --git a/store/source/lockbyte.cxx b/store/source/lockbyte.cxx index dfc7ad920a99..42662b8a4633 100644 --- a/store/source/lockbyte.cxx +++ b/store/source/lockbyte.cxx @@ -29,6 +29,7 @@ #include "object.hxx" #include "storbase.hxx" +#include <cstdlib> #include <memory> #include <string.h> diff --git a/store/source/storbase.hxx b/store/source/storbase.hxx index 0c21acbb3ad3..0c9f9c77d974 100644 --- a/store/source/storbase.hxx +++ b/store/source/storbase.hxx @@ -33,6 +33,7 @@ #include <store/types.h> +#include <cstdlib> #include <memory> #include <utility> commit 7b1903ce4a1818f71943850cd76ba88b7d710ace Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Sat Aug 10 11:33:58 2019 -0400 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:29:57 2020 +0100 sc: correctly update spelling markers on undo/redo (cherry picked from commit 62eb6210895b24855b077fb0d89f182beac4a43c) Reviewed-on: https://gerrit.libreoffice.org/85005 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Michael Meeks <michael.me...@collabora.com> (cherry picked from commit a05fd0eb30767471c6d20c93e0fb8b3ff7c7eaee) Change-Id: I946cede7967f454813f717273c4114d31167c8c6 Reviewed-on: https://gerrit.libreoffice.org/85681 Tested-by: Jenkins Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/sc/source/ui/undo/undoblk3.cxx b/sc/source/ui/undo/undoblk3.cxx index 1459e4dd482d..dbf07d4daabe 100644 --- a/sc/source/ui/undo/undoblk3.cxx +++ b/sc/source/ui/undo/undoblk3.cxx @@ -1234,6 +1234,10 @@ void ScUndoConversion::DoChange( ScDocument* pRefDoc, const ScAddress& rCursorPo pRefDoc->CopyToDocument( 0, 0, 0, rDoc.MaxCol(), rDoc.MaxRow(), nTabCount-1, InsertDeleteFlags::CONTENTS, bMulti, rDoc, &aMarkData ); + + // Reset the spell checking results to re-check on paint, otherwise + // we show the previous spelling markers (or lack thereof on misspellings). + ScDocShell::GetViewData()->GetActiveWin()->ResetAutoSpell(); pDocShell->PostPaintGridAll(); } else commit b310c4ebee2753211230265a7cba546f6a9c2a3e Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Sat Aug 10 12:26:40 2019 -0400 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:29:39 2020 +0100 sc: clear the spell checking markers after applying corrections Change-Id: Ia3fc3b5a21d21b9b945ce72f397350a57859f107 (cherry picked from commit afa5f2b0edb43b36b1b4f0ab8f70ecbe638435e5) Reviewed-on: https://gerrit.libreoffice.org/85006 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Michael Meeks <michael.me...@collabora.com> (cherry picked from commit e1ca2b989bf4d124588c35f7f4a0bc3da3c1f58e) Reviewed-on: https://gerrit.libreoffice.org/85682 Tested-by: Jenkins Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/sc/source/ui/view/spelldialog.cxx b/sc/source/ui/view/spelldialog.cxx index b3dd30f436c3..719a00b5e48f 100644 --- a/sc/source/ui/view/spelldialog.cxx +++ b/sc/source/ui/view/spelldialog.cxx @@ -92,7 +92,12 @@ void ScSpellDialogChildWindow::ApplyChangedSentence( const svx::SpellPortions& r { if( mxEngine.get() && mpViewData ) if( EditView* pEditView = mpViewData->GetSpellingView() ) + { mxEngine->ApplyChangedSentence( *pEditView, rChanged, bRecheck ); + + // Reset the spell checking results to clear the markers. + mpViewData->GetActiveWin()->ResetAutoSpell(); + } } void ScSpellDialogChildWindow::GetFocus() commit 0d86287ab8b42ba66f19404fe8ff55fc10ddc7bf Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Sat Jul 27 20:35:14 2019 -0400 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:29:24 2020 +0100 sc: spelldialog: lock before resetting the editview Painting also resets the view and that can happen asynchronously. When the spelldialog gets focus it resets the editview if it detects it has changed, which happens when, for example, another view is created. (cherry picked from commit d91d940ad0319e0cc6e0080a903ac0a7e9f6ba88) Reviewed-on: https://gerrit.libreoffice.org/85003 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Michael Meeks <michael.me...@collabora.com> (cherry picked from commit 550c0a666e119c911b311c96f9185c11d8465dcb) Change-Id: Ib23e332e4ba069e05787d1cc03ae23fe657b12c8 Reviewed-on: https://gerrit.libreoffice.org/85679 Tested-by: Jenkins Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/sc/source/ui/view/spelldialog.cxx b/sc/source/ui/view/spelldialog.cxx index 4e48cf7698cb..b3dd30f436c3 100644 --- a/sc/source/ui/view/spelldialog.cxx +++ b/sc/source/ui/view/spelldialog.cxx @@ -34,6 +34,7 @@ #include <undoblk.hxx> #include <gridwin.hxx> #include <refupdatecontext.hxx> +#include <vcl/svapp.hxx> SFX_IMPL_CHILDWINDOW_WITHID( ScSpellDialogChildWindow, SID_SPELL_DIALOG ) @@ -96,6 +97,8 @@ void ScSpellDialogChildWindow::ApplyChangedSentence( const svx::SpellPortions& r void ScSpellDialogChildWindow::GetFocus() { + SolarMutexGuard aGuard; + if( IsSelectionChanged() ) { Reset(); commit 44f0ec8a385adbe27dc922fa06e0034096e61092 Author: Tor Lillqvist <t...@collabora.com> AuthorDate: Tue Jan 21 13:52:31 2020 +0200 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:29:00 2020 +0100 tdf#129388: Avoid any actual painting of the slide sorter in the LOKit case The JS code in Online creates its own slides sorter thing anyway, the one is core is only needed as a data structure. Change-Id: I0eeb79c523b72f7b616f11443198d5af82b70643 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87184 Reviewed-by: Tor Lillqvist <t...@collabora.com> Tested-by: Tor Lillqvist <t...@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88444 Tested-by: Jenkins Reviewed-by: Andras Timar <andras.ti...@collabora.com> diff --git a/sd/source/ui/slidesorter/view/SlideSorterView.cxx b/sd/source/ui/slidesorter/view/SlideSorterView.cxx index 5923eb59dd97..a2ae43dae84d 100644 --- a/sd/source/ui/slidesorter/view/SlideSorterView.cxx +++ b/sd/source/ui/slidesorter/view/SlideSorterView.cxx @@ -41,6 +41,7 @@ #include <sdpage.hxx> #include <Window.hxx> +#include <comphelper/lok.hxx> #include <vcl/svapp.hxx> #include <vcl/scrbar.hxx> #include <vcl/settings.hxx> @@ -568,14 +569,18 @@ void SlideSorterView::CompleteRedraw ( sdr::contact::ViewObjectContactRedirector* pRedirector) { (void)pRedirector; + + if (comphelper::LibreOfficeKit::isActive()) + return; + + if (pDevice == nullptr || pDevice!=mrSlideSorter.GetContentWindow()) + return; + #ifdef DEBUG_TIMING const double nStartTime (gaTimer.getElapsedTime()); SAL_INFO("sd.timing", "SlideSorterView::CompleteRedraw start" << (mnLockRedrawSmph ? " locked" : "")); #endif - if (pDevice == nullptr || pDevice!=mrSlideSorter.GetContentWindow()) - return; - // The parent implementation of CompleteRedraw is called only when // painting is locked. We do all the painting ourself. When painting // is locked the parent implementation keeps track of the repaint commit 5770302e36ec2ca60ea72ff725cbfeb44db73340 Author: Tor Lillqvist <t...@collabora.com> AuthorDate: Tue Jan 21 13:43:07 2020 +0200 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:28:40 2020 +0100 Unbreak code compiled conditionally with DEBUG_TIMING But it will no doubt break eventually again, and not be noticed until next time somebody wants to try to see the information that defining DEBUG_TIMING produces in this file. This kind of conditionally compiled debug code is IMHO counter-productive. It is trivial to temporarily add timing information output to any function just locally in your own build. So I am not opposed to killing this DEBUG_TIMING stuff here. Change-Id: I940e0a40cf2bd9f319c53a6ccff7a2413d1323e5 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87183 Reviewed-by: Tor Lillqvist <t...@collabora.com> Tested-by: Tor Lillqvist <t...@collabora.com> (cherry picked from commit b81aa8a9930da5f93c074e7dd349c32ce49268de) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87255 Tested-by: Jenkins diff --git a/sd/source/ui/slidesorter/view/SlideSorterView.cxx b/sd/source/ui/slidesorter/view/SlideSorterView.cxx index 32a8e8c2bc6b..5923eb59dd97 100644 --- a/sd/source/ui/slidesorter/view/SlideSorterView.cxx +++ b/sd/source/ui/slidesorter/view/SlideSorterView.cxx @@ -558,7 +558,7 @@ static const size_t gFrameTimeCount (10); static size_t gFrameTimeIndex (0); static ::std::vector<double> gFrameTimes (gFrameTimeCount, 0); static double gFrameTimeSum (0); -static const Rectangle gFrameTimeBox (10,10,150,20); +static const ::tools::Rectangle gFrameTimeBox (10,10,150,20); static double gnLastFrameStart = 0; #endif @@ -570,9 +570,7 @@ void SlideSorterView::CompleteRedraw ( (void)pRedirector; #ifdef DEBUG_TIMING const double nStartTime (gaTimer.getElapsedTime()); - SAL_INFO("sd",("SlideSorterView::CompleteRedraw start at %f, %s", - nStartTime, - mnLockRedrawSmph ? "locked" : ""); + SAL_INFO("sd.timing", "SlideSorterView::CompleteRedraw start" << (mnLockRedrawSmph ? " locked" : "")); #endif if (pDevice == nullptr || pDevice!=mrSlideSorter.GetContentWindow()) @@ -597,16 +595,16 @@ void SlideSorterView::CompleteRedraw ( #ifdef DEBUG_TIMING const double nEndTime (gaTimer.getElapsedTime()); - SAL_INFO("sd",("SlideSorterView::CompleteRedraw end at %f after %fms", nEndTime, (nEndTime-nStartTime)*1000); + SAL_INFO("sd.timing", "SlideSorterView::CompleteRedraw end after " << (nEndTime-nStartTime)*1000 << " ms"); gFrameTimeSum -= gFrameTimes[gFrameTimeIndex]; gFrameTimes[gFrameTimeIndex] = nStartTime - gnLastFrameStart; gnLastFrameStart = nStartTime; gFrameTimeSum += gFrameTimes[gFrameTimeIndex]; gFrameTimeIndex = (gFrameTimeIndex+1) % gFrameTimeCount; - mrSlideSorter.GetContentWindow()->SetFillCOL_BLUE); + mrSlideSorter.GetContentWindow()->SetFillColor(COL_BLUE); mrSlideSorter.GetContentWindow()->DrawRect(gFrameTimeBox); - mrSlideSorter.GetContentWindow()->SetTextCOL_WHITE); + mrSlideSorter.GetContentWindow()->SetTextColor(COL_WHITE); mrSlideSorter.GetContentWindow()->DrawText( gFrameTimeBox, OUString::number(1 / (gFrameTimeSum / gFrameTimeCount)), commit 764b7dd63551a3f9d973e7c02d9086835f54e7ec Author: Mert Tumer <mert.tu...@collabora.com> AuthorDate: Wed Mar 18 13:17:32 2020 +0300 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:27:29 2020 +0100 mobile: fix hyperlinks are not clickable on impress Change-Id: I77a0ef263814359607d9c1d863f93c55f3493dbb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90687 Tested-by: Andras Timar <andras.ti...@collabora.com> Reviewed-by: Andras Timar <andras.ti...@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92211 Tested-by: Jenkins diff --git a/sd/source/ui/func/fusel.cxx b/sd/source/ui/func/fusel.cxx index b425727b67af..18616775ac9e 100644 --- a/sd/source/ui/func/fusel.cxx +++ b/sd/source/ui/func/fusel.cxx @@ -266,6 +266,14 @@ bool FuSelection::MouseButtonDown(const MouseEvent& rMEvt) { mpWindow->ReleaseMouse(); + // If tiled rendering, let client handles URL execution and early returns. + if (comphelper::LibreOfficeKit::isActive()) + { + SfxViewShell& rSfxViewShell = mpViewShell->GetViewShellBase(); + rSfxViewShell.libreOfficeKitViewCallback(LOK_CALLBACK_HYPERLINK_CLICKED, aVEvt.pURLField->GetURL().toUtf8().getStr()); + return true; + } + if (!lcl_followHyperlinkAllowed(rMEvt)) return true; @@ -275,14 +283,6 @@ bool FuSelection::MouseButtonDown(const MouseEvent& rMEvt) SfxViewFrame* pFrame = mpViewShell->GetViewFrame(); mpWindow->ReleaseMouse(); - // If tiled rendering, let client handles URL execution and early returns. - if (comphelper::LibreOfficeKit::isActive()) - { - SfxViewShell& rSfxViewShell = mpViewShell->GetViewShellBase(); - rSfxViewShell.libreOfficeKitViewCallback(LOK_CALLBACK_HYPERLINK_CLICKED, aVEvt.pURLField->GetURL().toUtf8().getStr()); - return true; - } - if (rMEvt.IsMod1()) { // Open in new frame commit 30a9c349e7b7db7a44c2f8bc7d282fb92d9c6766 Author: Dennis Francis <dennis.fran...@collabora.com> AuthorDate: Sun Feb 23 22:38:15 2020 +0530 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:26:32 2020 +0100 chart2: Fix the color uno command in ChartColorWrapper The uno command for color depends on the property name. For LineColor it should be .uno:XLineColor and the only other case is FillColor for which it should be .uno:FillColor. Without this fix, on selecting the chart for editing the first time, the sidebar line-color control is disabled as a side-effect. Change-Id: Ia71ed2f6d9e0f31523f1415f3ee089fd9d7d1b2d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89304 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Andras Timar <andras.ti...@collabora.com> (cherry picked from commit 9a80969f3115fc33778005861442f91127344dc0) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89677 Tested-by: Jenkins diff --git a/chart2/source/controller/sidebar/ChartColorWrapper.cxx b/chart2/source/controller/sidebar/ChartColorWrapper.cxx index 31808b1a292b..128fc6848342 100644 --- a/chart2/source/controller/sidebar/ChartColorWrapper.cxx +++ b/chart2/source/controller/sidebar/ChartColorWrapper.cxx @@ -96,12 +96,15 @@ void ChartColorWrapper::updateModel(const css::uno::Reference<css::frame::XModel void ChartColorWrapper::updateData() { + static const OUStringLiteral aLineColor = "LineColor"; + static const OUStringLiteral aCommands[2] = {".uno:XLineColor", ".uno:FillColor"}; + css::uno::Reference<css::beans::XPropertySet> xPropSet = getPropSet(mxModel); if (!xPropSet.is()) return; css::util::URL aUrl; - aUrl.Complete = ".uno:FillColor"; + aUrl.Complete = (maPropertyName == aLineColor) ? aCommands[0] : aCommands[1]; css::frame::FeatureStateEvent aEvent; aEvent.FeatureURL = aUrl; commit e3fe5d4e0e63d99cb984f431b2ae8e6e9dfb077d Author: Tor Lillqvist <t...@collabora.com> AuthorDate: Fri Dec 20 16:17:13 2019 +0200 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:24:58 2020 +0100 tdf#126966: Tweak to get the "color bar" below text color controls to show up (In iOS-only code, for now.) No idea where the size that the thing has to fit inside comes from; possibly one could alternatively have made that larger? (cherry picked from commit 4de843ee0de7f9211f536dc6c4ee0d05bf26cc52) Change-Id: Ifc1bec4c08c279403127056ecc61db4b80c03f96 diff --git a/svx/source/tbxctrls/tbxcolorupdate.cxx b/svx/source/tbxctrls/tbxcolorupdate.cxx index 9465e9aad2f6..547ae1c42757 100644 --- a/svx/source/tbxctrls/tbxcolorupdate.cxx +++ b/svx/source/tbxctrls/tbxcolorupdate.cxx @@ -93,7 +93,7 @@ namespace svx if (!mbWideButton) { // usually the normal size is 16 - const long nIOSSize = 40; + const long nIOSSize = 32; if (aItemSize.getWidth() < nIOSSize) { aItemSize.setWidth(nIOSSize); commit f355f6bc922cc633f20cae49c2560f77fb03c3d2 Author: Tor Lillqvist <t...@collabora.com> AuthorDate: Fri Dec 20 16:36:31 2019 +0200 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:24:36 2020 +0100 Get rid of two pointless #defines Change-Id: I7479f4f1c440bba1dd9f922c4df32ab877e0a803 Reviewed-on: https://gerrit.libreoffice.org/85619 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Tor Lillqvist <t...@collabora.com> (cherry picked from commit 83fedc02f2d831774c12d60097a99e968b56dc5e) diff --git a/svx/source/tbxctrls/tbxcolor.cxx b/svx/source/tbxctrls/tbxcolor.cxx index 5336f070aadf..3a17e38820a4 100644 --- a/svx/source/tbxctrls/tbxcolor.cxx +++ b/svx/source/tbxctrls/tbxcolor.cxx @@ -34,11 +34,8 @@ namespace svx using namespace ::com::sun::star::frame; using namespace ::com::sun::star::beans; - #define TOOLBAR_RESNAME "private:resource/toolbar/" - #define PROPNAME_LAYOUTMANAGER "LayoutManager" - ToolboxAccess::ToolboxAccess( const OUString& rToolboxName ) : - m_sToolboxResName ( TOOLBAR_RESNAME ) + m_sToolboxResName ( "private:resource/toolbar/" ) { m_sToolboxResName += rToolboxName; @@ -50,7 +47,7 @@ namespace svx Reference< XFrame > xFrame = SfxViewFrame::Current()->GetFrame().GetFrameInterface(); Reference< XPropertySet > xFrameProps( xFrame, UNO_QUERY ); if ( xFrameProps.is() ) - xFrameProps->getPropertyValue( PROPNAME_LAYOUTMANAGER ) >>= m_xLayouter; + xFrameProps->getPropertyValue( "LayoutManager" ) >>= m_xLayouter; } catch ( Exception const & ) { commit e910d27a297c9d05945ff160b2aac559f8703df5 Author: Muhammet Kara <muhammet.k...@collabora.com> AuthorDate: Thu Dec 19 11:35:13 2019 +0300 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:24:16 2020 +0100 lok: Improve hyperlink insertion to cell with content Change-Id: I520939dd5b44236cf835108ee2bc96c29f0a9677 Reviewed-on: https://gerrit.libreoffice.org/85509 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Marco Cecchetti <marco.cecche...@collabora.com> Tested-by: Marco Cecchetti <marco.cecche...@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89707 Tested-by: Jenkins Reviewed-by: Muhammet Kara <muhammet.k...@collabora.com> diff --git a/sc/source/ui/view/tabvwshe.cxx b/sc/source/ui/view/tabvwshe.cxx index fece06e7d8ec..d3fa448a6538 100644 --- a/sc/source/ui/view/tabvwshe.cxx +++ b/sc/source/ui/view/tabvwshe.cxx @@ -18,6 +18,7 @@ */ #include <comphelper/string.hxx> +#include <comphelper/lok.hxx> #include <editeng/eeitem.hxx> #include <editeng/editview.hxx> @@ -172,7 +173,11 @@ void ScTabViewShell::InsertURLField( const OUString& rName, const OUString& rURL ScInputHandler* pHdl = pScMod->GetInputHdl( rViewData.GetViewShell() ); bool bSelectFirst = false; - if ( !pScMod->IsEditMode() ) + bool bIsEditMode = pScMod->IsEditMode(); + int nSelInd = 1; + OUString sSeltext(GetSelectionText()); + + if ( !bIsEditMode ) { if ( !SelectionEditable() ) { @@ -189,12 +194,24 @@ void ScTabViewShell::InsertURLField( const OUString& rName, const OUString& rURL EditView* pTableView = pHdl->GetTableView(); OSL_ENSURE( pTopView || pTableView, "No EditView" ); + // Check if user selected a whole cell by single click, + // cell has content, and user didn't change the name/text + // of the link something different than the content via the hyperlink dialog. + // If true, assign the given hyperlink to the whole content + // instead of inserting a duplicate, or appending the url. + if (comphelper::LibreOfficeKit::isActive() && !bIsEditMode && !bSelectFirst + && pTableView && !sSeltext.isEmpty() && sSeltext == rName) + { + nSelInd = sSeltext.getLength(); + bSelectFirst = true; + } + if ( bSelectFirst ) { if ( pTopView ) pTopView->SetSelection( ESelection(0,0,0,1) ); if ( pTableView ) - pTableView->SetSelection( ESelection(0,0,0,1) ); + pTableView->SetSelection( ESelection(0,0,0,nSelInd) ); } pHdl->DataChanging(); commit 6e69240276aa8aa473e3417a819a7d5a8a6bc796 Author: Jan Holesovsky <ke...@collabora.com> AuthorDate: Wed Dec 18 23:07:11 2019 +0100 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:23:57 2020 +0100 android: Use correct path for the .mo resource files. They are read by boost::locale, so we cannot have them as normal assets, instead we have to have them in asset's 'unpack', so that they are unpacked to the filesystem and can be read as normal files. Change-Id: I5e902ee4b5294d1f693f51993b8bc2069f1c835a Reviewed-on: https://gerrit.libreoffice.org/85442 Reviewed-by: Michael Meeks <michael.me...@collabora.com> Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/85458 Tested-by: Michael Meeks <michael.me...@collabora.com> diff --git a/unotools/source/i18n/resmgr.cxx b/unotools/source/i18n/resmgr.cxx index 786c83df2e7b..197816b21fc3 100644 --- a/unotools/source/i18n/resmgr.cxx +++ b/unotools/source/i18n/resmgr.cxx @@ -51,6 +51,10 @@ #include <unordered_map> #include <memory> +#ifdef ANDROID +#include <osl/detail/android-bootstrap.h> +#endif + #if defined(_WIN32) && defined(DBG_UTIL) #include <o3tl/char16_t2wchar_t.hxx> #include <prewin.h> @@ -123,11 +127,15 @@ namespace Translate boost::locale::generator gen; gen.characters(boost::locale::char_facet); gen.categories(boost::locale::message_facet | boost::locale::information_facet); +#if defined(ANDROID) + OString sPath(OString(lo_get_app_data_dir()) + "/program/resource"); +#else OUString uri("$BRAND_BASE_DIR/$BRAND_SHARE_RESOURCE_SUBDIR/"); rtl::Bootstrap::expandMacros(uri); OUString path; osl::File::getSystemPathFromFileURL(uri, path); OString sPath(OUStringToOString(path, osl_getThreadTextEncoding())); +#endif gen.add_messages_path(sPath.getStr()); #if defined UNX && !defined MACOSX && !defined IOS && !defined ANDROID // allow gettext to find these .mo files e.g. so gtk dialogs can use them commit 8a1abb99996df357ea806248f6cc9e0ab476d4c1 Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Wed Dec 11 18:19:43 2019 -0500 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:23:40 2020 +0100 chart2: clear listener parents in dispose Change-Id: Icdab125589ab0f55255f683a1ea39e036de661b1 Reviewed-on: https://gerrit.libreoffice.org/84997 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/chart2/source/controller/main/ChartWindow.cxx b/chart2/source/controller/main/ChartWindow.cxx index 2a3bd54a85e6..0971fd079ca1 100644 --- a/chart2/source/controller/main/ChartWindow.cxx +++ b/chart2/source/controller/main/ChartWindow.cxx @@ -74,6 +74,7 @@ ChartWindow::~ChartWindow() void ChartWindow::dispose() { + m_pWindowController = nullptr; m_pViewShellWindow.clear(); vcl::Window::dispose(); } diff --git a/chart2/source/controller/sidebar/ChartSidebarModifyListener.cxx b/chart2/source/controller/sidebar/ChartSidebarModifyListener.cxx index b000e55c5aa6..0fa0b793b5b6 100644 --- a/chart2/source/controller/sidebar/ChartSidebarModifyListener.cxx +++ b/chart2/source/controller/sidebar/ChartSidebarModifyListener.cxx @@ -27,12 +27,17 @@ ChartSidebarModifyListener::~ChartSidebarModifyListener() void ChartSidebarModifyListener::modified(const css::lang::EventObject& /*rEvent*/) { - mpParent->updateData(); + if (mpParent) + mpParent->updateData(); } void ChartSidebarModifyListener::disposing(const css::lang::EventObject& /*rEvent*/) { + if (!mpParent) + return; + mpParent->modelInvalid(); + mpParent = nullptr; } } } diff --git a/chart2/source/controller/sidebar/ChartSidebarSelectionListener.cxx b/chart2/source/controller/sidebar/ChartSidebarSelectionListener.cxx index e037fc39aa5f..e7d74c656a55 100644 --- a/chart2/source/controller/sidebar/ChartSidebarSelectionListener.cxx +++ b/chart2/source/controller/sidebar/ChartSidebarSelectionListener.cxx @@ -41,6 +41,9 @@ ChartSidebarSelectionListener::~ChartSidebarSelectionListener() void ChartSidebarSelectionListener::selectionChanged(const css::lang::EventObject& rEvent) { + if (!mpParent) + return; + bool bCorrectObjectSelected = false; css::uno::Reference<css::frame::XController> xController(rEvent.Source, css::uno::UNO_QUERY); @@ -66,6 +69,10 @@ void ChartSidebarSelectionListener::selectionChanged(const css::lang::EventObjec void ChartSidebarSelectionListener::disposing(const css::lang::EventObject& /*rEvent*/) { + if (!mpParent) + return; + + mpParent = nullptr; } void ChartSidebarSelectionListener::setAcceptedTypes(const std::vector<ObjectType>& aTypes) commit 59d56949c6bb847e398489af00639f5144c288ff Author: Marco Cecchetti <marco.cecche...@collabora.com> AuthorDate: Mon Dec 9 10:48:15 2019 +0100 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:23:17 2020 +0100 lok: calc: zoom: object selection - is unreliable Change-Id: Ic8e61c56bdf38bbb74f179fa36075bdd3a601249 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90068 Tested-by: Jenkins Reviewed-by: Marco Cecchetti <marco.cecche...@collabora.com> diff --git a/sc/source/ui/view/gridwin3.cxx b/sc/source/ui/view/gridwin3.cxx index 1fbc48e0ae6f..7c6c6ad9a55e 100644 --- a/sc/source/ui/view/gridwin3.cxx +++ b/sc/source/ui/view/gridwin3.cxx @@ -41,6 +41,11 @@ bool ScGridWindow::DrawMouseButtonDown(const MouseEvent& rMEvt) FuPoor* pDraw = pViewData->GetView()->GetDrawFuncPtr(); if (pDraw && !pViewData->IsRefMode()) { + MapMode aDrawMode = GetDrawMapMode(); + MapMode aOldMode = GetMapMode(); + if ( comphelper::LibreOfficeKit::isActive() && aOldMode != aDrawMode ) + SetMapMode( aDrawMode ); + pDraw->SetWindow( this ); Point aLogicPos = PixelToLogic(rMEvt.GetPosPixel()); if ( pDraw->IsDetectiveHit( aLogicPos ) ) @@ -54,6 +59,9 @@ bool ScGridWindow::DrawMouseButtonDown(const MouseEvent& rMEvt) if ( bRet ) UpdateStatusPosSize(); } + + if ( comphelper::LibreOfficeKit::isActive() && aOldMode != aDrawMode ) + SetMapMode( aOldMode ); } // cancel draw with right key @@ -73,6 +81,11 @@ bool ScGridWindow::DrawMouseButtonUp(const MouseEvent& rMEvt) FuPoor* pDraw = pView->GetDrawFuncPtr(); if (pDraw && !pViewData->IsRefMode()) { + MapMode aDrawMode = GetDrawMapMode(); + MapMode aOldMode = GetMapMode(); + if ( comphelper::LibreOfficeKit::isActive() && aOldMode != aDrawMode ) + SetMapMode( aDrawMode ); + pDraw->SetWindow( this ); bRet = pDraw->MouseButtonUp( rMEvt ); @@ -89,6 +102,9 @@ bool ScGridWindow::DrawMouseButtonUp(const MouseEvent& rMEvt) if ( !pView->IsPaintBrushLocked() ) pView->ResetBrushDocument(); // end paint brush mode if not locked } + + if ( comphelper::LibreOfficeKit::isActive() && aOldMode != aDrawMode ) + SetMapMode( aOldMode ); } return bRet; @@ -99,10 +115,19 @@ bool ScGridWindow::DrawMouseMove(const MouseEvent& rMEvt) FuPoor* pDraw = pViewData->GetView()->GetDrawFuncPtr(); if (pDraw && !pViewData->IsRefMode()) { + MapMode aDrawMode = GetDrawMapMode(); + MapMode aOldMode = GetMapMode(); + if ( comphelper::LibreOfficeKit::isActive() && aOldMode != aDrawMode ) + SetMapMode( aDrawMode ); + pDraw->SetWindow( this ); bool bRet = pDraw->MouseMove( rMEvt ); if ( bRet ) UpdateStatusPosSize(); + + if ( comphelper::LibreOfficeKit::isActive() && aOldMode != aDrawMode ) + SetMapMode( aOldMode ); + return bRet; } else commit 3ad3024b84e53753865387c0af4a63e98d2f0559 Author: Henry Castro <hcas...@collabora.com> AuthorDate: Thu Aug 22 16:43:54 2019 -0400 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:22:29 2020 +0100 sd: define three columns for the slide transition pane Instead of calculating extra space Change-Id: I26e763e5db0ff05fb1bd6f64f3054d8ef88a2156 Reviewed-on: https://gerrit.libreoffice.org/82526 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Henry Castro <hcas...@collabora.com> (cherry picked from commit 58a3e6ad29eb7b616f03737265ef7a25afe83a36) Reviewed-on: https://gerrit.libreoffice.org/82673 Tested-by: Jenkins diff --git a/sd/source/ui/animations/SlideTransitionPane.cxx b/sd/source/ui/animations/SlideTransitionPane.cxx index 0a7459b66fea..05c11354f4b4 100644 --- a/sd/source/ui/animations/SlideTransitionPane.cxx +++ b/sd/source/ui/animations/SlideTransitionPane.cxx @@ -379,7 +379,7 @@ public: EnableFullItemMode( false ); set_hexpand( true ); set_vexpand( true ); - SetExtraSpacing( 2 ); + SetColCount(3); } virtual ~TransitionPane() override { disposeOnce(); } commit 9277ce2128f1aaacdcb41448797c83395603ca0c Author: Muhammet Kara <muhammet.k...@collabora.com> AuthorDate: Fri Nov 22 13:16:25 2019 +0300 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:22:10 2020 +0100 Add command .uno:EditChart This is an alias uno command which let's users switch to edit mode for the selected object. It is equivalent to the Edit > Object > Edit menu item, which is a parameterized variation of the .uno:ObjectMenue command, and is created dynamically each time user opens the Edit menu. Change-Id: I131fab1758d2689cecfa3adc79b7868abf67b5e0 Reviewed-on: https://gerrit.libreoffice.org/83463 Tested-by: Jenkins Reviewed-by: Muhammet Kara <muhammet.k...@collabora.com> diff --git a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu index 28ec0c674e3a..08ff909b5db5 100644 --- a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu +++ b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu @@ -7146,6 +7146,17 @@ <value>1</value> </prop> </node> + <node oor:name=".uno:EditChart" oor:op="replace"> + <prop oor:name="Label" oor:type="xs:string"> + <value xml:lang="en-US">~Edit Chart</value> + </prop> + <prop oor:name="TargetURL" oor:type="xs:string"> + <value>.uno:ObjectMenue?VerbID:short=-1</value> + </prop> + <prop oor:name="Properties" oor:type="xs:int"> + <value>1</value> + </prop> + </node> </node> </node> </oor:component-data> commit 2f32e19a6aca8e3361a05a4c333c39225775ef9e Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Sun May 26 14:59:53 2019 -0400 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:21:53 2020 +0100 svx: LOK: convert background child-dialogs to async (cherry picked from commit 625a8c33bd95b4068838caef19af28c5404c59ce) Reviewed-on: https://gerrit.libreoffice.org/83633 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> (cherry picked from commit 46a1c0a733f1e664823dcb483796dbcc8a743f4a) Change-Id: I682ab039ce3732aa973e0fe371c191f1ca1b9ef7 Reviewed-on: https://gerrit.libreoffice.org/85677 Tested-by: Jenkins Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/svx/source/dialog/hdft.cxx b/svx/source/dialog/hdft.cxx index c936e91ee740..b92dc46e8deb 100644 --- a/svx/source/dialog/hdft.cxx +++ b/svx/source/dialog/hdft.cxx @@ -624,54 +624,59 @@ IMPL_LINK_NOARG(SvxHFPage, BackgroundHdl, weld::Button&, void) *pBBSet, mbEnableDrawingLayerFillStyles)); - if(RET_OK == pDlg->Execute() && pDlg->GetOutputItemSet()) - { - SfxItemIter aIter(*pDlg->GetOutputItemSet()); - - for (const SfxPoolItem* pItem = aIter.GetCurItem(); pItem; pItem = aIter.NextItem()) + pDlg->StartExecuteAsync([&](sal_Int32 nResult) { + if (nResult == RET_OK && pDlg->GetOutputItemSet()) { - if(!IsInvalidItem(pItem)) - { - pBBSet->Put(*pItem); - } - } + SfxItemIter aIter(*pDlg->GetOutputItemSet()); - { - drawinglayer::attribute::SdrAllFillAttributesHelperPtr aFillAttributes; - - if(mbEnableDrawingLayerFillStyles) + for (const SfxPoolItem* pItem = aIter.GetCurItem(); pItem; pItem = aIter.NextItem()) { - // create FillAttributes directly from DrawingLayer FillStyle entries - aFillAttributes.reset(new drawinglayer::attribute::SdrAllFillAttributesHelper(*pBBSet)); + if(!IsInvalidItem(pItem)) + { + pBBSet->Put(*pItem); + } } - else + { - const sal_uInt16 nWhich = GetWhich(SID_ATTR_BRUSH); + drawinglayer::attribute::SdrAllFillAttributesHelperPtr aFillAttributes; - if(pBBSet->GetItemState(nWhich) == SfxItemState::SET) + if (mbEnableDrawingLayerFillStyles) { - // create FillAttributes from SvxBrushItem - const SvxBrushItem& rItem = static_cast< const SvxBrushItem& >(pBBSet->Get(nWhich)); - SfxItemSet aTempSet(*pBBSet->GetPool(), svl::Items<XATTR_FILL_FIRST, XATTR_FILL_LAST>{}); - - setSvxBrushItemAsFillAttributesToTargetSet(rItem, aTempSet); - aFillAttributes.reset(new drawinglayer::attribute::SdrAllFillAttributesHelper(aTempSet)); + // create FillAttributes directly from DrawingLayer FillStyle entries + aFillAttributes.reset( + new drawinglayer::attribute::SdrAllFillAttributesHelper(*pBBSet)); + } + else + { + const sal_uInt16 nWhich = GetWhich(SID_ATTR_BRUSH); + + if (pBBSet->GetItemState(nWhich) == SfxItemState::SET) + { + // create FillAttributes from SvxBrushItem + const SvxBrushItem& rItem + = static_cast<const SvxBrushItem&>(pBBSet->Get(nWhich)); + SfxItemSet aTempSet(*pBBSet->GetPool(), + svl::Items<XATTR_FILL_FIRST, XATTR_FILL_LAST>{}); + + setSvxBrushItemAsFillAttributesToTargetSet(rItem, aTempSet); + aFillAttributes.reset( + new drawinglayer::attribute::SdrAllFillAttributesHelper(aTempSet)); + } } - } - if(SID_ATTR_PAGE_HEADERSET == nId) - { - //m_aBspWin.SetHdColor(rItem.GetColor()); - m_aBspWin.setHeaderFillAttributes(aFillAttributes); - } - else - { - //m_aBspWin.SetFtColor(rItem.GetColor()); - m_aBspWin.setFooterFillAttributes(aFillAttributes); + if (SID_ATTR_PAGE_HEADERSET == nId) + { + //m_aBspWin.SetHdColor(rItem.GetColor()); + m_aBspWin.setHeaderFillAttributes(aFillAttributes); + } + else + { + //m_aBspWin.SetFtColor(rItem.GetColor()); + m_aBspWin.setFooterFillAttributes(aFillAttributes); + } } } - - } + }); UpdateExample(); } commit 08bf8101717ea293a1340b442c710de6a5dfc6dd Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Sat May 25 12:15:00 2019 -0400 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:21:38 2020 +0100 sd: LOK: don't access null dispatcher With SpellDialog invoked from multiple views the dispatcher can indeed be null in some cases. (cherry picked from commit b18c12f4dbe91379cef795d559c2d84341e32a75) Reviewed-on: https://gerrit.libreoffice.org/83631 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> (cherry picked from commit 188ff24ee209d4f5f8ec6a6ae82241e56f0be6d5) Change-Id: Ic2950e70383a77d63438341fac174d369748b36a Reviewed-on: https://gerrit.libreoffice.org/85676 Tested-by: Jenkins Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/sd/source/ui/view/Outliner.cxx b/sd/source/ui/view/Outliner.cxx index d3bd66724c7b..e67d823e9a11 100644 --- a/sd/source/ui/view/Outliner.cxx +++ b/sd/source/ui/view/Outliner.cxx @@ -1402,29 +1402,33 @@ void SdOutliner::EnterEditMode (bool bGrabFocus) // Make FuText the current function. SfxUInt16Item aItem (SID_TEXTEDIT, 1); std::shared_ptr<sd::ViewShell> pViewShell (mpWeakViewShell.lock()); - pViewShell->GetDispatcher()->ExecuteList(SID_TEXTEDIT, - SfxCallMode::SYNCHRON | SfxCallMode::RECORD, { &aItem }); - - if (mpView->IsTextEdit()) + if (pViewShell && pViewShell->GetDispatcher()) { - // end text edition before starting it again - mpView->SdrEndTextEdit(); - } + pViewShell->GetDispatcher()->ExecuteList( + SID_TEXTEDIT, SfxCallMode::SYNCHRON | SfxCallMode::RECORD, {&aItem}); - // To be consistent with the usual behaviour in the Office the text - // object that is put into edit mode would have also to be selected. - // Starting the text edit mode is not enough so we do it here by - // hand. - mpView->UnmarkAllObj (pPV); - mpView->MarkObj (mpSearchSpellTextObj, pPV); + if (mpView->IsTextEdit()) + { + // end text edition before starting it again + mpView->SdrEndTextEdit(); + } - mpSearchSpellTextObj->setActiveText( mnText ); + // To be consistent with the usual behaviour in the Office the text + // object that is put into edit mode would have also to be selected. + // Starting the text edit mode is not enough so we do it here by + // hand. + mpView->UnmarkAllObj(pPV); + mpView->MarkObj(mpSearchSpellTextObj, pPV); - // Turn on the edit mode for the text object. - mpView->SdrBeginTextEdit(mpSearchSpellTextObj, pPV, mpWindow, true, this, pOutlinerView, true, true, bGrabFocus); + mpSearchSpellTextObj->setActiveText(mnText); - SetUpdateMode(true); - mbFoundObject = true; + // Turn on the edit mode for the text object. + mpView->SdrBeginTextEdit(mpSearchSpellTextObj, pPV, mpWindow, true, this, + pOutlinerView, true, true, bGrabFocus); + + SetUpdateMode(true); + mbFoundObject = true; + } } ESelection SdOutliner::GetSearchStartPosition() const commit 0188224f0465856e19e861c637b41697ca828a8e Author: Muhammet Kara <muhammet.k...@collabora.com> AuthorDate: Mon Nov 25 21:55:31 2019 +0300 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:20:59 2020 +0100 tdf#94288: Show chart props sidebar on activation Change-Id: I2119b59e3b4e36a34c9925cb624558d2da49ecc7 Reviewed-on: https://gerrit.libreoffice.org/83715 Tested-by: Jenkins Reviewed-by: Muhammet Kara <muhammet.k...@collabora.com> diff --git a/framework/source/services/ContextChangeEventMultiplexer.cxx b/framework/source/services/ContextChangeEventMultiplexer.cxx index b64a4e131717..c6da6be6b0d1 100644 --- a/framework/source/services/ContextChangeEventMultiplexer.cxx +++ b/framework/source/services/ContextChangeEventMultiplexer.cxx @@ -163,11 +163,29 @@ void SAL_CALL ContextChangeEventMultiplexer::addContextChangeEventListener ( // the current context. if (rxEventFocus.is() && pFocusDescriptor!=nullptr) { + if (pFocusDescriptor->msCurrentApplicationName.isEmpty() && pFocusDescriptor->msCurrentContextName.isEmpty() + && rxEventFocus.is()) + { + Reference< lang::XServiceInfo > xServInfo( rxEventFocus, uno::UNO_QUERY_THROW ); + if( xServInfo.is() && xServInfo->getImplementationName() == "com.sun.star.comp.chart2.ChartController") + { + css::ui::ContextChangeEventObject aEvent ( + rxEventFocus, + "com.sun.star.chart2.ChartDocument", + "Chart"); + rxListener->notifyContextChangeEvent(aEvent); + + return; + } + + } + css::ui::ContextChangeEventObject aEvent ( - nullptr, - pFocusDescriptor->msCurrentApplicationName, - pFocusDescriptor->msCurrentContextName); + nullptr, + pFocusDescriptor->msCurrentApplicationName, + pFocusDescriptor->msCurrentContextName); rxListener->notifyContextChangeEvent(aEvent); + } } commit 9bc800ce75f0420ad279b67d2d1913bf62f772cc Author: Muhammet Kara <muhammet.k...@collabora.com> AuthorDate: Tue Nov 26 14:53:44 2019 +0300 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:20:46 2020 +0100 Comment the searchInMetaFile method Change-Id: Icbf1f852f8deb5392a820b46db4da19aef30f491 Reviewed-on: https://gerrit.libreoffice.org/83757 Tested-by: Jenkins Reviewed-by: Muhammet Kara <muhammet.k...@collabora.com> diff --git a/sfx2/inc/SfxRedactionHelper.hxx b/sfx2/inc/SfxRedactionHelper.hxx index 65835d75f8eb..d30284791439 100644 --- a/sfx2/inc/SfxRedactionHelper.hxx +++ b/sfx2/inc/SfxRedactionHelper.hxx @@ -98,6 +98,10 @@ public: * */ static PageMargins getPageMarginsForCalc(const css::uno::Reference<css::frame::XModel>& xModel); + /* + * Used to find the text portions to be redacted. Returns a list of rectangles to cover those + * areas to be redacted. Probably the most crucial part of the auto-redaction process. + * */ static void searchInMetaFile(const RedactionTarget* pRedactionTarget, const GDIMetaFile& rMtf, std::vector<tools::Rectangle>& aRedactionRectangles, const uno::Reference<XComponent>& xComponent); commit f858556a21163c3b86072803ebf0b69676fd36c4 Author: Jan Holesovsky <ke...@collabora.com> AuthorDate: Fri Nov 1 17:57:17 2019 +0100 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:19:56 2020 +0100 android: Fix linking of liblo-native-code.so on aarch64. Forces gold as the linker; but according to the switches that were used, I suppose gold was used previously anyway, so hopefully not really a problem. Change-Id: I7bc54abf0d3b4b3d86d53e4ea6f01146cf196216 Reviewed-on: https://gerrit.libreoffice.org/81905 Reviewed-by: Jan Holesovsky <ke...@collabora.com> Tested-by: Jan Holesovsky <ke...@collabora.com> (cherry picked from commit fa145a035fbab652ec8f3ee870e44e43dd1d770a) Reviewed-on: https://gerrit.libreoffice.org/81915 diff --git a/android/Bootstrap/Makefile.shared b/android/Bootstrap/Makefile.shared index 7126e9a2f506..00ad02285cf1 100644 --- a/android/Bootstrap/Makefile.shared +++ b/android/Bootstrap/Makefile.shared @@ -51,7 +51,7 @@ NSSLIBS = freebl3 \ WHOLELIBS = \ -Wl,--whole-archive \ $(addprefix -l,$(strip \ - juh \ + $(if $(ENABLE_JAVA),juh) \ )) \ -Wl,--no-whole-archive @@ -59,7 +59,7 @@ WHOLELIBS = \ $(OBJLOCAL)/liblo-native-code.so : native-code.cxx $(ALL_STATIC_LIBS) @echo "Linking $@" mkdir -p $(OBJLOCAL) - $(CXX) -Wl,--build-id=sha1 -Wl,--gc-sections -Wl,--version-script=../Bootstrap/version.map -Wl,--no-keep-files-mapped -Wl,--no-undefined -DANDROID -DDISABLE_DYNLOADING -shared -Wl,-soname,liblo-native-code.so -o $(OBJLOCAL)/liblo-native-code.so -I$(BUILDDIR)/config_host -I$(SRCDIR)/include native-code.cxx -L$(INSTDIR)/$(LIBO_LIB_FOLDER) $(WHOLELIBS) $(LIBS) -lc++_static -lc++abi -landroid_support $(if $(filter armeabi-v7a,$(ANDROID_APP_ABI)),-lunwind) $(addprefix -l,$(NSSLIBS)) -lGLESv2 -landroid -ljnigraphics -llog -lz + $(CXX) -fuse-ld=gold -Wl,--build-id=sha1 -Wl,--gc-sections -Wl,--version-script=../Bootstrap/version.map -Wl,--no-keep-files-mapped -Wl,--no-undefined -DANDROID -DDISABLE_DYNLOADING -shared -Wl,-soname,liblo-native-code.so -o $(OBJLOCAL)/liblo-native-code.so -I$(BUILDDIR)/config_host -I$(SRCDIR)/include native-code.cxx -L$(INSTDIR)/$(LIBO_LIB_FOLDER) $(WHOLELIBS) $(LIBS) -lc++_static -lc++abi $(if $(filter-out arm64-v8a,$(ANDROID_APP_ABI)),-landroid_support) $(if $(filter armeabi-v7a,$(ANDROID_APP_ABI)),-lunwind) $(addprefix -l,$(NSSLIBS)) -lGLESv2 -landroid -ljnigraphics -llog -lz $(SODEST)/liblo-native-code.so : $(OBJLOCAL)/liblo-native-code.so mkdir -p $(SODEST) commit 6fffa2d6ee64fce95fac204eb6d6db0f0cefc75d Author: Jan Holesovsky <ke...@collabora.com> AuthorDate: Fri Nov 1 17:56:01 2019 +0100 Commit: Michael Meeks <michael.me...@collabora.com> CommitDate: Sat May 16 19:19:42 2020 +0100 android: Add also aarch64 to avoid linking errors. Change-Id: Ifdd696dc37de541bc722807054cd4ba7b862c175 Reviewed-on: https://gerrit.libreoffice.org/81904 Reviewed-by: Jan Holesovsky <ke...@collabora.com> Tested-by: Jan Holesovsky <ke...@collabora.com> (cherry picked from commit 620f39b816e3f10a4e047bb4f5ed5f4fdad2b364) Reviewed-on: https://gerrit.libreoffice.org/81914 diff --git a/external/libpng/StaticLibrary_libpng.mk b/external/libpng/StaticLibrary_libpng.mk index cb8428ec4dae..fe2364141b3b 100644 --- a/external/libpng/StaticLibrary_libpng.mk +++ b/external/libpng/StaticLibrary_libpng.mk @@ -33,7 +33,7 @@ $(eval $(call gb_StaticLibrary_add_generated_cobjects,libpng,\ UnpackedTarball/libpng/pngwrite \ UnpackedTarball/libpng/pngwtran \ UnpackedTarball/libpng/pngwutil \ - $(if $(filter ARM ARM64,$(CPUNAME)),\ + $(if $(filter ARM AARCH64 ARM64,$(CPUNAME)),\ UnpackedTarball/libpng/arm/arm_init \ UnpackedTarball/libpng/arm/filter_neon_intrinsics \ UnpackedTarball/libpng/arm/palette_neon_intrinsics \ _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits