diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-05-03 14:09:20 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-05-03 15:32:53 +0200 |
| commit | c5a0b7af847a71fd50f713934b29305f8ce96c6b (patch) | |
| tree | d7c0193bc183250c36e467f830a4327ab94dc24e /filter | |
| parent | vcl: remove MetricVector (diff) | |
| download | core-c5a0b7af847a71fd50f713934b29305f8ce96c6b.tar.gz core-c5a0b7af847a71fd50f713934b29305f8ce96c6b.zip | |
loplugin:stringadd improvement for appending numbers
I was wrong, the Concat framework already optimised appending
numbers by stack-allocating small buffers, so include
them in the plugin
Change-Id: I922edbdde273c89abfe21d51c5d25dc01c97db25
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115037
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'filter')
| -rw-r--r-- | filter/source/svg/svgwriter.cxx | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/filter/source/svg/svgwriter.cxx b/filter/source/svg/svgwriter.cxx index 26421884212b..a5fe44536d32 100644 --- a/filter/source/svg/svgwriter.cxx +++ b/filter/source/svg/svgwriter.cxx @@ -1882,10 +1882,10 @@ OUString SVGActionWriter::GetPathString( const tools::PolyPolygon& rPolyPoly, bo if( nSize > 1 ) { aPolyPoint = rPoly[ 0 ]; - aPathData.append("M ") - .append( aPolyPoint.X() ) - .append(aComma) - .append( aPolyPoint.Y() ); + aPathData.append("M " + + OUString::number( aPolyPoint.X() ) + + aComma + + OUString::number( aPolyPoint.Y() )); char nCurrentMode = 0; const bool bClose(!bLine || rPoly[0] == rPoly[nSize - 1]); @@ -1906,9 +1906,9 @@ OUString SVGActionWriter::GetPathString( const tools::PolyPolygon& rPolyPoly, bo aPathData.append(aBlank); aPolyPoint = rPoly[ n++ ]; - aPathData.append( aPolyPoint.X() ) - .append(aComma) - .append( aPolyPoint.Y() ); + aPathData.append( OUString::number(aPolyPoint.X()) + + aComma + + OUString::number( aPolyPoint.Y() ) ); } } else @@ -1920,9 +1920,9 @@ OUString SVGActionWriter::GetPathString( const tools::PolyPolygon& rPolyPoly, bo } aPolyPoint = rPoly[ n++ ]; - aPathData.append( aPolyPoint.X() ) - .append(aComma) - .append( aPolyPoint.Y() ); + aPathData.append( OUString::number(aPolyPoint.X()) + + aComma + + OUString::number(aPolyPoint.Y()) ); } } |