sw/source/ui/misc/linenum.cxx | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-)
New commits: commit cc56044ea01538781f16576512b66538f2720394 Author: Muhammet Kara <[email protected]> Date: Thu Jun 16 19:28:24 2016 +0300 More efficient OUString concatanation in LineNumberingDialog According to https://goo.gl/jsVAwy: All the other alternatives, like explicit OUStringBuffer and repeated append() should be now worse in all possible aspects. In fact, this should result in just one OUString allocation, one data copy for anything and at most one length computation, so it should possibly beat even strcpy+strcat, while at the same time looking good. Change-Id: I23bf5b5a0b8d79b2d8995700eefc20ab3fb6f79c Reviewed-on: https://gerrit.libreoffice.org/26386 Tested-by: Jenkins <[email protected]> Reviewed-by: Noel Grandin <[email protected]> diff --git a/sw/source/ui/misc/linenum.cxx b/sw/source/ui/misc/linenum.cxx index 09c8bce..641804d 100644 --- a/sw/source/ui/misc/linenum.cxx +++ b/sw/source/ui/misc/linenum.cxx @@ -81,18 +81,18 @@ SwLineNumberingDlg::SwLineNumberingDlg(SwView *pVw) get(m_pNumberingOnCB, "shownumbering"); get(m_pNumberingOnFooterHeader, "showfooterheadernumbering"); - OUString sIntervalName = m_pDivIntervalFT->GetAccessibleName(); - sIntervalName += "("; - sIntervalName += m_pDivRowsFT->GetAccessibleName(); - sIntervalName += ")"; + OUString sIntervalName = m_pDivIntervalFT->GetAccessibleName() + + "(" + + m_pDivRowsFT->GetAccessibleName() + + ")"; m_pDivIntervalNF->SetAccessibleName(sIntervalName); vcl::Window *pNumIntervalFT = get<vcl::Window>("interval"); vcl::Window *pNumRowsFT = get<vcl::Window>("intervallines"); - sIntervalName = pNumIntervalFT->GetAccessibleName(); - sIntervalName += "("; - sIntervalName += pNumRowsFT->GetAccessibleName(); - sIntervalName += ")"; + sIntervalName = pNumIntervalFT->GetAccessibleName() + + "(" + + pNumRowsFT->GetAccessibleName() + + ")"; m_pNumIntervalNF->SetAccessibleName(sIntervalName); // char styles _______________________________________________ Libreoffice-commits mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
