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

Eike Rathke erack at redhat.com
Thu Dec 7 14:36:36 UTC 2017


 sc/source/ui/view/notemark.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 454d75e907b9e9e478f81388374ae09fb55f5c1c
Author: Eike Rathke <erack at redhat.com>
Date:   Thu Dec 7 13:03:59 2017 +0100

    Assert pDrawView even to spot bad usage
    
    Change-Id: Id33df9548085f8ab5cea6866664b4bed7c779c63
    Reviewed-on: https://gerrit.libreoffice.org/46023
    Reviewed-by: Eike Rathke <erack at redhat.com>
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/sc/source/ui/view/notemark.cxx b/sc/source/ui/view/notemark.cxx
index b92e7c278b70..47c9db0dacf5 100644
--- a/sc/source/ui/view/notemark.cxx
+++ b/sc/source/ui/view/notemark.cxx
@@ -108,6 +108,8 @@ IMPL_LINK_NOARG(ScNoteMarker, TimeHdl, Timer *, void)
                 // Need to include grid offset: GetCurrentBoundRect is removing it
                 // but we need to know actual rect position
                 aRect += mxObject->GetGridOffset();
+
+                assert(pDrawView && "this ended up with a wrong rectangle");
             }
 
             // Insert page so that the model recognise it and also deleted


More information about the Libreoffice-commits mailing list