summaryrefslogtreecommitdiffstats
path: root/svl
Commit message (Collapse)AuthorAgeFilesLines
* remove unnecessary use of OUString constructor when assigningNoel Grandin2013-11-192-4/+4
| | | | | | | | | change code like aStr = OUString("xxxx"); to aStr = "xxxx"; Change-Id: Ib981a5cc735677ec5dba76ef9279a107d22e99d4
* Band-aid fix for SvtBroadcaster::BroadcastStephan Bergmann2013-11-151-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
* Optimize SvtBroadcaster for insertion.Kohei Yoshida2013-11-141-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
* No need to individually remove listeners when being destroyed.Kohei Yoshida2013-11-141-2/+6
| | | | | | Otherwise a crash would ensue. Change-Id: I9a52524cc205765d059745e9f7e914b636667cb6
* Rework SvtListener and SvtBroadcaster internals.Kohei Yoshida2013-11-146-355/+92
| | | | | | The old code was simply awkward. Change-Id: I1a58a9af86c100be238d306570b40f70c5100314
* make l10n buildable separatelyBjoern Michaelsen2013-11-141-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
* remove unnecessary sal_Unicode casts in various placesNoel Grandin2013-11-144-72/+70
| | | | Change-Id: Ibf04062ca86ed866202d748c3b62a210d30ed6ec
* Mutex access to the global theIndexTable.Kohei Yoshida2013-11-111-20/+44
| | | | Change-Id: I31e2cf3a479e385aa0fca4678a3a2c7fa6cc4b5f
* convert OUString compareToAscii == 0 to equalsAsciiNoel Grandin2013-11-114-8/+8
| | | | | | | | | | Convert code like aStr.compareToAscii("XXX") == 0 to aStr.equalsAscii("XXX") which is both easier to read and faster. Change-Id: I448abf58f2fa0e7715dba53f8e8825ca0587c83f
* clean up #ifdef ICC codeMichael Stahl2013-11-103-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
* Make INetURLObject-from-OUString ctor explicitStephan Bergmann2013-11-081-12/+12
| | | | | | ...and clean up call sites. Change-Id: I7219a33652835e82fdc44e9e801a7e62868e4f38
* remove unnecessary use of OUString constructor in SVL moduleNoel Grandin2013-11-084-17/+16
| | | | Change-Id: Ib3c2f2b43895fe45e9efff888bbd81d8be5969e4
* Make TransformInput() a non-member function.Kohei Yoshida2013-11-062-6/+3
| | | | Change-Id: Ieb077b6ce661e2885d6010519f137235a048f9df
* Add mutex to guard the shared string pool content.Kohei Yoshida2013-11-061-0/+6
| | | | Change-Id: I0eb97d0fbeaefd8a1c86d240ed8bd7f208fb662e
* check string boundsCaolán McNamara2013-11-061-1/+1
| | | | | | as demonstrated by fdo54887-1.ods Change-Id: I2d5568126c58c831446b4c4b050ffed193f7921d
* fdo 70998: Termchange: Graphics->ImageSamuel Mehrbrodt2013-11-041-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>
* remove redundant calls to OUString constructor in if expressionNoel Grandin2013-11-041-2/+2
| | | | | | | | | Convert code like: if( aStr == OUString("xxxx") ) to this: if( aStr == "xxxx" ) Change-Id: I8d201f048477731eff590fb988259ef0935c080c
* remove redundant calls to OUString constructorNoel Grandin2013-11-041-7/+3
| | | | | | | | | Change code like this: aStr = OUString("xxxx"); into this: aStr = "xxxx"; Change-Id: I31cb92e21658d57bb9e14b65c179536eae8096f6
* Convert code that calls OUString::getStr()[] to use the [] operatorNoel Grandin2013-11-041-1/+1
| | | | | | This also means that this code now gets bounds checked in debug builds. Change-Id: Id777f85eaee6a737bbcb84625e6e110abe0e0f27
* Convert indexOf->startsWith and lastIndexOf->endsWithNoel Grandin2013-10-311-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
* fixincludeguards.sh: svlThomas Arnhold2013-10-2814-35/+36
| | | | Change-Id: I70ef44216fb7ddb4e12b9d90406b45cc594279f0
* stop looking for Jar files in solverMichael Stahl2013-10-251-0/+1
| | | | Change-Id: I4d2a93fa7395354fbf2893df9e254ab39fa365af
* fdo#54938: More uses of cppu::supportsServiceMarcos Paulo de Souza2013-10-251-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>
* clean up places accessing the NULL at the of an OUStringNoel Grandin2013-10-231-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
* clean up some include guardsThomas Arnhold2013-10-231-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>
* convert code to use OUString::endsWithNoel Grandin2013-10-233-7/+7
| | | | | | | | | Convert places that call aStr[aStr.getLength()-1] == 'x' to use the shorter form aStr.endsWith("x") Change-Id: I1b3a19c0e89b8989cdbeed440f95fc76f9a4b6b6
* Bin comments that claim to say why some header is includedTor Lillqvist2013-10-223-3/+3
| | | | | | They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
* drop unnecessary tools/string includesCaolán McNamara2013-10-201-1/+1
| | | | Change-Id: I4278999b9b7d184c26036bbe9e3b98420f461e8c
* some GetTokens hidden in windows only codeCaolán McNamara2013-10-181-5/+5
| | | | Change-Id: I45da175fe0f72996b26d66c29e0e6a1b4115b896
* fdo#54938: Adapt svtools and comphelper module ...Marcos Paulo de Souza2013-10-171-7/+2
| | | | | | | to use cppu::supportsService and other pieces. Change-Id: I16893b3d31a8055acd214ff23d01e63d38fe0826 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
* Simplify and avoid use of UNXTor Lillqvist2013-10-121-15/+7
| | | | Change-Id: Ia1f718341858de57d0e417d079981d91968f912b
* Compare by pointers first.Kohei Yoshida2013-10-111-0/+3
| | | | Change-Id: Iec697f7100c06da37158e564ac64f0248f923638
* Fix equality operator for SharedString & write test for it.Kohei Yoshida2013-10-112-7/+19
| | | | Change-Id: Ib592ca2fe359293da6c10aa9e1535a91627cfc43
* Formula tokens, formula cells and formula interpreters to use shared strings.Kohei Yoshida2013-10-111-0/+9
| | | | Change-Id: I5da99869fc7f61ce698180fa5daa9be9db9ac132
* Import shared formulas from xlsx without crashing.Kohei Yoshida2013-10-111-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
* Add isEmpty() to SharedString. This simplies a lot of its call sites.Kohei Yoshida2013-10-111-0/+5
| | | | Change-Id: I0ebc43abe59ac317c053a4f606dbe376d85c03b0
* No more getIdentifier*() calls because they are not efficient.Kohei Yoshida2013-10-082-22/+10
| | | | | | They shall never be used. Change-Id: I019c88b1511a67175d782777cd41e0ec0434f497
* Store svl::SharedString in document cell storage instead of OUString.Kohei Yoshida2013-10-081-0/+10
| | | | | | | With this, both ScColumn and ScMatrix store svl::SharedString as their string values, instead of OUString. Change-Id: I3faece94d98f774881fd72b3ed5f6143504cd350
* ContentInfo to store svl::SharedString instead of OUString.Kohei Yoshida2013-10-081-0/+17
| | | | Change-Id: I633e3bb633317eeb61cd804faf20e79312f569f7
* Re-implement interning in order to return both string arrays.Kohei Yoshida2013-10-084-55/+134
| | | | | | One is for the cased string and the other one for the non-cased one. Change-Id: I798687f2efecaaea73a09e0b3348f85a9d9e8c07
* This DEBUG_UNIT_TEST stuff is unusedTor Lillqvist2013-10-051-11/+0
| | | | Change-Id: I947713bda2c37e22199161a2c59e5d9ed00b37a2
* unused function 'operator<<'Tor Lillqvist2013-10-051-7/+0
| | | | Change-Id: I05d3877e2b0cb4bfccd3a2ae6f24abfd6507c46d
* Rename StringPool to SharedStringPool because that's what it is.Kohei Yoshida2013-10-043-14/+14
| | | | Change-Id: I2fc3ce4f0c2291d402cb470346d5561373fb51e7
* Let's just use sal_uIntPtr straight.Kohei Yoshida2013-10-042-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
* A little more test on shared string pool's life cycle management.Kohei Yoshida2013-10-041-0/+44
| | | | Change-Id: Ic676dd875c27ce60a0707903d7f22207764829e0
* No need to intern strings here; all OUString's are ref-counted.Kohei Yoshida2013-10-042-2/+2
| | | | | | | Calling intern() simply moves it to a global hash storage. Now the test passes. Change-Id: I0a93420abce1c3adaaa61d469dff5f359dd5ada4
* Add purge() method to purge unreferenced strings from the pool.Kohei Yoshida2013-10-042-1/+75
| | | | | | But this code needs more work. Change-Id: I538eebf5eb1738a2cfeebc22052b3d5db6001b6b
* The map actualy needs to store OUString to have it ref-counted automatically.Kohei Yoshida2013-10-041-3/+2
| | | | Change-Id: Iff6fcf3aba73f2d06ac0c885b39e69ac0febc49f
* Correct way to get case-insensitive string identifiers.Kohei Yoshida2013-10-042-5/+15
| | | | Change-Id: Ia343165941231fab34c4904b7a2fa10b07fa32bb
* Let's not expose the internal pointer. Define different ID type.Kohei Yoshida2013-10-042-23/+26
| | | | | | | To prevent the string ID's from being used to instantiate string objects, which can mess up shared string object's life cycles. Change-Id: Ibcd9a4fa9f591d5c27a9e1b50bc9f83ae230e86a