diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-10-01 18:25:40 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-10-02 08:47:24 +0200 |
| commit | 366eaa1a383a0f5f3f755f2b6a739d3d9c5b8343 (patch) | |
| tree | 0b7b4876a3e35d40a75818593214df7717e5f3b5 /writerfilter | |
| parent | 91c836acea76cffb0e2d5df0d15fc94d84bfdc5a (diff) | |
loplugin:stringadd in writerfilter..xmloff
Change-Id: Ib5292f4c702cc1e2994c736250a93e6fb18d1a20
Reviewed-on: https://gerrit.libreoffice.org/79988
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'writerfilter')
| -rw-r--r-- | writerfilter/source/dmapper/DomainMapper_Impl.hxx | 16 | ||||
| -rw-r--r-- | writerfilter/source/dmapper/NumberingManager.cxx | 3 | ||||
| -rw-r--r-- | writerfilter/source/ooxml/Handler.cxx | 7 |
3 files changed, 10 insertions, 16 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.hxx b/writerfilter/source/dmapper/DomainMapper_Impl.hxx index 55049dab7681..2df4e5ff505a 100644 --- a/writerfilter/source/dmapper/DomainMapper_Impl.hxx +++ b/writerfilter/source/dmapper/DomainMapper_Impl.hxx @@ -298,17 +298,17 @@ struct PermInsertPosition if (m_Ed.isEmpty()) { - bookmarkName += "permission-for-group:"; - bookmarkName += OUString::number(m_Id); - bookmarkName += ":"; - bookmarkName += m_EdGrp; + bookmarkName += "permission-for-group:" + + OUString::number(m_Id) + + ":" + + m_EdGrp; } else { - bookmarkName += "permission-for-user:"; - bookmarkName += OUString::number(m_Id); - bookmarkName += ":"; - bookmarkName += m_Ed; + bookmarkName += "permission-for-user:" + + OUString::number(m_Id) + + ":" + + m_Ed; } //todo: make sure the name is not used already! diff --git a/writerfilter/source/dmapper/NumberingManager.cxx b/writerfilter/source/dmapper/NumberingManager.cxx index 5f6bb6393f09..af5d2bde6f78 100644 --- a/writerfilter/source/dmapper/NumberingManager.cxx +++ b/writerfilter/source/dmapper/NumberingManager.cxx @@ -452,8 +452,7 @@ OUString ListDef::GetStyleName(sal_Int32 const nId, { if (xStyles.is()) { - OUString sStyleName( "WWNum" ); - sStyleName += OUString::number( nId ); + OUString sStyleName = "WWNum" + OUString::number( nId ); while (xStyles.is() && xStyles->hasByName(sStyleName)) // unique { diff --git a/writerfilter/source/ooxml/Handler.cxx b/writerfilter/source/ooxml/Handler.cxx index f7003b44e6f0..459e0adc480a 100644 --- a/writerfilter/source/ooxml/Handler.cxx +++ b/writerfilter/source/ooxml/Handler.cxx @@ -329,12 +329,7 @@ OOXMLHyperlinkHandler::~OOXMLHyperlinkHandler() void OOXMLHyperlinkHandler::writetext() { - OUString sReturn(" HYPERLINK \""); - - sReturn += mURL; - sReturn += "\""; - sReturn += mFieldCode; - + OUString sReturn = " HYPERLINK \"" + mURL + "\"" + mFieldCode; mpFastContext->text(sReturn); } |