summaryrefslogtreecommitdiff
path: root/comphelper
AgeCommit message (Collapse)AuthorFilesLines
2016-03-03loplugin:unuseddefaultparams comphelperNoel Grandin5-17/+9
Change-Id: I78280a13fc0d82fc87041b5dad0c3f2f7d462652 Reviewed-on: https://gerrit.libreoffice.org/22853 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-01OSingletonRegistration is dead nowMatúš Kukan1-14/+0
Change-Id: I64a5ffa8790496bd39ba26e0c789db9eedc51071 Reviewed-on: https://gerrit.libreoffice.org/22788 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Matúš Kukan <matus@libreoffice.org>
2016-02-24convert HighlighterLanguage to scoped enumNoel Grandin2-9/+9
Change-Id: Ibf0871601e52ea4fa49e9622176431943c55abdd
2016-02-24convert TokenTypes to scoped enumNoel Grandin2-32/+37
Change-Id: I17c0a616dd6cf48a22896b6cd6b0df157d1f9a9f
2016-02-24remove unused exc_handling enumNoel Grandin1-21/+12
Change-Id: I5e2e084114c8b0eedd0f2cd8327d6c6d68742462
2016-02-24convert PropertyDescription::LocationType to scoped enumNoel Grandin1-13/+13
Change-Id: Ifcd6bf1728c632ed10301c4a426dae57dbf0912a
2016-02-24convert DocPasswordRequestType to scoped enumNoel Grandin1-2/+2
Change-Id: I25d217ca2cf3e8cc4dfaccc6dc7d5453ffdc6cb6
2016-02-24convert PropertyOrigin to scoped enumNoel Grandin1-5/+5
Change-Id: I26c9929be8aad02030722508334e66f5028ffb37
2016-02-24convert DocPasswordVerifierResult to scoped enumNoel Grandin1-10/+10
Change-Id: Ib5a595819dec8511d284ed1022d8f7c624f8d8f9
2016-02-24convert EConfigurationModes to scoped enumNoel Grandin1-8/+8
Change-Id: I1e81c8d637e738f536f7efad8b67d0c9183e6483
2016-02-24convert ComponentMethodGuard::MethodType to scoped enumNoel Grandin1-1/+1
Change-Id: I1f1b4981c70c2b8ff8087a85f0af28b816b4654a
2016-02-24tdf#74608: Constructor function for SimpleLogRing singletonyeliztaneroglu5-46/+27
Change-Id: Ia8c2f214b635114ecac4a2ceb06628a2f18b6411 Reviewed-on: https://gerrit.libreoffice.org/22020 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-02-23new loplugin: commaoperatorNoel Grandin1-1/+1
Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
2016-02-17Typo: namesapce(s)->namespace(s)Julien Nabet1-1/+1
Change-Id: Ic7c14c2e39a5ade1f5622a8350f9197d84cf9cc8
2016-02-17use consistent #define checks for the Windows platformNoel Grandin1-1/+1
stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-11[API CHANGE] add operator==/!= to UNO structsNoel Grandin1-21/+0
this is useful now that we are storing UNO structs in std::vector Change-Id: Ic558bcd669bd2b3cdf9eb8393269eb906ac52369 Reviewed-on: https://gerrit.libreoffice.org/22257 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-09sequence->vector in OFOPXMLHelper_ImplNoel Grandin1-19/+15
Change-Id: I6c5806fefc57d82124652ecbb5bfa8e109f9a255 Reviewed-on: https://gerrit.libreoffice.org/22181 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-09sequence->vector in OPropertyChangeMultiplexerNoel Grandin1-5/+3
Change-Id: I4daa06fe14971ae34e089620ba07818d998764f0 Reviewed-on: https://gerrit.libreoffice.org/22179 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-09Remove excess newlinesChris Sherlock31-96/+0
A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-09sequence->vector in OPropertyArrayAggregationHelperNoel Grandin1-98/+23
Change-Id: Ib6ad60495736ea6da4612d3e7591b514465069a2 Reviewed-on: https://gerrit.libreoffice.org/22180 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-09sequence->vector in PropertyMapImplNoel Grandin1-16/+16
Change-Id: Id42d6d739f402c725325816c506caa369267ec8e Reviewed-on: https://gerrit.libreoffice.org/22178 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-08loplugin:unusedmethodsNoel Grandin3-127/+0
using an idea from dtardon: <dtardon> noelgrandin, hi. could you try to run the unusedmethods clang plugin with "make build-nocheck"? that would catch functions that are only used in tests. e.g., i just removed the whole o3tl::range class, which has not been used in many years, but htere was a test for it... <noelgrandin> dtardon, interesting idea! Sure, I can do that. Change-Id: I5653953a426a2186a1e43017212d87ffce520387 Reviewed-on: https://gerrit.libreoffice.org/22041 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-06sax: Use comphelper's AttributeListMatúš Kukan1-6/+8
Change-Id: I6eb0115dba8797025a513cf1e6cfbfbe0b6f7e50
2016-02-02boost::hash->std::hashNoel Grandin1-1/+0
Change-Id: If4d1e4071995f07212fad958b0226d5824d168f8 Reviewed-on: https://gerrit.libreoffice.org/21989 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-01tdf#74608 comphelper: Constructor feature for AnyCompareFactorykripton4-29/+8
Change-Id: If5d35828a7309b6916de65282e32ce1ad459a15f Reviewed-on: https://gerrit.libreoffice.org/21936 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-01-31tdf#74608 comphelper: Constructor feature for NamedPropertyValuesContainerkripton4-30/+8
Change-Id: I63390eca5c887bfd84846655cef6339ba9785e24 Reviewed-on: https://gerrit.libreoffice.org/21888 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Matúš Kukan <matus@libreoffice.org>
2016-01-31OSL_THIS_FUNC is a rip-off of BOOST_CURRENT_FUNCTION so we can use thatCaolán McNamara3-8/+2
as a direct drop in I guess Change-Id: I3add63f1459f4e659019bd6db54da2f5431958ce Reviewed-on: https://gerrit.libreoffice.org/21941 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-01-30tdf#74608 comphelper: Constructor feature for IndexedPropertyValuesContainerkripton4-31/+8
Change-Id: I0e9ea9e4c750918b33040dad2279fe55facd4cab Reviewed-on: https://gerrit.libreoffice.org/21858 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Matúš Kukan <matus@libreoffice.org>
2016-01-29sequence->vector in OInteractionRequestNoel Grandin1-5/+3
Change-Id: I995cb96e514e1aaa05a96f29344ef51e4ca83c64
2016-01-29sequence->vector in OSimpleLogRingNoel Grandin2-5/+6
Change-Id: Ief35ce33a11c93a4a78e50ccdd936ec7e17102a2
2016-01-29sequence->vector in AttachedObject_ImplNoel Grandin1-15/+9
Change-Id: I457de7e83554fbea0a9c5a50c628664fa3887df4
2016-01-25cleanup OInterfaceContainer2 unit testsNoel Grandin2-85/+34
remove unused code and actually hook up the tests in testifcontainer.cxx Change-Id: I9694ebe44f1e8eed3d72373d62424345f899c8e6
2016-01-25loplugin:overrideStephan Bergmann1-2/+2
Change-Id: I9a67ed46a67be521d00ed2368b4820737dcb15e5
2016-01-25loplugin:cstylecastStephan Bergmann1-6/+6
Change-Id: Ie2aeedac1b3fd90d6302be6e5917e99a782d76c7
2016-01-25InterfaceContainer2 with vector instead of SequenceNoel Grandin13-21/+715
create an InterfaceContainer2 class to replace InterfaceContainer. It uses a std::vector instead of a Sequence for the mutable listener list, which provides far better performance. Switch all our internal use-sites to the new class. Change-Id: I6b56cfa511ded2395faa22e68fab3b2f16c3cb88
2016-01-13tdf#97108: SwDocShell of clipboard document has no base URLMichael Stahl1-1/+2
... which triggers this assert; have to downgrade it to SAL_INFO. The startsWith("0x") check is obsolete anyway since commit 0f02bc189b6e89ca2e62d4bd6a2f407a9fd4ec3b. Change-Id: If47bc4496852ff85ba1bbeb4205ffc16be0782de
2016-01-11loplugin:unusedmethods unused return value in comphelperNoel Grandin2-5/+3
Change-Id: If95e3fa61a2c63c4ffc78ef5c27ea5c59088170d
2016-01-09crashtesting: fdo85994-1.doc assert on reexport to docCaolán McNamara1-5/+11
try setting DefaultParentBaseURL based on the parent BaseURL for objects created this way Change-Id: I1a660ff6e4874321e0308d556c7ce0e10a2605fe
2016-01-07Fix typosAndrea Gelmini5-7/+7
Change-Id: I90b04b8eda6fc3d530c9db72052720cbe9de0343 Reviewed-on: https://gerrit.libreoffice.org/21197 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2016-01-05new loplugin rangedforcopy - use reference in range based forAndrzej Hunt1-1/+1
Inspired by 6e6ae9803796b120e95f6e89575e03c5fd0ed3c2 Change-Id: Ia0f264d3a6bbf076aa5080e3398683e50bc6ef01 Reviewed-on: https://gerrit.libreoffice.org/20190 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2015-12-27tdf#74608: Constructor function for OfficeInstallationDirectories singletonMatúš Kukan5-63/+33
Change-Id: Ia0de503c50b6b1d568df27f6a8139fdcc8d0cfbf
2015-12-27tdf#74608: Constructor function for OfficeRestartManager singletonMatúš Kukan5-45/+29
Change-Id: I6c1383c292418b6744dacee95d4f289a6ca1c781
2015-12-24crashtesting: fdo73974-1.doc assert on reexport to docCaolán McNamara1-2/+7
try setting DefaultParentBaseURL based on the parent BaseURL for objects created this way Change-Id: Idfc44d90e4f73f23976e8648c504786955b4fce6
2015-12-24OModule::onFirstClient is unusedNoel Grandin1-9/+4
also move the MutexGuard in revokeClient. No need to hold it while doing an osl_atomic_increment. Save it until we actually need to call onLastClient. Change-Id: Ia87f24b66b0430604a2fe717c95223ff1f5ece6d Reviewed-on: https://gerrit.libreoffice.org/20919 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-12-23loplugin:unusedfields in chart2Noel Grandin3-9/+5
Change-Id: I8203f84ad8b138932c990032e17ea5d2daf384bb
2015-12-21loplugin:unusedmethodsNoel Grandin7-64/+0
Change-Id: Ifafdaf6da0225f244853a0042a6458643b570623
2015-12-15sw: set a BaseURL when copying embedded objectsMichael Stahl1-1/+2
Turns out that SfxObjectShell::CreateShellID() will produce the SfxMedium's base URL if it exists, so EmbeddedObjectContainer::CopyAndGetEmbeddedObject() already has parameters, sw just needs to set them. Change-Id: I36cedfde4e7c2e25c43c66a30d8ca572f099ad69
2015-12-15fix missing BaseURL when loading embedded objectsMichael Stahl1-6/+18
When the object is edited in the UI, the m_xClient is set to a SfxInPlaceClient and the DocumentBaseURL is retrieved from it. But if the object is not edited, it will be loaded during export via the API and without a m_xClient; in this case the DocumentBaseURL must have been set previously to be available during import. There appears to be no way to get the URL of the document via the API while it is being imported; SfxBaseModel's m_sURL is unfortunately only initialized from SfxObjectShell::FinishedLoading(). During ODF import, the SvXMLEmbeddedObjectHelper creates the embedded object, so let's make it pass in the parent's BaseURL. The "DefaultParentBaseURL" parameter already exists but was unused previously. Change-Id: I3d1ed29b3a2c0e77ec606a1d09f7bc07e7860733
2015-11-26That SB variable was a temporary debugging aidStephan Bergmann1-2/+4
...inadvertently checked in with 7bcf64c29a0f5d3d2d60fe98afadab6e48dbe8b3 "Add isReadOnly() support to simplified configuration access" Change-Id: I9b8b477f682605fcdcf65f8e1255524d010495be
2015-11-26mark UNO structs as SAL_WARN_UNUSED, where possibleNoel Grandin1-5/+2
Change-Id: Ie3de518f60c9f1313c68df54dbdc1fb2804f1f0d