[Libreoffice-commits] core.git: Branch 'libreoffice-7-2' - sc/source
Eike Rathke (via logerrit)
logerrit at kemper.freedesktop.org
Thu Jun 24 14:04:42 UTC 2021
sc/source/ui/inc/gridwin.hxx | 6 ---
sc/source/ui/view/gridwin.cxx | 4 --
sc/source/ui/view/gridwin4.cxx | 62 -----------------------------------------
sc/source/ui/view/tabview5.cxx | 7 ----
4 files changed, 79 deletions(-)
New commits:
commit 0be41797787622ebb5320acb6ec6cb65a7c8cdd4
Author: Eike Rathke <erack at redhat.com>
AuthorDate: Thu Jun 24 11:49:05 2021 +0200
Commit: Eike Rathke <erack at redhat.com>
CommitDate: Thu Jun 24 16:04:02 2021 +0200
Revert "tdf#124983 In calc make printable page borders initially visible"
This reverts commit ff3f749cd08851ee4eb0991eabee0327ca081a45.
Reason for revert: reportedly causes crashes and heap corruption on Windows, see discussion in https://gerrit.libreoffice.org/c/core/+/90101
Change-Id: I84429b55b5fa2b26afd21ef104606ae2f30aad5d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117727
Reviewed-by: Eike Rathke <erack at redhat.com>
Tested-by: Jenkins
diff --git a/sc/source/ui/inc/gridwin.hxx b/sc/source/ui/inc/gridwin.hxx
index daf6c35b652c..09e46d7a4e21 100644
--- a/sc/source/ui/inc/gridwin.hxx
+++ b/sc/source/ui/inc/gridwin.hxx
@@ -309,10 +309,6 @@ class SAL_DLLPUBLIC_RTTI ScGridWindow : public vcl::Window, public DropTargetHel
void InvalidateLOKViewCursor(const tools::Rectangle& rCursorRect,
const Fraction aScaleX, const Fraction aScaleY);
- Timer maShowPageBreaksTimer;
- bool bInitialPageBreaks;
- void SetupInitialPageBreaks(ScDocument& rDoc, SCTAB nTab, bool bSetup);
- DECL_LINK(InitiatePageBreaksTimer, Timer*, void);
protected:
virtual void PrePaint(vcl::RenderContext& rRenderContext) override;
virtual void Paint(vcl::RenderContext& rRenderContext, const tools::Rectangle& rRect) override;
@@ -494,8 +490,6 @@ public:
void updateLOKValListButton(bool bVisible, const ScAddress& rPos) const;
void updateLOKInputHelp(const OUString& title, const OUString& content) const;
- void initiatePageBreaks();
-
protected:
void ImpCreateOverlayObjects();
void ImpDestroyOverlayObjects();
diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index c2c115c4371c..b6412ae36fbb 100644
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -424,10 +424,6 @@ ScGridWindow::ScGridWindow( vcl::Window* pParent, ScViewData& rData, ScSplitPos
GetOutDev()->SetDigitLanguage( SC_MOD()->GetOptDigitLanguage() );
EnableRTL( false );
-
- bInitialPageBreaks = true;
- maShowPageBreaksTimer.SetInvokeHandler(LINK(this, ScGridWindow, InitiatePageBreaksTimer));
- maShowPageBreaksTimer.SetTimeout(1);
}
ScGridWindow::~ScGridWindow()
diff --git a/sc/source/ui/view/gridwin4.cxx b/sc/source/ui/view/gridwin4.cxx
index f1e53ed61cb8..9c1ff475449a 100644
--- a/sc/source/ui/view/gridwin4.cxx
+++ b/sc/source/ui/view/gridwin4.cxx
@@ -76,9 +76,6 @@
#include <vcl/virdev.hxx>
#include <svx/sdrpaintwindow.hxx>
#include <drwlayer.hxx>
-#include <columnspanset.hxx>
-#include <docfunc.hxx>
-#include <printfun.hxx>
static void lcl_LimitRect( tools::Rectangle& rRect, const tools::Rectangle& rVisible )
{
@@ -1270,30 +1267,6 @@ void ScGridWindow::DrawContent(OutputDevice &rDevice, const ScTableInfo& rTableI
if (mpNoteMarker)
mpNoteMarker->Draw(); // Above the cursor, in drawing map mode
-
- SetupInitialPageBreaks(rDoc, nTab, bPage&& bInitialPageBreaks);
-}
-
-
-void ScGridWindow::SetupInitialPageBreaks(ScDocument& rDoc, SCTAB nTab, bool bSetup)
-{
- // tdf#124983, if option LibreOfficeDev Calc/View/Visual Aids/Page breaks
- // is enabled, breaks should be visible. If the document is opened the first
- // time, the breaks are not calculated yet, so for this initialization
- // a timer will be triggered here.
- if (bSetup)
- {
- std::set<SCCOL> aColBreaks;
- std::set<SCROW> aRowBreaks;
- rDoc.GetAllColBreaks(aColBreaks, nTab, true, false);
- rDoc.GetAllRowBreaks(aRowBreaks, nTab, true, false);
- if (aColBreaks.size() == 0 || aRowBreaks.size() == 0)
- {
- maShowPageBreaksTimer.SetPriority(TaskPriority::DEFAULT_IDLE);
- maShowPageBreaksTimer.Start();
- bInitialPageBreaks = false;
- }
- }
}
namespace
@@ -2343,39 +2316,4 @@ void ScGridWindow::DataChanged( const DataChangedEvent& rDCEvt )
Invalidate();
}
-void ScGridWindow::initiatePageBreaks()
-{
- bInitialPageBreaks = true;
-}
-
-IMPL_LINK(ScGridWindow, InitiatePageBreaksTimer, Timer*, pTimer, void)
-{
- if (pTimer == &maShowPageBreaksTimer)
- {
- ScDocument& rDoc = mrViewData.GetDocument();
- const ScViewOptions& rOpts = mrViewData.GetOptions();
- bool bPage = rOpts.GetOption(VOPT_PAGEBREAKS);
- ScDocShell* pDocSh = mrViewData.GetDocShell();
- bool bModified = pDocSh->IsModified();
- // tdf#124983, if option LibreOfficeDev Calc/View/Visual Aids/Page breaks
- // is enabled, breaks should be visible. If the document is opened the first
- // time or a tab is activated the first time, the breaks are not calculated
- // yet, so this initialization is done here.
- if (bPage)
- {
- SCTAB nCurrentTab = mrViewData.GetTabNo();
- Size pagesize = rDoc.GetPageSize(nCurrentTab);
- if (pagesize.IsEmpty())
- {
- ScPrintFunc(pDocSh, pDocSh->GetPrinter(), nCurrentTab);
- rDoc.SetPageSize(nCurrentTab, rDoc.GetPageSize(nCurrentTab));
- }
- rDoc.UpdatePageBreaks(nCurrentTab);
- pDocSh->PostPaint(0, 0, nCurrentTab, rDoc.MaxCol(), rDoc.MaxRow(), nCurrentTab, PaintPartFlags::Grid);
- pDocSh->SetModified(bModified);
- }
- }
-}
-
-
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sc/source/ui/view/tabview5.cxx b/sc/source/ui/view/tabview5.cxx
index f0b6ed6c4f18..3de36af347d2 100644
--- a/sc/source/ui/view/tabview5.cxx
+++ b/sc/source/ui/view/tabview5.cxx
@@ -320,13 +320,6 @@ void ScTabView::TabChanged( bool bSameTabButMoved )
}
}
- for (int i = 0; i < 4; i++)
- if (pGridWin[i])
- {
- pGridWin[i]->initiatePageBreaks();
- }
-
-
if (!comphelper::LibreOfficeKit::isActive())
return;
More information about the Libreoffice-commits
mailing list