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

Peter Foley pefoley2 at pefoley.com
Sat Jan 16 02:53:03 PST 2016


 sd/source/ui/controller/slidelayoutcontroller.cxx |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

New commits:
commit 659526a33619a74730843a06158b22592ae06a0b
Author: Peter Foley <pefoley2 at pefoley.com>
Date:   Fri Jan 15 22:32:39 2016 -0500

    Fix ODR violations when compiling with LTO
    
    /home/peter/libreoffice/sd/source/ui/controller/slidelayoutcontroller.cxx:84:8: warning: type ‘struct snewfoil_value_info’ violates one definition rule [-Wodr]
     struct snewfoil_value_info
            ^
    /home/peter/libreoffice/sd/source/ui/controller/displaymodecontroller.cxx:71:8: note: a different type is defined in another translation unit
     struct snewfoil_value_info
            ^
    /home/peter/libreoffice/sd/source/ui/controller/slidelayoutcontroller.cxx:86:16: note: the first difference of corresponding definitions is field ‘mnBmpResId’
         sal_uInt16 mnBmpResId;
                    ^
    /home/peter/libreoffice/sd/source/ui/controller/displaymodecontroller.cxx:73:17: note: a field with different name is defined in another translation unit
         sal_uInt16  mnId;
                     ^
    
    Change-Id: Ic5f5354424188a0889e842e80cb70dd90ecaeff3
    Reviewed-on: https://gerrit.libreoffice.org/21504
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: jan iversen <jani at documentfoundation.org>
    Tested-by: jan iversen <jani at documentfoundation.org>

diff --git a/sd/source/ui/controller/slidelayoutcontroller.cxx b/sd/source/ui/controller/slidelayoutcontroller.cxx
index efe545b..15742ef 100644
--- a/sd/source/ui/controller/slidelayoutcontroller.cxx
+++ b/sd/source/ui/controller/slidelayoutcontroller.cxx
@@ -81,20 +81,20 @@ private:
     VclPtr<ValueSet> mpLayoutSet2;
 };
 
-struct snewfoil_value_info
+struct snewfoil_value_info_layout
 {
     sal_uInt16 mnBmpResId;
     sal_uInt16 mnStrResId;
     AutoLayout maAutoLayout;
 };
 
-static const snewfoil_value_info notes[] =
+static const snewfoil_value_info_layout notes[] =
 {
     {BMP_FOILN_01, STR_AUTOLAYOUT_NOTES, AUTOLAYOUT_NOTES},
     {0, 0, AUTOLAYOUT_NONE},
 };
 
-static const snewfoil_value_info handout[] =
+static const snewfoil_value_info_layout handout[] =
 {
     {BMP_FOILH_01, STR_AUTOLAYOUT_HANDOUT1, AUTOLAYOUT_HANDOUT1},
     {BMP_FOILH_02, STR_AUTOLAYOUT_HANDOUT2, AUTOLAYOUT_HANDOUT2},
@@ -105,7 +105,7 @@ static const snewfoil_value_info handout[] =
     {0, 0, AUTOLAYOUT_NONE},
 };
 
-static const snewfoil_value_info standard[] =
+static const snewfoil_value_info_layout standard[] =
 {
     {BMP_LAYOUT_EMPTY,    STR_AUTOLAYOUT_NONE,                 AUTOLAYOUT_NONE         },
     {BMP_LAYOUT_HEAD03,   STR_AUTOLAYOUT_TITLE,                AUTOLAYOUT_TITLE        },
@@ -122,7 +122,7 @@ static const snewfoil_value_info standard[] =
     {0, 0, AUTOLAYOUT_NONE}
 };
 
-static const snewfoil_value_info v_standard[] =
+static const snewfoil_value_info_layout v_standard[] =
 {
     // vertical
     {BMP_LAYOUT_VERTICAL02, STR_AL_VERT_TITLE_TEXT_CHART,      AUTOLAYOUT_VERTICAL_TITLE_TEXT_CHART       },
@@ -132,7 +132,7 @@ static const snewfoil_value_info v_standard[] =
     {0, 0, AUTOLAYOUT_NONE}
 };
 
-static void fillLayoutValueSet( ValueSet* pValue, const snewfoil_value_info* pInfo )
+static void fillLayoutValueSet( ValueSet* pValue, const snewfoil_value_info_layout* pInfo )
 {
     Size aLayoutItemSize;
     for( ; pInfo->mnBmpResId; pInfo++ )
@@ -186,7 +186,7 @@ LayoutToolbarMenu::LayoutToolbarMenu( SlideLayoutController& rController, const
     mpLayoutSet1 = createEmptyValueSetControl();
     mpLayoutSet1->SetSelectHdl( LINK( this, LayoutToolbarMenu, SelectValueSetHdl ) );
 
-    const snewfoil_value_info* pInfo = nullptr;
+    const snewfoil_value_info_layout* pInfo = nullptr;
     sal_Int16 nColCount = 4;
     switch( eMode )
     {


More information about the Libreoffice-commits mailing list