summaryrefslogtreecommitdiff
path: root/ucb
AgeCommit message (Collapse)Author
2017-04-13Removed duplicated includesAndrea Gelmini
No automatic tools. Manual checked and tested. Change-Id: Ife260fa4e1d786cf81f2917a901664cc54943754 Reviewed-on: https://gerrit.libreoffice.org/36371 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2017-04-10loplugin:inlinefields in ucb_impl::RegexpMapNoel Grandin
Change-Id: Id8cf4a9e7f85048dac37fc1256427214abe3ac8c Reviewed-on: https://gerrit.libreoffice.org/36265 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-04-10use unique_ptr in RegexpMapImplNoel Grandin
Change-Id: I150bd441281305346bf6be1844894ae857dd50f7 Reviewed-on: https://gerrit.libreoffice.org/36264 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-04-05tdf#106955: Open WebDAV resources on which PROPFIND failsMike Kaganski
When PROPFIND fails on a WebDAV resource, its IsDocument property stays undefined, and so stream creation fails. Proposed solution is to default to IsDocument=true for all WebDAV documents where we cannot get the property from server. Such resources also fail to return their locking options, so defaulting to server properties. When later locking is attempted on it, the attempt fails with user notification (a dialog saying that getting information from server failed). Proposed solution is to check Content-Disposition header in such resources, and in case it's attachment, disable lock on this resource. The rationale for this is that "In a regular HTTP response, the Content-Disposition response header is a header indicating if the content is expected to be displayed ... as an attachment, that is downloaded and saved locally" (see MDN: https://developer.mozilla.org/en/docs/Web/HTTP/Headers/Content-Disposition Also, Content::getProperties wasn't ready for PROPFIND returning empty result. Change-Id: I91dbffa8bdf0fe900c11d2f8c9c9394d2104bb49 Reviewed-on: https://gerrit.libreoffice.org/36090 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2017-04-03use actual UNO enums in test..vbahelperNoel Grandin
Change-Id: I2e38462045efedc31903589a3fa4e6f62d74d8de Reviewed-on: https://gerrit.libreoffice.org/36042 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-03-30NeonSession_NeonAuth(): avoid redundant OUString->OString conversionVitaliy Anderson
Change-Id: I6b8515c25d58a88a040d4e08e36ffbd442228828 Reviewed-on: https://gerrit.libreoffice.org/35915 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2017-03-25Fix typosAndrea Gelmini
Change-Id: Ieea40e1b7282267157810f9f58ca083e68ae6715 Reviewed-on: https://gerrit.libreoffice.org/35659 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2017-03-23loplugins:redundantcast teach it about c-style typedef castsNoel Grandin
Change-Id: I1ac11a2481c0f4d8be1e1fd7c7637ac0ece3d65c Reviewed-on: https://gerrit.libreoffice.org/35558 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-03-17cleanup double includes and declarationsJochen Nitschke
Change-Id: I770e09f48b1a3e8299f59a8e475ba3c18d436d7b Reviewed-on: https://gerrit.libreoffice.org/35312 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
2017-03-14try to fix webdav buildMarkus Mohrhard
Found by the "Random Condig" tb run #293. Change-Id: I8fa7b66444b9b57f289c963cb1da5e772341dc30 Reviewed-on: https://gerrit.libreoffice.org/35156 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2017-03-13tdf#95416 remove ../ in #include headers of sfx2 filesJeevan
Change-Id: I5f3e1a8fe559500aec7ea9e38edaba42dff3f98b Reviewed-on: https://gerrit.libreoffice.org/35028 Reviewed-by: Tor Lillqvist <tml@collabora.com> Tested-by: Tor Lillqvist <tml@collabora.com>
2017-03-03Remove redundant 'inline' keywordStephan Bergmann
...from function definitions occurring within class definitions. Done with a rewriting Clang plugin (to be pushed later). Change-Id: I9c6f2818a57ccdb361548895a7743107cbacdff8 Reviewed-on: https://gerrit.libreoffice.org/34874 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2017-03-02Fix typosAndrea Gelmini
Change-Id: I139e6948e92a7bcdce5a2197b9d3754cb9549a33 Reviewed-on: https://gerrit.libreoffice.org/34781 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-02-28new loplugin unoanyNoel Grandin
Change-Id: I5d6c4a67cb2a09e7cd5bd620c6b262d188701b89 Reviewed-on: https://gerrit.libreoffice.org/34714 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-02-25value returned from particular web server is not valid asciiCaolán McNamara
$1 = 0x1a24b90 "GET,HEAD,H\205\360),HEAD,POST,OPTIONS" from this url to a missing resource http://www.japan-net.ne.jp/%7Enagayama/image/14photo/omoide/0301fuji/01rot-s.jpg seen during soffice --headless --convert odt ooo26424-5.sxw Change-Id: If6429dc02ff42bae9372ae39cadf8b49f278f54d
2017-02-22Need <sys/stat.h> when HAVE_FEATURE_MACOSX_SANDBOXTor Lillqvist
Change-Id: I401679ae6cbc8f924126589cfb9f22fb244e7da2
2017-02-21fix the webdav=serf buildMarkus Mohrhard
Change-Id: I1e66f437d561e5a8838fd331c6add25f4e25655e Reviewed-on: https://gerrit.libreoffice.org/34525 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2017-02-21loplugin:subtlezeroinit: ucbStephan Bergmann
Change-Id: Ic3ad7a45def131d94e7e2c356a47478a63f1c9d9
2017-02-21try to fix "random config" tbMarkus Mohrhard
Fixes #228
2017-02-21try to fix random config tinderboxMarkus Mohrhard
Change-Id: I825d16eec294e23847f5150cdf1c7bebeb54a67e Reviewed-on: https://gerrit.libreoffice.org/34228 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2017-02-16external/libcmis: Silence -Wdynamic-exception-spec errorsStephan Bergmann
...with Clang in C++17 mode Change-Id: I50a0568c13e3e7a2d37ebd739d587985b2f9bc8a
2017-02-15try to fix "random config" boxMarkus Mohrhard
Test run #211. Change-Id: I438ba57d69bcd46b9114c80e9ca4b7f7ac731963 Reviewed-on: https://gerrit.libreoffice.org/34286 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2017-02-10remove unused enumerators from FTPErrors enumNoel Grandin
Change-Id: I30a2cc41d246b5048f2514eb38ee8e3bef0ed41c
2017-02-10convert InsertOperation to scoped enumNoel Grandin
and drop unused LINK enumerator Change-Id: I6c42424ae5a7503b3e14e5f04fcf7a49b595b416
2017-02-06Add missing #includesStephan Bergmann
...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
2017-02-01Revert "cancelCommandExecution - no need to pass exception by Any"Noel Grandin
This reverts commit 4739b31dafc5154a2c7d6b3f0ee90686863656f0. Apparently, passing a param of type css::uno::Exception to Any will record precisely a css::uno::Exception in that Any, losing any subtype information, which this commit changed.
2017-02-01cancelCommandExecution - no need to pass exception by AnyNoel Grandin
all the call sites are passing an uno::Exception subtype Change-Id: I6de1f00810e063e75ef620314561d7e2d6445ada Reviewed-on: https://gerrit.libreoffice.org/33657 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-31Revert "Avoid warning with Clang that doesn't have -Wunused-but-set-parameter"Stephan Bergmann
This reverts commit d69bc9c939b94718ac9e17a62f62ae7ae87776e5, which isn't needed any more now that 01aa52b5e91bac740ad21c31968c834c1530831e "Work around -Werror,-Wunknown-pragmas" makes the whole pragma block conditional on __GNUC__ >= 7 (which Clang does not fulfill).
2017-01-31Work around -Werror,-Wunknown-pragmasStephan Bergmann
...from GCC < 7 not knowing -Wunused-but-set-parameter Change-Id: I6494882085ad39df8d7b94fa062a7717bffacadb
2017-01-31Avoid warning with Clang that doesn't have -Wunused-but-set-parameterTor Lillqvist
Change-Id: I8d016e9959898a90bff9d1e4734cbef37592a97e
2017-01-31Silence GCC 7 warnings in libcmis/libcmis.hxxStephan Bergmann
Change-Id: If7114da10824300804a882fa31080a17e69fc36b
2017-01-31teach lolugin:stringconstant about calling constructorsNoel Grandin
so we can remove unnecessary calls to the OUString(literal) constructor when calling constructors like this: Foo(OUString("xxx"), 1) Change-Id: I1de60ef561437c86b27dc9cb095a5deb2e103b36 Reviewed-on: https://gerrit.libreoffice.org/33698 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-30NO_STREAM_CREATION_WITHIN_DOCUMENT_ROOT is always trueNoel Grandin
and has been for more than a decade Change-Id: I81dd8c6db0b3a961d2e2f2dfb4c0fb97704756d8
2017-01-28loplugin:stringconstant check for unnecessary OUString constructor..Noel Grandin
..calls when creating exceptions Change-Id: I3bc58a5aa4dc6f0508ecb88b3a843b96b8c7ebfe Reviewed-on: https://gerrit.libreoffice.org/33617 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-27Don't worry, Kai, it's still in the git repoStephan Bergmann
57558f61e9aa1a45f12a2f72d7a0359303d00be2 "#i108938# - commented out unused member fuction." Change-Id: I4972d0ff98f8fa21de4fa6506bc1bcbc7088f6d3
2017-01-26Remove dynamic exception specificationsStephan Bergmann
...(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>
2017-01-26loplugin: unnecessary destructor test..unotoolsNoel Grandin
Change-Id: I1fd8a3f39b875d1920759f42e37f4c9d6785d62e Reviewed-on: https://gerrit.libreoffice.org/33573 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-26loplugin:dynexcspecStephan Bergmann
Change-Id: I5e30befa9338bfd72e237e2bdb7c157526d32b85
2017-01-26loplugin:useuniqueptrStephan Bergmann
Change-Id: I5091c2b4550dee1f0b602a5337e17fe78b9d07f9
2017-01-26loplugin:useuniqueptrStephan Bergmann
Change-Id: I1fa141ee0d8d818f3ed5b31faa1ac578ba791e39
2017-01-26loplugin:salboolStephan Bergmann
Change-Id: I9c34c84d14084e84ab7d9fda78955cf2aebaf5dc
2017-01-20new loplugin: useuniqueptr: ucb..ucbhelperNoel Grandin
Change-Id: Ib19ca3225b96d1bfec8a43bb762e16597f33b690 Reviewed-on: https://gerrit.libreoffice.org/33297 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-19New loplugin:dynexcspec: Add @throws documentation, ucbStephan Bergmann
Change-Id: I39a50e3f066776ac670a99120e12624af39fa83a
2017-01-18use rtl::Reference in DynamicResultSetWrapperNoel Grandin
instead of storing both a raw pointer and an uno::Reference Change-Id: Ia155ebcb856884461871d5992380c364da07a1d3
2017-01-18use rtl::Reference in ContentResultSetWrapperNoel Grandin
instead of storing both a raw pointer and an uno::Reference Change-Id: Iba7317e37046c62411bae9d943f9e81f23ff3554
2017-01-18use rtl::Reference in CachedContentResultSetNoel Grandin
instead of storing both a raw pointer and an uno::Reference Change-Id: I868530ba1697baae88a4322eda6c467c9f8e8eb6
2017-01-17use rtl::Reference OFileAccessNoel Grandin
instead of storing both a raw pointer and an uno::Reference Change-Id: I36f1788510e7233ae3de4610370528e24ca03e1d Reviewed-on: https://gerrit.libreoffice.org/33170 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-17use rtl::Reference in SortedDynamicResultSetNoel Grandin
instead of storing both raw pointers and uno::Reference Change-Id: Ifcc9daf95d4b2583c9bc4b06deb65a2e618ac1e0 Reviewed-on: https://gerrit.libreoffice.org/33168 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-09ucb: cmis::DataSupplier::queryContent() looks rather questionableMichael Stahl
It's not immediately obvious if the caller is responsible for checking the index validity here, but all the other sub-classes of ResultSetDataSupplier that i looked at do check the index so do the same here. Change-Id: Ib0c5c38cb28282f08752cdb03202e4d6f3566693
2017-01-09ucb: ReacquireableGuard::reacquire() is a no-opMichael Stahl
So remove this junk and use osl::ResettableGuard instead which actually works. Change-Id: Ieb49fab16c94f0a2847ee5a1b95b52f2c141c674
generated by cgit (git 2.34.1) at 2025-02-06 03:00:40 +0000