summaryrefslogtreecommitdiff
path: root/svl
AgeCommit message (Collapse)AuthorFilesLines
2013-12-10editeng: fix more 32-bit Time breakageMichael Stahl1-3/+3
SfxDateTimeItem and SvxExtTimeField need to use 64-bit integer to store Time as well. These classes also have binary serialization Load()/Save() methods but they are unlikely to be used in a persistent way, just for the clipboard. The problem is easy to reproduce in Impress: Insert->Field->Time(fixed) (regression from 9830fd36dbdb72c79703b0c61efc027fba793c5a) Change-Id: I5946c5b94dd5a509805b6dc40461bbd910caffc4 (cherry picked from commit 7b9c61c7f20a679c5316a288c2ec2ffbf04b4200)
2013-11-19remove unnecessary use of OUString constructor when assigningNoel Grandin2-4/+4
change code like aStr = OUString("xxxx"); to aStr = "xxxx"; Change-Id: Ib981a5cc735677ec5dba76ef9279a107d22e99d4
2013-11-15Band-aid fix for SvtBroadcaster::BroadcastStephan Bergmann1-1/+2
...after 18d07b2affbbbfb2ff9152d3ad8d1744fe75ab47 "Rework SvtListener and SvtBroadcaster internals" broke the case where SvtBroadcaster::Add/Remove is called from within SvtBroadcaster::Broadcast, and at least CppunitTest_sc_ucalc started to fail. It should be evaluated whether the original SvtBroadcaster logic to handle this was better than taking a copy here. Change-Id: Ie47bccb8c5789e405349a9c8b89f30b1dba3cfba
2013-11-14Optimize SvtBroadcaster for insertion.Kohei Yoshida1-6/+24
This helps file load performance at the slight overhead during editing. That said, the overhead during editing is only in theory (not measured) and shouldn't be that much even in theory. Change-Id: If22ea34acb0cda311575ac3ed8ce1a8eb69ae33a
2013-11-14No need to individually remove listeners when being destroyed.Kohei Yoshida1-2/+6
Otherwise a crash would ensue. Change-Id: I9a52524cc205765d059745e9f7e914b636667cb6
2013-11-14Rework SvtListener and SvtBroadcaster internals.Kohei Yoshida6-355/+92
The old code was simply awkward. Change-Id: I1a58a9af86c100be238d306570b40f70c5100314
2013-11-14make l10n buildable separatelyBjoern Michaelsen1-1/+4
- this renames the 'almost' module target to non-l10n - and adds a l10n target which is intended to only build l10n parts of the product - packagers should then be able to build l10n and non-l10n parts of the product independently, thus: - enable quicker rebuilds - distribution of load - updates to l10n without a full rebuild - security fixes to binaries without rebuilding all l10n - the new targets are called build-l10n-only and build-non-l10n-only - note this is not intended to move a concept of split packages upstream -- while this exsists in distros, the number of test scenarios for this would explode upstream Change-Id: Ib8ccc9bc52718d9b0ebbfee76ad93dc29c260863 Conflicts: filter/Module_filter.mk
2013-11-14remove unnecessary sal_Unicode casts in various placesNoel Grandin4-72/+70
Change-Id: Ibf04062ca86ed866202d748c3b62a210d30ed6ec
2013-11-11Mutex access to the global theIndexTable.Kohei Yoshida1-20/+44
Change-Id: I31e2cf3a479e385aa0fca4678a3a2c7fa6cc4b5f
2013-11-11convert OUString compareToAscii == 0 to equalsAsciiNoel Grandin4-8/+8
Convert code like aStr.compareToAscii("XXX") == 0 to aStr.equalsAscii("XXX") which is both easier to read and faster. Change-Id: I448abf58f2fa0e7715dba53f8e8825ca0587c83f
2013-11-10clean up #ifdef ICC codeMichael Stahl3-34/+0
According to the dmake documentation, ICC refers to Visual Age C++ for OS/2, which is not a supported compiler (or platform). Change-Id: Ic9e23bc7c44de110a3a312bd007beda3b660927d
2013-11-08Make INetURLObject-from-OUString ctor explicitStephan Bergmann1-12/+12
...and clean up call sites. Change-Id: I7219a33652835e82fdc44e9e801a7e62868e4f38
2013-11-08remove unnecessary use of OUString constructor in SVL moduleNoel Grandin4-17/+16
Change-Id: Ib3c2f2b43895fe45e9efff888bbd81d8be5969e4
2013-11-06Make TransformInput() a non-member function.Kohei Yoshida2-6/+3
Change-Id: Ieb077b6ce661e2885d6010519f137235a048f9df
2013-11-06Add mutex to guard the shared string pool content.Kohei Yoshida1-0/+6
Change-Id: I0eb97d0fbeaefd8a1c86d240ed8bd7f208fb662e
2013-11-06check string boundsCaolán McNamara1-1/+1
as demonstrated by fdo54887-1.ods Change-Id: I2d5568126c58c831446b4c4b050ffed193f7921d
2013-11-04fdo 70998: Termchange: Graphics->ImageSamuel Mehrbrodt1-4/+4
Change-Id: If451dfb24567157c86bca9a4a8564eb8de231a38 Reviewed-on: https://gerrit.libreoffice.org/6524 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2013-11-04remove redundant calls to OUString constructor in if expressionNoel Grandin1-2/+2
Convert code like: if( aStr == OUString("xxxx") ) to this: if( aStr == "xxxx" ) Change-Id: I8d201f048477731eff590fb988259ef0935c080c
2013-11-04remove redundant calls to OUString constructorNoel Grandin1-7/+3
Change code like this: aStr = OUString("xxxx"); into this: aStr = "xxxx"; Change-Id: I31cb92e21658d57bb9e14b65c179536eae8096f6
2013-11-04Convert code that calls OUString::getStr()[] to use the [] operatorNoel Grandin1-1/+1
This also means that this code now gets bounds checked in debug builds. Change-Id: Id777f85eaee6a737bbcb84625e6e110abe0e0f27
2013-10-31Convert indexOf->startsWith and lastIndexOf->endsWithNoel Grandin1-1/+1
This is both an optimisation and a cleanup. This converts code like aStr.indexOf("XX") == 0 to aStr.startsWith("XX") and converts code like aStr.lastIndexOf("XXX") == aStr.getLength() - 3 to aStr.endsWith("XXX") Note that in general aStr.lastIndexOf("X") == aStr.getLength() - 1 converts to aStr.isEmpty() || aStr.endsWith("X") so I used the surrounding context to determine if aStr could be empty when modifying the code. Change-Id: I22cb8ca7c2a4d0288b001f72adb27fd63af87669
2013-10-28fixincludeguards.sh: svlThomas Arnhold14-35/+36
Change-Id: I70ef44216fb7ddb4e12b9d90406b45cc594279f0
2013-10-25stop looking for Jar files in solverMichael Stahl1-0/+1
Change-Id: I4d2a93fa7395354fbf2893df9e254ab39fa365af
2013-10-25fdo#54938: More uses of cppu::supportsServiceMarcos Paulo de Souza1-7/+2
Change-Id: Id6bed78d92eba52283a17ab3ca66e751c225e48d Reviewed-on: https://gerrit.libreoffice.org/6423 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2013-10-23clean up places accessing the NULL at the of an OUStringNoel Grandin1-2/+2
There were only a couple of real bugs fixed, but we're a little bit safer now. This also fixes the assert and the comment in OUString::operator[] about this. Change-Id: Ibe16b5794e0ba7ecd345fa0801586d25b015974c
2013-10-23clean up some include guardsThomas Arnhold1-2/+2
Conflicts: sw/source/ui/inc/content.hxx Change-Id: I58d81881271fc6e3320bf3b5f1321594b28614a6 Reviewed-on: https://gerrit.libreoffice.org/6388 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2013-10-23convert code to use OUString::endsWithNoel Grandin3-7/+7
Convert places that call aStr[aStr.getLength()-1] == 'x' to use the shorter form aStr.endsWith("x") Change-Id: I1b3a19c0e89b8989cdbeed440f95fc76f9a4b6b6
2013-10-22Bin comments that claim to say why some header is includedTor Lillqvist3-3/+3
They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
2013-10-20drop unnecessary tools/string includesCaolán McNamara1-1/+1
Change-Id: I4278999b9b7d184c26036bbe9e3b98420f461e8c
2013-10-18some GetTokens hidden in windows only codeCaolán McNamara1-5/+5
Change-Id: I45da175fe0f72996b26d66c29e0e6a1b4115b896
2013-10-17fdo#54938: Adapt svtools and comphelper module ...Marcos Paulo de Souza1-7/+2
to use cppu::supportsService and other pieces. Change-Id: I16893b3d31a8055acd214ff23d01e63d38fe0826 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2013-10-12Simplify and avoid use of UNXTor Lillqvist1-15/+7
Change-Id: Ia1f718341858de57d0e417d079981d91968f912b
2013-10-11Compare by pointers first.Kohei Yoshida1-0/+3
Change-Id: Iec697f7100c06da37158e564ac64f0248f923638
2013-10-11Fix equality operator for SharedString & write test for it.Kohei Yoshida2-7/+19
Change-Id: Ib592ca2fe359293da6c10aa9e1535a91627cfc43
2013-10-11Formula tokens, formula cells and formula interpreters to use shared strings.Kohei Yoshida1-0/+9
Change-Id: I5da99869fc7f61ce698180fa5daa9be9db9ac132
2013-10-11Import shared formulas from xlsx without crashing.Kohei Yoshida1-0/+5
The key here is to only use the shared formula ID's and ignore the ref range. The ref ranges are not correct half the time. Change-Id: If65f9b1b44ab6239db37977b6dfe3f822a9cf67e
2013-10-11Add isEmpty() to SharedString. This simplies a lot of its call sites.Kohei Yoshida1-0/+5
Change-Id: I0ebc43abe59ac317c053a4f606dbe376d85c03b0
2013-10-08No more getIdentifier*() calls because they are not efficient.Kohei Yoshida2-22/+10
They shall never be used. Change-Id: I019c88b1511a67175d782777cd41e0ec0434f497
2013-10-08Store svl::SharedString in document cell storage instead of OUString.Kohei Yoshida1-0/+10
With this, both ScColumn and ScMatrix store svl::SharedString as their string values, instead of OUString. Change-Id: I3faece94d98f774881fd72b3ed5f6143504cd350
2013-10-08ContentInfo to store svl::SharedString instead of OUString.Kohei Yoshida1-0/+17
Change-Id: I633e3bb633317eeb61cd804faf20e79312f569f7
2013-10-08Re-implement interning in order to return both string arrays.Kohei Yoshida4-55/+134
One is for the cased string and the other one for the non-cased one. Change-Id: I798687f2efecaaea73a09e0b3348f85a9d9e8c07
2013-10-05This DEBUG_UNIT_TEST stuff is unusedTor Lillqvist1-11/+0
Change-Id: I947713bda2c37e22199161a2c59e5d9ed00b37a2
2013-10-05unused function 'operator<<'Tor Lillqvist1-7/+0
Change-Id: I05d3877e2b0cb4bfccd3a2ae6f24abfd6507c46d
2013-10-04Rename StringPool to SharedStringPool because that's what it is.Kohei Yoshida3-14/+14
Change-Id: I2fc3ce4f0c2291d402cb470346d5561373fb51e7
2013-10-04Let's just use sal_uIntPtr straight.Kohei Yoshida2-6/+6
So that the user of this class won't have to include the header just to get the string ID type. Change-Id: I0ccbc18fe02644f69701f57b0b1b9c30fd141d83
2013-10-04A little more test on shared string pool's life cycle management.Kohei Yoshida1-0/+44
Change-Id: Ic676dd875c27ce60a0707903d7f22207764829e0
2013-10-04No need to intern strings here; all OUString's are ref-counted.Kohei Yoshida2-2/+2
Calling intern() simply moves it to a global hash storage. Now the test passes. Change-Id: I0a93420abce1c3adaaa61d469dff5f359dd5ada4
2013-10-04Add purge() method to purge unreferenced strings from the pool.Kohei Yoshida2-1/+75
But this code needs more work. Change-Id: I538eebf5eb1738a2cfeebc22052b3d5db6001b6b
2013-10-04The map actualy needs to store OUString to have it ref-counted automatically.Kohei Yoshida1-3/+2
Change-Id: Iff6fcf3aba73f2d06ac0c885b39e69ac0febc49f
2013-10-04Correct way to get case-insensitive string identifiers.Kohei Yoshida2-5/+15
Change-Id: Ia343165941231fab34c4904b7a2fa10b07fa32bb