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

Stephan Bergmann sbergman at redhat.com
Wed Feb 24 16:51:16 UTC 2016


 sd/source/ui/view/sdview3.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a3a7a70811123a9d7c812952496596d7be19135b
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Wed Feb 24 17:39:34 2016 +0100

    -Werror=nonnull-compare
    
    ...where "this" must be (derived from) View, in View::InsertData.
    
    Before 89d39bc100aabf5dccbe77c0b5c0c85736e85b39 "tdf#94559: 4th step to remove
    rtti.hxx," that was
    
      ISA( View )
    
    which had originally been introduced as
    
      ISA( SdView )
    
    into SdView::InsertData with 82453a8c6c84a22b6f940395d9b4c45516197ed3 "#80266#:
    added D&D functionality for slide view" in 2001 (and then, both in concert,
    SdView::InsertData changed to View::InsertData and the ISA( SdView ) changed to
    ISA( View ) with ee8db8ade5ac9205430eab6f1b701f683ea99eb1 "INTEGRATION: CWS
    impress1: #111996# Transition to stacked sub-shells. Introduction of namespace
    sd."
    
    So just assume it was always only a harmless glitch that is good to go.
    
    Change-Id: I5a99e401feb2a4c4455469b58c803ccab06694f5

diff --git a/sd/source/ui/view/sdview3.cxx b/sd/source/ui/view/sdview3.cxx
index 0a162e5..2818612 100644
--- a/sd/source/ui/view/sdview3.cxx
+++ b/sd/source/ui/view/sdview3.cxx
@@ -359,7 +359,7 @@ bool View::InsertData( const TransferableDataHelper& rDataHelper,
     {
         const View* pSourceView = pOwnData->GetView();
 
-        if( pOwnData->GetDocShell() && pOwnData->IsPageTransferable() && dynamic_cast< View *>( this) !=  nullptr )
+        if( pOwnData->GetDocShell() && pOwnData->IsPageTransferable() )
         {
             mpClipboard->HandlePageDrop (*pOwnData);
             bReturn = true;


More information about the Libreoffice-commits mailing list