summaryrefslogtreecommitdiffstats
path: root/connectivity
Commit message (Collapse)AuthorAgeFilesLines
* clang-cl loplugin: connectivityStephan Bergmann2016-10-1666-1201/+1201
| | | | | | | Change-Id: I025cf2ad63f08bb58d54983982d7a116fac0ffea Reviewed-on: https://gerrit.libreoffice.org/29874 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* tdf#88205 Adapt uses of css::uno::Sequence to use initializer_list ctornadith2016-10-141-4/+1
| | | | | | | Change-Id: I4abcff5e1bf050bd4e4cf0695f196c418d8229ef Reviewed-on: https://gerrit.libreoffice.org/29778 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#88205 Adapt uses of css::uno::Sequence to use initializer_list ctornadith2016-10-141-4/+1
| | | | | | | Change-Id: Ibfb981a58e61c72f3a292dd8e72895b98502d640 Reviewed-on: https://gerrit.libreoffice.org/29787 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#88205 Adapt uses of css::uno::Sequence to use initializer_list ctorAsela Dasanayaka2016-10-131-4/+1
| | | | | | | | | Fix a bug on connectivity/source/drivers/ado/AResultSet.cxx Change-Id: I164732100ff3bdc4e73f80e2495610cb53581b0d Reviewed-on: https://gerrit.libreoffice.org/29758 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:unnecessaryoverrideNoel Grandin2016-10-131-5/+0
| | | | | | | Change-Id: I08c55a3023ec2e8990098eeb60e91cd18556e7ae Reviewed-on: https://gerrit.libreoffice.org/29656 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* ADO: return correct css::sdbcx::CompareBookmark valuesStephan Bergmann2016-10-121-2/+3
| | | | | | | | | | | | | | | | css.sdbcx.XRowLocate's compareBookmarks is documented to return values from the css.sdbcx.ConpareBookmark constants group, which has LESS = -1, EQUAL = 0, GREATER = 1, NOT_EQUAL = 2, NOT_COMPARABLE = 3. So in case of equality return EQUAL (0) instead of sal_True (1). And, in the MS SDK's adoint.h, the CompareEnum enum os defined with enumeators adCompareLessThan = 0, adCompareEqual = 1, adCompareGreaterThan = 2, adCompareNotEqual = 3, adCOmpareNotComparable = 4, so mapping from the latter to the former needs to do -1, not +1. Change-Id: I6643da31f326e38153e4ebab9e680b33a6854918 Reviewed-on: https://gerrit.libreoffice.org/29735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:unnecessaryoverride in connectivity/cuiNoel Grandin2016-10-122-11/+0
| | | | | | | Change-Id: I5ec4eb2f74036e42851333e3d669dbea58c2490a Reviewed-on: https://gerrit.libreoffice.org/29687 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Replace OLEVariant operators with proper functions, for clarityStephan Bergmann2016-10-1113-111/+92
| | | | | | | | | | | | | | * Rename the original getDate (returning double) to getDateAsDouble, to make room for the new getDate (returning css::uno::Date). * Assume the difference between the original 'operator String' (first checking for VT_BSTR, then for isNull()) and getString (first checking for isNull(), only then calling 'operator String') was accidental rather than by design. Change-Id: I6261bdad5c328d6b10e83f2d1664a6c78ec1fed4 Reviewed-on: https://gerrit.libreoffice.org/29705 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* Replace OLEString operators with proper functions, for clarityStephan Bergmann2016-10-118-77/+77
| | | | | | | Change-Id: I1fa309d034e1ec94f38d07ede07e691f1484a4f5 Reviewed-on: https://gerrit.libreoffice.org/29697 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* Fix typosAndrea Gelmini2016-10-012-2/+2
| | | | | | | Change-Id: I8374d6d08f4eb4ae2821e213371c615b92d7e9ab Reviewed-on: https://gerrit.libreoffice.org/29432 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* TDE fix some clang plugin based warningsJan-Marek Glogowski2016-09-2324-252/+253
| | | | Change-Id: I3b44038e8a889e4c13126fd8d721e53e71ea9ec6
* loplugin:unusedmethods unused returnsNoel Grandin2016-09-222-3/+2
| | | | | | | Change-Id: I4da3374e31d3c7407b401d66275da7f56ae83d30 Reviewed-on: https://gerrit.libreoffice.org/29178 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Remove NumberFormatCodeWrapperMaarten Bosmans2016-09-222-2/+0
| | | | | | | | | This class only adds a level of indirection, without any useful functionality. Change-Id: I806e1b9241caf025c62c12c93aad3101daac874a Reviewed-on: https://gerrit.libreoffice.org/29134 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Remove obsolete workaround for SUNPRO 5 compilerStephan Bergmann2016-09-201-11/+14
| | | | | | ...copy/pasted into three more places Change-Id: I6661244d0248af509cd867fc3e8c451551068a78
* loplugin:singlevalfields in basic..idlNoel Grandin2016-09-1624-88/+57
| | | | | | | | | | | Also fix obvious bug in the initialisation of the connectivity::odbc::OConnection::m_bClosed field. Probably closes some kind of connection leak there. Change-Id: I04579cf91bcd6d6c51c697d83971da4142743a82 Reviewed-on: https://gerrit.libreoffice.org/28932 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* convert LogLevel constants to scoped enumJochen Nitschke2016-09-1512-88/+89
| | | | | | | | | | | rename some variables to distinguish them from type name use enumarray for log level strings remove unused DATA member Change-Id: Id3c38b82e679b3aede5ce790735443e769d5f236 Reviewed-on: https://gerrit.libreoffice.org/28920 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* connectivity: firebird: loplugin:nullptrMichael Stahl2016-09-151-0/+4
| | | | Change-Id: Ia711be1238de87ac5b52b02de077b70220369e4b
* replace sal_Size with std::size_t (or sal_uInt64 for SvStream pos)Michael Stahl2016-09-153-30/+30
| | | | | | | | ... except in include/rtl, include/sal, include/uno, where sal_Size is retained for compatibility, and where callers of rtl functions pass in pointers that are incompatible on MSVC. Change-Id: I8344453780689f5120ba0870e44965b6d292450c
* loplugin:overrideStephan Bergmann2016-09-149-10/+10
| | | | Change-Id: I0cd2007c9169fe480e28e684cfe1bd8b24da7123
* loplugin:constantparam in chart2..connectivityNoel Grandin2016-09-1419-66/+37
| | | | | | | Change-Id: Ic9e1bd36a11c7148fa7595a2b6c6de9bd7a8653d Reviewed-on: https://gerrit.libreoffice.org/28834 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-13104-112/+112
| | | | | | | | | | | | | | | | | 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
* coverity#1372406 Uncaught exceptionCaolán McNamara2016-09-011-1/+1
| | | | Change-Id: I24fe9cb3aa9002d4b94b6f934e39aff243c1e09d
* coverity#1372402 Uncaught exceptionCaolán McNamara2016-09-011-1/+1
| | | | Change-Id: I63743ccdeca1be66da5622bd286f08daebdd74ff
* coverity#1372398 Uncaught exceptionCaolán McNamara2016-09-012-2/+2
| | | | Change-Id: Icf6c0f8071a3ea2de81073dfcabc64fae6aeee9c
* coverity#1372392 Uncaught exceptionCaolán McNamara2016-09-012-3/+2
| | | | Change-Id: Ic8bb4748b0752796fcb76c1816441bb22b896809
* coverity#1372390 Uncaught exceptionCaolán McNamara2016-09-014-5/+4
| | | | Change-Id: I4bb01b0bb9865986d9da8ed294a46fa7a4488458
* coverity#1372389 Uncaught exceptionCaolán McNamara2016-09-012-2/+2
| | | | Change-Id: Ic788073adcaefb3de2bff7e249f17a64fad48b19
* coverity#1372388 Uncaught exceptionCaolán McNamara2016-09-014-7/+4
| | | | Change-Id: I55c4e2ad1f3384c4e8b0a6cc88b83fa96cf2dede
* coverity#1372387 Uncaught exceptionCaolán McNamara2016-09-012-6/+6
| | | | Change-Id: I563e7987d00a68373c05dc4f1f3d62cd3e52de70
* coverity#1372383 Uncaught exceptionCaolán McNamara2016-09-012-2/+2
| | | | Change-Id: Idfdd846d8439be91b059d53dab6591d202576fae
* declare default dtor as virtualCaolán McNamara2016-08-302-12/+1
| | | | Change-Id: Ic77dbe3ffb2d2856d437158914b877b913296e0e
* loplugin:stringconstant: adapt to improved OUStringLiteral1 (connectivity)Stephan Bergmann2016-08-301-1/+1
| | | | Change-Id: I1c9e33d3f4306199320e60cf0eb2970e687576ef
* Let OUStringLiteral1 take its arg as ctor arg, not template argStephan Bergmann2016-08-301-3/+3
| | | | | | | | ...which makes it more flexible, can now also be used on non-const arguments. The drawback of the argument no longer being a compile-time constant is remedied by making the ctor constexpr. Change-Id: Ia4903a2cc86791fece92eac0cb8406b6659dd19d
* Resolves: tdf#81883 dispose OCalcConnection's doc before shutdownCaolán McNamara2016-08-282-7/+77
| | | | | | | | | | | if noone else does it, then ensure the doc is disposed before XDesktop it torn down Change-Id: I3136802a40c9cfb43039307dc65949d3264b6f2b Reviewed-on: https://gerrit.libreoffice.org/28428 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* put StreamMode masks in scope of enum classJochen Nitschke2016-08-273-7/+7
| | | | | | | | Change-Id: I77682f7e289a59b986bb84edf014029a20266470 Reviewed-on: https://gerrit.libreoffice.org/28420 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* DocHolder->DocHolderCaolán McNamara2016-08-252-3/+3
| | | | Change-Id: I8eddff0791a7887e1aee0515e73d40c54b97a0ae
* cid#1371320 Missing move assignment operatorNoel Grandin2016-08-251-0/+17
| | | | | | | Change-Id: I34da999f516abedd65135f67e7479706e64e7b20 Reviewed-on: https://gerrit.libreoffice.org/28376 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Improve exception constructionStephan Bergmann2016-08-251-5/+3
| | | | Change-Id: I2c1d28bb7c841f594fe2fdaf5f34fcdd730d8ee2
* cid#1371770 Uncaught exceptionStephan Bergmann2016-08-251-2/+3
| | | | Change-Id: I147e98d0eef2568c78ea95aee9fa68656f2e8e75
* coverity#1371621 Uncaught exceptionCaolán McNamara2016-08-211-1/+13
| | | | Change-Id: I82e500ce3b6022f7783f645bbcd4e957588f7bae
* coverity#1371622 Uninitialized scalar fieldCaolán McNamara2016-08-211-7/+0
| | | | Change-Id: Id1408665b01889eabe449047f2d8f62229aeeb16
* Some clang-tidy misc-move-constructor-initStephan Bergmann2016-08-191-1/+1
| | | | | | ...by turning the relevant ctor parameters into "const &". Change-Id: Ia8d0aba5da10ad6b25f8689e2281e45b3d71c1fc
* tdf#72987 GSoC Use Firebird backup formatWastack2016-08-182-114/+227
| | | | | | | | | | | Store embedded database files as an archive (.fbk) file. The firebird database file is extracted when opening an odb file, and archived for each saving. Change-Id: I6c985f89a0fb01b2294f728b4581053521ca0c88 Reviewed-on: https://gerrit.libreoffice.org/28045 Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu> Tested-by: Jenkins <ci@libreoffice.org>
* tdf#69949 GSoC Firebird implement autoincrementWastack2016-08-1817-14/+511
| | | | | | | Change-Id: I6fe08b575f01b986f0a3c96b341f254279427b68 Reviewed-on: https://gerrit.libreoffice.org/28062 Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu> Tested-by: Jenkins <ci@libreoffice.org>
* tdf#91324 correct firebird date formatWastack2016-08-171-4/+4
| | | | | | | | | | | Store date inside the embedded firebird database as: Years as the years since 1900. Months from 0 to 11. Change-Id: Ic6269a183261c4c482f91f0d3871b42c54d4b99f Reviewed-on: https://gerrit.libreoffice.org/28173 Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu> Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
* tdf#73073 GSoC Firebird Set autoCommit trueWastack2016-08-171-0/+4
| | | | | | | | | Use autoCommit for embedded Firebird database. Change-Id: I5633960b8e3d4e6ee8f9b4f29406c7208a059521 Reviewed-on: https://gerrit.libreoffice.org/28047 Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu> Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
* GSoC Upgrade firebird to 3.0Wastack2016-08-171-2/+17
| | | | | | | | | | | | | | | | | | Embedded firebird is now version 3.0. Supports MSVC 14.0. Instead of fbembed, there are now two libraries: fbclient and Engine12. fbclient is linked as fbembed before, Engine12 is loaded at runtime from fbclient. fb now needs system libtommath, which is supplied as a new ExternalProject of LO. Change-Id: I132939bdee745795b22f675e4265e9590079c45f Reviewed-on: https://gerrit.libreoffice.org/27642 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
* loplugin:unusedenumconstants in basctl..editengNoel Grandin2016-08-172-17/+0
| | | | Change-Id: I49a23a197969d58a3a2d6b2be1fbe4a647c7fa08
* loplugin:stringstaticStephan Bergmann2016-08-161-4/+4
| | | | Change-Id: Ia5d7ca7864dbef1489294783438b9d718159614b
* new loplugin to check for static OUStringsNoel Grandin2016-08-167-37/+30
| | | | | | | | | that are better declared as OUStringLiteral Change-Id: Ifb5d9a12bb31a68641940bec16971a8181a46567 Reviewed-on: https://gerrit.libreoffice.org/27377 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>