[Libreoffice-commits] core.git: Branch 'libreoffice-5-1' - sc/source

Stephan Bergmann sbergman at redhat.com
Thu May 12 11:21:24 UTC 2016


 sc/source/ui/view/tabvwsh3.cxx |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit c82f0680283d3a6daaed5717cb791a5ee9692aea
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Wed May 11 15:51:34 2016 +0200

    Missing break in switch
    
    The case SID_CURRENTDOC used to harmlessly fall through to
    
      case SID_ATTR_SIZE://XXX ???
          break;
    
    which was removed with ebabf6d1fa648d62dd63529e9fe64dcb631caee8 "fdo#86018 Don't
    do anything when clicked on placeholder in statusbar", making it erroneously
    fall through to the case SID_PRINTPREVIEW instead.
    
    Change-Id: Ib7792ed4198ea3e25696a63f0d7ab4df78848e6a
    (cherry picked from commit 3c8f49d7f8d70282236bf4f3d6b5a700e169f8e8)
    Reviewed-on: https://gerrit.libreoffice.org/24919
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Eike Rathke <erack at redhat.com>

diff --git a/sc/source/ui/view/tabvwsh3.cxx b/sc/source/ui/view/tabvwsh3.cxx
index 957fa3f..4fd9a5c 100644
--- a/sc/source/ui/view/tabvwsh3.cxx
+++ b/sc/source/ui/view/tabvwsh3.cxx
@@ -494,6 +494,7 @@ void ScTabViewShell::Execute( SfxRequest& rReq )
 
                 rReq.Ignore();//XXX is handled by SFX
             }
+            break;
 
         case SID_PRINTPREVIEW:
             {


More information about the Libreoffice-commits mailing list