summaryrefslogtreecommitdiffstats
path: root/package
Commit message (Collapse)AuthorAgeFilesLines
...
* 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 syntax for Sequence<OUString>Noel Grandin2015-11-152-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>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-1026-92/+92
| | | | Change-Id: I1e5338558bc57afe51db57655550e9b7246f7214
* Don't check version field from local ZIP headerJan-Marek Glogowski2015-11-051-14/+8
| | | | | | | | | | | | | | | | | | | | | Our current Maven based Java toolchain produces JARs, which have a different "version needed to extract" in the ZIP local and central directory header. I had a look at 7zip and unzip and they already ignore the version but compare other data LO already ignores - sig. The "standard" document from PKWARE doesn't help. So just compare the file path and calculate the data offset and otherwise ignore all (duplicated) information from the local index and rely on a correct central directory entry. Various programs produce(d) "broken" ZIP files; even LO at some point (see git log). Change-Id: I8d63abb0d49a1087c7654f401b62355c147c3118 Reviewed-on: https://gerrit.libreoffice.org/19779 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
* use uno::Reference::set method instead of assignmentNoel Grandin2015-11-0411-92/+79
| | | | Change-Id: I1c7240fe2e2b5eb825f028ca7502e5ba8793046b
* com::sun::star->css in package,pyunoNoel Grandin2015-10-2344-1111/+1086
| | | | Change-Id: I7b7b0e7fea2d1a2b9f6f5501ad5e0b8c1b4a17b9
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-1227-324/+324
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* clang-analyzer-deadcode.DeadStoresStephan Bergmann2015-10-071-2/+3
| | | | Change-Id: I823aab2038eb08f4061e353b5fafc12b8e60e671
* loplugin:mergeclassesNoel Grandin2015-10-061-6/+3
| | | | | | | in this case, just rename the baseclass so that it matches the other places we do this Change-Id: I21704db7352f72e43636dde9e92c8bef61b7e251
* Fix typosAndrea Gelmini2015-09-302-3/+3
| | | | | | | Change-Id: Id27f5a357c1333dec5c24c4f5e0fdaed7a775774 Reviewed-on: https://gerrit.libreoffice.org/18963 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* Fix typosAndrea Gelmini2015-09-302-2/+2
| | | | | | | Change-Id: I013414cf10e25acd5216643855aaa230258a1b5f Reviewed-on: https://gerrit.libreoffice.org/18962 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* boost->stdCaolán McNamara2015-09-183-6/+4
| | | | | | | | Change-Id: Ie490bf2c6921f393bdeed96b1a8815996b701bf0 Reviewed-on: https://gerrit.libreoffice.org/18670 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* comphelper: fix MSVC --enable-mergedlibs build, OFOPXMLHelperMichael Stahl2015-09-011-0/+1
| | | | | | | | | | | The OFOPXMLHelper class causes duplicate definition link errors due to its WeakImplHelper base class. It turns out that the OFOPXMLHelper class itself is only used by other exported functions in comphelper itself so just hide the implementation detail. Change-Id: I3ac8c561af703193cc2609e2c799b630a0d43127
* package: tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe2015-08-2624-47/+46
| | | | | | | | | with the variadic variants. Change-Id: If88e1d741075e86997c91dc2c59eeebe45f67c1f Reviewed-on: https://gerrit.libreoffice.org/17980 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Consolidate isFileUrl checksStephan Bergmann2015-08-191-1/+2
| | | | Change-Id: I1b74fdfaa09c4d0d6c296253958e83e78b546a9a
* Remove newly unused ucbhelper/fileidentifierconverter.hxxStephan Bergmann2015-08-181-1/+0
| | | | Change-Id: I7c272383ecb115e19699ed96bf5622d979403a01
* Simplify ZipPackage::isLocalFile checkStephan Bergmann2015-08-181-14/+1
| | | | | | (vnd.sun.star.wfs is long gone) Change-Id: I3e25f1728504cbad07bb97187a7bac23b33695dc
* avoid possible leak on exceptionCaolán McNamara2015-08-171-13/+35
| | | | Change-Id: Id3c16e5fedc5e57c8daccafa25bdb2fbbd0131b0
* Related: tdf#88314 delete temp filesCaolán McNamara2015-08-073-0/+10
| | | | | | | | | | | clear to close the inputstream before attempting to delete the temp file, which is plausible as the windows-specific failure in earlier attempts at this Change-Id: I751aa3a098dd960f56b77a5f5fc93783c1060556 Reviewed-on: https://gerrit.libreoffice.org/17531 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Resolves: tdf#88314 close temp file after each thread completedCaolán McNamara2015-08-053-19/+65
| | | | | | | | Change-Id: Ic2eec30cfb5f61c53777eefeeb8bad6f552da2fc Reviewed-on: https://gerrit.libreoffice.org/17355 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Revert "Resolves: tdf#88314 close temp file after each thread completes"Norbert Thiebaud2015-07-263-47/+20
| | | | | | | | | This reverts commit 738cf411e9315d17c7eb8be47ded643a00dfe5c5. It brokes windows in https://gerrit.libreoffice.org/#/c/17289/ as per http://ci.libreoffice.org/job/lo_gerrit_master/4465/Gerrit=Gerrit,Platform=Windows/ was pushed despite that.. :-(
* Resolves: tdf#88314 close temp file after each thread completesCaolán McNamara2015-07-243-20/+47
| | | | | | | | | | | | and reopen them when we need their data. That way we don't have as many open files as substreams in the package, so we don't run out of file handles Change-Id: Ic124e275abf15f4578c77ee271d185f40cb844b1 Reviewed-on: https://gerrit.libreoffice.org/17289 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* loplugin:unusedmethods toolkit,packageNoel Grandin2015-07-141-15/+0
| | | | | | | Change-Id: I83618cd2fd12fb0c1691dc7255fc7192e68649cd Reviewed-on: https://gerrit.libreoffice.org/17031 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedmethods packageNoel Grandin2015-07-078-85/+0
| | | | | | | Change-Id: I19d6bbb9288d72b99d1023b4983b1c3fff7570e8 Reviewed-on: https://gerrit.libreoffice.org/16811 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2015-07-031-1/+1
| | | | | | | | Change-Id: I8e429d1f03aac7c7cdb7ff4b43b3f46d40292510 Reviewed-on: https://gerrit.libreoffice.org/16709 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com> Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
* drop yet another reimplementation of rtl::Reference (SotMutexHolderRef)Noel Grandin2015-06-2915-120/+48
| | | | Change-Id: I57f9dff931242405420bd45fae7ec5f13718707f
* Fix typosAndrea Gelmini2015-06-281-1/+1
| | | | | | | Change-Id: Ifd126a535f18db79a18aaec61b76cdf26ab33dff Reviewed-on: https://gerrit.libreoffice.org/16522 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* loplugin:stringconstant: handle OUString+=OUString(literal)Stephan Bergmann2015-06-261-3/+3
| | | | Change-Id: Ib9bf6e0919ee0cab02d826859317ea20cb64cabb
* remove unnecessary check for null when calling deleteNoel Grandin2015-06-151-10/+7
| | | | | | | | Idea originally from caolan. Found using the following command: find . -name *.cxx | xargs /opt/local/bin/grep -zlP '(?m)if\s*\(\s*\w+\s*\)\s*delete\s+\w+\;' Change-Id: I3338f4e22193a6dfd6219c8c75835224a3392763
* TyposJulien Nabet2015-06-131-1/+1
| | | | Change-Id: I21c352a63d668c174eef212dbfbe6346c678ce4d
* cppcheck: noExplicitConstructorCaolán McNamara2015-06-114-5/+5
| | | | Change-Id: Id6291335945a8f10cef3afd1a2b3bae65fdf4562
* convert expressions like 'size() == 0' to 'empty()'Noel Grandin2015-06-112-7/+7
| | | | Change-Id: Ia5c8c0f38a347f398d587970a22e03f29ffd37af
* convert 'it.begin() == it.end()' to 'it.empty()'Noel Grandin2015-06-111-1/+1
| | | | | | | Change-Id: I244a9eb6bce6b1c649653ec38ebb9a39f8b4c145 Reviewed-on: https://gerrit.libreoffice.org/16212 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2015-06-082-4/+4
| | | | Change-Id: I1a3766bba2c157f20422d3ffb0c08f631b537724
* WaE: unreferenced local variableCaolán McNamara2015-06-051-1/+1
| | | | Change-Id: Ie04966936a406fefbbcd1d96d7d3d299f98a31ae
* uncaught exceptions in parallel deflate threads on re-export of HTB23106 odgsCaolán McNamara2015-06-053-2/+20
| | | | Change-Id: Ib6aef478b8dcf1c7a7262ab60eef4d44da0de632
* loplugin:loopvartoosmallNoel Grandin2015-06-012-2/+2
| | | | Change-Id: I86ff38a90018a2ddfb2db3babf67168b0e6257a5
* Rephrase comparisons between bool and sal_BoolStephan Bergmann2015-05-081-1/+1
| | | | | | ...to cater for forthcoming loplugin:implicitboolconversion improvements Change-Id: I88c0c4681137022005c3a4c418e91cb17bc17148
* loplugin:simplifyboolStephan Bergmann2015-04-242-3/+3
| | | | Change-Id: Ia4177cf1a4b0c06ef614b54298fd6f9224956afc
* Improved loplugin:literaltoboolconversion looking into cond. exprs.Stephan Bergmann2015-04-232-3/+3
| | | | | | ...automatic rewriter fixes Change-Id: I6b04ca80f08f8a71ff94e309fd52f44d736751ee
* remove unnecessary use of void in function declarationsNoel Grandin2015-04-1516-19/+19
| | | | | | | | | | | | | | | | ie. void f(void); becomes void f(); I used the following command to make the changes: git grep -lP '\(\s*void\s*\)' -- *.cxx \ | xargs perl -pi -w -e 's/(\w+)\s*\(\s*void\s*\)/$1\(\)/g;' and ran it for both .cxx and .hxx files. Change-Id: I314a1b56e9c14d10726e32841736b0ad5eef8ddd
* unnecessary castCaolán McNamara2015-04-121-4/+3
| | | | Change-Id: I7c040b2539b9f7f923620a02d0a27e8e000fee4a
* loplugin:staticmethodsNoel Grandin2015-04-094-5/+5
| | | | Change-Id: Ie348778ea666c24e95e048386547f301083a0017
* A UNO Any can't contain an AnyStephan Bergmann2015-04-081-1/+1
| | | | | | | | ...and css::uno::makeAny<css::uno::Any>() was never meant to be used. Introduce css::uno::toAny for the (template-code) cases that shall return an Any for both Any and non-Any inputs. Change-Id: Ifa977d73f1da71b2fedde7e8140b19497c4a0257
* Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann2015-03-312-2/+2
| | | | Change-Id: I6979d9af3a97918593f0596df17c096bd3f4c4ad
* const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann2015-03-261-2/+2
| | | | Change-Id: I313d61fc6e7be3a5513da83d3de9202237306739
* tdf#90010 Partially revert "package: Do not bother with deflating jpeg images"Matúš Kukan2015-03-251-1/+2
| | | | | | | This reverts commit 0843ade74e5848c0babad509b10c334b01847be2 for the case, when the document is protected by password. Would be nice to fix properly later. Change-Id: If159639d6f74de4e106e9dad382faac9d5de4a09
* loplugin:constantfunction: variousNoel Grandin2015-03-241-4/+0
| | | | Change-Id: I6eddda9f4b31c7ce413c328b6a857a81bd222eed
* convert SFX_FILTER_ constants to enum classNoel Grandin2015-03-191-2/+2
| | | | Change-Id: I7e53cfc90cefd9da7d6ecd795b09214bd44b1613
* create new 'enum class' SotClipboardFormatId to unify typesNoel Grandin2015-03-181-2/+2
| | | | | | | | | | | | of which there are several. There are some issues here I am unsure of - the SW and SC and CHART2 modules essentially ignore the enum values and assign their own ids Perhaps I should change them to use the common values and create new enum values where necessary? - the sc/qa/ and sq/qa/ and starmath/qa/ code was doing some dodgy stuff. I translated the code to pass down the stuff numeric values to the underlying code, but perhaps further fixing is necessary? Change-Id: Ic06d723e404481e3f1bca67c43b70321b764d923