cui/source/inc/cuitabarea.hxx    |    6 ------
 cui/source/inc/cuitabline.hxx    |    3 ---
 cui/source/tabpages/tparea.cxx   |    6 ++----
 cui/source/tabpages/tpbitmap.cxx |    3 +--
 cui/source/tabpages/tpgradnt.cxx |    3 +--
 cui/source/tabpages/tphatch.cxx  |    3 +--
 cui/source/tabpages/tpline.cxx   |    3 +--
 cui/source/tabpages/tplnedef.cxx |    3 +--
 cui/source/tabpages/tplneend.cxx |    3 +--
 cui/source/tabpages/tpshadow.cxx |    3 +--
 10 files changed, 9 insertions(+), 27 deletions(-)

New commits:
commit ce5711267c09a8c85decbd2ad043149d584f8fe7
Author: Stephan Bergmann <sberg...@redhat.com>
Date:   Fri Jun 26 16:01:26 2015 +0200

    -fsanitize=vptr: wrong, unnecessary downcasts
    
    (and the pXPool members are not even necessary)
    
    Change-Id: I321f06387ca1f1e98d8f731ec2b070212c82c28a

diff --git a/cui/source/inc/cuitabarea.hxx b/cui/source/inc/cuitabarea.hxx
index a25966a..4b00e3e 100644
--- a/cui/source/inc/cuitabarea.hxx
+++ b/cui/source/inc/cuitabarea.hxx
@@ -140,7 +140,6 @@ class SvxTransparenceTabPage : public SvxTabPage
     VclPtr<SvxXRectPreview>    m_pCtlXRectPreview;
     bool                bBitmap;
 
-    XOutdevItemPool*    pXPool;
     XFillAttrSetItem    aXFillAttr;
     SfxItemSet&         rXFSet;
 
@@ -253,7 +252,6 @@ private:
 
     bool*               pbAreaTP;
 
-    XOutdevItemPool*    pXPool;
     XFillAttrSetItem    aXFillAttr;
     SfxItemSet&         rXFSet;
 
@@ -348,7 +346,6 @@ private:
 
     bool                bDisable;
 
-    XOutdevItemPool*    pXPool;
     XFillAttrSetItem    aXFillAttr;
     SfxItemSet&         rXFSet;
     SfxMapUnit          ePoolUnit;
@@ -420,7 +417,6 @@ private:
     sal_Int32*          pPos;
     bool*               pbAreaTP;
 
-    XOutdevItemPool*    pXPool;
     XFillStyleItem      aXFStyleItem;
     XFillGradientItem   aXGradientItem;
     XFillAttrSetItem    aXFillAttr;
@@ -497,7 +493,6 @@ private:
     sal_Int32*          pPos;
     bool*               pbAreaTP;
 
-    XOutdevItemPool*    pXPool;
     XFillStyleItem      aXFStyleItem;
     XFillHatchItem      aXHatchItem;
     XFillAttrSetItem    aXFillAttr;
@@ -584,7 +579,6 @@ private:
 
     bool                bBmpChanged;
 
-    XOutdevItemPool*    pXPool;
     XFillStyleItem      aXFStyleItem;
     XFillBitmapItem     aXBitmapItem;
     XFillAttrSetItem    aXFillAttr;
diff --git a/cui/source/inc/cuitabline.hxx b/cui/source/inc/cuitabline.hxx
index 678042b..d659409 100644
--- a/cui/source/inc/cuitabline.hxx
+++ b/cui/source/inc/cuitabline.hxx
@@ -150,7 +150,6 @@ private:
     RECT_POINT          eRP;
     bool            bObjSelected;
 
-    XOutdevItemPool*    pXPool;
     XLineStyleItem      aXLStyle;
     XLineWidthItem      aXWidth;
     XLineDashItem       aXDash;
@@ -267,7 +266,6 @@ private:
     XDash               aDash;
     bool            bObjSelected;
 
-    XOutdevItemPool*    pXPool;
     XLineStyleItem      aXLStyle;
     XLineWidthItem      aXWidth;
     XLineDashItem       aXDash;
@@ -349,7 +347,6 @@ private:
     const SdrObject*    pPolyObj;
     bool            bObjSelected;
 
-    XOutdevItemPool*    pXPool;
     XLineStyleItem      aXLStyle;
     XLineWidthItem      aXWidth;
     XLineColorItem      aXColor;
diff --git a/cui/source/tabpages/tparea.cxx b/cui/source/tabpages/tparea.cxx
index ae3f3f3..538c4612 100644
--- a/cui/source/tabpages/tparea.cxx
+++ b/cui/source/tabpages/tparea.cxx
@@ -258,8 +258,7 @@ SvxTransparenceTabPage::SvxTransparenceTabPage(vcl::Window* 
pParent, const SfxIt
     nPageType           (0),
     nDlgType            (0),
     bBitmap             ( false ),
-    pXPool              ( static_cast<XOutdevItemPool*>(rInAttrs.GetPool()) ),
-    aXFillAttr          ( pXPool ),
+    aXFillAttr          ( rInAttrs.GetPool() ),
     rXFSet              ( aXFillAttr.GetItemSet() )
 {
     get(m_pRbtTransOff,"RBT_TRANS_OFF");
@@ -624,8 +623,7 @@ SvxAreaTabPage::SvxAreaTabPage( vcl::Window* pParent, const 
SfxItemSet& rInAttrs
     // init with pointer to fixed bool
     pbAreaTP(&maFixed_sal_Bool),
 
-    pXPool              ( static_cast<XOutdevItemPool*>( rInAttrs.GetPool() ) 
),
-    aXFillAttr          ( pXPool ),
+    aXFillAttr          ( rInAttrs.GetPool() ),
     rXFSet              ( aXFillAttr.GetItemSet() ),
 
     ePoolUnit(SFX_MAPUNIT_100TH_MM),
diff --git a/cui/source/tabpages/tpbitmap.cxx b/cui/source/tabpages/tpbitmap.cxx
index 71c23e6..8c3427d 100644
--- a/cui/source/tabpages/tpbitmap.cxx
+++ b/cui/source/tabpages/tpbitmap.cxx
@@ -67,10 +67,9 @@ SvxBitmapTabPage::SvxBitmapTabPage(  vcl::Window* pParent, 
const SfxItemSet& rIn
 
     bBmpChanged         ( false ),
 
-    pXPool              ( static_cast<XOutdevItemPool*>( rInAttrs.GetPool() )),
     aXFStyleItem        ( drawing::FillStyle_BITMAP ),
     aXBitmapItem        ( OUString(), Graphic() ),
-    aXFillAttr          ( pXPool ),
+    aXFillAttr          ( rInAttrs.GetPool() ),
     rXFSet              ( aXFillAttr.GetItemSet() )
 {
     get(m_pBxPixelEditor,"maingrid");
diff --git a/cui/source/tabpages/tpgradnt.cxx b/cui/source/tabpages/tpgradnt.cxx
index 380f36a..d0c1241 100644
--- a/cui/source/tabpages/tpgradnt.cxx
+++ b/cui/source/tabpages/tpgradnt.cxx
@@ -60,10 +60,9 @@ SvxGradientTabPage::SvxGradientTabPage
     pPos                ( 0 ),
     pbAreaTP            ( 0 ),
 
-    pXPool              ( static_cast<XOutdevItemPool*>( rInAttrs.GetPool() )),
     aXFStyleItem        ( drawing::FillStyle_GRADIENT ),
     aXGradientItem      ( OUString(), XGradient( COL_BLACK, COL_WHITE ) ),
-    aXFillAttr          ( pXPool ),
+    aXFillAttr          ( rInAttrs.GetPool() ),
     rXFSet              ( aXFillAttr.GetItemSet() )
 {
     get(m_pLbGradientType, "gradienttypelb");
diff --git a/cui/source/tabpages/tphatch.cxx b/cui/source/tabpages/tphatch.cxx
index 0060b36..5b28712 100644
--- a/cui/source/tabpages/tphatch.cxx
+++ b/cui/source/tabpages/tphatch.cxx
@@ -63,10 +63,9 @@ SvxHatchTabPage::SvxHatchTabPage
     pPos                ( 0 ),
     pbAreaTP            ( 0 ),
 
-    pXPool              ( static_cast<XOutdevItemPool*>(rInAttrs.GetPool()) ),
     aXFStyleItem        ( drawing::FillStyle_HATCH ),
     aXHatchItem         ( OUString(), XHatch() ),
-    aXFillAttr          ( pXPool ),
+    aXFillAttr          ( rInAttrs.GetPool() ),
     rXFSet              ( aXFillAttr.GetItemSet() )
 
 {
diff --git a/cui/source/tabpages/tpline.cxx b/cui/source/tabpages/tpline.cxx
index e0440c7..01940a7 100644
--- a/cui/source/tabpages/tpline.cxx
+++ b/cui/source/tabpages/tpline.cxx
@@ -99,12 +99,11 @@ SvxLineTabPage::SvxLineTabPage
     eRP( RP_LT ),
     bObjSelected( false ),
 
-    pXPool              ( static_cast<XOutdevItemPool*>(rInAttrs.GetPool()) ),
     aXLStyle            ( drawing::LineStyle_DASH ),
     aXWidth             ( 1 ),
     aXDash              ( OUString(), XDash( css::drawing::DashStyle_RECT, 3, 
7, 2, 40, 15 ) ),
     aXColor             ( OUString(), COL_LIGHTRED ),
-    aXLineAttr          ( pXPool ),
+    aXLineAttr          ( rInAttrs.GetPool() ),
     rXLSet              ( aXLineAttr.GetItemSet() ),
      pnLineEndListState( 0 ),
     pnDashListState( 0 ),
diff --git a/cui/source/tabpages/tplnedef.cxx b/cui/source/tabpages/tplnedef.cxx
index 38a7c7f..07b904d 100644
--- a/cui/source/tabpages/tplnedef.cxx
+++ b/cui/source/tabpages/tplnedef.cxx
@@ -62,12 +62,11 @@ SvxLineDefTabPage::SvxLineDefTabPage
     rOutAttrs       ( rInAttrs ),
     bObjSelected    ( false ),
 
-    pXPool              ( static_cast<XOutdevItemPool*>(rInAttrs.GetPool()) ),
     aXLStyle            ( drawing::LineStyle_DASH ),
     aXWidth             ( XOUT_WIDTH ),
     aXDash              ( OUString(), XDash( css::drawing::DashStyle_RECT, 3, 
7, 2, 40, 15 ) ),
     aXColor             ( OUString(), COL_BLACK ),
-    aXLineAttr          ( pXPool ),
+    aXLineAttr          ( rInAttrs.GetPool() ),
     rXLSet              ( aXLineAttr.GetItemSet() ),
     pnDashListState(0),
     pPageType(0),
diff --git a/cui/source/tabpages/tplneend.cxx b/cui/source/tabpages/tplneend.cxx
index 478ab46..dbd9063 100644
--- a/cui/source/tabpages/tplneend.cxx
+++ b/cui/source/tabpages/tplneend.cxx
@@ -62,11 +62,10 @@ SvxLineEndDefTabPage::SvxLineEndDefTabPage
     pPolyObj            ( NULL ),
     bObjSelected        ( false ),
 
-    pXPool              ( static_cast<XOutdevItemPool*>(rInAttrs.GetPool()) ),
     aXLStyle            ( com::sun::star::drawing::LineStyle_SOLID ),
     aXWidth             ( XOUT_WIDTH ),
     aXColor             ( OUString(), COL_BLACK ),
-    aXLineAttr          ( pXPool ),
+    aXLineAttr          ( rInAttrs.GetPool() ),
     rXLSet              ( aXLineAttr.GetItemSet() ),
     pLineEndList        ( NULL ),
     pnLineEndListState  ( NULL ),
diff --git a/cui/source/tabpages/tpshadow.cxx b/cui/source/tabpages/tpshadow.cxx
index 10c0d0c..08a4d3c 100644
--- a/cui/source/tabpages/tpshadow.cxx
+++ b/cui/source/tabpages/tpshadow.cxx
@@ -58,8 +58,7 @@ SvxShadowTabPage::SvxShadowTabPage( vcl::Window* pParent, 
const SfxItemSet& rInA
     nDlgType            ( 0 ),
     pbAreaTP            ( 0 ),
     bDisable            ( false ),
-    pXPool              ( static_cast<XOutdevItemPool*>(rInAttrs.GetPool()) ),
-    aXFillAttr          ( pXPool ),
+    aXFillAttr          ( rInAttrs.GetPool() ),
     rXFSet              ( aXFillAttr.GetItemSet() )
 {
     get(m_pTsbShowShadow,"TSB_SHOW_SHADOW");
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to