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

Maxim Monastirsky momonasmon at gmail.com
Wed Oct 19 11:03:19 UTC 2016


 chart2/source/controller/main/ChartController_Window.cxx |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 42e89217fb3b8bda7cc084c8e14ba56efa8eae3f
Author: Maxim Monastirsky <momonasmon at gmail.com>
Date:   Wed Oct 19 13:59:56 2016 +0300

    No need to check here if menu name is empty
    
    We now do this on the ResourceMenuController side.
    
    Change-Id: Ic249b6a687fe059393ccf55bbc4534242c08e30e

diff --git a/chart2/source/controller/main/ChartController_Window.cxx b/chart2/source/controller/main/ChartController_Window.cxx
index b1daf04..4e7a791 100644
--- a/chart2/source/controller/main/ChartController_Window.cxx
+++ b/chart2/source/controller/main/ChartController_Window.cxx
@@ -1217,11 +1217,10 @@ void ChartController::execute_Command( const CommandEvent& rCEvt )
             }
         }
 
-        css::uno::Sequence< css::uno::Any > aArgs( aMenuName.isEmpty() ? 2 : 3 );
+        css::uno::Sequence< css::uno::Any > aArgs( 3 );
         aArgs[0] <<= comphelper::makePropertyValue( "IsContextMenu", true );
         aArgs[1] <<= comphelper::makePropertyValue( "Frame", m_xFrame );
-        if ( !aMenuName.isEmpty() )
-            aArgs[2] <<= comphelper::makePropertyValue( "Value", aMenuName );
+        aArgs[2] <<= comphelper::makePropertyValue( "Value", aMenuName );
 
         css::uno::Reference< css::frame::XPopupMenuController > xPopupController(
             m_xCC->getServiceManager()->createInstanceWithArgumentsAndContext(


More information about the Libreoffice-commits mailing list