summaryrefslogtreecommitdiffstats
path: root/unotools
Commit message (Collapse)AuthorAgeFilesLines
* nitpick, mention function name in assertEike Rathke2016-02-051-1/+1
| | | | Change-Id: Ifd5b544ad1374f13e3efc8c7a51601fadf1e14b6
* interface to new XTextSearch2 with SearchOptions2, tdf#72196Eike Rathke2016-02-051-15/+63
| | | | | | | | | | Places that had utl::TextSearch::UpgradeToSearchOptions2() introduced are worth an inspection if the new SearchAlgorithms2::WILDCARD search should be supported or at least use SearchOptions2 instead of SearchOptions to eliminate the small performance penalty that conversion involves. Change-Id: I565f73af2b551ae9ad0f488e672823dc6c5c1109
* OSL_THIS_FUNC is a rip-off of BOOST_CURRENT_FUNCTION so we can use thatCaolán McNamara2016-01-311-1/+0
| | | | | | | | | | as a direct drop in I guess Change-Id: I3add63f1459f4e659019bd6db54da2f5431958ce Reviewed-on: https://gerrit.libreoffice.org/21941 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* unnecessary use of OUString constructorNoel Grandin2016-01-311-2/+2
| | | | | | | Change-Id: Idd31b0a53c8318af69bbcd32f6798721ec8eb8e1 Reviewed-on: https://gerrit.libreoffice.org/21945 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin: unused return valuesNoel Grandin2016-01-182-27/+10
| | | | | | | Change-Id: I9c61a46c57894bc63a57740206c0bcb4a16553af Reviewed-on: https://gerrit.libreoffice.org/21571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedmethods unused return value in include/unotoolsNoel Grandin2016-01-153-26/+13
| | | | | | | Change-Id: I3b2281dc713bff92eb3d80fddef04a8a39838aa8 Reviewed-on: https://gerrit.libreoffice.org/21484 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* unotools: This list is constantTakeshi Abe2016-01-141-1/+1
| | | | | | | Change-Id: Ib97a824cf482ba48204447f25102d112b90d3064 Reviewed-on: https://gerrit.libreoffice.org/21463 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:unusedmethods unused return value unotools/Noel Grandin2016-01-134-12/+6
| | | | Change-Id: I5fbb0fe5910d4f430fe4885eae5689592c366500
* Fix typosAndrea Gelmini2016-01-101-1/+1
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* new loplugin rangedforcopy - use reference in range based forAndrzej Hunt2016-01-051-2/+2
| | | | | | | | | Inspired by 6e6ae9803796b120e95f6e89575e03c5fd0ed3c2 Change-Id: Ia0f264d3a6bbf076aa5080e3398683e50bc6ef01 Reviewed-on: https://gerrit.libreoffice.org/20190 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* let us try to use the pid as well on windowsMarkus Mohrhard2016-01-051-0/+6
| | | | | | | | | | I'm a bit worried that some of our random test failures are related to trying to reuse the same temp file in different tests. Change-Id: I0ca4ace088a96530e6a62d01f979fefe63822b22 Reviewed-on: https://gerrit.libreoffice.org/21090 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* remove unnecessary ;Markus Mohrhard2016-01-041-1/+1
| | | | Change-Id: I0e16305cac5cd5e4824a4e2bab7e4ac10d2bd469
* OModule::onFirstClient is unusedNoel Grandin2015-12-241-5/+0
| | | | | | | | | | | also move the MutexGuard in revokeClient. No need to hold it while doing an osl_atomic_increment. Save it until we actually need to call onLastClient. Change-Id: Ia87f24b66b0430604a2fe717c95223ff1f5ece6d Reviewed-on: https://gerrit.libreoffice.org/20919 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedfields unotools,unoxmlNoel Grandin2015-12-212-7/+5
| | | | Change-Id: I824193a9f4b0196ce1127c5cbf16b0064dbb3446
* loplugin:unusedmethodsNoel Grandin2015-12-211-15/+0
| | | | Change-Id: Ifafdaf6da0225f244853a0042a6458643b570623
* Silence warning C4334 under --enble-64-bit MSVCStephan Bergmann2015-12-141-1/+1
| | | | | | ("'<<' : result of 32-bit shift implicitly converted to 64 bits") Change-Id: I738fe94ec559e2a35ffc32fb91c22c06094cff9f
* loplugin:unusedfields variousNoel Grandin2015-11-261-1/+0
| | | | Change-Id: I18f94269a1172cf195ee402384f7144610e1e82d
* add const in ranged forAndrzej Hunt2015-11-251-1/+1
| | | | Change-Id: I0f55705e101e39e9e6e5286f21cad8ccb96e7fad
* Seems more natural to pass a homogenous list by initializer_listStephan Bergmann2015-11-201-2/+2
| | | | | | | ...than by template parameter pack (even if that requires using ServiceDecl*, as initializer_list cannot take reference types) Change-Id: Ia986201b52d8daedfe925f132ebc79bc2c0ba378
* attempt to fix tdf#95935 - CRASH on startup in o3tl vcl codeNoel Grandin2015-11-201-15/+14
| | | | | | | | I suspect that the problem is the shift and that converting it "1UL <<" will fix it Change-Id: I4808b833453a544aeb732192758d323f358c3ab0
* loplugin:sallogareasStephan Bergmann2015-11-201-1/+1
| | | | Change-Id: Ib1f06cb5f925535858bc14aab6f59ad7fd2a3a8d
* use comphelper::containerToSequenceNoel Grandin2015-11-191-4/+2
| | | | | | in chart2, we remove a local equivalent of the method Change-Id: I25129a3d1ea1dd724eb9cd38a57be37a78b3d100
* remove unused typedefs and inline use-once typedefsNoel Grandin2015-11-181-3/+1
| | | | | | | | | and improve the script a little Change-Id: I2792ea4dd5df3a50736fbe209225c3f16fb86b84 Reviewed-on: https://gerrit.libreoffice.org/20033 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* com::sun::star->css in unotools/Noel Grandin2015-11-1840-208/+194
| | | | | | | Change-Id: Ie36efe4abe7f5c505f564147eb52e0222c1ce955 Reviewed-on: https://gerrit.libreoffice.org/20019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* use unique_ptr for pImpl in unotools/Noel Grandin2015-11-173-21/+18
| | | | Change-Id: I4128ca0d4ff18f6e0c3c9f8ecad13b69c38c3157
* use initialiser list for Sequence<OUString>Noel Grandin2015-11-161-3/+1
| | | | | | | Change-Id: Ia5e47261d1fc6fac2d046656c05a1c5eedb07e02 Reviewed-on: https://gerrit.libreoffice.org/19978 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fast PCH generator and optimized PCH filesAshod Nakashian2015-11-151-136/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ported update_pch.sh to Python with improved performance and features. The new script is invoked from the same update_pch.sh which calls it for each library in parallel, although it can be invoked directly. The ported script (update_pch) updates all PCH files in ~15 seconds where the old script took ~4500 seconds. In addition, the new script supports 3-tiered headers (system, module, and local) and is very flexible to support other improvement. It has a per-library optimal configuration settings that can be updated using another new scripts (update_pch_autotune.sh) which finds optimal per-PCH settings. PCH files have been generated using the new scripts which builds significantly faster (2-3x, depending on module and configuration) and the intermediate binaries are noticably smaller (by several GBs). The new script stamps each generated PCH file with the command that generated it to make it trivial for users to update them, and also adds the command to invoke another script (update_pch_bisect) that helps find missing headers or conflicting headers that may break the build after updating the PCH. Finally update_pch has built-in unit-tests for makefile parsing and other core functionality. Change-Id: Ib933b50e50374d7e2e7e3e95ba8799b0cc8a27fa Reviewed-on: https://gerrit.libreoffice.org/19965 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* use initialiser for Sequence<OUString>Noel Grandin2015-11-153-6/+3
| | | | | | | | | | | | | replaced using: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<\s*OUString\s*> (\w+)\(\s*1\s*\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I20ad0489da887a9712982531c3b127339bb8b3b9 Reviewed-on: https://gerrit.libreoffice.org/19969 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* use initialiser for Sequence<OUString>Noel Grandin2015-11-152-12/+6
| | | | | | | | | | | | | performed using: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence<OUString> (\w+)\(1\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I4da56c80fa09bfc1e8f868794001e9921431e09f Reviewed-on: https://gerrit.libreoffice.org/19968 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* use initialiser syntax for Sequence<OUString>Noel Grandin2015-11-151-4/+2
| | | | | | | | | | | | | | replaced using the script: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence< OUString > (\w+)\(1\); .*\[0\] = (\S+);/Sequence< OUString > \1 { \2 };/g" Change-Id: I23688a91562051a8eed11fc2a85599545c285c34 Reviewed-on: https://gerrit.libreoffice.org/19967 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Namespace cleanup and disambiguationAshod Nakashian2015-11-122-4/+4
| | | | | | | | Change-Id: Ib6d2f8b4e71436c3a7c26bdfc9847152ebaf0739 Reviewed-on: https://gerrit.libreoffice.org/19900 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-1046-183/+183
| | | | Change-Id: Ib80c434598481ca0d05c800991fea9b30ef329f8
* new loplugin: oncevarNoel Grandin2015-11-091-3/+1
| | | | | | | Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:stringconstant: elide explicit ctor usage (manually due to macros)Stephan Bergmann2015-11-061-3/+3
| | | | Change-Id: Id65077d150fdb9785efeb9a2878d6ae643433b9d
* loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann2015-11-065-26/+26
| | | | Change-Id: I07e4b98ea03f9d5ddcf9c3f0bc0cba676a9155e8
* java: combine nested if statementsNoel Grandin2015-11-051-10/+6
| | | | Change-Id: I0457b81668e9427a3c8d6a4af93438b7fb2bb7ba
* use uno::Reference::set method instead of assignmentNoel Grandin2015-11-058-47/+44
| | | | Change-Id: Id57ccff7ea6cf5c7053b51268b1190f5459bb357
* complete initial compiler plugin for the ServiceDocumenterBjoern Michaelsen2015-11-051-1/+1
| | | | | | | | | | | | - this creates a set of static html-meta-redirect-pages linking the UNO implementation name to its C++ class, while doing a clang compile from scratch - the output is written to $(WORKDIR)/ServiceImplementations - still quite some corner cases missing, e.g: - anonymous namespaces - non-trivial getImplementationName() functions Change-Id: I19a0e8a3901277dc3811eb428cac9d00737f14a9
* yyyyyNoel Grandin2015-11-042-8/+8
| | | | Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
* UNO: no need to use OUString constructor when calling createInstanceNoel Grandin2015-10-303-5/+5
| | | | | | | Change-Id: I37da28539b94706574116d0fff5f008aabfb5526 Reviewed-on: https://gerrit.libreoffice.org/19682 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#48456 : when searching for a number, consider formattingDennis Francis2015-10-231-2/+13
| | | | | | | Change-Id: I4a1e3e269a269bb37e88aa8810a46d0f4ecfc518 Reviewed-on: https://gerrit.libreoffice.org/19420 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
* convert remaing DBG_WARNING to SAL_INFONoel Grandin2015-10-191-1/+1
| | | | | | and drop the macro Change-Id: I452ca373d0c277166e94928c1bce78bf37e39d7c
* calling IsSet() before Call() on Link<> is unnecessaryNoel Grandin2015-10-151-2/+1
| | | | | | | | | | | | the Call() already does a check Found with: git grep -A 1 -w 'IsSet()' | grep -B 1 '.Call(' | grep ':' | cut -d ':' -f 1 Change-Id: Ia7248f5d62640b75f705e539c3d1183e39c0d847
* cppcheck:variableScopeNoel Grandin2015-10-151-2/+1
| | | | | | | Change-Id: I9b671637fbe0f32e695d22b74bfb40a39a9fe884 Reviewed-on: https://gerrit.libreoffice.org/19364 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-1233-121/+121
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-125-10/+10
| | | | Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
* Move SolarMutex down from tools to comphelper/ to make life easier.Michael Meeks2015-10-091-3/+5
| | | | | | | Change-Id: I7dd21f30daa27e5de2848eb16aee9a610dd629d5 Reviewed-on: https://gerrit.libreoffice.org/19271 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
* loplugin:unusedmethodsNoel Grandin2015-10-061-17/+0
| | | | Change-Id: I150baadc442e57ee604563bc52965daa9d2e41af
* tdf#82744: fix WebDAV lock/unlock behaviour - part 3Giuseppe Castagno2015-10-051-6/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes done to the code in sfx2, ucbhelper, ucb, unotools in no particular order - add method helpers to call the ucb lock/unlock - add lock/unlock 'real' management - make DateChange property retrieval working for WebDAV as well - add check for changed content of a WebDAV file, in order to reload it correctly when 'Edit Mode' command is activated from GUI - Unlock WebDAV file while saving only if explicitly enabled Needed in order to avoid the small window of file unlocked state that opens while saving a file. When saving LO actually does as follows: - unlock the prevoius version of the file - prepares operations to save the modified version - lock the new file - save the new version - the lock method is enabled if the DAV resource supports it. In case the lock is not supported, for example example DAV with lock disabled, the lock method is disabled. Exception: when the resource is first created and the lock is not supported: a lock command is sent anyway, because if the resource is not yet present, there is no method to detect the lock/unlock availability in this case. - cppcheck:noExplicitConstructor Change-Id: I0aa876c4e3364d86e5740977b97f3db9a01e4491 Reviewed-on: https://gerrit.libreoffice.org/17189 Reviewed-by: Jan Holesovsky <kendy@collabora.com> Tested-by: Jan Holesovsky <kendy@collabora.com>
* clang-analyzer-deadcode.DeadStoresStephan Bergmann2015-10-051-1/+0
| | | | Change-Id: I6434b65e0607ef7a3551560e3ca1fd17e4f606fb