Kaydet (Commit) 65f2d5c9 authored tarafından Stephan Bergmann's avatar Stephan Bergmann

-Werror,-Wunused-private-field (Clang towards 3.2)

Change-Id: Idea1fd2769c2431906d4898c0ec935f20bddc391
üst 0a7000e0
...@@ -30,7 +30,6 @@ ...@@ -30,7 +30,6 @@
#include <rtl/ustrbuf.hxx> #include <rtl/ustrbuf.hxx>
#include <com/sun/star/uno/Reference.hxx> #include <com/sun/star/uno/Reference.hxx>
#include <com/sun/star/util/DateTime.hpp> #include <com/sun/star/util/DateTime.hpp>
#include <comphelper/mediadescriptor.hxx>
#include "oox/helper/binaryinputstream.hxx" #include "oox/helper/binaryinputstream.hxx"
#include "oox/helper/helper.hxx" #include "oox/helper/helper.hxx"
#include "oox/helper/storagebase.hxx" #include "oox/helper/storagebase.hxx"
...@@ -829,8 +828,7 @@ public: ...@@ -829,8 +828,7 @@ public:
const ::rtl::OUString& rFileName, const ::rtl::OUString& rFileName,
const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& rxContext, const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& rxContext,
const StorageRef& rxRootStrg, const StorageRef& rxRootStrg,
const ::rtl::OUString& rSysFileName, const ::rtl::OUString& rSysFileName );
::comphelper::MediaDescriptor& rMediaDesc );
virtual ~SharedConfigData(); virtual ~SharedConfigData();
...@@ -871,7 +869,6 @@ private: ...@@ -871,7 +869,6 @@ private:
::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext > mxContext; ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext > mxContext;
StorageRef mxRootStrg; StorageRef mxRootStrg;
::rtl::OUString maSysFileName; ::rtl::OUString maSysFileName;
::comphelper::MediaDescriptor& mrMediaDesc;
ConfigFileSet maConfigFiles; ConfigFileSet maConfigFiles;
ConfigDataMap maConfigData; ConfigDataMap maConfigData;
NameListMap maNameLists; NameListMap maNameLists;
...@@ -915,8 +912,7 @@ public: ...@@ -915,8 +912,7 @@ public:
const sal_Char* pcEnvVar, const sal_Char* pcEnvVar,
const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& rxContext, const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& rxContext,
const StorageRef& rxRootStrg, const StorageRef& rxRootStrg,
const ::rtl::OUString& rSysFileName, const ::rtl::OUString& rSysFileName );
::comphelper::MediaDescriptor& rMediaDesc );
virtual ~Config(); virtual ~Config();
...@@ -956,8 +952,7 @@ protected: ...@@ -956,8 +952,7 @@ protected:
const sal_Char* pcEnvVar, const sal_Char* pcEnvVar,
const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& rxContext, const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& rxContext,
const StorageRef& rxRootStrg, const StorageRef& rxRootStrg,
const ::rtl::OUString& rSysFileName, const ::rtl::OUString& rSysFileName );
::comphelper::MediaDescriptor& rMediaDesc );
virtual bool implIsValid() const; virtual bool implIsValid() const;
virtual const ::rtl::OUString* implGetOption( const ::rtl::OUString& rKey ) const; virtual const ::rtl::OUString* implGetOption( const ::rtl::OUString& rKey ) const;
......
...@@ -1303,11 +1303,10 @@ NameListRef NameListWrapper::getNameList( const Config& rCfg ) const ...@@ -1303,11 +1303,10 @@ NameListRef NameListWrapper::getNameList( const Config& rCfg ) const
SharedConfigData::SharedConfigData( const OUString& rFileName, SharedConfigData::SharedConfigData( const OUString& rFileName,
const Reference< XComponentContext >& rxContext, const StorageRef& rxRootStrg, const Reference< XComponentContext >& rxContext, const StorageRef& rxRootStrg,
const OUString& rSysFileName, MediaDescriptor& rMediaDesc ) : const OUString& rSysFileName ) :
mxContext( rxContext ), mxContext( rxContext ),
mxRootStrg( rxRootStrg ), mxRootStrg( rxRootStrg ),
maSysFileName( rSysFileName ), maSysFileName( rSysFileName ),
mrMediaDesc( rMediaDesc ),
mbLoaded( false ), mbLoaded( false ),
mbPwCancelled( false ) mbPwCancelled( false )
{ {
...@@ -1454,9 +1453,9 @@ Config::Config( const sal_Char* pcEnvVar, const FilterBase& rFilter ) ...@@ -1454,9 +1453,9 @@ Config::Config( const sal_Char* pcEnvVar, const FilterBase& rFilter )
construct( pcEnvVar, rFilter ); construct( pcEnvVar, rFilter );
} }
Config::Config( const sal_Char* pcEnvVar, const Reference< XComponentContext >& rxContext, const StorageRef& rxRootStrg, const OUString& rSysFileName, MediaDescriptor& rMediaDesc ) Config::Config( const sal_Char* pcEnvVar, const Reference< XComponentContext >& rxContext, const StorageRef& rxRootStrg, const OUString& rSysFileName )
{ {
construct( pcEnvVar, rxContext, rxRootStrg, rSysFileName, rMediaDesc ); construct( pcEnvVar, rxContext, rxRootStrg, rSysFileName );
} }
Config::~Config() Config::~Config()
...@@ -1471,14 +1470,14 @@ void Config::construct( const Config& rParent ) ...@@ -1471,14 +1470,14 @@ void Config::construct( const Config& rParent )
void Config::construct( const sal_Char* pcEnvVar, const FilterBase& rFilter ) void Config::construct( const sal_Char* pcEnvVar, const FilterBase& rFilter )
{ {
if( !rFilter.getFileUrl().isEmpty() ) if( !rFilter.getFileUrl().isEmpty() )
construct( pcEnvVar, rFilter.getComponentContext(), rFilter.getStorage(), rFilter.getFileUrl(), rFilter.getMediaDescriptor() ); construct( pcEnvVar, rFilter.getComponentContext(), rFilter.getStorage(), rFilter.getFileUrl() );
} }
void Config::construct( const sal_Char* pcEnvVar, const Reference< XComponentContext >& rxContext, const StorageRef& rxRootStrg, const OUString& rSysFileName, MediaDescriptor& rMediaDesc ) void Config::construct( const sal_Char* pcEnvVar, const Reference< XComponentContext >& rxContext, const StorageRef& rxRootStrg, const OUString& rSysFileName )
{ {
if( pcEnvVar && rxRootStrg.get() && !rSysFileName.isEmpty() ) if( pcEnvVar && rxRootStrg.get() && !rSysFileName.isEmpty() )
if( const sal_Char* pcFileName = ::getenv( pcEnvVar ) ) if( const sal_Char* pcFileName = ::getenv( pcEnvVar ) )
mxCfgData.reset( new SharedConfigData( OUString::createFromAscii( pcFileName ), rxContext, rxRootStrg, rSysFileName, rMediaDesc ) ); mxCfgData.reset( new SharedConfigData( OUString::createFromAscii( pcFileName ), rxContext, rxRootStrg, rSysFileName ) );
} }
const OUString& Config::getStringOption( const String& rKey, const OUString& rDefault ) const const OUString& Config::getStringOption( const String& rKey, const OUString& rDefault ) const
......
...@@ -36,7 +36,6 @@ using namespace ::com::sun::star::io; ...@@ -36,7 +36,6 @@ using namespace ::com::sun::star::io;
//using namespace ::com::sun::star::lang; //using namespace ::com::sun::star::lang;
using namespace ::com::sun::star::uno; using namespace ::com::sun::star::uno;
using ::comphelper::MediaDescriptor;
using ::oox::core::FilterBase; using ::oox::core::FilterBase;
using ::rtl::OUString; using ::rtl::OUString;
...@@ -122,8 +121,7 @@ Dumper::Dumper( const Reference< XComponentContext >& rxContext, const Reference ...@@ -122,8 +121,7 @@ Dumper::Dumper( const Reference< XComponentContext >& rxContext, const Reference
if( rxContext.is() && rxInStrm.is() ) if( rxContext.is() && rxInStrm.is() )
{ {
StorageRef xStrg( new ZipStorage( rxContext, rxInStrm ) ); StorageRef xStrg( new ZipStorage( rxContext, rxInStrm ) );
MediaDescriptor aMediaDesc; ConfigRef xCfg( new Config( DUMP_PPTX_CONFIG_ENVVAR, rxContext, xStrg, rSysFileName ) );
ConfigRef xCfg( new Config( DUMP_PPTX_CONFIG_ENVVAR, rxContext, xStrg, rSysFileName, aMediaDesc ) );
DumperBase::construct( xCfg ); DumperBase::construct( xCfg );
} }
} }
......
...@@ -47,13 +47,11 @@ protected: ...@@ -47,13 +47,11 @@ protected:
virtual sal_Bool SAL_CALL hasPropertyByName( const OUString& Name ) throw (RuntimeException); virtual sal_Bool SAL_CALL hasPropertyByName( const OUString& Name ) throw (RuntimeException);
private: private:
bool m_bIsFillColor;
OUString m_aColorPropName; OUString m_aColorPropName;
Property m_aColorProp; Property m_aColorProp;
}; };
lcl_ColorPropertySetInfo::lcl_ColorPropertySetInfo( bool bFillColor ) : lcl_ColorPropertySetInfo::lcl_ColorPropertySetInfo( bool bFillColor ) :
m_bIsFillColor( bFillColor ),
// note: length of FillColor and LineColor is 9 // note: length of FillColor and LineColor is 9
m_aColorPropName( (bFillColor ? "FillColor" : "LineColor"), 9, RTL_TEXTENCODING_ASCII_US ), m_aColorPropName( (bFillColor ? "FillColor" : "LineColor"), 9, RTL_TEXTENCODING_ASCII_US ),
m_aColorProp( m_aColorPropName, -1, m_aColorProp( m_aColorPropName, -1,
......
...@@ -21,19 +21,10 @@ ...@@ -21,19 +21,10 @@
#include <rtl/ustring.hxx> #include <rtl/ustring.hxx>
#include "oox/helper/attributelist.hxx" #include "oox/helper/attributelist.hxx"
using namespace ::oox::core;
using namespace ::com::sun::star::uno;
using namespace ::com::sun::star::xml::sax;
using ::rtl::OUString;
namespace oox { namespace ppt { namespace oox { namespace ppt {
BuildListContext::BuildListContext( FragmentHandler2& rParent, BuildListContext::BuildListContext( FragmentHandler2& rParent )
const Reference< XFastAttributeList >& /*xAttribs*/,
TimeNodePtrList & aTimeNodeList)
: FragmentHandler2( rParent ) : FragmentHandler2( rParent )
, maTimeNodeList( aTimeNodeList )
, mbInBldGraphic( false ) , mbInBldGraphic( false )
, mbBuildAsOne( false ) , mbBuildAsOne( false )
{ {
......
...@@ -22,7 +22,6 @@ ...@@ -22,7 +22,6 @@
#ifndef OOX_PPT_BUILDLISTCONTEXT #ifndef OOX_PPT_BUILDLISTCONTEXT
#define OOX_PPT_BUILDLISTCONTEXT #define OOX_PPT_BUILDLISTCONTEXT
#include "oox/ppt/timenode.hxx"
#include "oox/core/fragmenthandler2.hxx" #include "oox/core/fragmenthandler2.hxx"
namespace oox { namespace ppt { namespace oox { namespace ppt {
...@@ -33,9 +32,7 @@ namespace oox { namespace ppt { ...@@ -33,9 +32,7 @@ namespace oox { namespace ppt {
: public ::oox::core::FragmentHandler2 : public ::oox::core::FragmentHandler2
{ {
public: public:
BuildListContext( ::oox::core::FragmentHandler2& rParent, BuildListContext( ::oox::core::FragmentHandler2& rParent );
const ::com::sun::star::uno::Reference< ::com::sun::star::xml::sax::XFastAttributeList >& xAttribs,
TimeNodePtrList & aTimeNodeList);
~BuildListContext( ); ~BuildListContext( );
...@@ -43,7 +40,6 @@ namespace oox { namespace ppt { ...@@ -43,7 +40,6 @@ namespace oox { namespace ppt {
virtual ::oox::core::ContextHandlerRef onCreateContext( sal_Int32 aElementToken, const AttributeList& rAttribs ); virtual ::oox::core::ContextHandlerRef onCreateContext( sal_Int32 aElementToken, const AttributeList& rAttribs );
private: private:
TimeNodePtrList & maTimeNodeList;
bool mbInBldGraphic; bool mbInBldGraphic;
bool mbBuildAsOne; bool mbBuildAsOne;
}; };
......
...@@ -51,12 +51,12 @@ SlideTimingContext::~SlideTimingContext() throw() ...@@ -51,12 +51,12 @@ SlideTimingContext::~SlideTimingContext() throw()
} }
::oox::core::ContextHandlerRef SlideTimingContext::onCreateContext( sal_Int32 aElementToken, const AttributeList& rAttribs ) ::oox::core::ContextHandlerRef SlideTimingContext::onCreateContext( sal_Int32 aElementToken, const AttributeList& )
{ {
switch( aElementToken ) switch( aElementToken )
{ {
case PPT_TOKEN( bldLst ): case PPT_TOKEN( bldLst ):
return new BuildListContext( *this, rAttribs.getFastAttributeList(), maTimeNodeList ); return new BuildListContext( *this );
case PPT_TOKEN( extLst ): case PPT_TOKEN( extLst ):
return this; return this;
case PPT_TOKEN( tnLst ): case PPT_TOKEN( tnLst ):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment