summaryrefslogtreecommitdiffstats
path: root/connectivity/source/drivers/component/CDatabaseMetaData.cxx
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:ostr in connectivityNoel Grandin2024-05-071-8/+8
| | | | | | | Change-Id: Ice633719b05240ab5a052b62ca4eafe89f97e12f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167238 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#158072 Fails to apply AutoFilter in Mail Merge dialogNoel Grandin2023-11-231-65/+60
| | | | | | | | | | | | | | | regression from commit 66b0bc55854ba13b92b6c39b95f3f2d4ef76bf20 Author: Noel Grandin <noel.grandin@collabora.co.uk> Date: Fri Mar 10 10:35:33 2023 +0200 simplify initialisation in *DatabaseMetaData where I failed to notice that the supposedly static data was being std::move'd Change-Id: I6238fa8e20ee6ca9fe31c29a7e05262dc1e3bede Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159851 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* simplify initialisation in *DatabaseMetaDataNoel Grandin2023-03-101-12/+12
| | | | | | | Change-Id: I1d7e107e22a98489e4d1013e14bb780e20d87c40 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148594 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* remove ORowSetValue implicit conversion methodsNoel Grandin2021-11-231-2/+2
| | | | | | | | | | | | | | | | in favour of the existing get*() methods. The get*() methods 0 or false or empty in the case of "null", which is exactly the same behaviour as the conversion methods. These implicit conversion methods cause lookup problems when combined with some upcoming OUString changes. And the code looks cleaner this way too, and has less magic when calling methods. Change-Id: Ieb4756bf693e83b996a32667fc1b955f89193496 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125690 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:moveparam in connectivityNoel Grandin2021-10-081-2/+2
| | | | | | | Change-Id: Ie5c2f6ec1573ab2c47218dd4da8c9150b23bcd2e Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123247 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Directly initialize vector in connectivity (part 1)Julien Nabet2021-05-141-24/+22
| | | | | | | Change-Id: I3b0bf1ca710aba58d7d649687b5d791374be8d6e Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115610 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
* loplugin:refcounting in connectivityNoel2021-02-221-6/+4
| | | | | | | Change-Id: Iff2f096e7ee00d9cb29d47462dd7e1963c6b06e0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111302 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#42949 Fix IWYU warnings in connectivity/*/*cxxGabor Kelemen2020-01-141-5/+1
| | | | | | | | | | | | Except for platform specific drivers/ado and drivers/macab dirs Found with bin/find-unneeded-includes Only removal proposals are dealt with here. Change-Id: I24b741cded8995e29ac3d518aeaa0d60b3c55b56 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86317 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
* tdf#42949 Fix IWYU warnings in connectivity/*/*hxxGabor Kelemen2019-12-161-0/+2
| | | | | | | | | | | | | | Except for non Linux specific parts: connectivity/source/drivers/macab/ connectivity/source/inc/ado/ Found with bin/find-unneeded-includes Only removal proposals are dealt with here. Change-Id: Iba6d0cd0effdac777be6fbc8f12f2e92cae47887 Reviewed-on: https://gerrit.libreoffice.org/84174 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
* look for unnecessary calls to Reference::is() after an UNO_QUERY_THROWNoel Grandin2018-06-081-1/+0
| | | | | | | | | | | | | | | | | Since the previous call would throw if there was nothing to be assigned to the value. Idea from tml. Used the following script to find places: git grep -A3 -n UNO_QUERY_THROW | grep -B3 -F 'is()' Change-Id: I36ba7b00bcd014bdf16c0455ab91056f82194969 Reviewed-on: https://gerrit.libreoffice.org/55417 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* More loplugin:cstylecast: connectivityStephan Bergmann2018-01-121-9/+9
| | | | | | | | auto-rewrite with <https://gerrit.libreoffice.org/#/c/47798/> "Enable loplugin:cstylecast for some more cases" plus solenv/clang-format/reformat-formatted-files Change-Id: Idbefb16acae20ab137497e78c9bc5dc5634c07fe
* loplugin:includeform: connectivityStephan Bergmann2017-10-231-3/+3
| | | | Change-Id: I06596fac09f0568b8bab2e2e235a2b88bcd3fc7a
* connectivity DatabaseMetaData: pull out OComponent from OCalc/WriterMiklos Vajna2017-09-111-0/+247
Only getURL() and getTables() is custom, the rest can be shared. Change-Id: I9d282a30f722bce0c05b37c7d005c84193e01b7c Reviewed-on: https://gerrit.libreoffice.org/42152 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>