<div dir="ltr"><div dir="auto"><div dir="ltr">Hi Eike,  <div>thanks for your answer..</div><div>just a few question...</div><div dir="auto"><br></div><div dir="auto">-why there is no need to modify that function?</div><div dir="auto">there are many function with "fixme" near them (in core/sc/source/core/tool/scmat<wbr>rix.cxx), are they not to be modified/optimized?</div><div dir="auto"><br></div><div>- what is the main goal of this task (tdf#bug 89387) ?</div><div><br></div><div><br></div><div>thanks,</div><div><br></div><div><span style="font-size:12.8px">Kind regards</span><br></div><div><span style="font-size:12.8px">Filippo Giacchè</span></div><div><br></div><div><br></div></div><br><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">2016-12-13 19:24 GMT+00:00 Eike Rathke <span dir="ltr"><<a href="mailto:erack@redhat.com" target="_blank">erack@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Filippo,<br>
<span><br>
On Monday, 2016-12-12 16:40:50 +0100, Filippo giacchè wrote:<br>
<br>
> I ' m working on Bug 89387 - improve performance for some matrix operations.<br>
> I ' m looking the function ScFullMatrix::MatCopy in<br>
> /core/sc/source/core/tool/scma<wbr>trix.cxx and I wonder how can I try it?, how<br>
> can I test the performance of the function?<br>
<br>
</span>Just for completeness, as mentioned on IRC already, MatCopy() is used in<br>
the DDE spreadsheet function to copy result values,<br>
sc/source/core/tool/interpr2.c<wbr>xx ScInterpreter::ScDde()<br>
<br>
I wouldn't waste too much time to optimize that specific MatCopy<br>
function as that is the only place where it is used.<br>
<span class="gmail-m_-4372663479216212051gmail-m_-5323863943873918880m_-3087327623086224492HOEnZb"><font color="#888888"><br>
  Eike<br>
<br>
--<br>
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.<br>
GPG key "ID" 0x65632D3A - 2265 D7F3 A7B0 95CC 3918  630B 6A6C D5B7 6563 2D3A<br>
Better use 64-bit 0x6A6CD5B765632D3A here is why: <a href="https://evil32.com/" rel="noreferrer" target="_blank">https://evil32.com/</a><br>
Care about Free Software, support the FSFE <a href="https://fsfe.org/support/?erack" rel="noreferrer" target="_blank">https://fsfe.org/support/?erac<wbr>k</a><br>
</font></span></blockquote></div><br></div>
</div>
</div>