- 21 May, 2013 28 kayıt (commit)
-
-
Kohei Yoshida yazdı
Change-Id: Ib9b734ac9800b1ecf005dcd3a66beac6fcfc3a4f
-
Kohei Yoshida yazdı
UndoToDocument and its nested methods are tightly coupled with the CopyTo counterparts, and they both are used in the same undo / redo code.... Perhaps we can clean this up to reduce redundancy in the future. Change-Id: Id3aec60914ac3a87acbb53f889896a5850047c59
-
Kohei Yoshida yazdı
This removes O(n^2) slowness from undo and redo of a large array of cells with empty cells in between. Change-Id: Id87c8ac8986c97ddf305f849a4752f0f92cc5899
-
Kohei Yoshida yazdı
To reduce the number of function parameters lower. Change-Id: I816bd7c27070597f5fcdf69bdc7d981279390ca6
-
Kohei Yoshida yazdı
The basic idea is the same as the one I did for CopyFromClip. This prevents the O(n^2) algorithm from being invoked during undo object creation. Change-Id: Iac46663b0c25db2f00ffa3ab95a4dcf2857252ce
-
Kohei Yoshida yazdı
This alone makes AdjustRowHeight() fly on large ranges, by eliminating the silly O(n^2) algorithm. Change-Id: Id2693aab7fb9b2a5575c6390fcaae5a404549962
-
Kohei Yoshida yazdı
This change reduces the duration of this method from somewhere in the ballpark of 10 seconds down to a tiny fraction of a second. Change-Id: I0278dc06a4f134b43cd08bd94693b6dec4893f1f
-
Kohei Yoshida yazdı
Change-Id: Iad2e69e4eb61167ad85581a83b19e3798c5edfd0
-
Kohei Yoshida yazdı
Use block position hint to avoid re-starting search in the cell text attribute array. This change alone cuts additional 10 seconds off of the previously mentioned use case, by reducing the duration of HasEditCells() call from the previous ~10 seconds to a tiny fraction of a second. Change-Id: Id9f951cd235a88928c900619b6b66d7b8a057e0c
-
Kohei Yoshida yazdı
This avoids re-starting the search for insertion position in the cell text attribute array, on every single cell insertion. Instead of always searching from the first position, it starts the seaarch from the previous insertion position. This reduces the paste time from the previous 3.5 seconds to 1.7 seconds on my machine. Here is the scenario. 1) Put =A1 in B1, and leave B2 empty. 2) Select B1:B2 and copy it via Ctrl-C. 3) Select B3:B50000, and paste via Ctrl-V. Because of the empty cells interspersing the destination range, this makes the data array highly partitioned, which makes the position lookup very very expensive without the position hint. Still, I was hoping the duration would become a fraction of a second. 1.7 seconds still seems a bit too slow... Change-Id: I7742ce7e22935b6d0e082e4569d53dbd2072c4e5
-
Kohei Yoshida yazdı
so that I can reference it in another class. Change-Id: I035cc1d37ec219c3e3da2e180a0337fb3c6fba8a
-
Kohei Yoshida yazdı
Change-Id: I9f56c7a54405a805cb843321bdab660a3ab22fd6
-
Kohei Yoshida yazdı
This moderately improves the performance of pasting a large array of formula cells. Change-Id: I27a59ff348cb715df15db80693cb7d193e67ec2e
-
Kohei Yoshida yazdı
Change-Id: I0b4deec7f4b10c7b14e482f61c667972caa30bc8
-
Kohei Yoshida yazdı
Change-Id: Ida0542583242ebcdd7f1448931e31656aa542457
-
Kohei Yoshida yazdı
It's not used yet, and is currently empty. Change-Id: Ia377994ae042fc8b19d269adc2f94830422770f1
-
Kohei Yoshida yazdı
Change-Id: I5e108efa0219620696d15a304d2f4e3566d58d57
-
Kohei Yoshida yazdı
Change-Id: I2fdf38b23e74a9f4cad7abd9a943e8817f39474c
-
Kohei Yoshida yazdı
Change-Id: I90e419d19f4565f7f20603358308fd7aeba83190
-
Kohei Yoshida yazdı
Change-Id: I945c7ae834393c760d8a552043c391ff0eb9ce22
-
Kohei Yoshida yazdı
Change-Id: I615c9f9e95a44149c8079754e9aa6497f34c76cf
-
Kohei Yoshida yazdı
This massively speeds up the deletion of a large group of adjacent formula cells. As an example, on machine, deletion of formula cells over B2:B109101 (109100 cells in total) got reduced from 4.7 seconds to 0.09 seconds). Change-Id: Ib72da42a6644421601111907cf7c899d828c2996
-
Kohei Yoshida yazdı
This is to avoid repeated calls to set_empty() on the broadcaster array which causes array reallocation on each call. Instead, we can store the segments of broadcasters to be deleted later, and call set_empty() on those segments to reduce the number of calls to set_empty(). The meat of this is not implemented yet, as EndListeningContext is still empty. Change-Id: I914bc2881acee3902c4fec0f6c21aaf4d2489df8
-
Kohei Yoshida yazdı
Change-Id: I4c34cd4b352dfafe7f423ab8e85f0d93c0368349
-
Kohei Yoshida yazdı
I screwed this up earlier which made this operation take almost 50 seconds to complete. It should finish in 0.00001 seconds or less. The test checks against 1 second, which should be enough of a buffer for slower machines. Change-Id: I9923033045111c75a0740b6bb30a518fe93e01d2
-
Kohei Yoshida yazdı
Since they are always in sync... Change-Id: Ic07f57e1804d76ae801c7947d18df5fd0d729632
-
Kohei Yoshida yazdı
This block will be used to merge the text width and script type vectors that are currently separate. Text widths and script types should really be stored together. Change-Id: I6783769cc03180b513319e0a98a0773bd20ba941
-
Kohei Yoshida yazdı
This header hosts custom element types and blocks used in multi_type_vector. Change-Id: I32481a2354d8c66eb877b4005a260a79183cd714
-
- 20 May, 2013 12 kayıt (commit)
-
-
Rene Engelhard yazdı
Change-Id: I2c20a52e7f7688359172c9ccbd46b4b012bd4c02
-
Fridrich Štrba yazdı
Change-Id: I65a6018c442769f89f17ee0023510c45373407e7
-
Jian Fang Zhang yazdı
#i121134#, more fix for i119657, fallback to old logic to use current AbstractNum when the desired one can not find Found by: Li Lin Yi Patch by: Jinlong Wu,wujinlong@gmail.com Review by: zhangjf (cherry picked from commit aa53b457)
-
Jian Fang Zhang yazdı
that an AbstracNum references to a style, which references to another Num Found by: xiao ting xiao Patch by: Jinlong Wu,wujinlong@gmail.com Review by: zhangjf (cherry picked from commit d6870e14) Conflicts: writerfilter/source/dmapper/DomainMapper.cxx writerfilter/source/dmapper/NumberingManager.cxx Change-Id: I9be0b82d601bf43da31842edb02c4b0d4e50e84f
-
Andras Timar yazdı
Project: translations 6148d0296697632af52c5e39787cd09c2a748fd7
-
Khaled Hosny yazdı
No idea FontMetric::GetHeight() is supposed to mean, probably the code here meant GetLineHeight(). What it actually need is GetAscent(), so use it instead (which is the same as GetLineHeight() - GetDescent()). Change-Id: Ic467e56ed6b8d245ac7800f94e6d40030833328b Reviewed-on: https://gerrit.libreoffice.org/3980Reviewed-by:
Fridrich Strba <fridrich@documentfoundation.org> Tested-by:
Fridrich Strba <fridrich@documentfoundation.org>
-
Julien Nabet yazdı
69c52bc5 had some typos, and got reverted in 78123a50, this does the same, except those typos corrected, so vertical layout is still OK. Change-Id: I79fe610e4cc8bbbddbd8415198d6309a93f4ba18 Reviewed-on: https://gerrit.libreoffice.org/3929Reviewed-by:
Miklos Vajna <vmiklos@suse.cz> Reviewed-by:
Fridrich Strba <fridrich@documentfoundation.org> Tested-by:
Fridrich Strba <fridrich@documentfoundation.org>
-
Radu Ioan yazdı
- replaced RTL_CONTEXT_ with SAL_INFO - replace OSL_* with SAL_* Change-Id: I0054bc90fdcaa2fef7e1d98a0528277be4fc341e Reviewed-on: https://gerrit.libreoffice.org/3917Reviewed-by:
Fridrich Strba <fridrich@documentfoundation.org> Tested-by:
Fridrich Strba <fridrich@documentfoundation.org>
-
Joren De Cuyper yazdı
Change-Id: I991185f6b4efd305dc55c9ad85f92222f706f8a0 Reviewed-on: https://gerrit.libreoffice.org/3850Reviewed-by:
Fridrich Strba <fridrich@documentfoundation.org> Tested-by:
Fridrich Strba <fridrich@documentfoundation.org>
-
Joren De Cuyper yazdı
Placing "insert" commands under category "insert" is more logical Change-Id: Ib322aaace39cd160cc81ef4945abd2047e6c4c12 Reviewed-on: https://gerrit.libreoffice.org/3854Reviewed-by:
Fridrich Strba <fridrich@documentfoundation.org> Tested-by:
Fridrich Strba <fridrich@documentfoundation.org>
-
Mark Wielaard yazdı
Especially when valgrinding an optimized libreoffice build. Change-Id: I7d2ac5476ba8561ce6afbf8c9472550fbe959c76 Reviewed-on: https://gerrit.libreoffice.org/3966Reviewed-by:
Fridrich Strba <fridrich@documentfoundation.org> Tested-by:
Fridrich Strba <fridrich@documentfoundation.org>
-
Mark Wielaard yazdı
FcGetLangs() will return a new FCStrSet that needs to be freed after usage. Change-Id: Ie7fe0dd160fa59077d6a90878e70d0e034680812 Reviewed-on: https://gerrit.libreoffice.org/3967Reviewed-by:
Fridrich Strba <fridrich@documentfoundation.org> Tested-by:
Fridrich Strba <fridrich@documentfoundation.org>
-