summaryrefslogtreecommitdiff
path: root/scripting
AgeCommit message (Collapse)AuthorFilesLines
2014-03-29coverity#704287 Logically dead codeCaolán McNamara1-43/+40
Change-Id: Iab22d2d12319bc01f9aebc1dcf40f47881d40446
2014-03-27Second batch of adding SAL_OVERRIDE to overriding function declarationsStephan Bergmann11-104/+104
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: Ie656f9d653fc716f72ac175925272696d509038f
2014-03-26First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann7-187/+187
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
2014-03-18Find places where OUString and OString are passed by value.Noel Grandin1-2/+2
It's not very efficient, because we generally end up copying it twice - once into the parameter and again into the destination OUString. So I create a clang plugin that finds such places and generates a warning so that we can convert them to pass-by-reference. Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
2014-03-10scripting: simplify deprecated XTypeProvider.getImplementationIdStephan Bergmann2-3/+2
Change-Id: If0d9a6f1c56dc560df078eb59313aaba393ee372
2014-03-07Introduce com.sun.star.beans.theIntrospection singletonStephan Bergmann2-4/+4
...deprecating com.sun.star.beans.Introspection (single-instance) service. Change-Id: Ica2e3a3541c7dcb1aab79222c5abf40d6988c882
2014-03-04fdo#54938: Convert some places to use cppu::supportsServiceMarcos Paulo de Souza1-8/+2
The last cases are non obvious, so it's pratically done Change-Id: Icae1da8e238f516eaed0f7fbdf96fff778eac547 Reviewed-on: https://gerrit.libreoffice.org/8445 Reviewed-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Tested-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
2014-03-03fdo#54938: Convert some places to use cppu::supportsServiceDonizete Waterkemper1-1/+1
Change-Id: Ib941c6ec82d81b1da815561eee87ee91dc8de200 Reviewed-on: https://gerrit.libreoffice.org/8443 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
2014-02-27Remove unused ScriptingContextStephan Bergmann3-187/+0
Unused and not even built since 11ddfdb8563901bf515fa2eb07fa27244cc58ced/ 4516589f91e03fa5d8589a52ed8b72af8f9e4521 "INTEGRATION: CWS scriptingf2 (1.7.2); FILE MERGED." Change-Id: I06eec282877a91955b3f20109c3c1189b660f2da
2014-02-26cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann32-515/+515
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
2014-02-26Remove visual noise from scriptingAlexander Wilms36-243/+243
Change-Id: Id9f177a20d911ce1e041407aa556c9cf13f0efc8 Reviewed-on: https://gerrit.libreoffice.org/8305 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-02-24Replace deprecated std::auto_ptr with boost::scoped_ptrTakeshi Abe1-1/+3
Change-Id: I55292f5f0049e7c77d5bb0e5ef5fa187b815d159
2014-02-23Remove unneccessary commentsAlexander Wilms11-141/+141
Change-Id: I939160ae72fecbe3d4a60ce755730bd4c38497fb Reviewed-on: https://gerrit.libreoffice.org/8182 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-02-20Remove vcl/settings.hxx from vcl/svapp.hxx and vcl/outdev.hxxTobias Lippert1-1/+1
Added vcl/settings.hxx to all cxx files which require it. This helps to speed up compilation after changes to the settings. Conflicts: sc/source/ui/dbgui/pvlaydlg.cxx Change-Id: I211a0735c47f72d6879f6f15339355abfe0e3cf4 Reviewed-on: https://gerrit.libreoffice.org/7933 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-02-12Typo: "omited"/"ommitted"/"ommited" -> "omitted"Julien Nabet1-1/+1
Change-Id: Icc690b0ae1fc8f7165082774720d265798815faa
2014-02-06typo fixes in commentsAndras Timar1-1/+1
Change-Id: Idd49478d59cd062118fbf8e99d1c8bc5250013fc
2014-02-02fdo#54938 Convert bridges, editeng and others to cppu::supportsServiceAlexandre Vicenzi1-42/+9
Change-Id: I7ff5189473c3e0831c2f1e95264d1a04f3b716a9 Reviewed-on: https://gerrit.libreoffice.org/7761 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Marcos Souza <marcos.souza.org@gmail.com> Tested-by: Marcos Souza <marcos.souza.org@gmail.com>
2014-01-28bool improvementsStephan Bergmann3-5/+5
Change-Id: I0ef2adc0bd4a4764c38094b2b7d27c3453ecc117
2014-01-23Let C++ inline functions return bool instead of sal_BoolStephan Bergmann1-1/+1
...to improve diagnosing misuses of boolean expressions in client code (cf. compilerplugins/clang/implicitboolconversion.cxx). This change should be transparent to client code. Missing overloads of insert() for bool have been added to OStringBuffer and OUStringBuffer (which required dropping one !VALID_CONVERSION check that would now pick that overload, but would be flagged by compilerplugins/clang/pointertobool.cxx). Change-Id: I2d64cd923b8f47bfaa31e753def6515c29a3f8c9
2014-01-22bool improvementsStephan Bergmann3-5/+3
Change-Id: I75c137c1ed0c6089c5dfa8131ffde76cadda0134
2014-01-07remove unnecessary sal_Unicode casts in OUStringBuffer::append callsNoel Grandin1-12/+12
Convert code like: buf.append( static_cast<sal_Unicode>('!') ); to: buf.append( '!' ); Change-Id: Iacb03a61de65a895540940953b49620677b3d051
2013-12-20Spelling correction: s/retrive/retrieve/Tor Lillqvist2-7/+7
Change-Id: I96845d358765e2d2507763a9b15a30388b32bc6b
2013-12-20typo fixesAndras Timar3-3/+3
Change-Id: Ia5f104bfd707bcf4e159c78ca2764c861fb0b6d9
2013-12-17Adapt all (non-extension, SharedLibrary) .components to environment="..."Stephan Bergmann6-12/+12
Change-Id: I56f38bd786f3a026cb2908f28540dc9c4003af83
2013-12-17Remove unnecessary use of OUString constructor in OUStringBuffer::append callsNoel Grandin1-1/+1
Convert code like aStrBuffer.append(OUString(" AS ")); to aStrBuffer.append(" AS "); Change-Id: I8c1884b5a875f40f0b5e511b6ef38c6c8eeee656
2013-12-12Remove unnecessary macrosStephan Bergmann1-30/+27
Change-Id: I3a975ad9975c569b10f77aeee4105dec5c4f8c5e
2013-11-22remove unnecessary RTL_CONSTASCII_STRINGPARAMNoel Grandin2-2/+2
A final pass through the code, converting code to use the new OUString and OString methods that can detect string literals. Change-Id: Ifa6382335e5650a1c67e52006b26354e0692c710
2013-11-19remove unnecessary use of OUString constructor when assigningNoel Grandin5-23/+22
change code like aStr = OUString("xxxx"); to aStr = "xxxx"; Change-Id: Ib981a5cc735677ec5dba76ef9279a107d22e99d4
2013-11-14Move MediaDescriptor from comphelper to unotoolsStephan Bergmann2-4/+5
...so it will be able to use SvtSecurityOptions internally. Change-Id: Id7433247e8fc53651935578510bedbcca5aa2ac9
2013-11-11convert OUString compareToAscii == 0 to equalsAsciiNoel Grandin1-1/+1
Convert code like aStr.compareToAscii("XXX") == 0 to aStr.equalsAscii("XXX") which is both easier to read and faster. Change-Id: I448abf58f2fa0e7715dba53f8e8825ca0587c83f
2013-11-11remove unnecessary use of OUString constructorNoel Grandin1-1/+1
Change-Id: Ifb220af71857ddacd64e8204fb6d3e4aad8eef71
2013-11-11remove unnecessary use of OUString constructor in SCRIPTING moduleNoel Grandin6-21/+20
Change-Id: I87dd775949d1d3cc2c191e84e57b49cd66ecc750
2013-11-06Drop unnecessary #includesTakeshi Abe1-1/+0
Change-Id: I9659279233067a8946a9e54be2f22439854a961e
2013-11-04remove redundant calls to OUString constructorNoel Grandin3-12/+12
Change code like this: aStr = OUString("xxxx"); into this: aStr = "xxxx"; Change-Id: I31cb92e21658d57bb9e14b65c179536eae8096f6
2013-11-04Convert code that calls OUString::getStr()[] to use the [] operatorNoel Grandin1-3/+3
This also means that this code now gets bounds checked in debug builds. Change-Id: Id777f85eaee6a737bbcb84625e6e110abe0e0f27
2013-10-31Convert indexOf->startsWith and lastIndexOf->endsWithNoel Grandin6-7/+7
This is both an optimisation and a cleanup. This converts code like aStr.indexOf("XX") == 0 to aStr.startsWith("XX") and converts code like aStr.lastIndexOf("XXX") == aStr.getLength() - 3 to aStr.endsWith("XXX") Note that in general aStr.lastIndexOf("X") == aStr.getLength() - 1 converts to aStr.isEmpty() || aStr.endsWith("X") so I used the surrounding context to determine if aStr could be empty when modifying the code. Change-Id: I22cb8ca7c2a4d0288b001f72adb27fd63af87669
2013-10-28gbuild: set Package default target to INSTDIRMichael Stahl7-14/+0
Change-Id: I2bc45e4ba63f5faaee7389bcd9d7b3f563503186
2013-10-25stop looking for Jar files in solverMichael Stahl1-1/+1
Change-Id: I4d2a93fa7395354fbf2893df9e254ab39fa365af
2013-10-22Bin comments that claim to say why some header is includedTor Lillqvist1-1/+1
They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
2013-10-22fdo#54938: Adapt supportsService implementations..Marcos Paulo de Souza7-99/+18
to cppu::supportsService Change-Id: I0b03d3910f094f2183bf9859db9d5bcaf35d1f14 Reviewed-on: https://gerrit.libreoffice.org/6370 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2013-10-18fdo#36964 Wording change: "Remove Filter" -> "Reset Filter"Samuel Mehrbrodt1-1/+1
As discussed in the bug report. I changed all occurrences, not only the one in the Calc menu. Change-Id: Ia652cb10ec0123b0a79a719dda59e6d2f54f0680 Reviewed-on: https://gerrit.libreoffice.org/6123 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2013-10-18random OUString to String conversionsNoel Grandin1-1/+1
Change-Id: I9cfb8e7183b9cce7c690f3a43a64b61a2aa8c754
2013-10-02WaE: unused variableTor Lillqvist1-1/+0
Change-Id: If37c7b6a5de8277ecc25538b06197cdf6168878c
2013-10-02-Werror,-Wunused-const-variableStephan Bergmann1-3/+1
Change-Id: I5e00ce258e2dc2b13dc0f7a38f5a92bd1235e81e
2013-09-26typo fixes in commentsAndras Timar1-1/+1
Change-Id: Iaadec33715f8e0e0c6595c5e684606905274fdab
2013-09-23Try to fix cross-compilationTor Lillqvist7-7/+7
Add more FOO_FOR_BUILD variables and some gb_Foo_for_build functions. Get rid of gb_INSTROOT and gb_DEVINSTALLROOT, just use INSTROOT. Change-Id: Iee531b02d14fae41edb68ad589a5dec829a60255
2013-09-12gbuild: remove gb_Rdb__get_final_targetMichael Stahl2-6/+2
... by replacing gb_Rdb_install with a separate constructor so the right target can be registered at the module. There is still an ugly special case for the ure/services. Change-Id: I81c004143f201aaf38daca99819888313ee24f49
2013-09-11Towards a working instdir for Mac OS XStephan Bergmann7-7/+7
Introduced gb_INSTROOT, which is the same as $(INSTDIR) except for Mac OS X, where it is $(INSTDIR)/LibreOffice.app/Contents. Most stuff ends up there (so most occurrences of $(INSTDIR) have been replaced with $(gb_INSTROOT)), but SDK- related stuff goes to $(INSTDIR)/$(gb_Package_SDKDIRNAME). (And GeneratedPackage needed to be made more flexible, to allow for packages that go into either of those two places.) For Android and iOS, gb_INSTROOT probably still needs to be set. The most obvious missing thing yet to make instdir work for Mac OS X is the instdir/*/LibreOffice.app/Contents/ure/ vs. instdir/*/LibreOffice.app/Contents/ure-link/ split. Change-Id: I4478edd27b14c92c96d92d5169bdca3ec50d78f5
2013-09-09gbuild: install rdb files directly in module they come fromMatúš Kukan2-0/+4
Change-Id: I3a9cb4ce71cfb02b7378289a36aa0eb6e3a42f26
2013-08-30ENABLE_SCRIPTING_* Harmonize ENABLE_* variable to TRUE/<nothing>Norbert Thiebaud1-2/+2
Change-Id: I937967889da75062c792cf377ce4e13c67526162 Reviewed-on: https://gerrit.libreoffice.org/5702 Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com> Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>