ODFstudio/odfstudio
2
0
Fork 0
Code Issues Packages Activity

loplugin:passstuffbyref

Browse Source 
Change-Id: I330e0ab6c9955939dad313f9d472f93e39dbd313
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109924
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
Noel
2021-01-26 08:30:42 +01:00
committed by Noel Grandin
parent a553c85cbd
commit c83166ce71
73 changed files with 91 additions and 89 deletions
Download Patch File Download Diff File Expand all files Collapse all files
avmedia/source/framework/mediaitem.cxx
+1 -1
View File
@@ -268,7 +268,7 @@ void MediaItem::setGraphic(const Graphic& rGraphic)
m_pImpl->m_aGraphic = rGraphic;
}
Graphic MediaItem::getGraphic() const { return m_pImpl->m_aGraphic; }
const Graphic & MediaItem::getGraphic() const { return m_pImpl->m_aGraphic; }
void MediaItem::setState( MediaState eState )
{
basic/source/inc/buffer.hxx
+2 -2
View File
@@ -45,8 +45,8 @@ public:
void operator += (sal_Int32); // save integer
std::vector<sal_uInt8>&& GetBuffer() { return std::move(m_aBuf); } // pass ownership
sal_uInt32 GetSize() const { return m_aBuf.size(); }
ErrCode GetErrCode() const { return m_aErrCode; }
OUString GetErrMessage() const { return m_sErrMsg; }
const ErrCode & GetErrCode() const { return m_aErrCode; }
const OUString & GetErrMessage() const { return m_sErrMsg; }
};
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
chart2/source/controller/inc/CommandDispatchContainer.hxx
+1 -1
View File
@@ -102,7 +102,7 @@ public:
const css::uno::Reference< css::frame::XController > & xChartController,
const css::util::URL & rURL );
css::uno::Reference< css::frame::XDispatch > getChartDispatcher() const { return m_xChartDispatcher; }
const css::uno::Reference< css::frame::XDispatch > & getChartDispatcher() const { return m_xChartDispatcher; }
void setDrawCommandDispatch( DrawCommandDispatch* pDispatch );
DrawCommandDispatch* getDrawCommandDispatch() { return m_pDrawCommandDispatch; }
chart2/source/view/axes/Tickmarks.cxx
+1 -1
View File
@@ -203,7 +203,7 @@ sal_Int32 TickFactory2D::getTickScreenDistance( TickIter& rIter )
return pFirstTickInfo->getScreenDistanceBetweenTicks( *pSecondTickInfo );
}
B2DVector TickFactory2D::getXaxisStartPos() const
const B2DVector & TickFactory2D::getXaxisStartPos() const
{
return m_aAxisStartScreenPosition2D;
}
chart2/source/view/axes/Tickmarks.hxx
+1 -1
View File
@@ -134,7 +134,7 @@ public:
bool isHorizontalAxis() const;
bool isVerticalAxis() const;
::basegfx::B2DVector getXaxisStartPos() const;
const ::basegfx::B2DVector & getXaxisStartPos() const;
private:
::basegfx::B2DVector getTickScreenPosition2D( double fScaledLogicTickValue ) const;
connectivity/source/drivers/firebird/DatabaseMetaData.cxx
+1 -1
View File
@@ -1475,7 +1475,7 @@ uno::Reference< XResultSet > SAL_CALL ODatabaseMetaData::getImportedKeys(
return ODatabaseMetaData::lcl_getKeys(true, table);
}
uno::Reference< XResultSet > ODatabaseMetaData::lcl_getKeys(const bool& bIsImport, std::u16string_view table )
uno::Reference< XResultSet > ODatabaseMetaData::lcl_getKeys(const bool bIsImport, std::u16string_view table )
{
ODatabaseMetaDataResultSet* pResultSet = new
ODatabaseMetaDataResultSet(ODatabaseMetaDataResultSet::eImportedKeys);
connectivity/source/drivers/firebird/DatabaseMetaData.hxx
+1 -1
View File
@@ -42,7 +42,7 @@ namespace connectivity::firebird
{
::rtl::Reference<Connection> m_pConnection;
private:
css::uno::Reference< css::sdbc::XResultSet > lcl_getKeys( const bool& bIsImport, std::u16string_view table );
css::uno::Reference< css::sdbc::XResultSet > lcl_getKeys( bool bIsImport, std::u16string_view table );
public:
explicit ODatabaseMetaData(Connection* _pCon);
cui/source/dialogs/AdditionsDialog.cxx
+1 -1
View File
@@ -289,7 +289,7 @@ void LoadImage(const OUString& rPreviewFile, std::shared_ptr<AdditionsItem> pCur
} // End of the anonymous namespace
SearchAndParseThread::SearchAndParseThread(AdditionsDialog* pDialog, const bool& isFirstLoading)
SearchAndParseThread::SearchAndParseThread(AdditionsDialog* pDialog, const bool isFirstLoading)
: Thread("cuiAdditionsSearchThread")
, m_pAdditionsDialog(pDialog)
, m_bExecute(true)
cui/source/inc/AdditionsDialog.hxx
+1 -1
View File
@@ -152,7 +152,7 @@ private:
virtual void execute() override;
public:
SearchAndParseThread(AdditionsDialog* pDialog, const bool& bIsFirstLoading);
SearchAndParseThread(AdditionsDialog* pDialog, bool bIsFirstLoading);
void StopExecution() { m_bExecute = false; }
};
cui/source/inc/cuihyperdlg.hxx
+1 -1
View File
@@ -116,7 +116,7 @@ public:
void AddTabPage(const OString &rId, CreatePage pCreateFunc /* != NULL */);
void SetCurPageId( const OString& rId ) { msCurrentPageId = rId; SwitchPage(rId ); }
OString GetCurPageId() const { return msCurrentPageId; }
const OString & GetCurPageId() const { return msCurrentPageId; }
void ShowPage( const OString& rId );
/// gives via map converted local slots if applicable
dbaccess/source/ui/control/opendoccontrols.cxx
+1 -1
View File
@@ -186,7 +186,7 @@ namespace dbaui
return sURL;
}
OpenDocumentListBox::StringPair OpenDocumentListBox::impl_getDocumentAtIndex( sal_uInt16 _nListIndex ) const
const OpenDocumentListBox::StringPair & OpenDocumentListBox::impl_getDocumentAtIndex( sal_uInt16 _nListIndex ) const
{
return m_aURLs[_nListIndex];
}
dbaccess/source/ui/inc/opendoccontrols.hxx
+1 -1
View File
@@ -68,7 +68,7 @@ namespace dbaui
void connect_changed(const Link<weld::ComboBox&, void>& rLink) { m_xControl->connect_changed(rLink); }
private:
StringPair impl_getDocumentAtIndex( sal_uInt16 _nListIndex ) const;
const StringPair & impl_getDocumentAtIndex( sal_uInt16 _nListIndex ) const;
void impl_init( const char* _pAsciiModuleName );
};
editeng/source/editeng/editview.cxx
+1 -1
View File
@@ -1655,7 +1655,7 @@ void EditView::SetLOKSpecialOutputArea(const tools::Rectangle& rOutputArea)
pImpEditView->SetLOKSpecialOutputArea(rOutputArea);
}
tools::Rectangle EditView::GetLOKSpecialOutputArea() const
const tools::Rectangle & EditView::GetLOKSpecialOutputArea() const
{
return pImpEditView->GetLOKSpecialOutputArea();
}
editeng/source/editeng/impedit.cxx
+1 -1
View File
@@ -2686,7 +2686,7 @@ void ImpEditView::SetLOKSpecialOutputArea(const tools::Rectangle& rOutputArea)
mpLOKSpecialPositioning->SetOutputArea(rOutputArea);
}
tools::Rectangle ImpEditView::GetLOKSpecialOutputArea() const
const tools::Rectangle & ImpEditView::GetLOKSpecialOutputArea() const
{
assert(mpLOKSpecialPositioning);
return mpLOKSpecialPositioning->GetOutputArea();
editeng/source/editeng/impedit.hxx
+1 -1
View File
@@ -460,7 +460,7 @@ public:
void InitLOKSpecialPositioning(MapUnit eUnit, const tools::Rectangle& rOutputArea,
const Point& rVisDocStartPos);
void SetLOKSpecialOutputArea(const tools::Rectangle& rOutputArea);
tools::Rectangle GetLOKSpecialOutputArea() const;
const tools::Rectangle & GetLOKSpecialOutputArea() const;
void SetLOKSpecialVisArea(const tools::Rectangle& rVisArea);
tools::Rectangle GetLOKSpecialVisArea() const;
bool HasLOKSpecialPositioning() const;
framework/source/services/desktop.cxx
+1 -1
View File
@@ -1762,7 +1762,7 @@ rtl::Reference<framework::Desktop> createDesktop(
}
rtl::Reference<framework::Desktop> framework::getDesktop(
const rtl::Reference<framework::Desktop> & framework::getDesktop(
css::uno::Reference<css::uno::XComponentContext> const & context)
{
static auto const instance = createDesktop(context);
framework/source/uiconfiguration/ImageList.cxx
+1 -1
View File
@@ -158,7 +158,7 @@ sal_uInt16 ImageList::GetImageId( sal_uInt16 nPos ) const
return maImages[ nPos ]->mnId;
}
OUString ImageList::GetImageName( sal_uInt16 nPos ) const
const OUString & ImageList::GetImageName( sal_uInt16 nPos ) const
{
return maImages[ nPos ]->maName;
}
framework/source/uiconfiguration/ImageList.hxx
+1 -1
View File
@@ -59,7 +59,7 @@ public:
sal_uInt16 GetImageId( sal_uInt16 nPos ) const;
OUString GetImageName( sal_uInt16 nPos ) const;
const OUString & GetImageName( sal_uInt16 nPos ) const;
void GetImageNames( ::std::vector< OUString >& rNames ) const;
private:
i18nlangtag/source/languagetag/languagetag.cxx
+2 -2
View File
@@ -291,7 +291,7 @@ private:
OUString getRegion() const;
OUString const & getVariants() const;
bool hasScript() const;
OUString getGlibcLocaleString() const;
OUString const & getGlibcLocaleString() const;
void setScriptType(LanguageTag::ScriptType st);
LanguageTag::ScriptType getScriptType() const;
@@ -1927,7 +1927,7 @@ OUString LanguageTag::getVariants() const
return aRet;
}
OUString LanguageTagImpl::getGlibcLocaleString() const
OUString const & LanguageTagImpl::getGlibcLocaleString() const
{
if (mbCachedGlibcString)
return maCachedGlibcString;
i18npool/inc/breakiteratorImpl.hxx
+1 -1
View File
@@ -116,7 +116,7 @@ private:
/// @throws css::uno::RuntimeException
bool createLocaleSpecificBreakIterator( const OUString& aLocaleName );
/// @throws css::uno::RuntimeException
css::uno::Reference < XBreakIterator > getLocaleSpecificBreakIterator( const css::lang::Locale& rLocale );
const css::uno::Reference < XBreakIterator > & getLocaleSpecificBreakIterator( const css::lang::Locale& rLocale );
};
i18npool/source/breakiterator/breakiteratorImpl.cxx
+1 -1
View File
@@ -605,7 +605,7 @@ bool BreakIteratorImpl::createLocaleSpecificBreakIterator(const OUString& aLocal
return false;
}
Reference < XBreakIterator >
const Reference < XBreakIterator > &
BreakIteratorImpl::getLocaleSpecificBreakIterator(const Locale& rLocale)
{
if (xBI.is() && rLocale == aLocale)
include/avmedia/mediaitem.hxx
+1 -1
View File
@@ -118,7 +118,7 @@ public:
void setMimeType( const OUString& rMimeType );
OUString getMimeType() const;
void setGraphic(const Graphic& rGraphic);
Graphic getGraphic() const;
const Graphic & getGraphic() const;
const OUString& getTempURL() const;
const OUString& getReferer() const;
include/editeng/editview.hxx
+1 -1
View File
@@ -354,7 +354,7 @@ public:
void InitLOKSpecialPositioning(MapUnit eUnit, const tools::Rectangle& rOutputArea,
const Point& rVisDocStartPos);
void SetLOKSpecialOutputArea(const tools::Rectangle& rOutputArea);
tools::Rectangle GetLOKSpecialOutputArea() const;
const tools::Rectangle & GetLOKSpecialOutputArea() const;
void SetLOKSpecialVisArea(const tools::Rectangle& rVisArea);
tools::Rectangle GetLOKSpecialVisArea() const;
bool HasLOKSpecialPositioning() const;
include/framework/desktop.hxx
+1 -1
View File
@@ -446,7 +446,7 @@ class FWK_DLLPUBLIC Desktop final : private cppu::BaseMutex,
}; // class Desktop
FWK_DLLPUBLIC rtl::Reference<Desktop> getDesktop(
FWK_DLLPUBLIC const rtl::Reference<Desktop> & getDesktop(
css::uno::Reference<css::uno::XComponentContext> const & context);
} // namespace framework
include/oox/dump/dumperbase.hxx
+1 -1
View File
@@ -726,7 +726,7 @@ public:
/*implicit*/ NameListWrapper( const NameListRef& rxList ) : mxList( rxList ) {}
bool isEmpty() const { return !mxList && !maName.has(); }
NameListRef getNameList( const Config& rCfg ) const;
const NameListRef & getNameList( const Config& rCfg ) const;
private:
String maName;
include/sfx2/docfile.hxx
+1 -1
View File
@@ -113,7 +113,7 @@ public:
SfxItemSet * GetItemSet() const;
void SetArgs(const css::uno::Sequence<css::beans::PropertyValue>& rArgs);
css::uno::Sequence<css::beans::PropertyValue> GetArgs() const;
const css::uno::Sequence<css::beans::PropertyValue> & GetArgs() const;
void Close(bool bInDestruction = false);
void CloseAndRelease();
void ReOpen();
include/sfx2/lokhelper.hxx
+1 -1
View File
@@ -64,7 +64,7 @@ public:
/// Get the document id for a view
static int getDocumentIdOfView(int nViewId);
/// Get the default language that should be used for views
static LanguageTag getDefaultLanguage();
static const LanguageTag & getDefaultLanguage();
/// Set language of the given view.
static void setViewLanguage(int nId, const OUString& rBcp47LanguageTag);
/// Set the default language for views.
include/svx/svdobj.hxx
+1 -1
View File
@@ -334,7 +334,7 @@ public:
sal_Int16 GetRelativeHeightRelation() const;
void SetDiagramData(std::shared_ptr<DiagramDataInterface> pDiagramData);
std::shared_ptr<DiagramDataInterface> GetDiagramData() const;
const std::shared_ptr<DiagramDataInterface> & GetDiagramData() const;
/// @param bNotMyself = true: set only ObjList to dirty, don't mark this object as dirty.
///
include/vcl/SwapFile.hxx
+1 -1
View File
@@ -43,7 +43,7 @@ public:
utl::UCBContentHelper::Kill(maSwapURL.GetMainURL(INetURLObject::DecodeMechanism::NONE));
}
INetURLObject getSwapURL() const { return maSwapURL; }
const INetURLObject& getSwapURL() const { return maSwapURL; }
OUString getSwapURLString() const
{
include/vcl/event.hxx
+1 -1
View File
@@ -127,7 +127,7 @@ public:
maLogicPosition = aLogicPosition;
}
std::optional<Point> getLogicPosition() const
const std::optional<Point> & getLogicPosition() const
{
return maLogicPosition;
}
include/vcl/filter/PDFiumLibrary.hxx
+1 -1
View File
@@ -65,7 +65,7 @@ public:
PDFium();
~PDFium();
OUString getLastError() const { return maLastError; }
const OUString& getLastError() const { return maLastError; }
std::unique_ptr<PDFiumDocument> openDocument(const void* pData, int nSize);
std::unique_ptr<PDFiumBitmap> createBitmap(int nWidth, int nHeight, int nAlpha);
include/vcl/graph.hxx
+1 -1
View File
@@ -182,7 +182,7 @@ public:
bool IsDummyContext() const;
void SetGfxLink(const std::shared_ptr<GfxLink>& rGfxLink);
std::shared_ptr<GfxLink> GetSharedGfxLink() const;
const std::shared_ptr<GfxLink> & GetSharedGfxLink() const;
GfxLink GetGfxLink() const;
bool IsGfxLink() const;
include/vcl/notebookbar/notebookbar.hxx
+1 -1
View File
@@ -49,7 +49,7 @@ public:
bool IsWelded() const { return m_bIsWelded; }
VclPtr<vcl::Window>& GetMainContainer() { return m_xVclContentArea; }
OUString GetUIFilePath() const { return m_sUIXMLDescription; }
const OUString & GetUIFilePath() const { return m_sUIXMLDescription; }
void SetDisposeCallback(const Link<const SfxViewShell*, void> rDisposeCallback, const SfxViewShell* pViewShell);
private:
include/vcl/uitest/logger.hxx
+1 -1
View File
@@ -60,7 +60,7 @@ public:
void setAppName(OUString name) { app_name = name; }
OUString getAppName() const { return app_name; }
const OUString& getAppName() const { return app_name; }
};
#endif
linguistic/source/hyphdsp.hxx
+4 -4
View File
@@ -55,9 +55,9 @@ class HyphenatorDispatcher :
HyphenatorDispatcher(const HyphenatorDispatcher &) = delete;
HyphenatorDispatcher & operator = (const HyphenatorDispatcher &) = delete;
inline css::uno::Reference< css::linguistic2::XLinguProperties >
inline const css::uno::Reference< css::linguistic2::XLinguProperties > &
GetPropSet();
inline css::uno::Reference< css::linguistic2::XSearchableDictionaryList >
inline const css::uno::Reference< css::linguistic2::XSearchableDictionaryList > &
GetDicList();
void ClearSvcList();
@@ -108,7 +108,7 @@ public:
};
inline css::uno::Reference< css::linguistic2::XLinguProperties >
inline const css::uno::Reference< css::linguistic2::XLinguProperties > &
HyphenatorDispatcher::GetPropSet()
{
if (!xPropSet.is())
@@ -117,7 +117,7 @@ inline css::uno::Reference< css::linguistic2::XLinguProperties >
}
inline css::uno::Reference< css::linguistic2::XSearchableDictionaryList >
inline const css::uno::Reference< css::linguistic2::XSearchableDictionaryList > &
HyphenatorDispatcher::GetDicList()
{
if (!xDicList.is())
linguistic/source/spelldsp.hxx
+4 -4
View File
@@ -61,9 +61,9 @@ class SpellCheckerDispatcher :
inline linguistic::SpellCache & GetCache() const;
inline css::uno::Reference< css::linguistic2::XLinguProperties >
inline const css::uno::Reference< css::linguistic2::XLinguProperties > &
GetPropSet();
inline css::uno::Reference< css::linguistic2::XSearchableDictionaryList >
inline const css::uno::Reference< css::linguistic2::XSearchableDictionaryList > &
GetDicList();
/// @throws css::uno::RuntimeException
@@ -118,7 +118,7 @@ inline linguistic::SpellCache & SpellCheckerDispatcher::GetCache() const
}
inline css::uno::Reference< css::linguistic2::XLinguProperties >
inline const css::uno::Reference< css::linguistic2::XLinguProperties > &
SpellCheckerDispatcher::GetPropSet()
{
if (!m_xPropSet.is())
@@ -127,7 +127,7 @@ inline css::uno::Reference< css::linguistic2::XLinguProperties >
}
inline css::uno::Reference< css::linguistic2::XSearchableDictionaryList >
inline const css::uno::Reference< css::linguistic2::XSearchableDictionaryList > &
SpellCheckerDispatcher::GetDicList()
{
if (!m_xDicList.is())
linguistic/source/thesdsp.hxx
+2 -2
View File
@@ -48,7 +48,7 @@ class ThesaurusDispatcher :
ThesaurusDispatcher(const ThesaurusDispatcher &) = delete;
ThesaurusDispatcher & operator = (const ThesaurusDispatcher &) = delete;
inline css::uno::Reference< css::linguistic2::XLinguProperties >
inline const css::uno::Reference< css::linguistic2::XLinguProperties > &
GetPropSet();
void ClearSvcList();
@@ -78,7 +78,7 @@ public:
};
inline css::uno::Reference< css::linguistic2::XLinguProperties >
inline const css::uno::Reference< css::linguistic2::XLinguProperties > &
ThesaurusDispatcher::GetPropSet()
{
if (!xPropSet.is())
oox/source/drawingml/diagram/diagram.hxx
+1 -1
View File
@@ -149,7 +149,7 @@ public:
void addTo( const ShapePtr & pShape );
css::uno::Sequence<css::beans::PropertyValue> getDomsAsPropertyValues() const;
ShapePtr getShape() const { return mpShape; }
const ShapePtr & getShape() const { return mpShape; }
private:
ShapePtr mpShape;
oox/source/dump/dumperbase.cxx
+1 -1
View File
@@ -1247,7 +1247,7 @@ void UnitConverter::implIncludeList( const NameListBase& /*rList*/ )
{
}
NameListRef NameListWrapper::getNameList( const Config& rCfg ) const
const NameListRef & NameListWrapper::getNameList( const Config& rCfg ) const
{
if (!mxList)
mxList = rCfg.getNameList( maName );
sdext/source/presenter/PresenterToolBar.cxx
+1 -1
View File
@@ -1897,7 +1897,7 @@ void PresentationTimeLabel::TimeHasChanged (const oslDateTime& rCurrentTime)
TimeValue pauseTime_(0, 0);
setPauseTimeValue(pauseTime_);
}
}
}
else
{
sfx2/source/doc/docfile.cxx
+1 -1
View File
@@ -3294,7 +3294,7 @@ void SfxMedium::SetArgs(const uno::Sequence<beans::PropertyValue>& rArgs)
pImpl->m_aArgs = aArgsMap.getAsConstPropertyValueList();
}
uno::Sequence<beans::PropertyValue> SfxMedium::GetArgs() const { return pImpl->m_aArgs; }
const uno::Sequence<beans::PropertyValue> & SfxMedium::GetArgs() const { return pImpl->m_aArgs; }
SfxMedium::SfxMedium( const uno::Reference < embed::XStorage >& rStor, const OUString& rBaseURL, const std::shared_ptr<SfxItemSet>& p ) :
pImpl(new SfxMedium_Impl)
sfx2/source/view/lokhelper.cxx
+1 -1
View File
@@ -253,7 +253,7 @@ int SfxLokHelper::getDocumentIdOfView(int nViewId)
return -1;
}
LanguageTag SfxLokHelper::getDefaultLanguage()
const LanguageTag & SfxLokHelper::getDefaultLanguage()
{
return g_defaultLanguageTag;
}
starmath/inc/ElementsDockingWindow.hxx
+1 -1
View File
@@ -152,7 +152,7 @@ public:
void SetSelectHdl(const Link<SmElement&,void>& rLink) { maSelectHdlLink = rLink; }
rtl::Reference<AccessibleSmElementsControl> GetAccessible() const { return m_xAccessible; }
const rtl::Reference<AccessibleSmElementsControl> & GetAccessible() const { return m_xAccessible; }
static Color GetTextColor();
static Color GetControlBackground();
svx/source/svdraw/presetooxhandleadjustmentrelations.cxx
+1 -1
View File
@@ -282,7 +282,7 @@ const HandleAdjRel aHandleAdjRelArray[]
{ "wedgeRoundRectCallout_0", "RefX|adj1|RefY|adj2" } };
}
static sal_Int32 lcl_getAdjIndexFromToken(const sal_Int32& nTokenPos, const OUString& rMapValue)
static sal_Int32 lcl_getAdjIndexFromToken(const sal_Int32 nTokenPos, const OUString& rMapValue)
{
OUString sAdjRef = rMapValue.getToken(nTokenPos, '|');
OUString sNumber; // number part from "adj1", "adj2" etc.
svx/source/svdraw/svdobj.cxx
+1 -1
View File
@@ -540,7 +540,7 @@ void SdrObject::SetDiagramData(std::shared_ptr<DiagramDataInterface> pDiagramDat
mpImpl->mpDiagramData = pDiagramData;
}
std::shared_ptr<DiagramDataInterface> SdrObject::GetDiagramData() const
const std::shared_ptr<DiagramDataInterface> & SdrObject::GetDiagramData() const
{
return mpImpl->mpDiagramData;
}
sw/inc/fldbas.hxx
+1 -1
View File
@@ -261,7 +261,7 @@ public:
m_wXFieldMaster = xFieldMaster;
}
static OUString GetTypeStr( SwFieldTypesEnum nTypeId );
static const OUString & GetTypeStr( SwFieldTypesEnum nTypeId );
/// Only in derived classes.
virtual OUString GetName() const;
sw/source/core/fields/fldbas.cxx
+1 -1
View File
@@ -118,7 +118,7 @@ namespace
}
OUString SwFieldType::GetTypeStr(SwFieldTypesEnum nTypeId)
const OUString & SwFieldType::GetTypeStr(SwFieldTypesEnum nTypeId)
{
if (!s_pFieldNames)
GetFieldName_();
sw/source/uibase/fldui/fldmgr.cxx
+1 -1
View File
@@ -524,7 +524,7 @@ SwFieldTypesEnum SwFieldMgr::GetTypeId(sal_uInt16 nPos)
return aSwFields[ nPos ].nTypeId;
}
OUString SwFieldMgr::GetTypeStr(sal_uInt16 nPos)
const OUString & SwFieldMgr::GetTypeStr(sal_uInt16 nPos)
{
OSL_ENSURE(nPos < ::GetPackCount(), "forbidden TypeId");
sw/source/uibase/inc/fldmgr.hxx
+1 -1
View File
@@ -174,7 +174,7 @@ public:
// TypeId for a concrete position in the list
static SwFieldTypesEnum GetTypeId(sal_uInt16 nPos);
// name of the type in the list of fields
static OUString GetTypeStr(sal_uInt16 nPos);
static const OUString & GetTypeStr(sal_uInt16 nPos);
// Pos in the list of fields
static sal_uInt16 GetPos(SwFieldTypesEnum nTypeId);
sw/source/uibase/sidebar/WriterInspectorTextPanel.cxx
+2 -2
View File
@@ -367,7 +367,7 @@ static svx::sidebar::TreeNode LocaleToTreeNode(const OUString& rName, const css:
static svx::sidebar::TreeNode
PropertyToTreeNode(const css::beans::Property& rProperty,
const uno::Reference<beans::XPropertySet>& xPropertiesSet, const bool& rIsGrey)
const uno::Reference<beans::XPropertySet>& xPropertiesSet, const bool rIsGrey)
{
const OUString& rPropName = rProperty.Name;
svx::sidebar::TreeNode aCurNode;
@@ -401,7 +401,7 @@ PropertyToTreeNode(const css::beans::Property& rProperty,
static void InsertValues(const css::uno::Reference<css::uno::XInterface>& rSource,
std::unordered_map<OUString, bool>& rIsDefined,
svx::sidebar::TreeNode& rNode, const bool& isRoot,
svx::sidebar::TreeNode& rNode, const bool isRoot,
const std::vector<OUString>& rHiddenProperty)
{
uno::Reference<beans::XPropertySet> xPropertiesSet(rSource, uno::UNO_QUERY_THROW);
uui/source/iahndl.hxx
+1 -1
View File
@@ -88,7 +88,7 @@ public:
explicit UUIInteractionHelper(
css::uno::Reference< css::uno::XComponentContext > const & rxContext);
css::uno::Reference<css::awt::XWindow> GetParentWindow() const { return m_xWindowParam; }
const css::uno::Reference<css::awt::XWindow> & GetParentWindow() const { return m_xWindowParam; }
void SetParentWindow(const css::uno::Reference<css::awt::XWindow>& rWindow) { m_xWindowParam = rWindow; }
~UUIInteractionHelper();
vcl/inc/image.h
+1 -1
View File
@@ -47,7 +47,7 @@ public:
return maStockName.getLength() > 0;
}
OUString getStock() const
const OUString & getStock() const
{
return maStockName;
}
vcl/inc/impgraph.hxx
+1 -1
View File
@@ -192,7 +192,7 @@ private:
void setDummyContext( bool value ) { mbDummyContext = value; }
bool isDummyContext() const { return mbDummyContext; }
void setGfxLink( const std::shared_ptr<GfxLink>& );
std::shared_ptr<GfxLink> getSharedGfxLink() const;
const std::shared_ptr<GfxLink> & getSharedGfxLink() const;
GfxLink getGfxLink();
bool isGfxLink() const;
vcl/inc/sft.hxx
+1 -1
View File
@@ -741,7 +741,7 @@ public:
sal_uInt32 horzMetricCount() const { return m_nHorzMetrics; }
sal_uInt32 vertMetricCount() const { return m_nVertMetrics; }
sal_uInt32 unitsPerEm() const { return m_nUnitsPerEm; }
FontCharMapRef GetCharMap() const { return m_xCharMap; }
const FontCharMapRef & GetCharMap() const { return m_xCharMap; }
virtual bool hasTable(sal_uInt32 ord) const = 0;
virtual const sal_uInt8* table(sal_uInt32 ord, sal_uInt32& size) const = 0;
vcl/inc/unx/freetype_glyphcache.hxx
+1 -1
View File
@@ -72,7 +72,7 @@ public:
void AnnounceFont( PhysicalFontCollection* );
FontCharMapRef GetFontCharMap() const;
const FontCharMapRef & GetFontCharMap() const;
bool GetFontCapabilities(vcl::FontCapabilities&) const;
private:
vcl/inc/unx/glyphcache.hxx
+1 -1
View File
@@ -122,7 +122,7 @@ public:
void GetFontMetric(ImplFontMetricDataRef const &) const;
const unsigned char* GetTable( const char* pName, sal_uLong* pLength ) const;
FontCharMapRef GetFontCharMap() const;
const FontCharMapRef & GetFontCharMap() const;
bool GetFontCapabilities(vcl::FontCapabilities &) const;
bool GetGlyphBoundRect(sal_GlyphId, tools::Rectangle&, bool) const;
vcl/inc/verticaltabctrl.hxx
+2 -2
View File
@@ -54,11 +54,11 @@ public:
sal_uInt16 GetPageCount() const { return m_xChooser->GetEntryCount(); }
OString GetCurPageId() const { return m_sCurrentPageId; }
const OString& GetCurPageId() const { return m_sCurrentPageId; }
void SetCurPageId(const OString& rId);
sal_uInt16 GetPagePos(std::string_view rPageId) const;
OString GetPageId(sal_uInt16 nIndex) const;
const OString& GetPageId(sal_uInt16 nIndex) const;
VclPtr<vcl::Window> GetPage(std::string_view rPageId);
void RemovePage(std::string_view rPageId);
vcl/source/control/ivctrl.cxx
+1 -1
View File
@@ -547,7 +547,7 @@ void VerticalTabControl::SetCurPageId(const OString& rId)
collectUIInformation(get_id(),OStringToOUString(m_sCurrentPageId,RTL_TEXTENCODING_UTF8), GetPageParent()->get_id());
}
OString VerticalTabControl::GetPageId(sal_uInt16 nIndex) const
const OString & VerticalTabControl::GetPageId(sal_uInt16 nIndex) const
{
return maPageList[nIndex]->sId;
}
vcl/source/gdi/graph.cxx
+1 -1
View File
@@ -500,7 +500,7 @@ void Graphic::SetGfxLink( const std::shared_ptr<GfxLink>& rGfxLink )
mxImpGraphic->setGfxLink(rGfxLink);
}
std::shared_ptr<GfxLink> Graphic::GetSharedGfxLink() const
const std::shared_ptr<GfxLink> & Graphic::GetSharedGfxLink() const
{
return mxImpGraphic->getSharedGfxLink();
}
vcl/source/gdi/impgraph.cxx
+1 -1
View File
@@ -1659,7 +1659,7 @@ void ImpGraphic::setGfxLink(const std::shared_ptr<GfxLink>& rGfxLink)
mpGfxLink = rGfxLink;
}
std::shared_ptr<GfxLink> ImpGraphic::getSharedGfxLink() const
const std::shared_ptr<GfxLink> & ImpGraphic::getSharedGfxLink() const
{
return mpGfxLink;
}
vcl/source/gdi/pdfbuildin_fonts.cxx
+1 -1
View File
@@ -41,7 +41,7 @@ OString BuildinFont::getNameObject() const
return aBuf.makeStringAndClear();
}
FontCharMapRef BuildinFont::GetFontCharMap() const
const FontCharMapRef& BuildinFont::GetFontCharMap() const
{
assert(false && "pdf::BuildinFont doesn't provide correct char maps!");
if (m_xFontCharMap.is())
vcl/source/gdi/pdfbuildin_fonts.hxx
+1 -1
View File
@@ -41,7 +41,7 @@ struct BuildinFont
mutable FontCharMapRef m_xFontCharMap;
OString getNameObject() const;
FontCharMapRef GetFontCharMap() const;
const FontCharMapRef& GetFontCharMap() const;
FontAttributes GetFontAttributes() const;
};
vcl/source/uitest/logger.cxx
+4 -2
View File
@@ -345,7 +345,8 @@ OUString StringMapToOUString(const std::map<OUString, OUString>& rParameters)
return aParameterString.makeStringAndClear();
}
OUString GetValueInMapWithIndex(const std::map<OUString, OUString>& rParameters, sal_Int32 index)
const OUString& GetValueInMapWithIndex(const std::map<OUString, OUString>& rParameters,
sal_Int32 index)
{
sal_Int32 j = 0;
@@ -359,7 +360,8 @@ OUString GetValueInMapWithIndex(const std::map<OUString, OUString>& rParameters,
return itr->second;
}
OUString GetKeyInMapWithIndex(const std::map<OUString, OUString>& rParameters, sal_Int32 index)
const OUString& GetKeyInMapWithIndex(const std::map<OUString, OUString>& rParameters,
sal_Int32 index)
{
sal_Int32 j = 0;
vcl/source/window/NotebookBarAddonsMerger.cxx
+1 -1
View File
@@ -65,7 +65,7 @@ static void GetAddonNotebookBarItem(const css::uno::Sequence<css::beans::Propert
static void CreateNotebookBarToolBox(vcl::Window* pNotebookbarToolBox,
const css::uno::Reference<css::frame::XFrame>& m_xFrame,
const AddonNotebookBarItem& aAddonNotebookBarItem,
const std::vector<Image>& aImageVec, const tools::ULong& nIter)
const std::vector<Image>& aImageVec, const tools::ULong nIter)
{
sal_uInt16 nItemId = 0;
ToolBox* pToolbox = dynamic_cast<ToolBox*>(pNotebookbarToolBox);
vcl/unx/generic/glyphs/freetype_glyphcache.cxx
+2 -2
View File
@@ -673,7 +673,7 @@ bool FreetypeFont::GetAntialiasAdvice() const
// determine unicode ranges in font
FontCharMapRef FreetypeFont::GetFontCharMap() const
const FontCharMapRef & FreetypeFont::GetFontCharMap() const
{
return mxFontInfo->GetFontCharMap();
}
@@ -683,7 +683,7 @@ bool FreetypeFont::GetFontCapabilities(vcl::FontCapabilities &rFontCapabilities)
return mxFontInfo->GetFontCapabilities(rFontCapabilities);
}
FontCharMapRef FreetypeFontInfo::GetFontCharMap() const
const FontCharMapRef & FreetypeFontInfo::GetFontCharMap() const
{
// check if the charmap is already cached
if( mxFontCharMap.is() )
writerfilter/source/dmapper/DomainMapperTableManager.cxx
+1 -1
View File
@@ -413,7 +413,7 @@ TablePositionHandler* DomainMapperTableManager::getCurrentTableRealPosition()
return nullptr;
}
TableParagraphVectorPtr DomainMapperTableManager::getCurrentParagraphs( )
const TableParagraphVectorPtr & DomainMapperTableManager::getCurrentParagraphs( )
{
return m_aParagraphsToEndTable.top( );
}
writerfilter/source/dmapper/DomainMapperTableManager.hxx
+1 -1
View File
@@ -89,7 +89,7 @@ public:
IntVectorPtr const & getCurrentGrid( );
IntVectorPtr const & getCurrentCellWidths( );
TableParagraphVectorPtr getCurrentParagraphs( );
const TableParagraphVectorPtr & getCurrentParagraphs( );
/// Turn the attributes collected so far in m_aTableLook into a property and clear the container.
void finishTableLook();
writerfilter/source/dmapper/NumberingManager.cxx
+1 -1
View File
@@ -414,7 +414,7 @@ ListDef::~ListDef( )
{
}
OUString ListDef::GetStyleName(sal_Int32 const nId,
const OUString & ListDef::GetStyleName(sal_Int32 const nId,
uno::Reference<container::XNameContainer> const& xStyles)
{
if (xStyles.is())
writerfilter/source/dmapper/NumberingManager.hxx
+2 -2
View File
@@ -193,8 +193,8 @@ public:
const AbstractListDef::Pointer& GetAbstractDefinition( ) const { return m_pAbstractDef; };
// Mapping functions
OUString GetStyleName() const { return m_StyleName; };
OUString GetStyleName(sal_Int32 nId, css::uno::Reference<css::container::XNameContainer> const& xStyles);
const OUString & GetStyleName() const { return m_StyleName; };
const OUString & GetStyleName(sal_Int32 nId, css::uno::Reference<css::container::XNameContainer> const& xStyles);
sal_Int16 GetDefaultParentLevels() const { return m_nDefaultParentLevels; };
writerfilter/source/dmapper/SettingsTable.cxx
+2 -2
View File
@@ -693,12 +693,12 @@ sal_Int16 SettingsTable::GetHyphenationZone() const
return m_pImpl->m_nHyphenationZone;
}
OUString SettingsTable::GetDecimalSymbol() const
const OUString & SettingsTable::GetDecimalSymbol() const
{
return m_pImpl->m_sDecimalSymbol;
}
OUString SettingsTable::GetListSeparator() const
const OUString & SettingsTable::GetListSeparator() const
{
return m_pImpl->m_sListSeparator;
}
writerfilter/source/dmapper/SettingsTable.hxx
+2 -2
View File
@@ -79,8 +79,8 @@ public:
bool GetNoHyphenateCaps() const;
sal_Int16 GetHyphenationZone() const;
OUString GetDecimalSymbol() const;
OUString GetListSeparator() const;
const OUString& GetDecimalSymbol() const;
const OUString& GetListSeparator() const;
css::uno::Sequence<css::beans::PropertyValue> const& GetThemeFontLangProperties() const;
xmlsecurity/source/xmlsec/nss/nssinitializer.cxx
+1 -1
View File
@@ -168,7 +168,7 @@ bool lcl_pathExists(const OUString& sPath)
} // namespace
OUString ONSSInitializer::getMozillaCurrentProfile(const css::uno::Reference< css::uno::XComponentContext > &rxContext, bool bSetActive)
const OUString & ONSSInitializer::getMozillaCurrentProfile(const css::uno::Reference< css::uno::XComponentContext > &rxContext, bool bSetActive)
{
if (m_bIsNSSinitialized)
return m_sNSSPath;
xmlsecurity/source/xmlsec/nss/nssinitializer.hxx
+1 -1
View File
@@ -47,7 +47,7 @@ public:
virtual ~ONSSInitializer() override;
static bool initNSS( const css::uno::Reference< css::uno::XComponentContext > &rxContext );
static OUString getMozillaCurrentProfile(const css::uno::Reference< css::uno::XComponentContext > &rxContext, bool bSetActive = false);
static const OUString & getMozillaCurrentProfile(const css::uno::Reference< css::uno::XComponentContext > &rxContext, bool bSetActive = false);
/* XNSSInitializer */
virtual OUString SAL_CALL getNSSPath() override;