summaryrefslogtreecommitdiff
path: root/javaunohelper/source
AgeCommit message (Collapse)AuthorFilesLines
2017-12-11loplugin:salcall fix functionsNoel Grandin1-1/+1
since cdecl is the default calling convention on Windows for such functions, the annotation is redundant. Change-Id: I1a85fa27e5ac65ce0e04a19bde74c90800ffaa2d Reviewed-on: https://gerrit.libreoffice.org/46164 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-10-23loplugin:includeform: javaunohelperStephan Bergmann7-36/+36
Change-Id: If8f65b5942fd41c7023a14f9e321568a0d6c7946
2017-10-04add << operator for css::uno::ExceptionNoel Grandin1-2/+2
Change-Id: Ia23dafd07133779144965682df3b7125a3214235 Reviewed-on: https://gerrit.libreoffice.org/43046 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Jenkins <ci@libreoffice.org>
2017-05-09cleanup osl/diagnose.h includesJochen Nitschke1-1/+0
with command > git grep -l osl/diagnose.h *.cxx | xargs grep -L -w 'OSL_\w*' | xargs sed -i '/#include *\(<\|\"\)osl\/diagnose.h\(>\|\"\).*/d' headers need more work Change-Id: I906519ebbd47a04703b4fa5943b2f7abea7a97ab Reviewed-on: https://gerrit.libreoffice.org/37350 Tested-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2017-05-07revert OSL_ASSERT changesChris Sherlock1-2/+2
Change-Id: I365d140446bd2a62cf8256acbfdd53fe72987380
2017-05-07tdf#43157: convert javaunohelper from OSL_ASSERT to assertChris Sherlock1-2/+2
Change-Id: Ia000b8e1406954cc9c65a9c4e2ad25f656db0dbd
2017-01-31teach lolugin:stringconstant about calling constructorsNoel Grandin1-6/+2
so we can remove unnecessary calls to the OUString(literal) constructor when calling constructors like this: Foo(OUString("xxx"), 1) Change-Id: I1de60ef561437c86b27dc9cb095a5deb2e103b36 Reviewed-on: https://gerrit.libreoffice.org/33698 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-26Remove dynamic exception specificationsStephan Bergmann1-6/+2
...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-12-13OSL_TRACE->SAL in framework..salNoel Grandin1-2/+2
Change-Id: I9a897af88aa9f6f7ca98ce521c69b5a4ee8462e9 Reviewed-on: https://gerrit.libreoffice.org/31903 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2015-11-24cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
2015-11-17Don't assume sal_Unicode is unsigned shortStephan Bergmann2-3/+3
Change-Id: I083a33e71d1bb9fcef3b287e1db15910779312ab
2015-11-11coverity#1338592 Explicit null dereferencedCaolán McNamara1-4/+2
Change-Id: Ief5c2effbddebe29fc7a4d38ed0af4fed72de40e
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann3-21/+21
Change-Id: I89aa17e66c502e81089f42578b45097f44de4ea9
2015-11-09new loplugin: oncevarNoel Grandin1-8/+4
Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-30UNO: no need to use OUString constructor when calling createInstanceNoel Grandin1-2/+2
Change-Id: I37da28539b94706574116d0fff5f008aabfb5526 Reviewed-on: https://gerrit.libreoffice.org/19682 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-23com::sun::star->css in lingucomponentNoel Grandin1-2/+2
Change-Id: Iceef10c51db88b3e85f118a366de45f0f70bbf21
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann1-3/+3
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-09-15Resolves: tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe1-2/+2
with the variadic variants. Change-Id: I7154f9472f02fdf47d27ba715db55bb1ec669a8a Reviewed-on: https://gerrit.libreoffice.org/18580 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-26loplugin:stringconstant: handle OUString+=OUString(literal)Stephan Bergmann1-1/+1
Change-Id: I3561a665cffbdf063c5a8de1225f4ddefd71718f
2015-05-08jboolean-related clean-upStephan Bergmann1-3/+3
Change-Id: Iecd03d0343b9b7c3a8b23b5a1e9654148b94d44c
2015-04-27More loplugin:simplifyboolStephan Bergmann1-1/+1
Change-Id: I4749145f12a8f22bc8cd13722ad38096a2c4720e
2015-04-01Replace remaining getCppuType et al with cppu::UnoTypeStephan Bergmann1-1/+1
Change-Id: Ib1b59e56a32c5675fc728624584fb16b79f03469
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann2-4/+4
Change-Id: Ieb830d8216badd75899666f4db565a83e37d11fc
2015-01-20Some more loplugin:cstylecast: javaunohelperStephan Bergmann3-15/+15
Change-Id: If0728af19ba48896678cc3196c922185e403a885
2015-01-07javaunohelper: missing includeMiklos Vajna1-0/+1
Change-Id: Ie8427e9ad7a2a03d6698ee95ce0f11fb458c00e3
2015-01-07WaE: -Werror=unused-variable with #define DISABLE_DYNLOADINGCaolán McNamara1-8/+7
and also a leak in that mode with a missing ReleaseStringChars after GetStringChars in one use, and a use after free in the other use where ReleaseStringChars is called Change-Id: I2da0310429daf8519554fd7aaf47bba9f6f0e85c
2014-12-15silence coverity#705667 Resource leakCaolán McNamara1-9/+7
still leaks on usual success case of course Change-Id: Ia6e0f61b5a08271c03690bbb1c0af59081bea663
2014-10-30coverity#705665 osl_Module->osl::ModuleCaolán McNamara1-11/+13
and coverity#705666 Resource leak still actually fails to dlclose module, but explicitly does that Change-Id: Idbf5608b052e3a7c5a465a73d8ad067a0a9494c5
2014-09-29loplugin: cstylecastNoel Grandin1-2/+2
Change-Id: I53b69a488c70769cbb841db519bc28fd211dc087
2014-05-23Prefer cppu::UnoType<T>::get() to ::getCppuType((T*)0) part20Julien Nabet1-5/+5
Change-Id: If87cdfb2c605254f6d69baa4ca5aec09091caa68
2014-05-23remove boilerplate in UNO Exception constructor callsNoel Grandin2-10/+4
Now that we have default values for Exception constructor params, remove lots of boilerplate code. Change-Id: I620bd641eecfed38e6123873b3b94aaf47922e74
2014-04-23javaunohelper: sal_Bool->boolNoel Grandin1-2/+2
Change-Id: I60650a10cc561046c411791aa85cdb8956aa61ed
2014-03-26First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann1-3/+3
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
2014-02-26cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann1-4/+4
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
2014-02-26Remove visual noise from javaunohelperAlexander Wilms2-4/+4
Change-Id: Ib359d536070456c1ef403642eba7913a834d365d Reviewed-on: https://gerrit.libreoffice.org/8275 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-02-23Remove unneccessary commentsAlexander Wilms1-1/+1
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>
2013-12-16Clean-up uno/lbnames.hStephan Bergmann2-0/+2
Change-Id: I4bd729499aa8be58f04194656e35c1f79d5d4919
2013-05-06cleanup whitespaces due to RTL_CONSTASCII_USTRINGPARAM removalPhilipp Riemer1-9/+4
In e2e2cc61144cb22227eebfadff0ea24b51ccfbd0 the method was removed automatically leaving several line breaks etc. as visual noise.
2013-05-06remove usage of RTL_CONSTASCII_USTRINGPARAMLuboš Luňák2-7/+6
Mechanical removal of usage together with OUString ctor, done by compiler plugin. Change-Id: I554227f76df0dac620b1b46fca32516f78b462c5
2013-04-18Let juhx.dll export undecorated symbols on WindowsStephan Bergmann5-19/+112
...otherwise preload.cxx in juh.dll will not be able to find them via osl_getFunctionSymbol. What apparently happens is that JNICALL expanding to __stdcall decorates symbol names with _...@NN. This likely was hidden in pre-gbuild times thanks to the use of def files. (On a side note, the JVM appears to contain special code to find syms for native methods in both decorated and undecorated form; that explains why it picks up the decorated symbols from juh.dll just fine.) There is no need for the functions in juhx.dll (called from the juh.dll wrapper) to adhere to JNICALL (in fact, things would likely be easier to maintain if the juhx.dll functions also used different names than their juh.dll wrappers). However, what complicates this patch is that for DISABLE_DYNLOADING, the juh wrapper and its preload.cxx is elided, and the code that would normally go into the juhx library goes into the juh library (and thus needs to stick to JNICALL, and also needs to use the right function names). Change-Id: I66611648f1f79f57f0c1b23fb7a801da2d7b86c5
2013-04-11Remove createRegistryServiceFactory preload left-overStephan Bergmann1-22/+0
...forgotten from f3051abde54e7a0bc3e2d184e31ac011e96d3e6b "API CHANGE: Remove deprecated cppuhelper/servicefactory.hxx and Java equiv." Change-Id: Iccdcb68c6051a64a09ae7918eecb8d3e2c53a085
2013-04-07mass removal of rtl:: prefixes for O(U)String*Luboš Luňák4-16/+11
Modules sal, salhelper, cppu, cppuhelper, codemaker (selectively) and odk have kept them, in order not to break external API (the automatic using declaration is LO-internal). Change-Id: I588fc9e0c45b914f824f91c0376980621d730f09
2013-04-02drop prefix from ::cssThomas Arnhold1-1/+1
as css is already ::com::sun::star Change-Id: I86b43843e4b74f990b6e05dee37184a002072d12
2013-03-19automated removal of RTL_CONSTASCII_USTRINGPARAM for quoted OUStringsThomas Arnhold1-8/+4
Done with a perl regex: s/OUString\s*\(\s*RTL_CONSTASCII_USTRINGPARAM\s*\((\s*"[^")]*?"\s*)\)\s*\)/OUString\($1\)/gms Change-Id: Idf28320817cdcbea6d0f7ec06a9bf51bd2c3b3ec Reviewed-on: https://gerrit.libreoffice.org/2832 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2013-01-30Removed several useless macros: A2OU, A2S, C2U, C2S, OUSTR, OUSTRINGJean-Noël Rouvignac3-18/+12
Change-Id: Ie859cb2dfdc7103c379fce56be88eef8fe390afd Reviewed-on: https://gerrit.libreoffice.org/1924 Tested-by: Luboš Luňák <l.lunak@suse.cz> Reviewed-by: Luboš Luňák <l.lunak@suse.cz>
2012-12-21API CHANGE: Remove deprecated cppuhelper/servicefactory.hxx and Java equiv.Stephan Bergmann1-91/+0
... com.sun.star.comp.helper.RegistryServiceFactory (juh.jar). Superseded by default bootstrap mechanisms. An aborting stub for non-inline cppu::createRegistryServiceFactory is left in cppuhelper/srouce/compat.cxx to avoid having to incompatibly change cppuhelper/soruce/gcc3.map. Change-Id: I590e50b8f57e86d4bb3e00d157c9e5907c02f267
2012-11-30c++ API: use css alias in generated headers, adds global css declThorsten Behrens1-2/+0
This changes all generated API headers (.hpp and .hdl) to use a namespace alias 'css' instead of the pointlessly long com::sun::star Makes the change in cppumaker & associated tools, adds a global namespace alias definition in sal/types.h, and removes a kiloton of local, now pointless-to-harmful versions of that alias from all over the code. Change-Id: Ice5a644a6b971a981f01dc0589d48f5add31cc0f
2012-11-09Introdude SAL_JNI_EXPORT and use that instead of JNIEXPORTTor Lillqvist2-4/+4
Use it in the cases where I yesterday changed SAL_DLLPUBLIC_EXPORT to JNIEXPORT. It turns out that on Linux JNIEXPORT does not enforce "default" visibility, but expands to empty. Change-Id: I033b3cf538715fb596e965e17f3da12fb987df63
2012-11-09Use JNIEXPORT to really export these for AndroidTor Lillqvist2-4/+4
Now with DISABLE_DYNLOADING, SAL_DLLPUBLIC_EXPORT actually means hidden visibilty. Which is OK in general as with a single DSO (or a single executable, for iOS), none of our "normal" entry points need to be visible froom the outside. So for the JNI entry points use JNIEXPORT. On "normal" platforms it should be equivalent to SAL_DLLPUBLIC_EXPORT. Change-Id: Iad634950e635ac03a0e90cae6d00afd9fb4eeb64
2012-10-07DISABLE_DYNLOADING tentative fixTor Lillqvist1-2/+20
Change-Id: Ic23dcc0b45a90e845fc4fba1861bcee03354da01
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-24 04:09:33 +0000