summaryrefslogtreecommitdiffstats
path: root/io
Commit message (Collapse)AuthorAgeFilesLines
* io: warn if OConnector can't connect to pipeMichael Stahl2017-02-101-5/+4
| | | | Change-Id: I536177e7fbc2a8bb120db786722b71cc77c039a4
* Add missing #includesStephan Bergmann2017-02-0612-2/+22
| | | | | | | | | ...and remove some unncessary using directives/declarations, in preparation of removing now-unnecessary #includes from cppumaker-generated files, post e57ca02849c3d87142ff5ff9099a212e72b8139c "Remove dynamic exception specifications". Change-Id: Iaf1f268871e2ee1d1c76cf90f03557527ebc9067
* unnecessary use of OUStringBuffer in throwing exceptionsNoel Grandin2017-02-022-46/+29
| | | | | | | Change-Id: Iec1473264426f19c31e72260dfce9494389e474f Reviewed-on: https://gerrit.libreoffice.org/33788 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Remove dynamic exception specificationsStephan Bergmann2017-01-2617-724/+330
| | | | | | | | | | | | | | | | | | | | | | | | | ...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* New loplugin:dynexcspec: Add @throws documentation, ioStephan Bergmann2017-01-196-1/+23
| | | | Change-Id: I2d8b69454217bbbc9542cced8ce85269e3872662
* new loplugin: useuniqueptr: helpcompiler..ioNoel Grandin2017-01-172-23/+10
| | | | | | | Change-Id: I6b394163c144e6b5540cb160abb613d56fe327de Reviewed-on: https://gerrit.libreoffice.org/33165 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* OSL_TRACE->SAL in framework..salNoel Grandin2016-12-132-20/+1
| | | | | | | Change-Id: I9a897af88aa9f6f7ca98ce521c69b5a4ee8462e9 Reviewed-on: https://gerrit.libreoffice.org/31903 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unnecessaryoverride (dtors) in ioStephan Bergmann2016-12-054-35/+1
| | | | Change-Id: I5c787d70fb3388e649e751d2b801765d567249c7
* loplugin:unusedmethods in chart2..svxNoel Grandin2016-09-222-20/+0
| | | | | | | Change-Id: Ifb6045885049733415895f58cdd911256f48323c Reviewed-on: https://gerrit.libreoffice.org/29187 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-1312-17/+17
| | | | | | | | | | | | | | | | | 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:unnecessaryvirtualNoel Grandin2016-08-081-1/+1
| | | | | | | Change-Id: If25d9307efda5f57b0f80a0cf5c2c5cab6a752d6 Reviewed-on: https://gerrit.libreoffice.org/27981 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* new loplugin unnecessary overrideNoel Grandin2016-07-151-2/+0
| | | | | | | Change-Id: I88d3e33823d68745b98625050a8a274f9ef04bcb Reviewed-on: https://gerrit.libreoffice.org/27135 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* remove (un-)definitions of NDEBUG from source filesMichael Stahl2016-05-251-16/+0
| | | | | | | The build system already defines NDEBUG only if OSL_DEBUG_LEVEL=0 AND ASSERT_ALWAYS_ABORT=FALSE so the C++ code shouldn't override that. Change-Id: Ie67a576646cc7e18c3ffac67a3c81b80747ea438
* Include cppumaker-generated .hpp, not .hdlStephan Bergmann2016-05-111-4/+4
| | | | Change-Id: Ib0451f26edbfbcdfdca78caa97bd09bc65ec2782
* Fix typo in codeAndrea Gelmini2016-05-011-4/+4
| | | | | | | Change-Id: I670b016fb937fd4e87939afedd6d2047533812bf Reviewed-on: https://gerrit.libreoffice.org/24531 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* new plugin stylepoliceNoel Grandin2016-04-211-3/+3
| | | | | | | check for local variables which follow our member field naming convention, which is highly confusing Change-Id: Idacedf7145d09843e96a584237b385f7662eea10
* clang-tidy performance-unnecessary-value-param in variousNoel Grandin2016-04-142-8/+8
| | | | | | | Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907 Reviewed-on: https://gerrit.libreoffice.org/24019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* clean upStephan Bergmann2016-04-071-2/+2
| | | | Change-Id: Ibf0df59d2773a3356b21e1b4c70889511b373e56
* Revert "reduce unnecessary realloc'ing"Stephan Bergmann2016-04-073-18/+8
| | | | | | This reverts commit 9f6e6fabcd5718e0b65437c5ce398e520f47aae1, which is wrong, as it would cause the functions to now try to read aReadBytes.getLength() bytes if that is initially larger than nBytesToRead.
* reduce unnecessary realloc'ingNoel Grandin2016-04-013-8/+18
| | | | | | | Change-Id: Ic597814706573576a0ba330a69a7a38aa97e5224 Reviewed-on: https://gerrit.libreoffice.org/23694 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Remove excess newlinesChris Sherlock2016-02-0914-99/+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>
* fix OSX buildNoel Grandin2016-01-252-8/+6
| | | | | | | | | | by reverting part of commit commit 752cd07d085ac0aadc99bd512d49072843139032 "InterfaceContainer2 with vector instead of Sequence" cannot add comphelper as a dependency to a module that is part of URE Change-Id: I40aeeff1fa39988af911b00a06fb1ac8edeb6f03
* InterfaceContainer2 with vector instead of SequenceNoel Grandin2016-01-252-6/+8
| | | | | | | | | | 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-102-2/+2
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* loplugin:unusedfieldsNoel Grandin2015-11-301-1/+0
| | | | Change-Id: Icac4ac1a2614e72bc9ff070819533e09eeb1a864
* cppcheck: noExplicitConstructorCaolán McNamara2015-11-2411-16/+16
| | | | Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
* use initialiser list for Sequence<OUString>Noel Grandin2015-11-162-9/+3
| | | | | | | Change-Id: Ia5e47261d1fc6fac2d046656c05a1c5eedb07e02 Reviewed-on: https://gerrit.libreoffice.org/19978 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* use initialiser for Sequence<OUString>Noel Grandin2015-11-151-2/+1
| | | | | | | | | | | | 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>
* use initialiser for Sequence<OUString>Noel Grandin2015-11-156-20/+10
| | | | | | | | | | | | | 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-153-6/+3
| | | | | | | | | | | | | | 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>
* Replaced for_each and algorithm include by range based loop and exception.Mario J. Rugiero2015-11-102-4/+6
| | | | | | | | | | Since the only thing used from algorithm was std::for_each, and std::exception was being used without including it. Change-Id: Ida415bfbcae2b14811183a6afa46b6f6e1ac6ab2 Reviewed-on: https://gerrit.libreoffice.org/19864 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-108-27/+27
| | | | Change-Id: I04ceb5b1514a1d0bf123a1cded5e1aac3e8dde2f
* 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 (automatic rewrite)Stephan Bergmann2015-11-061-5/+5
| | | | Change-Id: Ifcfb6792138802f4150ccb966927818fbd3f64e3
* use uno::Reference::set method instead of assignmentNoel Grandin2015-11-047-18/+17
| | | | Change-Id: I11822c50fa66d038a3d6f38054ab35c2e613f077
* Fix create/destroy mismatchStephan Bergmann2015-11-031-2/+2
| | | | Change-Id: Ibff6caedae0c5ed84cf4864c7a52af56a7cea527
* no need to use OUString constructor in call to createInstanceNoel Grandin2015-11-011-2/+1
| | | | | | | Change-Id: Iaf3d83ba1490cb1d97a5bd4d1f7cd6943d4a7296 Reviewed-on: https://gerrit.libreoffice.org/19704 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* UNO: no need to use OUString constructor when calling createInstanceNoel Grandin2015-10-304-16/+14
| | | | | | | Change-Id: I37da28539b94706574116d0fff5f008aabfb5526 Reviewed-on: https://gerrit.libreoffice.org/19682 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* com::sun::star->css in ioNoel Grandin2015-10-239-125/+125
| | | | Change-Id: I4a0cd4b29db91584c23148d659846c71b90b0d65
* coverity#1327454 Uncaught exceptionCaolán McNamara2015-10-174-126/+39
| | | | | | and a boatload more along that vein Change-Id: Ic7d458d3b8e5c99df1995251f75d3ad57d0df967
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-1213-217/+217
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* loplugin:mergeclassesNoel Grandin2015-10-122-38/+20
| | | | Change-Id: I672d277a8d2dc6b57b299679e7e98348f9dbde30
* loplugin:mergeclassesNoel Grandin2015-10-122-46/+28
| | | | Change-Id: I31fe981dac14bd732ee68ee1a2e475dd1b0498a2
* clang-analyzer-deadcode.DeadStoresStephan Bergmann2015-10-071-1/+0
| | | | Change-Id: I22b423b99c87070ab1b8ce7f3eff928384951947
* Fix typosAndrea Gelmini2015-09-301-2/+2
| | | | | | | Change-Id: I44bdb63bef76e9686e995661a24a3bef7cd50ab7 Reviewed-on: https://gerrit.libreoffice.org/18959 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* loplugin:stringconstant: OUStringBuffer: appendAscii -> appendStephan Bergmann2015-08-313-26/+26
| | | | Change-Id: I3adb8d5664ecd8fafccc7bae0656f15100484422
* io: tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe2015-08-1916-45/+40
| | | | | | | | | with the variadic variants. Change-Id: Iad03db6b729a785ab8b29a69943fa45f5a36b21b Reviewed-on: https://gerrit.libreoffice.org/17849 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Some oslCondition -> osl::ConditionStephan Bergmann2015-08-181-9/+7
| | | | Change-Id: Ib2c98db9ffd5871b6422c05f0230bbe27d1ba729
* loplugin: defaultparamsNoel Grandin2015-08-144-8/+4
| | | | Change-Id: I320eb6149793689c496b14090e8e0243397d01ba
* inline some use-once typedefsNoel Grandin2015-08-032-6/+2
| | | | | | | Change-Id: I55cc82c8e180cce371c996690608090b1bfdfda4 Reviewed-on: https://gerrit.libreoffice.org/17494 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
generated by cgit (git 2.34.1) at 2024-08-09 13:23:43 +0000