don't use heap for elements in ScRangeList

no need to store small objects like this out of line.

Also
 - add move constructor and move assignment operator
 - drop Assign method since it now has the same
   signature as push_back

Change-Id: I9a8647d3a11f24166a83d399a358a2bce3b2cb79
Reviewed-on: https://gerrit.libreoffice.org/50899
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/sc/source/ui/drawfunc/fuins2.cxx b/sc/source/ui/drawfunc/fuins2.cxx
index a041c7b8f98e..b494a11952d7 100644
--- a/sc/source/ui/drawfunc/fuins2.cxx
+++ b/sc/source/ui/drawfunc/fuins2.cxx
@@ -462,10 +462,10 @@ FuInsertChart::FuInsertChart(ScTabViewShell* pViewSh, vcl::Window* pWin, ScDrawV
             // get "total" range for positioning
             if ( !aRanges.empty() )
             {
-                aPositionRange = *aRanges[ 0 ];
+                aPositionRange = aRanges[ 0 ];
                 for ( size_t i = 1, nCount = aRanges.size(); i < nCount; ++i )
                 {
-                    aPositionRange.ExtendTo( *aRanges[ i ] );
+                    aPositionRange.ExtendTo( aRanges[ i ] );
                 }
             }