compilerplugins/clang/stringbuffer.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit 6e1cb2e9dd406fb2883460cefaa4660622996005 Author: Noel Grandin <[email protected]> AuthorDate: Tue Sep 17 09:27:41 2019 +0200 Commit: Noel Grandin <[email protected]> CommitDate: Tue Sep 17 10:13:47 2019 +0200 disable stringbuffer plugin as pointed out by mike kaganski: the 'aCellStr + u"\x0001"' expression gives an OUStringConcat which holds the references to the strings and doesn't allocate any memory; the OUStringBuffer has a dedicated overload of append() for the OUStringConcat, which allocates only once for the combined concat length and then copies everything from the concat, instead of doing the append twice for each element... why the strange warning for a more efficient method? Change-Id: I6492e99c335e2375cfe78547a87b3079a86890f0 Reviewed-on: https://gerrit.libreoffice.org/79036 Reviewed-by: Noel Grandin <[email protected]> Tested-by: Noel Grandin <[email protected]> diff --git a/compilerplugins/clang/stringbuffer.cxx b/compilerplugins/clang/stringbuffer.cxx index ad8b2eb792a0..b68a8ba32700 100644 --- a/compilerplugins/clang/stringbuffer.cxx +++ b/compilerplugins/clang/stringbuffer.cxx @@ -66,7 +66,7 @@ bool StringBuffer::VisitCXXMemberCallExpr(CXXMemberCallExpr const* memberCallExp return true; } -loplugin::Plugin::Registration<StringBuffer> stringbuffer("stringbuffer"); +loplugin::Plugin::Registration<StringBuffer> stringbuffer("stringbuffer", false); } // namespace _______________________________________________ Libreoffice-commits mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
