Kaydet (Commit) 70b09d93 authored tarafından Caolán McNamara's avatar Caolán McNamara

cppcheck: noExplicitConstructor

Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
üst cbc5dc14
......@@ -47,7 +47,7 @@ struct HBox
* Construct a HBox object with parameter hch.
* @param hch 16bit character being able to have Korean character.
*/
HBox( hchar hch );
explicit HBox( hchar hch );
virtual ~HBox();
/**
* @returns The Size of HBox object
......@@ -75,7 +75,7 @@ struct SkipData: public HBox
hchar dummy;
char *data_block;
SkipData(hchar);
explicit SkipData(hchar);
virtual ~SkipData();
virtual bool Read(HWPFile &hwpf) override;
};
......@@ -321,7 +321,7 @@ struct FBox: public HBox
FBox *prev, *next;
FBox( hchar hch );
explicit FBox( hchar hch );
virtual ~FBox();
};
......
......@@ -80,7 +80,7 @@ class HStreamIODev : public HIODev
std::unique_ptr<HStream> _stream;
gz_stream *_gzfp;
public:
HStreamIODev(HStream* stream);
explicit HStreamIODev(HStream* stream);
virtual ~HStreamIODev();
/**
* Check whether the stream is available
......
......@@ -31,7 +31,7 @@ struct EmPicture
char type[16];
uchar *data;
EmPicture(size_t size);
explicit EmPicture(size_t size);
~EmPicture(void);
bool Read(HWPFile& hwpf);
......@@ -60,7 +60,7 @@ struct OlePicture
#else
char *pis;
#endif
OlePicture(int tsize);
explicit OlePicture(int tsize);
~OlePicture(void);
bool Read(HWPFile& hwpf);
......
......@@ -70,7 +70,7 @@ struct ColumnInfo{
int start_page;
bool bIsSet;
ColumnDef *coldef;
ColumnInfo(int num){
explicit ColumnInfo(int num){
start_page = num;
bIsSet = false;
coldef = NULL;
......
......@@ -43,7 +43,7 @@ class LinkedList
public:
/// construct list with one element (pItem) or no element (pItem == NULL)
LinkedList( T* pItem = 0 );
explicit LinkedList( T* pItem = 0 );
~LinkedList();
};
......@@ -59,7 +59,7 @@ class LinkedListIterator
public:
/// construct list with single element
LinkedListIterator( LinkedList<T>* pList = 0 );
explicit LinkedListIterator( LinkedList<T>* pList = 0 );
~LinkedListIterator();
// bug-compatible with original LinkedList.h/cxx: Ignore parameter!
......
......@@ -62,7 +62,7 @@ enum IDLIST {
class Node{
public:
Node(int _id) : id(_id)
explicit Node(int _id) : id(_id)
{
value = NULL;
child = NULL;
......
......@@ -1058,7 +1058,7 @@ bool LanguageTagImpl::canonicalize()
struct dumper
{
lt_tag_t** mpp;
dumper( lt_tag_t** pp ) : mpp( *pp ? NULL : pp) {}
explicit dumper( lt_tag_t** pp ) : mpp( *pp ? NULL : pp) {}
~dumper() { if (mpp && *mpp) lt_tag_dump( *mpp); }
};
dumper aDumper( &mpImplLangtag);
......
......@@ -136,7 +136,7 @@ private:
public:
HebrewDate(sal_Int32 m, sal_Int32 d, sal_Int32 y) { month = m; day = d; year = y; }
HebrewDate(sal_Int32 d) { // Computes the Hebrew date from the absolute date.
explicit HebrewDate(sal_Int32 d) { // Computes the Hebrew date from the absolute date.
year = (d + HebrewEpoch) / 366; // Approximation from below.
// Search forward for year from the approximation.
while (d >= HebrewDate(7,1,year + 1))
......@@ -214,7 +214,7 @@ private:
public:
GregorianDate(int m, int d, int y) { month = m; day = d; year = y; }
GregorianDate(int d) { // Computes the Gregorian date from the absolute date.
explicit GregorianDate(int d) { // Computes the Gregorian date from the absolute date.
// Search forward year by year from approximate year
year = d/366;
while (d >= GregorianDate(1,1,year+1))
......
......@@ -71,7 +71,7 @@ class Attr {
Sequence <OUString> value;
public:
Attr (const Reference< XAttributeList > & attr);
explicit Attr (const Reference< XAttributeList > & attr);
const OUString& getValueByName (const sal_Char *str) const;
sal_Int32 getLength() const;
const OUString& getTypeByIndex (sal_Int32 idx) const;
......
......@@ -113,7 +113,7 @@ class TextSearch: public cppu::WeakImplHelper
throw(css::uno::RuntimeException);
public:
TextSearch(
explicit TextSearch(
const css::uno::Reference < css::uno::XComponentContext >& rxContext );
virtual ~TextSearch();
......
......@@ -42,7 +42,7 @@ namespace io_acceptor
public MyPipeConnection
{
public:
PipeConnection( const OUString &sConnectionDescription);
explicit PipeConnection( const OUString &sConnectionDescription);
virtual ~PipeConnection();
virtual sal_Int32 SAL_CALL read( Sequence< sal_Int8 >& aReadBytes, sal_Int32 nBytesToRead )
......
......@@ -69,7 +69,7 @@ namespace io_acceptor {
{
public:
SocketConnection( const OUString & sConnectionDescription );
explicit SocketConnection( const OUString & sConnectionDescription );
virtual ~SocketConnection();
virtual sal_Int32 SAL_CALL read( css::uno::Sequence< sal_Int8 >& aReadBytes,
......@@ -135,7 +135,7 @@ namespace io_acceptor {
struct callError {
const Any & any;
callError(const Any & any);
explicit callError(const Any & any);
void operator () (Reference<XStreamListener> xStreamListener);
};
......
......@@ -49,7 +49,7 @@ namespace io_acceptor
class OAcceptor : public WeakImplHelper< XAcceptor, XServiceInfo >
{
public:
OAcceptor(const Reference< XComponentContext > & xCtx);
explicit OAcceptor(const Reference< XComponentContext > & xCtx);
virtual ~OAcceptor();
public:
// Methods
......
......@@ -52,7 +52,7 @@ namespace stoc_connector
Reference< XMultiComponentFactory > _xSMgr;
Reference< XComponentContext > _xCtx;
public:
OConnector(const Reference< XComponentContext > &xCtx);
explicit OConnector(const Reference< XComponentContext > &xCtx);
virtual ~OConnector();
// Methods
virtual Reference< XConnection > SAL_CALL connect(
......
......@@ -60,7 +60,7 @@ namespace stoc_connector
{
public:
PipeConnection( const OUString &sConnectionDescription );
explicit PipeConnection( const OUString &sConnectionDescription );
virtual ~PipeConnection();
virtual sal_Int32 SAL_CALL read( css::uno::Sequence< sal_Int8 >& aReadBytes,
......@@ -89,7 +89,7 @@ namespace stoc_connector
{
public:
SocketConnection( const OUString & sConnectionDescription );
explicit SocketConnection( const OUString & sConnectionDescription );
virtual ~SocketConnection();
virtual sal_Int32 SAL_CALL read( css::uno::Sequence< sal_Int8 >& aReadBytes,
......
......@@ -56,7 +56,7 @@ namespace stoc_connector {
struct callError {
const Any & any;
callError(const Any & any);
explicit callError(const Any & any);
void operator () (Reference<XStreamListener> xStreamListener);
};
......
......@@ -1178,7 +1178,7 @@ class OObjectInputStream:
XObjectInputStream, XMarkableStream >
{
public:
OObjectInputStream( const Reference < XComponentContext > &r)
explicit OObjectInputStream( const Reference < XComponentContext > &r)
: m_rSMgr( r->getServiceManager() )
, m_rCxt( r )
, m_bValidMarkable(false)
......
......@@ -63,7 +63,7 @@ class ODataStreamTest :
public WeakImplHelper< XSimpleTest >
{
public:
ODataStreamTest( const Reference < XMultiServiceFactory > & rFactory ) :
explicit ODataStreamTest( const Reference < XMultiServiceFactory > & rFactory ) :
m_rFactory( rFactory )
{}
......@@ -374,7 +374,7 @@ public:
m_c( 429 ),
m_s( OUString( "foo" ) )
{}
MyPersistObject( const OUString & sServiceName ) : m_sServiceName( sServiceName )
explicit MyPersistObject( const OUString & sServiceName ) : m_sServiceName( sServiceName )
{}
......@@ -628,7 +628,7 @@ class OObjectStreamTest :
public ODataStreamTest
{
public:
OObjectStreamTest( const Reference < XMultiServiceFactory > &r) : ODataStreamTest(r) {}
explicit OObjectStreamTest( const Reference < XMultiServiceFactory > &r) : ODataStreamTest(r) {}
public:
virtual void SAL_CALL testInvariant(const OUString& TestName,
......
......@@ -45,7 +45,7 @@ using namespace ::com::sun::star::test;
class OMarkableOutputStreamTest : public WeakImplHelper< XSimpleTest >
{
public:
OMarkableOutputStreamTest( const Reference< XMultiServiceFactory > & rFactory );
explicit OMarkableOutputStreamTest( const Reference< XMultiServiceFactory > & rFactory );
~OMarkableOutputStreamTest();
public: // implementation names
......@@ -403,7 +403,7 @@ OUString OMarkableOutputStreamTest_getImplementationName() throw ()
class OMarkableInputStreamTest : public WeakImplHelper< XSimpleTest >
{
public:
OMarkableInputStreamTest( const Reference< XMultiServiceFactory > & rFactory );
explicit OMarkableInputStreamTest( const Reference< XMultiServiceFactory > & rFactory );
~OMarkableInputStreamTest();
public: // implementation names
......
......@@ -94,7 +94,7 @@ private:
class OPipeTest : public WeakImplHelper < XSimpleTest >
{
public:
OPipeTest( const Reference< XMultiServiceFactory > & rFactory );
explicit OPipeTest( const Reference< XMultiServiceFactory > & rFactory );
~OPipeTest();
public: // implementation names
......
......@@ -63,7 +63,7 @@ static void mywait()
class OPumpTest : public WeakImplHelper < XSimpleTest >
{
public:
OPumpTest( const Reference< XMultiServiceFactory > & rFactory );
explicit OPumpTest( const Reference< XMultiServiceFactory > & rFactory );
~OPumpTest();
public: // implementation names
......
......@@ -47,7 +47,7 @@ protected:
virtual void SAL_CALL disposing() override;
public:
inline SingletonFactory( ::rtl::Reference< ::jvmaccess::UnoVirtualMachine > const & vm_access )
explicit SingletonFactory( ::rtl::Reference< ::jvmaccess::UnoVirtualMachine > const & vm_access )
: t_impl( m_mutex ),
m_vm_access( vm_access )
{}
......
......@@ -82,8 +82,8 @@ protected:
PreRelease m_preRelease;
public:
SunVersion(const char * szVer);
SunVersion(const OUString& usVer);
explicit SunVersion(const char * szVer);
explicit SunVersion(const OUString& usVer);
~SunVersion();
/**
......
......@@ -222,7 +222,7 @@ rtl::Bootstrap * getBootstrap()
class FileHandleGuard: private boost::noncopyable
{
public:
inline FileHandleGuard(oslFileHandle & rHandle):
explicit FileHandleGuard(oslFileHandle & rHandle):
m_rHandle(rHandle) {}
inline ~FileHandleGuard();
......@@ -255,7 +255,7 @@ public:
RESULT_ERROR
};
inline FileHandleReader(oslFileHandle & rHandle):
explicit FileHandleReader(oslFileHandle & rHandle):
m_aGuard(rHandle), m_nSize(0), m_nIndex(0), m_bLf(false) {}
Result readLine(OString * pLine);
......@@ -339,7 +339,7 @@ class AsynchReader: public salhelper::Thread
void execute() override;
public:
AsynchReader(oslFileHandle & rHandle);
explicit AsynchReader(oslFileHandle & rHandle);
/** only call this function after this thread has finished.
......
......@@ -65,7 +65,7 @@ bool makeDriveLetterSame(OUString * fileURL);
struct InfoFindSame
{
OUString sJava;
InfoFindSame(const OUString& sJavaHome):sJava(sJavaHome){}
explicit InfoFindSame(const OUString& sJavaHome):sJava(sJavaHome){}
bool operator () (const rtl::Reference<VendorBase> & aVendorInfo)
{
......@@ -76,7 +76,7 @@ struct InfoFindSame
struct SameOrSubDirJREMap
{
OUString s1;
SameOrSubDirJREMap(const OUString& s):s1(s){
explicit SameOrSubDirJREMap(const OUString& s):s1(s){
}
bool operator () (const std::pair<const OUString, rtl::Reference<VendorBase> > & s2)
......
......@@ -49,7 +49,7 @@ class CXPathContextPtr
CXPathContextPtr & operator = (const CXPathContextPtr&) = delete;
public:
CXPathContextPtr();
CXPathContextPtr(xmlXPathContextPtr aContext);
explicit CXPathContextPtr(xmlXPathContextPtr aContext);
CXPathContextPtr & operator = (xmlXPathContextPtr pObj);
~CXPathContextPtr();
xmlXPathContext* operator -> () { return _object;}
......@@ -66,7 +66,7 @@ class CXmlDocPtr
public:
CXmlDocPtr & operator = (const CXmlDocPtr&);
CXmlDocPtr();
CXmlDocPtr(xmlDoc* aDoc);
explicit CXmlDocPtr(xmlDoc* aDoc);
/** Takes ownership of xmlDoc
*/
CXmlDocPtr & operator = (xmlDoc* pObj);
......@@ -84,8 +84,8 @@ class CXmlCharPtr
CXmlCharPtr & operator = (const CXmlCharPtr&) = delete;
public:
CXmlCharPtr();
CXmlCharPtr(xmlChar* aDoc);
CXmlCharPtr(const OUString &);
explicit CXmlCharPtr(xmlChar* aDoc);
explicit CXmlCharPtr(const OUString &);
~CXmlCharPtr();
CXmlCharPtr & operator = (xmlChar* pObj);
operator xmlChar* () const { return _object;}
......
......@@ -67,7 +67,7 @@ public:
int m_nPositionPixel;
TiledBarType m_eType;
TiledRowColumnBar(TiledBarType eType);
explicit TiledRowColumnBar(TiledBarType eType);
static gboolean draw(GtkWidget* pWidget, cairo_t* pCairo, gpointer pData);
gboolean drawImpl(GtkWidget* pWidget, cairo_t* pCairo);
static gboolean docConfigureEvent(GtkWidget* pWidget, GdkEventConfigure* pEvent, gpointer pData);
......
......@@ -242,7 +242,7 @@ struct LOEvent
///@}
/// Constructor to instantiate an object of type `type`.
LOEvent(int type)
explicit LOEvent(int type)
: m_nType(type)
, m_pCommand(nullptr)
, m_pArguments(nullptr)
......
......@@ -41,7 +41,7 @@ class Meaning :
Meaning & operator = (const Meaning &) = delete;
public:
Meaning(const OUString &rTerm);
explicit Meaning(const OUString &rTerm);
virtual ~Meaning();
// XMeaning
......
......@@ -83,7 +83,7 @@ private:
class LwpFribRubyFrame : public LwpFrib
{
public:
LwpFribRubyFrame( LwpPara* pPara) : LwpFrib(pPara){}
explicit LwpFribRubyFrame(LwpPara* pPara) : LwpFrib(pPara){}
virtual ~LwpFribRubyFrame(){}
void Read(LwpObjectStream* pObjStrm, sal_uInt16 len) override;
LwpRubyLayout* GetLayout();
......
......@@ -58,7 +58,7 @@ namespace remotebridges_officeclient {
class OfficeClientMain : public WeakImplHelper1< XMain >
{
public:
OfficeClientMain( const Reference< XMultiServiceFactory > &r ) :
explicit OfficeClientMain( const Reference< XMultiServiceFactory > &r ) :
m_xSMgr( r )
{}
public: // Methods
......
......@@ -65,7 +65,7 @@ struct resource {
class IncreaseThread : public Thread
{
public:
IncreaseThread( struct resource *pData ): pResource( pData ) { }
explicit IncreaseThread( struct resource *pData ): pResource( pData ) { }
virtual ~IncreaseThread( )
{
......@@ -93,7 +93,7 @@ protected:
class DecreaseThread : public Thread
{
public:
DecreaseThread( struct resource *pData ): pResource( pData ) { }
explicit DecreaseThread( struct resource *pData ): pResource( pData ) { }
virtual ~DecreaseThread( )
{
......@@ -130,7 +130,7 @@ class PutThread : public Thread
{
public:
//get the struct pointer to write data to buffer
PutThread( struct chain* pData ): pChain( pData ) { }
explicit PutThread( struct chain* pData ): pChain( pData ) { }
virtual ~PutThread( )
{
......@@ -168,7 +168,7 @@ class HoldThread : public Thread
{
public:
//get the Mutex pointer to operate
HoldThread( Mutex* pMutex ): pMyMutex( pMutex ) { }
explicit HoldThread( Mutex* pMutex ): pMyMutex( pMutex ) { }
virtual ~HoldThread( )
{
......@@ -190,7 +190,7 @@ class WaitThread : public Thread
{
public:
//get the Mutex pointer to operate
WaitThread( Mutex* pMutex ): pMyMutex( pMutex ) { }
explicit WaitThread( Mutex* pMutex ): pMyMutex( pMutex ) { }
virtual ~WaitThread( )
{
......@@ -525,7 +525,7 @@ class GuardThread : public Thread
{
public:
//get the Mutex pointer to operate
GuardThread( Mutex* pMutex ): pMyMutex( pMutex ) { }
explicit GuardThread( Mutex* pMutex ): pMyMutex( pMutex ) { }
virtual ~GuardThread( )
{
......@@ -606,7 +606,7 @@ class ClearGuardThread : public Thread
{
public:
//get the Mutex pointer to operate
ClearGuardThread( Mutex* pMutex ): pMyMutex( pMutex ) {}
explicit ClearGuardThread( Mutex* pMutex ): pMyMutex( pMutex ) {}
virtual ~ClearGuardThread( )
{
......@@ -743,7 +743,7 @@ class ResetGuardThread : public Thread
{
public:
//get the Mutex pointer to operate
ResetGuardThread( Mutex* pMutex ): pMyMutex( pMutex ) {}
explicit ResetGuardThread( Mutex* pMutex ): pMyMutex( pMutex ) {}
virtual ~ResetGuardThread( )
{
......
......@@ -160,7 +160,7 @@ class ThreadSafeValue
T m_nFlag;
Mutex m_aMutex;
public:
ThreadSafeValue(T n = 0): m_nFlag(n) {}
explicit ThreadSafeValue(T n = 0): m_nFlag(n) {}
T getValue()
{
//block if already acquired by another thread.
......@@ -1749,7 +1749,7 @@ public:
// a public char member for test result checking
char m_Char_Test;
// for pass thread-special data to thread
myKeyThread(const char cData)
explicit myKeyThread(const char cData)
: m_Char_Test(0)
{
m_nData = cData;
......
......@@ -997,7 +997,7 @@ class CopyTextAttrToClipHandler
sc::CellTextAttrStoreType::iterator miPos;
public:
CopyTextAttrToClipHandler( sc::CellTextAttrStoreType& rAttrs ) :
explicit CopyTextAttrToClipHandler( sc::CellTextAttrStoreType& rAttrs ) :
mrDestAttrs(rAttrs), miPos(mrDestAttrs.begin()) {}
void operator() ( const sc::CellTextAttrStoreType::value_type& aNode, size_t nOffset, size_t nDataSize )
......@@ -2305,7 +2305,7 @@ class FormulaGroupPicker : public SharedTopFormulaCellPicker
std::vector<sc::FormulaGroupEntry>& mrGroups;
public:
FormulaGroupPicker( std::vector<sc::FormulaGroupEntry>& rGroups ) : mrGroups(rGroups) {}
explicit FormulaGroupPicker( std::vector<sc::FormulaGroupEntry>& rGroups ) : mrGroups(rGroups) {}
virtual ~FormulaGroupPicker() {}
......@@ -2596,7 +2596,7 @@ class UpdateCompileHandler
{
bool mbForceIfNameInUse:1;
public:
UpdateCompileHandler(bool bForceIfNameInUse) :
explicit UpdateCompileHandler(bool bForceIfNameInUse) :
mbForceIfNameInUse(bForceIfNameInUse) {}
void operator() (size_t /*nRow*/, ScFormulaCell* pCell)
......@@ -2609,7 +2609,7 @@ class TabNoSetter
{
SCTAB mnTab;
public:
TabNoSetter(SCTAB nTab) : mnTab(nTab) {}
explicit TabNoSetter(SCTAB nTab) : mnTab(nTab) {}
void operator() (size_t /*nRow*/, ScFormulaCell* pCell)
{
......@@ -2621,7 +2621,7 @@ class UsedRangeNameFinder
{
std::set<sal_uInt16>& mrIndexes;
public:
UsedRangeNameFinder(std::set<sal_uInt16>& rIndexes) : mrIndexes(rIndexes) {}
explicit UsedRangeNameFinder(std::set<sal_uInt16>& rIndexes) : mrIndexes(rIndexes) {}
void operator() (size_t /*nRow*/, const ScFormulaCell* pCell)
{
......@@ -2668,7 +2668,7 @@ class SetDirtyOnRangeHandler
sc::SingleColumnSpanSet maValueRanges;
ScColumn& mrColumn;
public:
SetDirtyOnRangeHandler(ScColumn& rColumn) : mrColumn(rColumn) {}
explicit SetDirtyOnRangeHandler(ScColumn& rColumn) : mrColumn(rColumn) {}
void operator() (size_t /*nRow*/, ScFormulaCell* p)
{
......@@ -2712,7 +2712,7 @@ class SetTableOpDirtyOnRangeHandler
sc::SingleColumnSpanSet maValueRanges;
ScColumn& mrColumn;
public:
SetTableOpDirtyOnRangeHandler(ScColumn& rColumn) : mrColumn(rColumn) {}
explicit SetTableOpDirtyOnRangeHandler(ScColumn& rColumn) : mrColumn(rColumn) {}
void operator() (size_t /*nRow*/, ScFormulaCell* p)
{
......@@ -2800,7 +2800,7 @@ class CompileAllHandler
{
sc::CompileFormulaContext& mrCxt;
public:
CompileAllHandler( sc::CompileFormulaContext& rCxt ) : mrCxt(rCxt) {}
explicit CompileAllHandler( sc::CompileFormulaContext& rCxt ) : mrCxt(rCxt) {}
void operator() (size_t /*nRow*/, ScFormulaCell* pCell)
{
......@@ -2914,7 +2914,7 @@ class FindEditCellsHandler
sc::CellStoreType::iterator miCellPos;
public:
FindEditCellsHandler(ScColumn& rCol) :
explicit FindEditCellsHandler(ScColumn& rCol) :
mrColumn(rCol),
miAttrPos(rCol.GetCellAttrStore().begin()),
miCellPos(rCol.GetCellStore().begin()) {}
......
......@@ -1120,7 +1120,7 @@ class TestTabRefAbsHandler
SCTAB mnTab;
bool mbTestResult;
public:
TestTabRefAbsHandler(SCTAB nTab) : mnTab(nTab), mbTestResult(false) {}
explicit TestTabRefAbsHandler(SCTAB nTab) : mnTab(nTab), mbTestResult(false) {}
void operator() (size_t /*nRow*/, const ScFormulaCell* pCell)
{
......@@ -2906,7 +2906,7 @@ class FindUsedRowsHandler
UsedRowsType& mrUsed;
UsedRowsType::const_iterator miUsed;
public:
FindUsedRowsHandler(UsedRowsType& rUsed) : mrUsed(rUsed), miUsed(rUsed.begin()) {}
explicit FindUsedRowsHandler(UsedRowsType& rUsed) : mrUsed(rUsed), miUsed(rUsed.begin()) {}
void operator() (const sc::CellStoreType::value_type& node, size_t nOffset, size_t nDataSize)
{
......@@ -3024,7 +3024,7 @@ class CompileDBFormulaHandler
sc::CompileFormulaContext& mrCxt;
public:
CompileDBFormulaHandler( sc::CompileFormulaContext& rCxt ) :
explicit CompileDBFormulaHandler( sc::CompileFormulaContext& rCxt ) :
mrCxt(rCxt) {}
void operator() (size_t, ScFormulaCell* p)
......@@ -3037,7 +3037,7 @@ struct CompileColRowNameFormulaHandler
{
sc::CompileFormulaContext& mrCxt;
public:
CompileColRowNameFormulaHandler( sc::CompileFormulaContext& rCxt ) : mrCxt(rCxt) {}
explicit CompileColRowNameFormulaHandler( sc::CompileFormulaContext& rCxt ) : mrCxt(rCxt) {}
void operator() (size_t, ScFormulaCell* p)
{
......@@ -3122,7 +3122,7 @@ class UpdateSubTotalHandler
}
public:
UpdateSubTotalHandler(ScFunctionData& rData) : mrData(rData) {}
explicit UpdateSubTotalHandler(ScFunctionData& rData) : mrData(rData) {}
void operator() (size_t /*nRow*/, double fVal)
{
......
......@@ -294,8 +294,8 @@ class AttachFormulaCellsHandler
sc::StartListeningContext& mrCxt;
public:
AttachFormulaCellsHandler( sc::StartListeningContext& rCxt ) :
mrCxt(rCxt) {}
explicit AttachFormulaCellsHandler(sc::StartListeningContext& rCxt)
: mrCxt(rCxt) {}
void operator() (size_t /*nRow*/, ScFormulaCell* pCell)
{
......@@ -2937,8 +2937,8 @@ class GroupFormulaCells
std::vector<ScAddress>* mpGroupPos;
public:
GroupFormulaCells( std::vector<ScAddress>* pGroupPos ) :
mpGroupPos(pGroupPos) {}
explicit GroupFormulaCells(std::vector<ScAddress>* pGroupPos)
: mpGroupPos(pGroupPos) {}
void operator() (sc::CellStoreType::value_type& node)
{
......
......@@ -378,8 +378,8 @@ namespace sd
class TransitionPane : public ValueSet
{
public:
TransitionPane( vcl::Window *pParent ) :
ValueSet( pParent, WB_TABSTOP | WB_3DLOOK | WB_BORDER |
explicit TransitionPane( vcl::Window *pParent )
: ValueSet( pParent, WB_TABSTOP | WB_3DLOOK | WB_BORDER |
WB_ITEMBORDER | WB_FLATVALUESET | WB_VSCROLL )
{
EnableFullItemMode( false );
......
......@@ -32,7 +32,7 @@ namespace sd
class DisplayModeController : public svt::PopupWindowController
{
public:
DisplayModeController( const css::uno::Reference< css::uno::XComponentContext >& rxContext );
explicit DisplayModeController( const css::uno::Reference< css::uno::XComponentContext >& rxContext );
virtual VclPtr<vcl::Window> createPopupWindow( vcl::Window* pParent ) override;
......
......@@ -436,8 +436,8 @@ namespace {
struct compareIgnoreAsciiCase
{
compareIgnoreAsciiCase(const OUString& rString):
maString(rString)
explicit compareIgnoreAsciiCase(const OUString& rString)
: maString(rString)
{
}
......