summaryrefslogtreecommitdiffstats
path: root/io/qa
Commit message (Collapse)AuthorAgeFilesLines
* tdf#42949 Fix IWYU warnings in io/Gabor Kelemen2019-04-251-4/+0
| | | | | | | | | | Found with bin/find-unneeded-includes Only removal proposals are dealt with here. Change-Id: I16dfde335816053e5821d5e975b27b6e4819b1a8 Reviewed-on: https://gerrit.libreoffice.org/71135 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
* Better fix for missing [[noreturn]] of CPPUNIT_FAILStephan Bergmann2019-02-011-1/+1
| | | | | | | | | | | This includes a revert of commit 9808486a89c6368f836579f8d8c0dda63fd0063c "Avoid -Werror,-Wimplicit-fallthrough with clang-cl", and also undoes older MSVC workarounds (that would start to cause "warning C4702: unreachable code" now). Change-Id: If72bf336e12e0a2db589857e8875003dae861977 Reviewed-on: https://gerrit.libreoffice.org/67248 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* remove more rtl::OUString and OString prefixesNoel Grandin2018-10-241-2/+2
| | | | | | | | | which seem to have snuck back in since the great rounds of removals. Change-Id: I85f7f5f4801c0b48dae8b50f51f83595b286d6a1 Reviewed-on: https://gerrit.libreoffice.org/62229 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:includeform: ioStephan Bergmann2017-10-231-5/+5
| | | | Change-Id: I68e0ebc5a1585d2f55a4061edd8ab197f437b6a2
* Remove dynamic exception specificationsStephan Bergmann2017-01-261-22/+4
| | | | | | | | | | | | | | | | | | | | | | | | | ...(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>
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-131-1/+1
| | | | | | | | | | | | | | | | | 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
* Include cppumaker-generated .hpp, not .hdlStephan Bergmann2016-05-111-4/+4
| | | | Change-Id: Ib0451f26edbfbcdfdca78caa97bd09bc65ec2782
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-121-5/+5
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* io: tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe2015-08-191-2/+2
| | | | | | | | | 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>
* Fix C4716: 'anonymous namespace'::Input::readBytes : must return a valueTor Lillqvist2014-11-261-1/+1
| | | | Change-Id: I1d660da4cdd3f612ac1478e37e613446d4253fae
* Unit test for fdo#79941 (handle short reads)Stephan Bergmann2014-11-241-0/+148
...done as a subsequentcheck as doing it as a BootstrapFixtureBase (which is the easiest way) makes it depend on later modules in the dependency chain. Change-Id: I9588bae409b38aa373ccfa855042f598b6e2bb2b
generated by cgit (git 2.34.1) at 2024-05-13 19:24:51 +0000