[PATCH] Remove unused code (oox)

2012-04-12 Thread Santiago Martinez
This patch removes unused code as listed in unusedcode.easy


0001-Remove-unused-code-in-oox.patch
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PUSHED][PATCH] Remove unused code (oox)

2012-04-10 Thread Caolán McNamara
On Sun, 2012-04-08 at 21:56 +0200, Santiago Martinez wrote:
> This patch removes unused code as listed in unusedcode.easy 

Yup, pushed now. Well, actually Julien removed the same code and got in
first.

C.

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


[PATCH] Remove unused code (oox)

2012-04-08 Thread Santiago Martinez
This patch removes unused code as listed in unusedcode.easy


0001-Remove-unused-code-in-oox.patch
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PUSHED] Re: [PATCH] remove unused code (oox)

2012-04-05 Thread Jan Holesovsky
Hi Petr,

Petr Vorel píše v Po 02. 04. 2012 v 16:18 +0200:

> removed unused classes from oox.

Pushed:

http://cgit.freedesktop.org/libreoffice/core/commit/?id=07ff032c805835355ff0dff75b8ad534dfb770e9

Thank you! :-)

Regards,
Kendy

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


[PATCH] remove unused code (oox)

2012-04-02 Thread Petr Vorel
Hi there,

removed unused classes from oox.

Regards,
Petr
>From b4350501d1fe85da2cce5750b480e33eb6409301 Mon Sep 17 00:00:00 2001
From: Petr Vorel 
Date: Mon, 2 Apr 2012 11:40:06 +0200
Subject: [PATCH] remove unused code (oox)

---
 oox/inc/oox/dump/dumperbase.hxx |   18 --
 oox/inc/oox/dump/oledumper.hxx  |   21 ---
 oox/source/dump/dumperbase.cxx  |  107 
 oox/source/dump/oledumper.cxx   |  114 ---
 unusedcode.easy |   17 --
 5 files changed, 0 insertions(+), 277 deletions(-)

diff --git a/oox/inc/oox/dump/dumperbase.hxx b/oox/inc/oox/dump/dumperbase.hxx
index 7f18220..a099886 100644
--- a/oox/inc/oox/dump/dumperbase.hxx
+++ b/oox/inc/oox/dump/dumperbase.hxx
@@ -395,7 +395,6 @@ public:
 // token list -
 
 static void appendToken( ::rtl::OUStringBuffer& rStr, const ::rtl::OUString& rToken, sal_Unicode cSep = OOX_DUMP_LISTSEP );
-static void appendToken( ::rtl::OUStringBuffer& rStr, sal_Int64 nToken, sal_Unicode cSep = OOX_DUMP_LISTSEP );
 
 static void appendIndex( ::rtl::OUStringBuffer& rStr, const ::rtl::OUString& rIdx );
 static void appendIndex( ::rtl::OUStringBuffer& rStr, sal_Int64 nIdx );
@@ -1074,9 +1073,6 @@ public:
 voidwriteColRowRange( sal_Int32 nColRow1, sal_Int32 nColRow2 );
 voidwriteColRange( sal_Int32 nCol1, sal_Int32 nCol2 );
 voidwriteRowRange( sal_Int32 nRow1, sal_Int32 nRow2 );
-voidwriteAddress( const Address& rPos );
-voidwriteRange( const Range& rRange );
-voidwriteRangeList( const RangeList& rRanges );
 
 template< typename Type >
 inline void writeDec( Type nData, sal_Int32 nWidth = 0, sal_Unicode cFill = ' ' )
@@ -1349,17 +1345,6 @@ protected:
 voidwriteArrayItem( const String& rName, const sal_uInt8* pnData, sal_Size nSize, sal_Unicode cSep = OOX_DUMP_LISTSEP );
 voidwriteDateTimeItem( const String& rName, const ::com::sun::star::util::DateTime& rDateTime );
 voidwriteGuidItem( const String& rName, const ::rtl::OUString& rGuid );
-voidwriteColIndexItem( const String& rName, sal_Int32 nCol );
-voidwriteRowIndexItem( const String& rName, sal_Int32 nRow );
-voidwriteColRangeItem( const String& rName, sal_Int32 nCol1, sal_Int32 nCol2 );
-voidwriteRowRangeItem( const String& rName, sal_Int32 nRow1, sal_Int32 nRow2 );
-voidwriteAddressItem( const String& rName, const Address& rPos );
-voidwriteRangeItem( const String& rName, const Range& rRange );
-voidwriteRangeListItem( const String& rName, const RangeList& rRanges );
-voidwriteTokenAddressItem( const String& rName, const TokenAddress& rPos, bool bNameMode );
-voidwriteTokenAddress3dItem( const String& rName, const ::rtl::OUString& rRef, const TokenAddress& rPos, bool bNameMode );
-voidwriteTokenRangeItem( const String& rName, const TokenRange& rRange, bool bNameMode );
-voidwriteTokenRange3dItem( const String& rName, const ::rtl::OUString& rRef, const TokenRange& rRange, bool bNameMode );
 
 template< typename Type >
 voidaddNameToItem( Type nData, const NameListWrapper& rListWrp );
@@ -1753,9 +1738,6 @@ protected:
 const OutputObjectBase& rParent,
 const BinaryInputStreamRef& rxStrm,
 rtl_TextEncoding eTextEnc );
-voidconstruct(
-const InputObjectBase& rParent,
-rtl_TextEncoding eTextEnc );
 
 virtual boolimplIsValid() const;
 virtual voidimplDump();
diff --git a/oox/inc/oox/dump/oledumper.hxx b/oox/inc/oox/dump/oledumper.hxx
index 4d3ceaa..64d3d5c 100644
--- a/oox/inc/oox/dump/oledumper.hxx
+++ b/oox/inc/oox/dump/oledumper.hxx
@@ -84,27 +84,6 @@ protected:
 };
 
 // 
-
-class StdHlinkObject : public OleInputObjectBase
-{
-public:
-explicitStdHlinkObject( const InputObjectBase& rParent );
-
-protected:
-virtual voidimplDump();
-
-private:
-::rtl::OUString dumpHyperlinkString( const String& rName, bool bUnicode );
-
-booldumpGuidAndMoniker();
-voiddumpUrlMon

Re: [PUSHED][PATCH] remove unused code (oox)

2012-03-29 Thread Caolán McNamara
On Thu, 2012-03-29 at 15:23 +0200, Petr Vorel wrote:
> Hi there,
> 
> removed unused methods

pushed, thanks for these

C.

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


[PATCH] remove unused code (oox)

2012-03-29 Thread Petr Vorel
Hi there,

removed unused methods, hope better this time.

Regards,
Petr
>From 2d8c47fede9ec6842a98663f16c4efa70f833e22 Mon Sep 17 00:00:00 2001
From: Petr Vorel 
Date: Thu, 29 Mar 2012 11:54:36 +0200
Subject: [PATCH] remove unused code (oox)

---
 oox/inc/oox/dump/dffdumper.hxx   |1 -
 oox/inc/oox/dump/dumperbase.hxx  |   10 --
 oox/inc/oox/dump/oledumper.hxx   |1 -
 oox/inc/oox/ole/vbaprojectfilter.hxx |5 ---
 oox/source/dump/dffdumper.cxx|6 
 oox/source/dump/dumperbase.cxx   |   53 --
 oox/source/dump/oledumper.cxx|5 ---
 oox/source/ole/vbaprojectfilter.cxx  |5 ---
 unusedcode.easy  |   12 ---
 9 files changed, 0 insertions(+), 98 deletions(-)

diff --git a/oox/inc/oox/dump/dffdumper.hxx b/oox/inc/oox/dump/dffdumper.hxx
index 10f84f3..de0913f 100644
--- a/oox/inc/oox/dump/dffdumper.hxx
+++ b/oox/inc/oox/dump/dffdumper.hxx
@@ -49,7 +49,6 @@ protected:
 inline explicit DffStreamObject() {}
 
 using   SequenceRecordObjectBase::construct;
-voidconstruct( const OutputObjectBase& rParent, const BinaryInputStreamRef& rxStrm );
 
 virtual boolimplReadRecordHeader( BinaryInputStream& rBaseStrm, sal_Int64& ornRecId, sal_Int64& ornRecSize );
 virtual voidimplWriteExtHeader();
diff --git a/oox/inc/oox/dump/dumperbase.hxx b/oox/inc/oox/dump/dumperbase.hxx
index a8b16b0..0dd1abc 100644
--- a/oox/inc/oox/dump/dumperbase.hxx
+++ b/oox/inc/oox/dump/dumperbase.hxx
@@ -231,7 +231,6 @@ struct ItemFormat
 explicitItemFormat();
 
 voidset( DataType eDataType, FormatType eFmtType, const ::rtl::OUString& rItemName );
-voidset( DataType eDataType, FormatType eFmtType, const ::rtl::OUString& rItemName, const ::rtl::OUString& rListName );
 
 /** Initializes the struct from a vector of strings containing the item format.
 
@@ -398,7 +397,6 @@ public:
 static void appendToken( ::rtl::OUStringBuffer& rStr, const ::rtl::OUString& rToken, sal_Unicode cSep = OOX_DUMP_LISTSEP );
 static void appendToken( ::rtl::OUStringBuffer& rStr, sal_Int64 nToken, sal_Unicode cSep = OOX_DUMP_LISTSEP );
 static void prependToken( ::rtl::OUStringBuffer& rStr, const ::rtl::OUString& rToken, sal_Unicode cSep = OOX_DUMP_LISTSEP );
-static void prependToken( ::rtl::OUStringBuffer& rStr, sal_Int64 nToken, sal_Unicode cSep = OOX_DUMP_LISTSEP );
 
 static void appendIndex( ::rtl::OUStringBuffer& rStr, const ::rtl::OUString& rIdx );
 static void appendIndex( ::rtl::OUStringBuffer& rStr, sal_Int64 nIdx );
@@ -1047,11 +1045,9 @@ public:
 
 voidincIndent();
 voiddecIndent();
-voidresetIndent();
 
 voidstartTable( sal_Int32 nW1 );
 voidstartTable( sal_Int32 nW1, sal_Int32 nW2 );
-voidstartTable( sal_Int32 nW1, sal_Int32 nW2, sal_Int32 nW3 );
 voidstartTable( sal_Int32 nW1, sal_Int32 nW2, sal_Int32 nW3, sal_Int32 nW4 );
 voidstartTable( size_t nColCount, const sal_Int32* pnColWidths );
 voidtab();
@@ -1209,8 +1205,6 @@ public:
 explicitStorageIterator( const StorageRef& rxStrg );
 virtual ~StorageIterator();
 
-size_t  getElementCount() const;
-
 StorageIterator&operator++();
 
 ::rtl::OUString getName() const;
@@ -1251,8 +1245,6 @@ protected:
 
 // 
 
-voidreconstructConfig( const ConfigRef& rxConfig );
-
 inline Config&  cfg() const { return *mxConfig; }
 
 private:
@@ -1564,8 +1556,6 @@ protected:
 ::rtl::OUString dumpNullCharArray( const String& rName, rtl_TextEncoding eTextEnc );
 ::rtl::OUString dumpNullUnicodeArray( const String& rName );
 
-double  dumpRk( const String& rName = EMPTY_STRING );
-sal_Int32   dumpColorABGR( const String& rName = EMPTY_STRING );
 ::com::sun::star::util::DateTime dumpFileTime( const String& rName = EMPTY_STRING );
 ::rtl::OUString dumpGuid( const String& rName = EMPTY_STRING );
 
diff --git a/oox/inc/oox/dump/oledumper.hxx b/oox/inc/oox/dump/oledumper.hxx
index 42abf2d..4d3ceaa 100644
--- a/oox/inc/oox/dump/oledumper.hxx
+++ b/oox/inc/oox/dump/oledumper.hxx
@@ -178,7 +178,6 @@ protected:
 
 using   StorageObjectBase::construct;
 voidconstruct( const ObjectBase& rParent, const StorageRef& rxStrg, const ::rtl::OUString& rSysPath );
-voidconstruct( const ObjectBase& rParent );
 
 virtual voidimplDumpStream(

Re: [PUSHED][PATCH] remove unused code (oox, sc)

2012-03-28 Thread Petr Vorel
> FWIW, if a class has only one constructor and that constructor is unused
> then it indicates that the whole class (except perhaps for any static
> methods it might have) should be possible to be removed.
Sorry, I'll think more next time.
Petr
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PUSHED][PATCH] remove unused code (oox, sc)

2012-03-28 Thread Caolán McNamara
On Wed, 2012-03-28 at 16:23 +0200, Petr Vorel wrote:
> Hi there,
> 
> yet another remove unused code patch.

Pushed now, thanks for this.

FWIW, if a class has only one constructor and that constructor is unused
then it indicates that the whole class (except perhaps for any static
methods it might have) should be possible to be removed.

C.

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


[PATCH] remove unused code (oox, sc)

2012-03-28 Thread Petr Vorel
Hi there,

yet another remove unused code patch.

Regards,
Petr
>From 38a7f343736f7049d28aae132307eba02a019f27 Mon Sep 17 00:00:00 2001
From: Petr Vorel 
Date: Wed, 28 Mar 2012 11:20:18 +0200
Subject: [PATCH] remove unused code (oox, sc)

---
 oox/inc/oox/drawingml/textparagraphproperties.hxx |4 -
 oox/inc/oox/dump/dffdumper.hxx|2 -
 oox/inc/oox/dump/dumperbase.hxx   |   13 
 oox/source/drawingml/textparagraphproperties.cxx  |   25 
 oox/source/dump/dffdumper.cxx |   11 
 oox/source/dump/dumperbase.cxx|   65 -
 sd/source/filter/eppt/epptooxml.hxx   |1 -
 sd/source/filter/eppt/pptx-epptooxml.cxx  |   14 -
 unusedcode.easy   |   14 -
 9 files changed, 0 insertions(+), 149 deletions(-)

diff --git a/oox/inc/oox/drawingml/textparagraphproperties.hxx b/oox/inc/oox/drawingml/textparagraphproperties.hxx
index b2c5096..e037ff2 100644
--- a/oox/inc/oox/drawingml/textparagraphproperties.hxx
+++ b/oox/inc/oox/drawingml/textparagraphproperties.hxx
@@ -114,10 +114,6 @@ public:
 size can be zero and the default value is returned. */
 float   getCharHeightPoints( float fDefault ) const;
 
-#if OSL_DEBUG_LEVEL > 0
-void dump() const;
-#endif
-
 protected:
 
 TextCharacterProperties maTextCharacterProperties;
diff --git a/oox/inc/oox/dump/dffdumper.hxx b/oox/inc/oox/dump/dffdumper.hxx
index a23d071..10f84f3 100644
--- a/oox/inc/oox/dump/dffdumper.hxx
+++ b/oox/inc/oox/dump/dffdumper.hxx
@@ -49,7 +49,6 @@ protected:
 inline explicit DffStreamObject() {}
 
 using   SequenceRecordObjectBase::construct;
-voidconstruct( const ObjectBase& rParent, const BinaryInputStreamRef& rxStrm, const ::rtl::OUString& rSysFileName );
 voidconstruct( const OutputObjectBase& rParent, const BinaryInputStreamRef& rxStrm );
 
 virtual boolimplReadRecordHeader( BinaryInputStream& rBaseStrm, sal_Int64& ornRecId, sal_Int64& ornRecSize );
@@ -61,7 +60,6 @@ private:
 voidconstructDffObj();
 
 sal_uInt32  dumpDffSimpleColor( const String& rName );
-sal_uInt32  dumpDffColor( const String& rName );
 
 voiddumpDffOpt();
 sal_uInt16  dumpDffOptPropHeader();
diff --git a/oox/inc/oox/dump/dumperbase.hxx b/oox/inc/oox/dump/dumperbase.hxx
index 5989c1e..68a4848 100644
--- a/oox/inc/oox/dump/dumperbase.hxx
+++ b/oox/inc/oox/dump/dumperbase.hxx
@@ -480,26 +480,18 @@ static const String EMPTY_STRING;
 class FormulaStack
 {
 public:
-explicitFormulaStack();
-
 inline const ::rtl::OUString& getFormulaString() const { return getString( maFmlaStack ); }
 inline const ::rtl::OUString& getClassesString() const { return getString( maClassStack ); }
 
 voidpushOperand( const String& rOp, const ::rtl::OUString& rTokClass );
-voidpushOperand( const String& rOp );
-voidpushUnaryOp( const String& rLOp, const String& rROp );
-voidpushBinaryOp( const String& rOp );
-voidpushFuncOp( const String& rFunc, const ::rtl::OUString& rTokClass, sal_uInt8 nParamCount );
 
 inline void setError() { mbError = true; }
-voidreplaceOnTop( const ::rtl::OUString& rOld, const ::rtl::OUString& rNew );
 
 private:
 typedef ::std::stack< ::rtl::OUString > StringStack;
 
 inline bool check( bool bCond ) { return (mbError |= !bCond) == false; }
 
-const ::rtl::OUString& getString( const StringStack& rStack ) const;
 voidpushUnaryOp( StringStack& rStack, const ::rtl::OUString& rLOp, const ::rtl::OUString& rROp );
 voidpushBinaryOp( StringStack& rStack, const ::rtl::OUString& rOp );
 voidpushFuncOp( StringStack& rStack, const ::rtl::OUString& rOp, sal_uInt8 nParamCount );
@@ -612,9 +604,6 @@ private:
 ::rtl::OUString& orKey,
 ::rtl::OUString& orData ) const;
 
-LineTypereadConfigLine(
-TextInputStream& rStrm ) const;
-
 voidprocessConfigItem(
 TextInputStream& rStrm,
 const ::rtl::OUString& rKey,
@@ -1013,7 +1002,6 @@ public:
 template< typename Type >
 boolhasName( const NameListWrapper& rListWrp, Type nKey ) const;
 
-::com::sun::star::uno::Sequence< ::com::sun::star::beans::NamedValue > requestEncryptionData( ::comphelper::IDocPasswordVerifier& rVerifier );
 boolisPasswordCanc

Re: [PUSHED][PATCH] remove unused code (oox, sc)

2012-03-27 Thread Caolán McNamara
On Tue, 2012-03-27 at 16:27 +0200, Petr Vorel wrote:
> Hi there,
> 
> another bits of unused code removed.

looks good, pushed now, thanks for these.

C.


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


[PATCH] remove unused code (oox, sc)

2012-03-27 Thread Petr Vorel
Hi there,

another bits of unused code removed.

Regards,
Petr

>From e31c211d2ebb7cde8a260ec0bcef5359c0c1db8f Mon Sep 17 00:00:00 2001
From: Petr Vorel 
Date: Tue, 27 Mar 2012 11:03:09 +0200
Subject: [PATCH] remove unused code (oox, sc)

---
 oox/inc/oox/dump/dumperbase.hxx   |4 --
 oox/inc/oox/mathml/importutils.hxx|4 --
 oox/inc/oox/ppt/pptshape.hxx  |1 -
 oox/source/dump/dumperbase.cxx|5 ---
 oox/source/mathml/importutils.cxx |   10 -
 oox/source/ppt/pptshape.cxx   |7 
 sc/source/filter/inc/addressconverter.hxx |1 -
 sc/source/filter/inc/biffoutputstream.hxx |   14 
 sc/source/filter/inc/sheetdatabuffer.hxx  |   10 -
 sc/source/filter/oox/addressconverter.cxx |6 ---
 sc/source/filter/oox/biffoutputstream.cxx |   23 -
 sc/source/filter/oox/sheetdatabuffer.cxx  |   52 +
 unusedcode.easy   |   12 ---
 13 files changed, 1 insertions(+), 148 deletions(-)

diff --git a/oox/inc/oox/dump/dumperbase.hxx b/oox/inc/oox/dump/dumperbase.hxx
index 534776b..5989c1e 100644
--- a/oox/inc/oox/dump/dumperbase.hxx
+++ b/oox/inc/oox/dump/dumperbase.hxx
@@ -1860,10 +1860,6 @@ public:
 const BinaryInputStreamRef& rxStrm,
 const ::rtl::OUString& rSysFileName );
 
-explicitXmlStreamObject(
-const OutputObjectBase& rParent,
-const BinaryInputStreamRef& rxStrm );
-
 protected:
 virtual voidimplDumpText( TextInputStream& rTextStrm );
 };
diff --git a/oox/inc/oox/mathml/importutils.hxx b/oox/inc/oox/mathml/importutils.hxx
index 9b90681..fc0e276 100644
--- a/oox/inc/oox/mathml/importutils.hxx
+++ b/oox/inc/oox/mathml/importutils.hxx
@@ -213,10 +213,6 @@ public:
 */
 bool findTag( int token );
 /**
- Skips the given element (i.e. reads up to and including the matching closing tag).
-*/
-void skipElement( int token );
-/**
  Handle the current (unexpected) tag.
 */
 void handleUnexpectedTag();
diff --git a/oox/inc/oox/ppt/pptshape.hxx b/oox/inc/oox/ppt/pptshape.hxx
index e06fda6..07ab723 100644
--- a/oox/inc/oox/ppt/pptshape.hxx
+++ b/oox/inc/oox/ppt/pptshape.hxx
@@ -67,7 +67,6 @@ public:
 
 static oox::drawingml::ShapePtr findPlaceholder( const sal_Int32 nMasterPlaceholder, std::vector< oox::drawingml::ShapePtr >& rShapes );
 static oox::drawingml::ShapePtr findPlaceholderByIndex( const sal_Int32 nIdx, std::vector< oox::drawingml::ShapePtr >& rShapes );
-static oox::drawingml::ShapePtr findPlaceholder( sal_Int32 nFirstPlaceholder, sal_Int32 nSecondPlaceholder, std::vector< oox::drawingml::ShapePtr >& rShapes );
 
 static oox::drawingml::TextListStylePtr getSubTypeTextListStyle( const SlidePersist& rSlidePersist, sal_Int32 nSubType );
 
diff --git a/oox/source/dump/dumperbase.cxx b/oox/source/dump/dumperbase.cxx
index f078111..366090c 100644
--- a/oox/source/dump/dumperbase.cxx
+++ b/oox/source/dump/dumperbase.cxx
@@ -2920,11 +2920,6 @@ XmlStreamObject::XmlStreamObject( const ObjectBase& rParent,
 TextStreamObjectBase::construct( rParent, rxStrm, RTL_TEXTENCODING_UTF8, rSysFileName );
 }
 
-XmlStreamObject::XmlStreamObject( const OutputObjectBase& rParent, const BinaryInputStreamRef& rxStrm )
-{
-TextStreamObjectBase::construct( rParent, rxStrm, RTL_TEXTENCODING_UTF8 );
-}
-
 void XmlStreamObject::implDumpText( TextInputStream& rTextStrm )
 {
 /*  Buffers a start element and the following element text. Needed to dump
diff --git a/oox/source/mathml/importutils.cxx b/oox/source/mathml/importutils.cxx
index ab9d224..41a254d 100644
--- a/oox/source/mathml/importutils.cxx
+++ b/oox/source/mathml/importutils.cxx
@@ -114,11 +114,6 @@ static OUString tokenToString( int token )
 
 } // namespace
 
-bool XmlStream::AttributeList::hasAttribute( int token ) const
-{
-return attrs.find( token ) != attrs.end();
-}
-
 rtl::OUString XmlStream::AttributeList::attribute( int token, const rtl::OUString& def ) const
 {
 std::map< int, rtl::OUString >::const_iterator find = attrs.find( token );
@@ -304,11 +299,6 @@ bool XmlStream::findTagInternal( int token, bool silent )
 return false;
 }
 
-void XmlStream::skipElement( int token )
-{
-return skipElementInternal( token, true ); // no debug about skipping if called from outside
-}
-
 void XmlStream::skipElementInternal( int token, bool silent )
 {
 int closing = ( token & ~TAG_OPENING ) | TAG_CLOSING; // make it a closing tag
diff --git a/oox/source/ppt/pptshape.cxx b/oox/source/ppt/pptshape.cxx
index a72a10c..2b017f0 100644
--- a/oox/source/ppt/pptshape.cxx
+++ b/oox/source/ppt/pptshape.cxx
@@ -423,13 +423,6 @@ oox::drawingml::ShapePtr PPTShape::findPlaceholderByIndex( const sal_Int32 nIdx,
 return aShapePtr;
 }
 
-

Re: [PUSHED][PATCH] remove unused code (oox)

2012-03-26 Thread Caolán McNamara
On Sun, 2012-03-25 at 20:43 +0200, Petr Vorel wrote:
> Hi there,
> 
> removed some code from oox.

Pushed, looks good. I suspect that this will have a bit of a cascade
effect and a pile more stuff will become available for removal on the
next run.

C.

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


[PATCH] remove unused code (oox)

2012-03-25 Thread Petr Vorel
Hi there,

removed some code from oox.

Regards,
Petr
>From e04049e2554a94ce80dc21dfe390e91c8fd8bc41 Mon Sep 17 00:00:00 2001
From: Petr Vorel 
Date: Thu, 22 Mar 2012 20:14:13 +0100
Subject: [PATCH] remove unused code (oox)

---
 oox/source/ole/vbaprojectfilter.cxx   |   13 -
 sc/source/filter/inc/biffinputstream.hxx  |   27 +---
 sc/source/filter/inc/biffoutputstream.hxx |5 --
 sc/source/filter/inc/drawingmanager.hxx   |4 --
 sc/source/filter/oox/biffdetector.cxx |5 --
 sc/source/filter/oox/biffinputstream.cxx  |   11 -
 sc/source/filter/oox/biffoutputstream.cxx |   16 ---
 sc/source/filter/oox/drawingmanager.cxx   |   71 -
 sc/source/filter/oox/excelfilter.cxx  |   27 ---
 sc/source/filter/oox/ooxformulaparser.cxx |5 --
 unusedcode.easy   |   13 -
 11 files changed, 1 insertions(+), 196 deletions(-)

diff --git a/oox/source/ole/vbaprojectfilter.cxx b/oox/source/ole/vbaprojectfilter.cxx
index e152598..f4698fa 100644
--- a/oox/source/ole/vbaprojectfilter.cxx
+++ b/oox/source/ole/vbaprojectfilter.cxx
@@ -77,19 +77,6 @@ OUString SAL_CALL WordVbaProjectFilter_getImplementationName() throw()
 return CREATE_OUSTRING( "com.sun.star.comp.oox.WordVbaProjectFilter" );
 }
 
-Sequence< OUString > SAL_CALL WordVbaProjectFilter_getSupportedServiceNames() throw()
-{
-Sequence< OUString > aSeq( 1 );
-aSeq[ 0 ] = CREATE_OUSTRING( "com.sun.star.document.ImportFilter" );
-return aSeq;
-}
-
-Reference< XInterface > SAL_CALL WordVbaProjectFilter_createInstance(
-const Reference< XComponentContext >& rxContext ) throw( Exception )
-{
-return static_cast< ::cppu::OWeakObject* >( new WordVbaProjectFilter( rxContext ) );
-}
-
 // 
 
 WordVbaProjectFilter::WordVbaProjectFilter( const Reference< XComponentContext >& rxContext ) throw( RuntimeException ) :
diff --git a/sc/source/filter/inc/biffinputstream.hxx b/sc/source/filter/inc/biffinputstream.hxx
index 9f01b02..47ee2b0 100644
--- a/sc/source/filter/inc/biffinputstream.hxx
+++ b/sc/source/filter/inc/biffinputstream.hxx
@@ -129,16 +129,7 @@ private:
 Trying to read over the record limits results in a stream error. The
 isValid() function indicates that by returning false. From now on the data
 returned by the read functions is undefined. The error state will be reset,
-if the record is reset (with the function resetRecord()), or if the next
-record is started.
-
-To switch off the automatic lookup of CONTINUE records, use resetRecord()
-with false parameter. This is useful e.g. on import of drawing layer data,
-where sometimes solely CONTINUE records will occur. The automatic lookup
-keeps switched off until the method resetRecord() is called with parameter
-true. All other settings done on the stream (e.g. alternative CONTINUE
-record identifier, enabled decryption, NUL substitution character) will be
-reset to default values, if a new record is started.
+if the next record is started.
 
 The import stream supports decrypting the stream data. The contents of a
 record (not the record header) will be encrypted by Excel if the file has
@@ -189,22 +180,6 @@ public:
  */
 boolstartRecordByHandle( sal_Int64 nRecHandle );
 
-/** Sets stream pointer to begin of record content.
-
-@param bContLookup
-Automatic CONTINUE lookup on/off. In difference to other stream
-settings, this setting is persistent until next call of this
-function (because it is wanted to receive the next CONTINUE records
-separately).
-@param nAltContId
-Sets an alternative record identifier for content continuation.
-This value is reset automatically when a new record is started with
-startNextRecord().
- */
-voidresetRecord(
-bool bContLookup,
-sal_uInt16 nAltContId = BIFF_ID_UNKNOWN );
-
 /** Sets stream pointer before current record and invalidates stream.
 
 The next call to startNextRecord() will start again the current record.
diff --git a/sc/source/filter/inc/biffoutputstream.hxx b/sc/source/filter/inc/biffoutputstream.hxx
index a38bfdf..f7a6462 100644
--- a/sc/source/filter/inc/biffoutputstream.hxx
+++ b/sc/source/filter/inc/biffoutputstream.hxx
@@ -110,8 +110,6 @@ public:
 
 /** Returns the absolute position in the wrapped binary stream. */
 sal_Int64   tellBase() const;
-/** Returns the total size of the wrapped binary stream. */
-sal_Int64   sizeBase() const;
 
 // BinaryOutputStream interface (stream write access) -
 
@@ -120,9 +118,6 @@ public:
 /** Writes nBytes bytes from the passed buf

Re: [PUSHED][PATCH] remove unused code (oox, sc)

2012-03-20 Thread Caolán McNamara
On Sun, 2012-03-18 at 14:40 +0100, Petr Vorel wrote:
> Hi there,
> 
> some unused code removed (oox, sc).

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

C.

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


[PATCH] remove unused code (oox, sc)

2012-03-18 Thread Petr Vorel
Hi there,

some unused code removed (oox, sc).

Regards,
Petr

>From 3c03ddd6eb8e0cefa229f3874e3da35bd0a57788 Mon Sep 17 00:00:00 2001
From: Petr Vorel 
Date: Sat, 17 Mar 2012 20:25:52 +0100
Subject: [PATCH 1/2] remove unused code (oox, sc)

---
 oox/inc/oox/dump/dumperbase.hxx  |7 ---
 oox/inc/oox/dump/oledumper.hxx   |5 -
 oox/inc/oox/ole/vbacontrol.hxx   |2 --
 oox/source/dump/dumperbase.cxx   |   15 ---
 oox/source/dump/oledumper.cxx|7 ---
 oox/source/ole/vbacontrol.cxx|5 -
 sc/source/filter/inc/formulabase.hxx |3 ---
 sc/source/filter/oox/formulabase.cxx |9 -
 unusedcode.easy  |7 ---
 9 files changed, 0 insertions(+), 60 deletions(-)

diff --git a/oox/inc/oox/dump/dumperbase.hxx b/oox/inc/oox/dump/dumperbase.hxx
index 7cba7ae..534776b 100644
--- a/oox/inc/oox/dump/dumperbase.hxx
+++ b/oox/inc/oox/dump/dumperbase.hxx
@@ -993,8 +993,6 @@ public:
 inline const StorageRef& getRootStorage() const { return mxCfgData->getRootStorage(); }
 inline const ::rtl::OUString& getSysFileName() const { return mxCfgData->getSysFileName(); }
 
-voidsetStringOption( const String& rKey, const String& rData );
-
 const ::rtl::OUString& getStringOption( const String& rKey, const ::rtl::OUString& rDefault ) const;
 boolgetBoolOption( const String& rKey, bool bDefault ) const;
 template< typename Type >
@@ -1005,7 +1003,6 @@ public:
 
 template< typename ListType >
 ::boost::shared_ptr< ListType > createNameList( const String& rListName );
-voidsetNameList( const String& rListName, const NameListRef& rxList );
 voideraseNameList( const String& rListName );
 NameListRef getNameList( const String& rListName ) const;
 
@@ -1792,10 +1789,6 @@ public:
 const BinaryInputStreamRef& rxStrm,
 const ::rtl::OUString& rSysFileName );
 
-explicitBinaryStreamObject(
-const OutputObjectBase& rParent,
-const BinaryInputStreamRef& rxStrm );
-
 protected:
 voiddumpBinaryStream( bool bShowOffset = true );
 
diff --git a/oox/inc/oox/dump/oledumper.hxx b/oox/inc/oox/dump/oledumper.hxx
index 1a94e77..42abf2d 100644
--- a/oox/inc/oox/dump/oledumper.hxx
+++ b/oox/inc/oox/dump/oledumper.hxx
@@ -331,11 +331,6 @@ protected:
 const String& rPropNameList,
 bool b64BitPropFlags = false );
 voidconstruct(
-const OutputObjectBase& rParent,
-const BinaryInputStreamRef& rxStrm,
-const String& rPropNameList,
-bool b64BitPropFlags = false );
-voidconstruct(
 const InputObjectBase& rParent,
 const String& rPropNameList,
 bool b64BitPropFlags = false );
diff --git a/oox/inc/oox/ole/vbacontrol.hxx b/oox/inc/oox/ole/vbacontrol.hxx
index 4647245..b4d2afb 100644
--- a/oox/inc/oox/ole/vbacontrol.hxx
+++ b/oox/inc/oox/ole/vbacontrol.hxx
@@ -65,8 +65,6 @@ public:
 inline const AxPairData& getPosition() const { return maPos; }
 /** Returns the unique identifier of this control. */
 inline sal_Int32getId() const { return mnId; }
-/** Returns true, if the control is visible. */
-boolisVisible() const;
 /** Returns true, if this control is a container control. */
 boolisContainer() const;
 /** Returns the length of the stream data for stream based controls. */
diff --git a/oox/source/dump/dumperbase.cxx b/oox/source/dump/dumperbase.cxx
index 54159b440..f078111 100644
--- a/oox/source/dump/dumperbase.cxx
+++ b/oox/source/dump/dumperbase.cxx
@@ -1699,11 +1699,6 @@ void Config::construct( const sal_Char* pcEnvVar, const Reference< XComponentCon
 mxCfgData.reset( new SharedConfigData( OUString::createFromAscii( pcFileName ), rxContext, rxRootStrg, rSysFileName, rMediaDesc ) );
 }
 
-void Config::setStringOption( const String& rKey, const String& rData )
-{
-mxCfgData->setOption( rKey, rData );
-}
-
 const OUString& Config::getStringOption( const String& rKey, const OUString& rDefault ) const
 {
 const OUString* pData = implGetOption( rKey );
@@ -1726,11 +1721,6 @@ bool Config::isImportEnabled() const
 return getBoolOption( "enable-import", true );
 }
 
-void Config::setNameList( const String& rListName, const NameListRef& rxList )
-{
-mxCfgData->setNameList( rListName, rxList );
-}
-
 void Config::eraseNameList( const String& rListName )
 {
 mxCfgData->eraseNameList( rListName );
@@ -2837,11 +2827,6 @@ BinaryStreamObject::BinaryStreamObject( const ObjectBase& rParent, const BinaryI
 InputObjectBase::construct

[PATCH] remove unused code (oox, sd)

2012-03-15 Thread Petr Vorel
Hi there,

remove some unused code in oox and sd.

Regards,
Petr
>From 2cc831282d53c3a14dd10fc8cb190f1d15df3354 Mon Sep 17 00:00:00 2001
From: Petr Vorel 
Date: Thu, 15 Mar 2012 11:40:15 +0100
Subject: [PATCH] remove unused code (oox, sd)

---
 oox/inc/oox/helper/attributelist.hxx |6 --
 oox/inc/oox/helper/containerhelper.hxx   |   16 
 oox/inc/oox/helper/graphichelper.hxx |2 --
 oox/source/helper/attributelist.cxx  |   10 --
 oox/source/helper/containerhelper.cxx|   18 --
 oox/source/helper/graphichelper.cxx  |5 -
 sd/source/filter/eppt/epptooxml.hxx  |1 -
 sd/source/filter/eppt/pptx-epptooxml.cxx |   10 --
 unusedcode.easy  |5 -
 9 files changed, 0 insertions(+), 73 deletions(-)

diff --git a/oox/inc/oox/helper/attributelist.hxx b/oox/inc/oox/helper/attributelist.hxx
index d0a0600..c60c6f8 100644
--- a/oox/inc/oox/helper/attributelist.hxx
+++ b/oox/inc/oox/helper/attributelist.hxx
@@ -66,12 +66,6 @@ public:
 
 /** Returns the 32-bit signed integer value from the passed string (hexadecimal). */
 static sal_Int32decodeIntegerHex( const ::rtl::OUString& rValue );
-
-/** Returns the 32-bit unsigned integer value from the passed string (hexadecimal). */
-static sal_uInt32   decodeUnsignedHex( const ::rtl::OUString& rValue );
-
-/** Returns the 64-bit signed integer value from the passed string (hexadecimal). */
-static sal_Int64decodeHyperHex( const ::rtl::OUString& rValue );
 };
 
 // 
diff --git a/oox/inc/oox/helper/containerhelper.hxx b/oox/inc/oox/helper/containerhelper.hxx
index 5c9dc88..eef9429 100644
--- a/oox/inc/oox/helper/containerhelper.hxx
+++ b/oox/inc/oox/helper/containerhelper.hxx
@@ -173,22 +173,6 @@ public:
 static ::com::sun::star::uno::Reference< ::com::sun::star::container::XIndexContainer >
 createIndexContainer( const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& rxContext );
 
-/** Inserts an object into an indexed container.
-
-@param rxIndexContainer  com.sun.star.container.XIndexContainer
-interface of the indexed container.
-
-@param nIndex  Insertion index for the object.
-
-@param rObject  The object to be inserted.
-
-@return  True = object successfully inserted.
- */
-static bool insertByIndex(
-const ::com::sun::star::uno::Reference< ::com::sun::star::container::XIndexContainer >& rxIndexContainer,
-sal_Int32 nIndex,
-const ::com::sun::star::uno::Any& rObject );
-
 // com.sun.star.container.XNameContainer --
 
 /** Creates a new name container object from scratch. */
diff --git a/oox/inc/oox/helper/graphichelper.hxx b/oox/inc/oox/helper/graphichelper.hxx
index d3507fa..9881200 100644
--- a/oox/inc/oox/helper/graphichelper.hxx
+++ b/oox/inc/oox/helper/graphichelper.hxx
@@ -93,8 +93,6 @@ public:
 sal_Int32   convertScreenPixelXToHmm( double fPixelX ) const;
 /** Converts the passed value from vertical screen pixels to 1/100 mm. */
 sal_Int32   convertScreenPixelYToHmm( double fPixelY ) const;
-/** Converts the passed point from screen pixels to 1/100 mm. */
-::com::sun::star::awt::Point convertScreenPixelToHmm( const ::com::sun::star::awt::Point& rPixel ) const;
 /** Converts the passed size from screen pixels to 1/100 mm. */
 ::com::sun::star::awt::Size convertScreenPixelToHmm( const ::com::sun::star::awt::Size& rPixel ) const;
 
diff --git a/oox/source/helper/attributelist.cxx b/oox/source/helper/attributelist.cxx
index 33f4ed5..f1d114d 100644
--- a/oox/source/helper/attributelist.cxx
+++ b/oox/source/helper/attributelist.cxx
@@ -122,16 +122,6 @@ sal_Int32 AttributeConversion::decodeIntegerHex( const OUString& rValue )
 return rValue.toInt32( 16 );
 }
 
-sal_uInt32 AttributeConversion::decodeUnsignedHex( const OUString& rValue )
-{
-return getLimitedValue< sal_uInt32, sal_Int64 >( rValue.toInt64( 16 ), 0, SAL_MAX_UINT32 );
-}
-
-sal_Int64 AttributeConversion::decodeHyperHex( const OUString& rValue )
-{
-return rValue.toInt64( 16 );
-}
-
 // 
 
 AttributeList::AttributeList( const Reference< XFastAttributeList >& rxAttribs ) :
diff --git a/oox/source/helper/containerhelper.cxx b/oox/source/helper/containerhelper.cxx
index 23bad22..a19e9b4 100644
--- a/oox/source/helper/containerhelper.cxx
+++ b/oox/source/helper/containerhelper.cxx
@@ -119,24 +119,6 @@ Reference< XIndexContainer > ContainerHelper::createIndexContainer( const Refere
 return xContainer;
 }
 
-bool Contai