summaryrefslogtreecommitdiff
path: root/dbaccess
AgeCommit message (Collapse)AuthorFilesLines
2016-10-10tdf#89307: Removed SvRef::operator T*()Jacek Fraczek2-3/+3
Conditional statements are using SvRef::Is() method. Changed static_cast<T*>(svRef<T>) occurances to svRef.get(). Added operator == and != to SvRef. SbxObject::Execute is using SbxVariableRef internally. SbxObject::FindQualified is using SbxVariableRef internally. Change-Id: I45b553e35d8fca9bf71163e6eefc60802a066395 Reviewed-on: https://gerrit.libreoffice.org/29621 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-10-08de-src solo PushButton resourceCaolán McNamara4-8/+6
Change-Id: Id6cd01c8bd4bf9ed36c17fd75ecce34eb7f36cc7
2016-10-06I can't see any evidence of any remaining magenta mask using image resourcesCaolán McNamara4-34/+0
Change-Id: I12865d743daa3e6479edb60584d5bd01f88e818d
2016-10-06ImageList just reads and discards the MaskColor, so drop itCaolán McNamara2-3/+0
Change-Id: I243e739fea28e3f38b91534182cc54c2aa660f0e
2016-10-05now ditch WB_INFOCaolán McNamara1-1/+1
Change-Id: I3b5505e06989c6bb8df5977b6690d32ca5f161e2
2016-10-05loplugin:unnecessaryoverride in dbaccessNoel Grandin23-114/+1
Change-Id: I458092403faa46f80957a55b48374678e196de11
2016-10-05convert MapUnit to scoped enumNoel Grandin22-42/+42
I left a prefix on the names "Map" so that I would not have to re-arrange each name too much, since I can't start identifiers with digits like "100thMM" And remove RSC_EXTRAMAPUNIT, which doesn't seem to be doing anything anymore. Change-Id: I5187824aa87e30caf5357b51b5384b5ab919d224 Reviewed-on: https://gerrit.libreoffice.org/29096 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-10-05merge dbaccess::StorageInputStream with dbaccess::StorageXMLInputStreamNoel Grandin4-47/+16
Change-Id: I05b54b8a2fe4ff0c8b9f89e3d29e864e8665b7b8 Reviewed-on: https://gerrit.libreoffice.org/29519 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-10-05Remove _TYPED suffix from tools/link.hxx macrosStephan Bergmann129-376/+376
...which was introduced with 3ead3ad52f9bb2f9d1d6cf8dfc73a0a25e6778ed "Gradually typed Link" to distinguish the new, typed versions from the old, untyped ones, but is no longer necessary since 382eb1a23c390154619c385414bdbe6f6e461173 "remove untyped Link<>" removed the old versions. Change-Id: I494025df486a16a45861fcd8192dfe0275b1103c
2016-09-25remove unused parameterJochen Nitschke1-1/+1
bTotalRanges is never used. SFX_ITEMSET_NO_DEFAULT_CTOR is no where else used. Change-Id: Ia35ea875f16a8ca04c2173b01074113f1825f565 Reviewed-on: https://gerrit.libreoffice.org/29248 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-09-23perf: eliminate SfxSimpleHint and move to SfxHint, tdf#87101 relatedEike Rathke1-1/+0
There were over 150 places in *::Notify() functions that did some dynamic_cast<SfxSimpleHint*> of which ~98% were unnecessary because the base class SfxHint passed was an SfxSimpleHint anyway. dynamic_cast operations come with quite some cost, so avoid if possible. Specifically for ScFormulaCell::Notify() that created a bottleneck in scenarios where cells were notified that already handled a previous notification. In mass operations doing the dynamic_cast before it could be decided whether having to act on it or not this made 2/3 of all time spent in the Notify() call. To get rid of that rename/move SfxSimpleHint to SfxHint and let classes derive from SfxHint instead of SfxSimpleHint. This comes only with a slight cost that an additional sal_uInt32 is transported in such hints, initialized to 0, but this is neglectable compared to the huge gain. For the rare cases where a Notify() actually expects both, an SfxHint (formerly SfxSimpleHint) and a derived hint, this changed order of the dynamic_cast involved so the simple SfxHint::GetId() is handled last. Modules using such combinations can further optimize by treating the simple SfxHint::GetId() first once verified that none of the other derived hints use an ID not equal to zero respectively none of the ID values the simple hint uses. Change-Id: I9fcf723e3a4487ceb92336189d23a62c344cf0ce Reviewed-on: https://gerrit.libreoffice.org/29205 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2016-09-22convert SIZING constants to scoped typed_flagsNoel Grandin2-41/+39
Change-Id: Ibd504cd7059bcaae0a37c0df25d805efff573b99
2016-09-22convert CARDINALITY constants to scoped enumNoel Grandin3-16/+16
Change-Id: I35cc98086850e62a2ea8703fc4b086ed2adff66a
2016-09-19loplugin:unusedenumvaluesNoel Grandin1-1/+0
Change-Id: I9dcc9f73af6db5b4f7cc946bc28931e5c230b34c Reviewed-on: https://gerrit.libreoffice.org/29012 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-09-15replace sal_Size with std::size_t (or sal_uInt64 for SvStream pos)Michael Stahl3-3/+3
... 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
2016-09-15Make OGenericUnoDialog::destoryDialog non-virtualStephan Bergmann2-9/+3
...it is only ever called: * either from dtors, where the virtual-ness doesn't acutally help (and would trigger loplugin:fragiledestructor if that were enabled) * or, in module filter, from executeDialog members of (final) classes directly derived from OGenericUnODialog without overriding destroyDialog Change-Id: If9a7370385a27952189423e5632b3f01e18ddbb5
2016-09-14loplugin:constantparam in chart2..connectivityNoel Grandin1-1/+1
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>
2016-09-13loplugin:dllprivateStephan Bergmann10-13/+13
Change-Id: I1fe70a39c50aba8b84c117653185fc37dbbfeab0
2016-09-13loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann209-263/+263
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
2016-09-13loplugin:constantparam in dbaccess..editengNoel Grandin29-79/+51
Change-Id: I1d0dc3d89933d86cd229e503b350fccc1523dedc Reviewed-on: https://gerrit.libreoffice.org/28833 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-09-10Typo: (un)kown->(un)knownJulien Nabet1-1/+1
Change-Id: I7455c928293cd8d2d2ff16aa85fe02c9a54959d2 Reviewed-on: https://gerrit.libreoffice.org/28801 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2016-09-09loplugin:constantparam in svtoolsNoel Grandin1-1/+1
Change-Id: I04caae0c9ae621c55e16d3bdc014a4729617feb3 Reviewed-on: https://gerrit.libreoffice.org/28757 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-09-05convert ScrollType to scoped enumNoel Grandin1-2/+2
Change-Id: I6dd02d4f7df028dada6cfd5d767a6ec1b1c1efe1
2016-09-03coverity#1372446 Uncaught exceptionCaolán McNamara2-2/+2
Change-Id: I831c0d2e6449eae76383163f08e7b16c7decc92f
2016-09-02dbaccess: fix loplugin:cppunitassertequals warningsMiklos Vajna2-20/+20
Change-Id: Ia450aa3170a21424fce641d3c8dee43b49f8ef61 Reviewed-on: https://gerrit.libreoffice.org/28609 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2016-09-01coverity#1372405 Uncaught exceptionCaolán McNamara2-2/+2
Change-Id: I9d2029be969d767eb096137017e8a7e646b69d2d
2016-09-01coverity#1372404 Uncaught exceptionCaolán McNamara4-4/+4
Change-Id: Id78404a2fae06004a35c9bf07136bd4806a12e02
2016-09-01coverity#1372403 Uncaught exceptionCaolán McNamara2-2/+2
Change-Id: I9bdb04e55f28c5470e13debde9845c984954092f
2016-09-01coverity#1372401 Uncaught exceptionCaolán McNamara4-4/+4
Change-Id: I9624eeacbe8dc32fd22560737c3d69cf022b2724
2016-09-01coverity#1372400 Uncaught exceptionCaolán McNamara2-2/+2
Change-Id: I2319602a3b04bc0c5a6d4082ac62f98f7a697cf8
2016-09-01coverity#1372395 Uncaught exceptionCaolán McNamara4-4/+4
Change-Id: Ia384d4f7186e3ea870a6bb06fb8227ae0321715e
2016-09-01coverity#1372393 Uncaught exceptionCaolán McNamara4-4/+4
Change-Id: I310fb547b81d89e9000760f6628e07d2e5c7effc
2016-09-01coverity#1372391 Uncaught exceptionCaolán McNamara2-2/+2
Change-Id: I62358c3705c437dfe1e3e370a2cc857a21144ca2
2016-09-01coverity#1372385 Uncaught exceptionCaolán McNamara4-4/+4
Change-Id: Ic8a33ef10d627bc5868a77aa7f14b46682d735d2
2016-09-01coverity#1372382 Uncaught exceptionCaolán McNamara4-4/+4
Change-Id: I7c3e11542d2b82af212d2f52bc1b66f0e6ce187c
2016-09-01std::list<sal_Int32> -> std::vectorNoel Grandin2-9/+10
Change-Id: Ie1d062bd4dabf7ca7de08193597225462447b076
2016-08-30Translate some German comments and messagesMaarten Bosmans1-1/+1
Change-Id: Id15e1afd991f3476e260ba40a8c45c7261113577 Reviewed-on: https://gerrit.libreoffice.org/28493 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Jenkins <ci@libreoffice.org>
2016-08-30convert DbGridControl::Option to scoped enumNoel Grandin2-9/+9
Change-Id: I160fb53b585bb7eb04348dd208ec5c705d595279
2016-08-26convert SV_ITEM_ID_LBOX defines to scoped enumJochen Nitschke8-14/+14
and remove unused SV_ITEM_ID_EXTENDRLBOXSTRING Change-Id: Ic84d9341d0225b01b9ef46814483c66b1af307b5 Reviewed-on: https://gerrit.libreoffice.org/28397 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-08-25convert DataAccessDescriptorProperty to scoped enumNoel Grandin6-73/+73
Change-Id: Ifb8fb1d28464f0638ec1485d77ad333b1acc667c
2016-08-19Resolves: coverity#705366 Mixing enum typesCaolán McNamara1-1/+1
and coverity#705367 Mixing enum types coverity#705371 Mixing enum types coverity#982694 Mixing enum types coverity#1027717 Mixing enum types coverity#1371228 Mixing enum types coverity#1371242 Mixing enum types coverity#1371280 Mixing enum types coverity#1371310 Mixing enum types MapUnit and SfxMapUnit share the same values and are freely cast from one to the other. Now that commit d30a4298bdb5ba53cd1fe659f2b742f218a2e527 Date: Thu Aug 11 15:02:19 2016 +0200 loplugin:unusedenumconstants in package..svtools removed the SfxMapUnit entries that were directly unused, they don't match anymore and casting from one to the other is dangerous. Why there was two of these anyway escapes me, get rid of SfxMapUnit and just use MapUnit universally Change-Id: I4db5dcd04b59be2f85b62b728f96c90afe00c57e Reviewed-on: https://gerrit.libreoffice.org/28234 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-08-18cppcheck: passedByValueCaolán McNamara1-1/+1
Change-Id: I3ffcfe1294a1cbb06aadd3a1a79e9259876e38cc
2016-08-18screenshots: add new global make targetArmin Le Grand1-1/+2
Up to now the screenshot creation was added/dependent of target slowcheck. Since quite some modules have added screenshot creations now, I added an own target 'screenshot' to allow to keep current slowcheck and screenshot creation separated Change-Id: I80a49a0db607edf8e0405672d570f624d29912e7
2016-08-18screenshots: unify dumping of dialog test batch fileArmin Le Grand1-74/+2
Change-Id: Ia1e79216d9537c65f8b7362778a2e1ad3ec6c2c2
2016-08-18screenshots: add dialog test cases for dbaccessArmin Le Grand4-0/+323
One dialog makes the execution hang, seems to wait for some interaction Change-Id: I68ef4f9b2a1575a6c70238f82eb27ee87aaae336
2016-08-17loplugin:unusedenumconstants in basctl..editengNoel Grandin4-15/+0
Change-Id: I49a23a197969d58a3a2d6b2be1fbe4a647c7fa08
2016-08-16loplugin:stringstaticStephan Bergmann1-6/+4
Change-Id: I00cb0310c90601472472c570e6c3452605258059
2016-08-15tdf#89329: use unique_ptr for pImpl in odbcconfigArnold Dumas2-2/+1
Change-Id: Ieba9b60dfa8e61d41a18096e1bddc3c8c70b2e2f Reviewed-on: https://gerrit.libreoffice.org/28113 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-08-15tdf#89329: use unique_ptr for pImpl in dlgsaveArnold Dumas2-4/+5
Change-Id: I3743637b2ad5747073e61a46cdb28604dd9fb95f Reviewed-on: https://gerrit.libreoffice.org/28114 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-08-11tdf#96015 (part) links to Askbot in the Help menuOlivier Hallot5-0/+5
Add the entry "~Get help online" to the modules Help menu. on click, the system browser opens http://hub.libreoffice.org/forum/?&LOlang=<lang> <lang> is obtained from .getLocale() That in turn will redirect to e.g. askbot (but only few languages are using it at the moment) or other nl-specific resources. (nl-projects can request corresponding targets from infa, without the need to change the code – cf rdm#1983) The link can/should be extended to also include the version of LO and the currently used module to allow finer grained redirects. Change-Id: I56108dba4dd2684405623c4586103fc8ff9bea56 Reviewed-on: https://gerrit.libreoffice.org/27028 Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> Tested-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-04-28 06:19:36 +0000