[PATCH] Removing unused code

2012-04-16 Thread Laure Garcí­a Linde

From ade2a4ad180eac9e72e5104fd8e0ef9014847bec Mon Sep 17 00:00:00 2001
From: Laureano G. Linde laur...@gmail.com
Date: Sun, 15 Apr 2012 18:47:57 +0200
Subject: [PATCH] Removed StgHeader::SetClassId(ClsId const )

---
 sot/source/sdstor/stgelem.cxx |6 --
 sot/source/sdstor/stgelem.hxx |1 -
 2 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/sot/source/sdstor/stgelem.cxx b/sot/source/sdstor/stgelem.cxx
index ff73a59..c51dc1b 100644
--- a/sot/source/sdstor/stgelem.cxx
+++ b/sot/source/sdstor/stgelem.cxx
@@ -187,12 +187,6 @@ void StgHeader::SetFATPage( short n, sal_Int32 nb )
 }
 }
 
-void StgHeader::SetClassId( const ClsId r )
-{
-if( memcmp( aClsId, r, sizeof( ClsId ) ) )
-bDirty = sal_True, memcpy( aClsId, r, sizeof( ClsId ) );
-}
-
 void StgHeader::SetTOCStart( sal_Int32 n )
 {
 if( n != nTOCstrm ) bDirty = sal_True, nTOCstrm = n;
diff --git a/sot/source/sdstor/stgelem.hxx b/sot/source/sdstor/stgelem.hxx
index 7792608..6ea349e 100644
--- a/sot/source/sdstor/stgelem.hxx
+++ b/sot/source/sdstor/stgelem.hxx
@@ -87,7 +87,6 @@ public:
 void  SetMasters( sal_Int32 n );
 short GetFAT1Size() const   { return 109;   }
 const ClsId GetClassId() const { return aClsId;}
-void  SetClassId( const ClsId );
 sal_Int32 GetFATPage( short ) const;
 void  SetFATPage( short, sal_Int32 );
 };
-- 
1.7.0.4

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PUSHED][PATCH] Removing unused code

2012-04-16 Thread Caolán McNamara
On Sun, 2012-04-15 at 19:47 +0200, Laure Garcí­a Linde wrote:
 ___

Looks good, pushed now. Thanks for this. I think this is your first
patch, so can you confirm this patch is under our preferred LGPLv3+/MPL
combo ?

Typical procedure is to send a sort of blanket all submissions unless I
say otherwise are under LGPLv3+/MPL to this list and add yourself to
http://wiki.documentfoundation.org/Development/Developers with a link to
the mail archive of that mail

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Removing unused code

2012-02-13 Thread Santiago Martinez
This patch removes unused code as listed in unusedcode.easy
From 193f0208edeba24a820e9fe098456f5412d5e3a4 Mon Sep 17 00:00:00 2001
From: Santiago Martinez smvar...@gmail.com
Date: Mon, 13 Feb 2012 22:31:23 +0100
Subject: [PATCH] Remove unused code.

---
 editeng/inc/editeng/numitem.hxx  |1 -
 editeng/source/items/numitem.cxx |   17 -
 unusedcode.easy  |1 -
 3 files changed, 0 insertions(+), 19 deletions(-)

diff --git a/editeng/inc/editeng/numitem.hxx b/editeng/inc/editeng/numitem.hxx
index 1084ea1..ea34a7f 100644
--- a/editeng/inc/editeng/numitem.hxx
+++ b/editeng/inc/editeng/numitem.hxx
@@ -278,7 +278,6 @@ public:
 const SvxNumberFormat*  Get(sal_uInt16 nLevel)const;
 const SvxNumberFormat  GetLevel(sal_uInt16 nLevel)const;
 voidSetLevel(sal_uInt16 nLevel, const SvxNumberFormat rFmt, sal_Bool bIsValid = sal_True);
-voidSetLevel(sal_uInt16 nLevel, const SvxNumberFormat* pFmt);
 
 sal_BoolIsContinuousNumbering()const
 {return bContinuousNumbering;}
diff --git a/editeng/source/items/numitem.cxx b/editeng/source/items/numitem.cxx
index 01fc463..ccb3455 100644
--- a/editeng/source/items/numitem.cxx
+++ b/editeng/source/items/numitem.cxx
@@ -759,23 +759,6 @@ void SvxNumRule::SetLevel( sal_uInt16 i, const SvxNumberFormat rNumFmt, sal_Boo
 }
 }
 
-void SvxNumRule::SetLevel(sal_uInt16 nLevel, const SvxNumberFormat* pFmt)
-{
-DBG_ASSERT(nLevel  SVX_MAX_NUM, Wrong Level );
-
-if( nLevel  SVX_MAX_NUM )
-{
-aFmtsSet[nLevel] = 0 != pFmt;
-if(pFmt)
-SetLevel(nLevel, *pFmt);
-else
-{
-delete aFmts[nLevel];
-aFmts[nLevel] = 0;
-}
-}
-}
-
 String  SvxNumRule::MakeNumString( const SvxNodeNum rNum, sal_Bool bInclStrings ) const
 {
 String aStr;
diff --git a/unusedcode.easy b/unusedcode.easy
index 9f4bf3e..95f65eb 100755
--- a/unusedcode.easy
+++ b/unusedcode.easy
@@ -369,7 +369,6 @@ SvxMSDffShapeTxBxSort::Insert(SvxMSDffShapeOrder* const*, unsigned short)
 SvxMSDffShapeTxBxSort::Insert(SvxMSDffShapeTxBxSort const*, unsigned short, unsigned short)
 SvxMSDffShapeTxBxSort::Remove(SvxMSDffShapeOrder* const, unsigned short)
 SvxMSDffShapeTxBxSort::Remove(unsigned short, unsigned short)
-SvxNumRule::SetLevel(unsigned short, SvxNumberFormat const*)
 SvxOpenGraphicDialog::SetControlHelpIds(short const*, char const**)
 SvxPluginFileDlg::IsAvailable(unsigned short)
 SvxPopupWindowListBox::StartSelection()
-- 
1.7.7.6

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PUSHED][PATCH] Removing unused code

2012-01-26 Thread Caolán McNamara
On Thu, 2012-01-19 at 13:19 +0100, Alexander Bergmann wrote:
 ---
  basegfx/inc/basegfx/polygon/b2dpolygonclipper.hxx |7 --
  basegfx/inc/basegfx/polygon/b3dpolygonclipper.hxx |   14 ---
  basegfx/source/polygon/b2dpolygonclipper.cxx  |   66 -
  basegfx/source/polygon/b3dpolygonclipper.cxx  |  102 
 -
  unusedcode.easy   |6 -
  5 files changed, 0 insertions(+), 195 deletions(-)

Mathu pushed this, just marking it as such by mail thread

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Removing unused code

2012-01-26 Thread Caolán McNamara
On Thu, 2012-01-19 at 13:13 +0100, Alexander Bergmann wrote:
 ---
  .../inc/basegfx/polygon/b2dpolygoncutandtouch.hxx  |   10 --
  basegfx/source/polygon/b2dpolygoncutandtouch.cxx   |  112 
 
  unusedcode.easy|4 -
  3 files changed, 0 insertions(+), 126 deletions(-)

Mathu pushed this, just marking it as such by mail thread

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PUSHED][PATCH] Removing unused code

2012-01-26 Thread Caolán McNamara
On Thu, 2012-01-26 at 16:38 +, Caolán McNamara wrote:
 On Thu, 2012-01-19 at 13:13 +0100, Alexander Bergmann wrote:
  ---
   .../inc/basegfx/polygon/b2dpolygoncutandtouch.hxx  |   10 --
   basegfx/source/polygon/b2dpolygoncutandtouch.cxx   |  112 
  
   unusedcode.easy|4 -
   3 files changed, 0 insertions(+), 126 deletions(-)
 
 Mathu pushed this, just marking it as such by mail thread

sigh, spam spam, added PUSHED now.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PATCH] Removing unused code

2012-01-19 Thread Alexander Bergmann
---
 .../inc/basegfx/polygon/b2dpolygoncutandtouch.hxx  |   10 --
 basegfx/source/polygon/b2dpolygoncutandtouch.cxx   |  112 
 unusedcode.easy|4 -
 3 files changed, 0 insertions(+), 126 deletions(-)

diff --git a/basegfx/inc/basegfx/polygon/b2dpolygoncutandtouch.hxx 
b/basegfx/inc/basegfx/polygon/b2dpolygoncutandtouch.hxx
index 0c6296d..8997c54 100644
--- a/basegfx/inc/basegfx/polygon/b2dpolygoncutandtouch.hxx
+++ b/basegfx/inc/basegfx/polygon/b2dpolygoncutandtouch.hxx
@@ -53,28 +53,18 @@ namespace basegfx
 // not change or add points to rMask.
 BASEGFX_DLLPUBLIC B2DPolygon addPointsAtCutsAndTouches(const 
B2DPolyPolygon rMask, const B2DPolygon rCandidate);
 
-// look for intersections of rCandidate with all polygons from rMask 
and add extra points there. Do
-// not change or add points to rMask.
-BASEGFX_DLLPUBLIC B2DPolyPolygon addPointsAtCutsAndTouches(const 
B2DPolyPolygon rMask, const B2DPolyPolygon rCandidate);
-
 // look for intersections of rCandidate with the edge from rStart to 
rEnd and add extra points there.
 // Points are only added in the range of the edge, not on the endless 
vector.
 BASEGFX_DLLPUBLIC B2DPolygon addPointsAtCuts(const B2DPolygon 
rCandidate, const B2DPoint rStart, const B2DPoint rEnd);
-BASEGFX_DLLPUBLIC B2DPolyPolygon addPointsAtCuts(const B2DPolyPolygon 
rCandidate, const B2DPoint rStart, const B2DPoint rEnd);
 
 // look for intersections of rCandidate with the mask Polygon and add 
extra points there.
 // The mask polygon is assumed to be closed, even when it's not 
explicitly.
 BASEGFX_DLLPUBLIC B2DPolygon addPointsAtCuts(const B2DPolygon 
rCandidate, const B2DPolyPolygon rMask);
-BASEGFX_DLLPUBLIC B2DPolyPolygon addPointsAtCuts(const B2DPolyPolygon 
rCandidate, const B2DPolyPolygon rMask);
 
 // look for self-intersections in given polygon and add extra points 
there. Result will have no
 // intersections on an edge
 BASEGFX_DLLPUBLIC B2DPolygon addPointsAtCuts(const B2DPolygon 
rCandidate);
 
-// add points at all self-intersections of single polygons (depends on 
bSelfIntersections)
-// and at polygon-polygon intersections
-BASEGFX_DLLPUBLIC B2DPolyPolygon addPointsAtCuts(const B2DPolyPolygon 
rCandidate, bool bSelfIntersections = true);
-
 } // end of namespace tools
 } // end of namespace basegfx
 
diff --git a/basegfx/source/polygon/b2dpolygoncutandtouch.cxx 
b/basegfx/source/polygon/b2dpolygoncutandtouch.cxx
index 011ecff..9ff9fe1 100644
--- a/basegfx/source/polygon/b2dpolygoncutandtouch.cxx
+++ b/basegfx/source/polygon/b2dpolygoncutandtouch.cxx
@@ -1033,20 +1033,6 @@ namespace basegfx
 
 

 
-B2DPolyPolygon addPointsAtCutsAndTouches(const B2DPolyPolygon rMask, 
const B2DPolyPolygon rCandidate)
-{
-B2DPolyPolygon aRetval;
-
-for(sal_uInt32 a(0L); a  rCandidate.count(); a++)
-{
-aRetval.append(addPointsAtCutsAndTouches(rMask, 
rCandidate.getB2DPolygon(a)));
-}
-
-return aRetval;
-}
-
-

-
 B2DPolygon addPointsAtCuts(const B2DPolygon rCandidate, const 
B2DPoint rStart, const B2DPoint rEnd)
 {
 const sal_uInt32 nCount(rCandidate.count());
@@ -1094,18 +1080,6 @@ namespace basegfx
 return rCandidate;
 }
 
-B2DPolyPolygon addPointsAtCuts(const B2DPolyPolygon rCandidate, const 
B2DPoint rStart, const B2DPoint rEnd)
-{
-B2DPolyPolygon aRetval;
-
-for(sal_uInt32 a(0); a  rCandidate.count(); a++)
-{
-aRetval.append(addPointsAtCuts(rCandidate.getB2DPolygon(a), 
rStart, rEnd));
-}
-
-return aRetval;
-}
-
 

 
 B2DPolygon addPointsAtCuts(const B2DPolygon rCandidate, const 
B2DPolyPolygon rPolyMask)
@@ -1189,18 +1163,6 @@ namespace basegfx
 return rCandidate;
 }
 
-B2DPolyPolygon addPointsAtCuts(const B2DPolyPolygon rCandidate, const 
B2DPolyPolygon rMask)
-{
-B2DPolyPolygon aRetval;
-
-for(sal_uInt32 a(0); a  rCandidate.count(); a++)
-{
-aRetval.append(addPointsAtCuts(rCandidate.getB2DPolygon(a), 
rMask));
-}
-
-return aRetval;
-}
-
 B2DPolygon addPointsAtCuts(const B2DPolygon rCandidate)
 {
 if(rCandidate.count())
@@ -1217,80 +1179,6 @@ namespace basegfx
 }
 }
 
-B2DPolyPolygon addPointsAtCuts(const B2DPolyPolygon rCandidate, bool 
bSelfIntersections)
-{
-

[Libreoffice] [PATCH] Removing unused code

2012-01-19 Thread Alexander Bergmann
---
 basegfx/inc/basegfx/polygon/b2dpolygonclipper.hxx |7 --
 basegfx/inc/basegfx/polygon/b3dpolygonclipper.hxx |   14 ---
 basegfx/source/polygon/b2dpolygonclipper.cxx  |   66 -
 basegfx/source/polygon/b3dpolygonclipper.cxx  |  102 -
 unusedcode.easy   |6 -
 5 files changed, 0 insertions(+), 195 deletions(-)

diff --git a/basegfx/inc/basegfx/polygon/b2dpolygonclipper.hxx 
b/basegfx/inc/basegfx/polygon/b2dpolygonclipper.hxx
index 172996a..2556b34 100644
--- a/basegfx/inc/basegfx/polygon/b2dpolygonclipper.hxx
+++ b/basegfx/inc/basegfx/polygon/b2dpolygonclipper.hxx
@@ -57,13 +57,6 @@ namespace basegfx
 BASEGFX_DLLPUBLIC B2DPolyPolygon clipPolyPolygonOnRange(const 
B2DPolyPolygon rCandidate, const B2DRange rRange, bool bInside, bool bStroke);
 BASEGFX_DLLPUBLIC B2DPolyPolygon clipPolygonOnRange(const B2DPolygon 
rCandidate, const B2DRange rRange, bool bInside, bool bStroke);
 
-// Clip given PolyPolygon against the endless edge (ray) defined by 
the given two points. bAbove defines on which side
-// of the edge the result will be together with the definition of the 
edge. If the edge is seen as a vector
-// from A to B and bAbove is true, the result will contain the 
geometry left of the vector.
-// The switch bStroke decides if the polygon is interpreted as area 
(false) or strokes (true).
-BASEGFX_DLLPUBLIC B2DPolyPolygon clipPolyPolygonOnEdge(const 
B2DPolyPolygon rCandidate, const B2DPoint rPointA, const B2DPoint rPointB, 
bool bAbove, bool bStroke);
-BASEGFX_DLLPUBLIC B2DPolyPolygon clipPolygonOnEdge(const B2DPolygon 
rCandidate, const B2DPoint rPointA, const B2DPoint rPointB, bool bAbove, bool 
bStroke);
-
 // Clip given PolyPolygon against given clipping polygon.
 // The switch bStroke decides if the polygon is interpreted as area 
(false) or strokes (true).
 // With stroke polygons, You get all line snippets inside rCip.
diff --git a/basegfx/inc/basegfx/polygon/b3dpolygonclipper.hxx 
b/basegfx/inc/basegfx/polygon/b3dpolygonclipper.hxx
index 0c66967..f34effe 100644
--- a/basegfx/inc/basegfx/polygon/b3dpolygonclipper.hxx
+++ b/basegfx/inc/basegfx/polygon/b3dpolygonclipper.hxx
@@ -62,26 +62,12 @@ namespace basegfx
 // version for Polygons
 BASEGFX_DLLPUBLIC B3DPolyPolygon clipPolygonOnOrthogonalPlane(const 
B3DPolygon rCandidate, B3DOrientation ePlaneOrthogonal, bool bClipPositive, 
double fPlaneOffset, bool bStroke);
 
-// Clip the given PolyPolygon against the given range. bInside defines 
if the result will contain the
-// parts which are contained in the range or vice versa.
-// The switch bStroke decides if the polygon is interpreted as area 
(false) or strokes (true).
-BASEGFX_DLLPUBLIC B3DPolyPolygon clipPolyPolygonOnRange(const 
B3DPolyPolygon rCandidate, const B3DRange rRange, bool bInside, bool bStroke);
-
 // version for Polygons
 BASEGFX_DLLPUBLIC B3DPolyPolygon clipPolygonOnRange(const B3DPolygon 
rCandidate, const B3DRange rRange, bool bInside, bool bStroke);
 
 // versions for B2DRange, clips only against X,Y
-BASEGFX_DLLPUBLIC B3DPolyPolygon clipPolyPolygonOnRange(const 
B3DPolyPolygon rCandidate, const B2DRange rRange, bool bInside, bool bStroke);
 BASEGFX_DLLPUBLIC B3DPolyPolygon clipPolygonOnRange(const B3DPolygon 
rCandidate, const B2DRange rRange, bool bInside, bool bStroke);
 
-// Clip the given PolyPolygon against given plane in 3D. The plane is 
defined by a plane normal and a point on the plane.
-// The value bClipPositive defines on which side the return value will 
be (true - on positive side of plane).
-// The switch bStroke decides if the polygon is interpreted as area 
(false) or strokes (true).
-BASEGFX_DLLPUBLIC B3DPolyPolygon clipPolyPolygonOnPlane(const 
B3DPolyPolygon rCandidate, const B3DPoint rPointOnPlane, const B3DVector 
rPlaneNormal, bool bClipPositive, bool bStroke);
-
-// version for Polygons
-BASEGFX_DLLPUBLIC B3DPolyPolygon clipPolygonOnPlane(const B3DPolygon 
rCandidate, const B3DPoint rPointOnPlane, const B3DVector rPlaneNormal, bool 
bClipPositive, bool bStroke);
-
 } // end of namespace tools
 } // end of namespace basegfx
 
diff --git a/basegfx/source/polygon/b2dpolygonclipper.cxx 
b/basegfx/source/polygon/b2dpolygonclipper.cxx
index 8514fd0..9343c2e 100644
--- a/basegfx/source/polygon/b2dpolygonclipper.cxx
+++ b/basegfx/source/polygon/b2dpolygonclipper.cxx
@@ -346,72 +346,6 @@ namespace basegfx
 return aRetval;
 }
 
-B2DPolyPolygon clipPolygonOnEdge(const B2DPolygon rCandidate, const 
B2DPoint rPointA, const B2DPoint rPointB, bool bAbove, bool bStroke)
-{
-B2DPolyPolygon aRetval;
-
-if(rPointA.equal(rPointB))
-{
-// edge has no length, return polygon
-  

[Libreoffice] [PATCH] Removing unused code

2012-01-17 Thread Alexander Bergmann
---
 unusedcode.easy |8 
 vcl/inc/vcl/pdfextoutdevdata.hxx|   62 
 vcl/inc/vcl/pdfwriter.hxx   |1 -
 vcl/source/gdi/pdfextoutdevdata.cxx |   36 
 vcl/source/gdi/pdfwriter.cxx|5 --
 vcl/source/gdi/pdfwriter_impl.cxx   |   77 ---
 vcl/source/gdi/pdfwriter_impl.hxx   |1 -
 7 files changed, 0 insertions(+), 190 deletions(-)

diff --git a/unusedcode.easy b/unusedcode.easy
index b1dcbcd..9ce0c21 100644
--- a/unusedcode.easy
+++ b/unusedcode.easy
@@ -1889,13 +1889,5 @@ 
vcl::MatrixArranger::addChild(boost::shared_ptrvcl::WindowArranger const, uns
 vcl::MatrixArranger::addWindow(Window*, unsigned int, unsigned int, int, Size 
const)
 vcl::MatrixArranger::remove(Window*)
 vcl::MatrixArranger::remove(boost::shared_ptrvcl::WindowArranger const)
-vcl::PDFExtOutDevData::EndGroup()
-vcl::PDFExtOutDevData::GetFormsFormat() const
-vcl::PDFExtOutDevData::SetAutoAdvanceTime(unsigned int, int)
-vcl::PDFExtOutDevData::SetOutlineItemDest(int, int)
-vcl::PDFExtOutDevData::SetOutlineItemParent(int, int)
-vcl::PDFExtOutDevData::SetOutlineItemText(int, rtl::OUString const)
-vcl::PDFWriter::DrawPixel(Polygon const, Color const*)
-vcl::PDFWriterImpl::drawPolyPolygon(PolyPolygon const, int, bool)
 
vcl::unx::GtkPrintWrapper::print_operation_set_has_selection(_GtkPrintOperation*,
 int) const
 
vcl::unx::GtkPrintWrapper::print_operation_set_support_selection(_GtkPrintOperation*,
 int) const
diff --git a/vcl/inc/vcl/pdfextoutdevdata.hxx b/vcl/inc/vcl/pdfextoutdevdata.hxx
index c1be576..665303b 100644
--- a/vcl/inc/vcl/pdfextoutdevdata.hxx
+++ b/vcl/inc/vcl/pdfextoutdevdata.hxx
@@ -131,7 +131,6 @@ public :
 sal_BoolGetIsExportFormFields() const;
 voidSetIsExportFormFields( const sal_Bool bExportFormFields );
 
-sal_Int32   GetFormsFormat() const;
 voidSetFormsFormat( const sal_Int32 nFormsFormat );
 
 sal_BoolGetIsExportBookmarks() const;
@@ -166,13 +165,6 @@ public :
 
 /** End render output
 
-This method ends grouped render output without
-further actions.
- */
-voidEndGroup();
-
-/** End render output
-
 This method ends grouped render output, that can be
 represented by a GfxLink. This is typically used for
 external graphic files, such as JPEGs, EPS files etc.
@@ -321,49 +313,6 @@ public :
 */
 sal_Int32 CreateOutlineItem( sal_Int32 nParent = 0, const rtl::OUString 
rText = rtl::OUString(), sal_Int32 nDestID = -1 );
 
-/** Set an outline item's parent
-
-@param nItem
-specififies which item should be reparented.
-
-@param nNewParent
-specifies which outline item will be the item's new parent.
-Use 0 for reparenting to top level.
-
-@returns
--1 if the item does not exist
--2 if the new parent does not exist, item will be reparented to top level.
-*/
-sal_Int32 SetOutlineItemParent( sal_Int32 nItem, sal_Int32 nNewParent );
-
-/** Set an outline item's title text
-
-@param nItem
-specififies which item should get a new text
-
-@param rText
-sets the title text of the item
-
-@returns
-0 if the item exists and the text was changed
--1 if the item does not exist
-*/
-sal_Int32 SetOutlineItemText( sal_Int32 nItem, const rtl::OUString rText 
);
-
-/** Set an outline item's destination
-
-@param nItem
-specififies which item should get a new dest
-
-@param nDestID
-specifies the item's new destination
-
-@returns
--1 if the item does not exist
--2 if the new dest does not exist, dest will remain unchanged
-*/
-sal_Int32 SetOutlineItemDest( sal_Int32 nItem, sal_Int32 nDestID );
-
 /** Create a new note on a page
 
 @param rRect
@@ -518,17 +467,6 @@ public :
 */
 void SetAlternateText( const String rText );
 
-/** Sets the time in seconds a page will appear before the next
-page is shown automatically
-
-@param nSeconds
-time in seconds the current page will be shown; pass 0 for manual 
advancement
-
-@param nPageNr
-the page number to apply the autoadvance time to; -1 denotes the 
current page
-*/
-void SetAutoAdvanceTime( sal_uInt32 nSeconds, sal_Int32 nPageNr = -1 );
-
 /** Sets the transitional effect to be applied when the current page gets 
shown.
 
 @param eType
diff --git a/vcl/inc/vcl/pdfwriter.hxx b/vcl/inc/vcl/pdfwriter.hxx
index b14ebd5..c4a7479 100644
--- a/vcl/inc/vcl/pdfwriter.hxx
+++ b/vcl/inc/vcl/pdfwriter.hxx
@@ -763,7 +763,6 @@ The following structure describes the permissions used in 
PDF security
 voidDrawPixel( const Point rPt, const Color rColor );
 voidDrawPixel( const Point rPt )
 { DrawPixel( rPt, Color( COL_TRANSPARENT ) ); }
-voidDrawPixel( const Polygon rPts, const Color* pColors = 
NULL );
 void  

Re: [Libreoffice] [PATCH] Removing unused code

2012-01-17 Thread David Tardon
On Tue, Jan 17, 2012 at 01:22:55PM +0100, Alexander Bergmann wrote:
 -vcl::PDFExtOutDevData::EndGroup()
 -vcl::PDFExtOutDevData::GetFormsFormat() const
 -vcl::PDFExtOutDevData::SetAutoAdvanceTime(unsigned int, int)
 -vcl::PDFExtOutDevData::SetOutlineItemDest(int, int)
 -vcl::PDFExtOutDevData::SetOutlineItemParent(int, int)
 -vcl::PDFExtOutDevData::SetOutlineItemText(int, rtl::OUString const)
 -vcl::PDFWriter::DrawPixel(Polygon const, Color const*)
 -vcl::PDFWriterImpl::drawPolyPolygon(PolyPolygon const, int, bool)

At least some of these look like they should be used... I am going to
look at it.

D.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PATCH] Removing unused code

2012-01-16 Thread Alexander Bergmann
---
 sax/source/tools/fshelper.cxx |   48 -
 unusedcode.easy   |4 ---
 2 files changed, 0 insertions(+), 52 deletions(-)

diff --git a/sax/source/tools/fshelper.cxx b/sax/source/tools/fshelper.cxx
index 89c2b1a..6cdd39d 100644
--- a/sax/source/tools/fshelper.cxx
+++ b/sax/source/tools/fshelper.cxx
@@ -29,49 +29,6 @@ FastSerializerHelper::~FastSerializerHelper()
 delete mpSerializer;
 }
 
-void FastSerializerHelper::startElement(const char* elementName, ...)
-{
-FastAttributeList* pAttrList = new FastAttributeList( mxTokenHandler );
-va_list args;
-va_start(args, elementName);
-while (true)
-{
-const char* pName = va_arg(args, const char*);
-if (!pName)
-break;
-const char* pValue = va_arg(args, const char*);
-if (pValue)
-pAttrList-addUnknown(pName, pValue);
-}
-va_end(args);
-const 
com::sun::star::uno::Referencecom::sun::star::xml::sax::XFastAttributeList 
xAttrList(pAttrList);
-mpSerializer-startUnknownElement(::rtl::OUString(), 
::rtl::OUString::createFromAscii(elementName), xAttrList);
-}
-
-void FastSerializerHelper::singleElement(const char* elementName, ...)
-{
-FastAttributeList* pAttrList = new FastAttributeList( mxTokenHandler );
-va_list args;
-va_start(args, elementName);
-while (true)
-{
-const char* pName = va_arg(args, const char*);
-if (!pName)
-break;
-const char* pValue = va_arg(args, const char*);
-if (pValue)
-pAttrList-addUnknown(pName, pValue);
-}
-va_end(args);
-const 
com::sun::star::uno::Referencecom::sun::star::xml::sax::XFastAttributeList 
xAttrList(pAttrList);
-mpSerializer-singleUnknownElement(::rtl::OUString(), 
::rtl::OUString::createFromAscii(elementName), xAttrList);
-}
-
-void FastSerializerHelper::endElement(const char* elementName)
-{
-mpSerializer-endUnknownElement(::rtl::OUString(), 
::rtl::OUString::createFromAscii(elementName));
-}
-
 void FastSerializerHelper::startElementV(sal_Int32 elementTokenId, va_list 
args)
 {
 FastAttributeList* pAttrList = new FastAttributeList( mxTokenHandler );
@@ -119,11 +76,6 @@ void FastSerializerHelper::startElementV(sal_Int32 
elementTokenId, XFastAttribut
 }
 
 
-void FastSerializerHelper::singleElement(const char* elementName, 
XFastAttributeListRef xAttrList)
-{
-mpSerializer-singleUnknownElement(::rtl::OUString(), 
::rtl::OUString::createFromAscii(elementName), xAttrList);
-}
-
 void FastSerializerHelper::singleElementV(sal_Int32 elementTokenId, 
XFastAttributeListRef xAttrList)
 {
 mpSerializer-singleFastElement(elementTokenId, xAttrList);
diff --git a/unusedcode.easy b/unusedcode.easy
index 26f593d..65495408 100644
--- a/unusedcode.easy
+++ b/unusedcode.easy
@@ -1777,10 +1777,6 @@ sax::Converter::clearUndefinedChars(rtl::OUString, 
rtl::OUString const)
 sax::Converter::convertAny(com::sun::star::uno::Any, rtl::OUString const, 
rtl::OUString const)
 sax::Converter::convertDouble(double, rtl::OUString const, short)
 sax::Converter::convertNumber64(rtl::OUStringBuffer, long)
-sax_fastparser::FastSerializerHelper::endElement(char const*)
-sax_fastparser::FastSerializerHelper::singleElement(char const*, ...)
-sax_fastparser::FastSerializerHelper::singleElement(char const*, 
com::sun::star::uno::Referencecom::sun::star::xml::sax::XFastAttributeList)
-sax_fastparser::FastSerializerHelper::startElement(char const*, ...)
 sd::CopyDlg::LinkStubReset(void*, void*)
 sd::LeftDrawPaneShell::RegisterInterface(SfxModule*)
 sd::LeftImpressPaneShell::RegisterInterface(SfxModule*)
-- 
1.7.8.3

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PUSHED][PATCH] Removing unused code

2012-01-16 Thread Caolán McNamara
On Mon, 2012-01-16 at 12:26 +0100, Alexander Bergmann wrote:
 ---
  sax/source/tools/fshelper.cxx |   48 
 -
  unusedcode.easy   |4 ---
  2 files changed, 0 insertions(+), 52 deletions(-)

Looks good to me. Pushed now, thanks for these.

As an aside, make sure to remove them from the header files as well,
though it'll be pretty obvious fast if someone tries to use them at some
future point after they're gone.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice