diff options
| author | Paul Kmiec <[email protected]> | 2023-05-05 08:58:17 -0700 |
|---|---|---|
| committer | Paul Kmiec <[email protected]> | 2023-05-05 09:29:58 -0700 |
| commit | 67aefd7705df82e43a8670102400a5abab49f6e8 (patch) | |
| tree | 6872755fccedb331f4553c72b642c6c86a700bff /lib/axlsx/drawing/line_series.rb | |
| parent | 1c355c83a9603f835dfe59ef4473df2b8cc3534c (diff) | |
| download | caxlsx-67aefd7705df82e43a8670102400a5abab49f6e8.tar.gz caxlsx-67aefd7705df82e43a8670102400a5abab49f6e8.zip | |
Pipe output directly to str and avoid additional memory allocations
Currently, there are lots of examples of code like this,
```
str << ('<tag ' << foo << ' ' << bar << '/>')
```
which create the string for the tag in memory before piping to str.
We can avoid creating all of these intermediate strings by dropping
the paranthesis and piping directly to str.
This relies on the `str` passed around to handle lots of small
appends. This is a problem when using RubyZip, but that is solved
in the next commit.
Diffstat (limited to 'lib/axlsx/drawing/line_series.rb')
| -rw-r--r-- | lib/axlsx/drawing/line_series.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/axlsx/drawing/line_series.rb b/lib/axlsx/drawing/line_series.rb index 42c86873..bbd3957e 100644 --- a/lib/axlsx/drawing/line_series.rb +++ b/lib/axlsx/drawing/line_series.rb @@ -75,11 +75,11 @@ module Axlsx super(str) do if color str << '<c:spPr><a:solidFill>' - str << (+'<a:srgbClr val="' << color << '"/>') + str << '<a:srgbClr val="' << color << '"/>' str << '</a:solidFill>' str << '<a:ln w="28800">' str << '<a:solidFill>' - str << (+'<a:srgbClr val="' << color << '"/>') + str << '<a:srgbClr val="' << color << '"/>' str << '</a:solidFill>' str << '</a:ln>' str << '<a:round/>' @@ -94,7 +94,7 @@ module Axlsx @labels.to_xml_string(str) unless @labels.nil? @data.to_xml_string(str) unless @data.nil? - str << (+'<c:smooth val="' << ((smooth) ? '1' : '0') << '"/>') + str << '<c:smooth val="' << ((smooth) ? '1' : '0') << '"/>' end end |
