summaryrefslogtreecommitdiffstats
path: root/ucbhelper
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:expandablemethods in ucb..ucbhelperNoel Grandin2016-10-262-7/+31
| | | | | | | Change-Id: I42d077d7ebcfb77447d011644f8d64e80eae69c2 Reviewed-on: https://gerrit.libreoffice.org/30266 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Fix typosAndrea Gelmini2016-10-031-1/+1
| | | | | | | Change-Id: Ie75beb4e282a4d1b784a5847262e39cf9c851527 Reviewed-on: https://gerrit.libreoffice.org/29440 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* convert CONTINUATION constants to typed_flags_setNoel Grandin2016-09-261-52/+19
| | | | Change-Id: I38333e5d229aa520fbe0a8ad72007c503853956e
* loplugin:unusedfieldsNoel Grandin2016-09-201-5/+2
| | | | | | | Change-Id: I852e98b16fdcb88b04e39d11e3101d502c918c24 Reviewed-on: https://gerrit.libreoffice.org/29078 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-133-3/+3
| | | | | | | | | | | | | | | | | The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark overriding destructors as 'virtual'" appears to no longer be a problem with MSVC 2013. (The little change in the rewriting code of compilerplugins/clang/override.cxx was necessary to prevent an endless loop when adding "override" to OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager(); in chart2/source/inc/LifeTime.hxx, getting stuck in the leading OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.) Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
* loplugin:constantparam in framework..vbahelperNoel Grandin2016-09-072-11/+7
| | | | Change-Id: I592f73012ae8a4df8488ef04dc0956d9400625bc
* cid#1371163 Missing move assignment operatorNoel Grandin2016-08-291-0/+9
| | | | Change-Id: Ie050cfae965adf7bc43c91f366904cf6876783c0
* ucbhelper::Std{In,Out}putStream are only used in ucb/source/ucb/cmis/Stephan Bergmann2016-08-223-297/+0
| | | | | | | | ...where their use of boost::shared_ptr (instead of std::shared_ptr) matches the use in libcmis, but makes them unlikely to be useful anywhere else. So move them into ucb/source/ucb/cmis/. Change-Id: I68359be6b43d6889af4f241dcdcbdc0d9d70d717
* Some clang-tidy misc-move-constructor-initStephan Bergmann2016-08-192-2/+2
| | | | | | ...by turning the relevant ctor parameters into "const &". Change-Id: Ia8d0aba5da10ad6b25f8689e2281e45b3d71c1fc
* improve passstuffbyref return analysisNoel Grandin2016-07-271-1/+1
| | | | | | | Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c Reviewed-on: https://gerrit.libreoffice.org/27317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:singlevalfields in variousNoel Grandin2016-06-281-6/+2
| | | | | | | Change-Id: Ia0d8f463a4dba9ec63aa0159441e3e607dd3bf5e Reviewed-on: https://gerrit.libreoffice.org/26738 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* ucbhelper: get rid of pointless scope in ContentImplHelper::releaseMichael Stahl2016-06-241-4/+2
| | | | Change-Id: I5a35674115388288190e3a6496010badc32cf589
* loplugin:singlevalfields in ucbhelperNoel Grandin2016-06-243-97/+22
| | | | | | | Change-Id: I738bda6e6e161fb5ea96bd496a21ccf72680cd7f Reviewed-on: https://gerrit.libreoffice.org/26604 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* remove some more (void) style function definitions/declarationsNoel Grandin2016-06-231-7/+3
| | | | | | | | | | found with git grep -nP '^\s*void\s*\)' Change-Id: I1206031cf957cb6f957cae64ec5cae1280bda694 Reviewed-on: https://gerrit.libreoffice.org/26590 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* remove some manual ref-countingNoel Grandin2016-06-061-8/+4
| | | | | | | | | | | | | triggered when I noticed a class doing acquire() in the constructor and then release() in the destructor. found mostly by git grep -n -B5 -e '->release()' Change-Id: I96e43a3d30ffd9ae9a34275f24cd914d8f7b026f Reviewed-on: https://gerrit.libreoffice.org/25806 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Revert "remove some manual ref-counting"Noel Grandin2016-05-241-4/+8
| | | | | | | until I have a better understanding of the UNO reference counting. This reverts commit 111de438ea3e512a541281dc0716cc728ea8d152.
* remove some manual ref-countingNoel Grandin2016-05-241-8/+4
| | | | | | | | | | | | | triggered when I noticed a class doing acquire() in the constructor and then release() in the destructor. found mostly by git grep -n -B5 -e '->release()' Change-Id: Ie1abeaed75c1f861df185e3bde680272dbadc97f Reviewed-on: https://gerrit.libreoffice.org/25363 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:passstuffbyrefStephan Bergmann2016-05-121-1/+1
| | | | Change-Id: Iba108c1007f33de2e7207e25d7eb2dfc18d1c22c
* Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann2016-05-101-1/+1
| | | | | | | | | | | | | | | | ...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in preparation of enabling -Wimplicit-fallthrough. (This is only relevant for C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.) Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but that would require adding back in dependencies on boost_headers to many libraries where we carefully removed any remaining Boost dependencies only recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its future evolution will not have any impact on the stable URE interface.) C++17 will have a proper [[fallthroug]], eventually removing the need for a macro altogether. Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
* teach passstuffbyref plugin to check for..Noel Grandin2016-05-061-1/+1
| | | | | | | | | | | | unnecessarily passing primitives by const ref. Suggested by Tor Lillqvist Change-Id: I445e220542969ca3e252581e5953fb01cb2b2be6 Reviewed-on: https://gerrit.libreoffice.org/24672 Reviewed-by: Tor Lillqvist <tml@collabora.com> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-204-42/+42
| | | | Change-Id: Ie47a53763332119d290ebe5646af987ddd44be9d
* loplugin:passstuffbyref in ucbhelperNoel Grandin2016-04-141-2/+1
| | | | Change-Id: I680e4ca0d20d5ddabdc875a1d6bec27322956f5e
* clang-tidy performance-unnecessary-value-param in ucbhelperNoel Grandin2016-04-111-1/+1
| | | | Change-Id: I595485e8804d6f2e4e0f9bc8a78c8cb132411f7d
* loplugin:constantfunction in ucbhelperNoel Grandin2016-03-164-26/+14
| | | | | | | Change-Id: I80e00b5ac2621378801f89532ed88b377ef72b60 Reviewed-on: https://gerrit.libreoffice.org/23297 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:constantparamNoel Grandin2016-03-151-2/+2
| | | | Change-Id: I270e068b3c83e966e741b0a072fecce9d92d53f5
* tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar2016-03-151-2/+2
| | | | | | | | | | I simply replaced OSL_DEBUG_LEVEL > 1 with OSL_DEBUG_LEVEL > 0 as suggested in easy hack Change-Id: Id30d93fa439fa4cddf4a56c90b6054203c7882b1 Reviewed-on: https://gerrit.libreoffice.org/23233 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* new loplugin: commaoperatorNoel Grandin2016-02-231-1/+4
| | | | Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
* Remove excess newlinesChris Sherlock2016-02-0912-239/+0
| | | | | | | | | | | | | | | | | | | A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* InterfaceContainer2 with vector instead of SequenceNoel Grandin2016-01-252-9/+2
| | | | | | | | | | create an InterfaceContainer2 class to replace InterfaceContainer. It uses a std::vector instead of a Sequence for the mutable listener list, which provides far better performance. Switch all our internal use-sites to the new class. Change-Id: I6b56cfa511ded2395faa22e68fab3b2f16c3cb88
* 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>
* loplugin:unusedfields variousNoel Grandin2015-11-261-1/+0
| | | | Change-Id: I18f94269a1172cf195ee402384f7144610e1e82d
* use unique_ptr for pImpl in ucbhelper/Noel Grandin2015-11-177-11/+3
| | | | Change-Id: Ibae75fc4d843bd38179d4c7afc6ddb532835c7a6
* use initialiser for Sequence<OUString>Noel Grandin2015-11-151-4/+2
| | | | | | | | | | | | using variations of: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<OUString> (\w+)\(1\)\; \s*OUString\* pArray.*; .*\[0\]\s*=\s*(\S+)\;/Sequence<OUString> \1 { \2 };/g" Change-Id: I03c64334ff30ee14dce0d17b67f5122a3893bbe3 Reviewed-on: https://gerrit.libreoffice.org/19971 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* com::sun::star->css in ucbhelper/Noel Grandin2015-11-1111-156/+141
| | | | | | | Change-Id: I01e3046caefdcfbf630f4a794f12e8890be6db40 Reviewed-on: https://gerrit.libreoffice.org/19902 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-109-35/+35
| | | | Change-Id: I3003db8a411740db48c2e0c6ecd81a7fd9209e6c
* loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann2015-11-061-2/+2
| | | | Change-Id: I970a03643f3c460110bc7caf1dc33c6dde1d78d5
* use uno::Reference::set method instead of assignmentNoel Grandin2015-11-053-5/+3
| | | | Change-Id: Id57ccff7ea6cf5c7053b51268b1190f5459bb357
* no need to use OUString constructor in call to createInstanceNoel Grandin2015-11-011-3/+2
| | | | | | | Change-Id: Iaf3d83ba1490cb1d97a5bd4d1f7cd6943d4a7296 Reviewed-on: https://gerrit.libreoffice.org/19704 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-124-27/+27
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* loplugin:stringconstantStephan Bergmann2015-10-051-2/+2
| | | | Change-Id: I68c63d560cfd7ec82802cd3f073cdd0544cff99b
* tdf#82744: fix WebDAV lock/unlock behaviour - part 3Giuseppe Castagno2015-10-051-0/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* Fix typosAndrea Gelmini2015-09-302-2/+2
| | | | | | | Change-Id: I3926eca56ac9f54f9ddbf88610888ce97cf00b34 Reviewed-on: https://gerrit.libreoffice.org/18990 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Resolves: tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe2015-09-152-4/+4
| | | | | | | | | with the variadic variants. Change-Id: I7154f9472f02fdf47d27ba715db55bb1ec669a8a Reviewed-on: https://gerrit.libreoffice.org/18580 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:stringconstant: OUStringBuffer: appendAscii -> appendStephan Bergmann2015-08-311-5/+5
| | | | Change-Id: If38423c29c7a9f4d740c244d350b83c471523797
* Remove newly unused ucbhelper/fileidentifierconverter.hxxStephan Bergmann2015-08-182-78/+0
| | | | Change-Id: I7c272383ecb115e19699ed96bf5622d979403a01
* Remove obsolete getLocalFileURLStephan Bergmann2015-08-181-18/+0
| | | | | | ...vnd.sun.star.wfs is long gone Change-Id: I64da15a6c16e429aeda57c435e353891fb28f04d
* loplugin: defaultparamsNoel Grandin2015-08-111-1/+1
| | | | Change-Id: I50ba6a836473961d952ed88e56532501469c5368
* inline some use-once typedefsNoel Grandin2015-08-031-8/+1
| | | | | | | Change-Id: I02cbbba56a2ad83e0ac3d806265a7e0d6a29594d Reviewed-on: https://gerrit.libreoffice.org/17495 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#84190 : when auth fails, don't remember passwords for cmisSzymon Kłos2015-07-161-9/+23
| | | | Change-Id: Ia1e0d553556693e0efa6de0bfc6f8b0ae9d40b5d
* loplugin:stringconstant: handle OUString+=OUString(literal)Stephan Bergmann2015-06-261-3/+3
| | | | Change-Id: I6e534537883ef76db88926203bf21b6d0be0aab1
generated by cgit (git 2.34.1) at 2024-05-30 00:34:09 +0000