Date: prev next · Thread: first prev next last
2016 Archives by date, by thread · List index


Hi Eike,
thanks for your answer..
just a few question...

-why there is no need to modify that function?
there are many function with "fixme" near them (in
core/sc/source/core/tool/scmatrix.cxx), are they not to be
modified/optimized?

- what is the main goal of this task (tdf#bug 89387) ?


thanks,

Kind regards
Filippo Giacchè




2016-12-13 19:24 GMT+00:00 Eike Rathke <erack@redhat.com>:

Hi Filippo,

On Monday, 2016-12-12 16:40:50 +0100, Filippo giacchè wrote:

I ' m working on Bug 89387 - improve performance for some matrix
operations.
I ' m looking the function ScFullMatrix::MatCopy in
/core/sc/source/core/tool/scmatrix.cxx and I wonder how can I try it?,
how
can I test the performance of the function?

Just for completeness, as mentioned on IRC already, MatCopy() is used in
the DDE spreadsheet function to copy result values,
sc/source/core/tool/interpr2.cxx ScInterpreter::ScDde()

I wouldn't waste too much time to optimize that specific MatCopy
function as that is the only place where it is used.

  Eike

--
LibreOffice Calc developer. Number formatter stricken i18n
transpositionizer.
GPG key "ID" 0x65632D3A - 2265 D7F3 A7B0 95CC 3918  630B 6A6C D5B7 6563
2D3A
Better use 64-bit 0x6A6CD5B765632D3A here is why: https://evil32.com/
Care about Free Software, support the FSFE https://fsfe.org/support/?erack


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.