[Libreoffice-commits] core.git: sc/source

Tor Lillqvist (via logerrit) logerrit at kemper.freedesktop.org
Tue Jun 30 06:19:35 UTC 2020


 sc/source/ui/unoobj/docuno.cxx |   11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

New commits:
commit ca1df564c3199a569c681d5814e1fddc66c48522
Author:     Tor Lillqvist <tml at iki.fi>
AuthorDate: Thu Jun 25 16:20:45 2020 +0300
Commit:     Tor Lillqvist <tml at collabora.com>
CommitDate: Tue Jun 30 08:18:52 2020 +0200

    Let's not run on hope
    
    Change-Id: If2fb2c55d53321ec2e5beff0ade9974c02811f7f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97120
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Tor Lillqvist <tml at collabora.com>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97408
    Tested-by: Jenkins

diff --git a/sc/source/ui/unoobj/docuno.cxx b/sc/source/ui/unoobj/docuno.cxx
index fc9903bb2e3c..d0ec84d8cea5 100644
--- a/sc/source/ui/unoobj/docuno.cxx
+++ b/sc/source/ui/unoobj/docuno.cxx
@@ -500,15 +500,14 @@ void ScModelObj::paintTile( VirtualDevice& rDevice,
                             int nTilePosX, int nTilePosY,
                             long nTileWidth, long nTileHeight )
 {
-    // There seems to be no clear way of getting the grid window for this
-    // particular document, hence we need to hope we get the right window.
-    // FIXME: "hope"? Are you kidding me?
-    ScViewData* pViewData = ScDocShell::GetViewData();
+    ScTabViewShell* pViewShell = pDocShell->GetBestViewShell(false);
 
-    // FIXME
-    if (!pViewData)
+    // FIXME: Can this happen? What should we do?
+    if (!pViewShell)
         return;
 
+    ScViewData* pViewData = &pViewShell->GetViewData();
+
     ScGridWindow* pGridWindow = pViewData->GetActiveWin();
 
     // update the size of the area we are painting


More information about the Libreoffice-commits mailing list