summaryrefslogtreecommitdiff
path: root/bridges
AgeCommit message (Collapse)Author
2014-12-11Fix JNI calls to AsynchronousFinalizerStephan Bergmann
...make sure the class is actually found, etc. Change-Id: I5459d531be39b07594a975ae708a7611d1667a2f
2014-12-11Store JNI_info in JniUnoEnvironmentData instead of BridgeStephan Bergmann
...to have it available during JNI-UNO's uno_initEnvironment (see next) Change-Id: I7a2f27b512fc74f418b4648d92dafbf0304eaa96
2014-12-11better documentationStephan Bergmann
Change-Id: Ife9a98cfe2166ccc7aac3904c7be4ea71443d857
2014-12-11Move JniUnoEnvironmentData into its own include fileStephan Bergmann
Change-Id: I8e6fb1fc0acff781dd6e6b62018c7ccd5d0e2307
2014-12-11Rename Context -> JniUnoEnvironmentDataStephan Bergmann
Change-Id: I97879d250ed0ed20d5e129ff3af6dbc2f5759078
2014-12-11bridges: fix gcc3_linux_arm buildMiklos Vajna
Change-Id: I3bfaeab9dd9f2d8cd603c655ec3aa7c4f508c673
2014-12-11reduce scope of local variablesMichael Weghorn
This addresses some cppcheck warnings. Change-Id: I1122494e295af756ef3cc32717fe204505aeb9e3 Reviewed-on: https://gerrit.libreoffice.org/13335 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-11java: reduce visibility of fields and methodsNoel Grandin
found by PMD Change-Id: Id6737916b68ccbdbdeec5d314747a38410923ac6 Reviewed-on: https://gerrit.libreoffice.org/13409 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-10java: improve use of WrappedTargetException to set cause properlyNoel Grandin
since we introduced the new constructors that pass the cause all the way up to java.lang.Throwable. Also simplify some exeception printing sites, because Throwable will correctly print out child exceptions for us. Change-Id: Ibbecce3c6f971fbc80d6de2052ab4f33a4503c0a
2014-12-10Typo: nuber=>numberJulien Nabet
Change-Id: Idc87ab05f817a21e491537a1fa4c014c5a313bf6
2014-12-09Uno bridge: Apply debug info setting to noopt codeDavid Ostrovsky
Change-Id: Id949718baf77bb7e2d276d3db08f68149c114796 Reviewed-on: https://gerrit.libreoffice.org/13364 Reviewed-by: David Ostrovsky <david@ostrovsky.org> Tested-by: David Ostrovsky <david@ostrovsky.org>
2014-12-09Fix base+offset pointer calculations when synthesizing exception dataStephan Bergmann
...but the code does not yet work: Care must be taken to allocate the various data structures close enough together so that the offset calculations at the four places now marked with assert(...); //TODO actually succeed. Change-Id: I1fedf7d2d3cdde5035842b4ad5eca9ad9ccf2d44
2014-12-09Fix mscx_filterCppException base+offset pointer calculationsStephan Bergmann
Change-Id: I9fbe8d7eba181cbfcab704761e6feebd78120644
2014-12-09rhbz#1036877: Join Java AsynchronousFinalizer thread well before exitStephan Bergmann
AsynchronousFinalizer was originally added as 870a4401c05beec3d31c1f6055a64591edd0a9d9 "INTEGRATION: CWS mtg1: #i57753# Avoid long-running finalize methods" referring to <https://issues.apache.org/ooo/show_bug.cgi?id=57753> " Fix JNI-UNO bridge so that the JVM doesn't run out of memory when a destructor locks the SolarMutex." It is unclear to me how relevant "If JVMs are getting more mature and should no longer have problems with long-running finalize methods, this class could be removed again" really is in practice. After all, advice on hotspot-gc-devel is to avoid finalize() if possible (<http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2014-June/010215.html> "Re: History of finalizer execution and gc progress?"). So stick with this approach of home-grown draining for now (where a home-grown approach using PhantomReferencens would need a dedicated draining thread, too, so would not have much benefit over the existing code in practice). Timely termination of AsynchronousFinalizer threads is achieved by using a dedicated thread per bridge and joining it in the remote bridge's dispose() resp. the JNI environment's new java_env_dispose. Change-Id: Idcef2dbf361a1de22f60db73828f59e85711aea7
2014-12-08Mark classes as noncopyableStephan Bergmann
Change-Id: I4262fb56d681c70a089638b876dc07b1f472f583
2014-12-03Fold URE: WindowsStephan Bergmann
...assuming the delayLoadHook in cli_ure/source/native/native_bootstrap.cxx is no longer necessary and loading of cppuhelper from the program dir cannot fail regardless in whatever scenario the cli_cppuhelper library itself is loaded. Change-Id: I13f32b327bca4cce9780864f5e57cdad3860afe5
2014-12-02except.cxx: Fix pTypeDescr is unknown in this context errorDavid Ostrovsky
Change-Id: Ic47f69b01cf17a55901e9e3541419d9f477d9585 Reviewed-on: https://gerrit.libreoffice.org/13210 Tested-by: David Ostrovsky <david@ostrovsky.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2014-12-02MSVC 2013: Compute uno bridge exception offset for X64David Ostrovsky
Change-Id: I34b308db422d861098fdf93cff8fea63128ba47a Reviewed-on: https://gerrit.libreoffice.org/13211 Tested-by: David Ostrovsky <david@ostrovsky.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-28Fold URE: Linux ure/share/java/* -> program/classes/Stephan Bergmann
Change-Id: I86864f832c0377d307cfa0b2c137f452e43797eb
2014-11-28Fold URE: Linux ure/lib/* -> program/Stephan Bergmann
The ../../../program/ links in the URE jar Class-Paths are a temporary kludge (and juh.jar had lacked adaption for Mac OS X). Change-Id: I2542d8a582866485dd61c05df3fc6b4b39a8403d
2014-11-23gcc3_linux_powerpc64 bridges: assert -> static_assert to fix buildRene Engelhard
Change-Id: I9cd345de6895b38fc766b0fe16e218a146bfb7b9
2014-11-23aarch63 bridges: add missing #include <osl/mutex.hxx>Rene Engelhard
Change-Id: Ie422682f38752a5f92336106d40c79d2bf9006c7
2014-11-21bridges: oops some semicolons got lostMichael Stahl
Change-Id: I5c2f9286067237ec1a9df6a47009e25e82ea7f63
2014-11-21bridges: oops, static_assert requires 2nd string parameterMichael Stahl
Change-Id: I09b461473f514f2b7c4f256aedbe129372bae9e3
2014-11-21bridges: forgot to remove some include diagnose.hMichael Stahl
Change-Id: Ia4fb18a36769a3eecc20f92a6f5719b841964765
2014-11-21ibridges: convert all legacy osl/diagnose.h assertions in source/cppu_unoMichael Stahl
Change-Id: I4e78fac76b6bb4923e3b680d910afe5bb9640c33
2014-11-20Include <osl/diagnose.h>Tor Lillqvist
Change-Id: I67cb459b1e9ab5638a5b405430d3e0a42fc6bfc3
2014-11-18cppu: clean up public headers with include-what-you-useMichael Stahl
Unfortunately iwyu gets quite confused by the weird cyclic dependencies between various foo.h/foo.hxx and cppumaker generated headers, so it's not obvious if any improvement here is realistic... Change-Id: I0bc66f98b146712e28cabc18d56c11c08418c721
2014-11-18java: fix some raw types warningsNoel Grandin
Change-Id: I0e00739ac36ccc8704ad3c4e4adfa377a6b01f43
2014-11-18java: make fields final where possibleNoel Grandin
found by PMD Change-Id: I87780366119c141cd2dafe6ca1bf2d9798b10aec
2014-11-17Blind fix attempt: include <osl/diagnose.h>Tor Lillqvist
Change-Id: Ic734b56852212d4cee352fc3669dbb16838be8ae
2014-11-17missing includeMatúš Kukan
Change-Id: Iac01fc756a74cbaeebd1d9a01ddea74bbaf18f4b
2014-11-17bridges: get at least the x86 bridge to buildMichael Stahl
Change-Id: Idd2f223e0c83e0a6a19e70bd677058b6899bf766
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl
Sadly cannot forward declare "struct {...} TimeValue;". rtl/(u)?string.hxx still include sal/log.hxx but removing osl/diagnose.h was painful enough for now... Change-Id: Id41e17f3870c4f24c53ce7b11f2c40a3d14d1f05
2014-11-17bridges, vcl: turns out there is sal/alloca.h; replace #ifdeferyMichael Stahl
Change-Id: I2eda8dab51e66af7856b55f2d5f297620e07ccdb
2014-11-14fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-12Fix common typos. No automatic tools. Handmade…Andrea Gelmini
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-12java: convert fields to local variables where possibleNoel Grandin
found by PMD Change-Id: I05b45382b8fb1b734657ce9421a20e6ef6fbe542 Reviewed-on: https://gerrit.libreoffice.org/12376 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-12java: last statement in finalize() method should be call to super.finalize()Noel Grandin
Change-Id: I1785c6cef1fe7c1990207a76c263cff388cbb7e1 Reviewed-on: https://gerrit.libreoffice.org/12375 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-11Avoid -fsanitize=signed-integer-overflowStephan Bergmann
Change-Id: Icec67e6eb57605102cabf494d3e2bb6249d785c7
2014-11-06Revert "use the new OUString::fromUtf8 method"Stephan Bergmann
This reverts commit 05050cdb23de586870bf479a9df5ced06828d498, not all places that use e.g. OStringToOUString to convert potential UTF-8 are guaranteed to fulfil the prerequisites necessary to use fromUtf8 (and some places like e.g. in codemaker are happy with the best-effort effect of OStringToOUString's OSTRING_TO_OUSTRING_CVTFLAGS).
2014-11-06use the new OUString::fromUtf8 methodNoel Grandin
Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
2014-11-05markup with event type not checker typeCaolán McNamara
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
2014-11-03some coverity false positive markup effortsCaolán McNamara
Change-Id: Id17c3426cb4a5913881201a0bf2f569ca5002303
2014-11-03fix sparc buildNoel Grandin
after my commit e7bc3cab019fbf040f9fb8b53ae2cf3f977d200b "remove boilerplate in UNO Exception constructor calls" Change-Id: Iccf79e3c41c860e15078b791fed551a9526aa453
2014-10-30Fixed typos. No automatic tools (sed, and so on).Andrea Gelmini
Change-Id: Ia43976d84eede6f699381bc4f3daf89b95e4cb4f Reviewed-on: https://gerrit.libreoffice.org/12150 Reviewed-by: Bryan Quigley <gquigs@gmail.com> Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-10-30coverity#704134 silence Resource leak in objectCaolán McNamara
Change-Id: I7c2400e9f340562380423eb3300bbcf3eb4c4871
2014-10-29remove unnecessary 'using namespace rtl' declarationsNoel Grandin
It turns out that almost none of them were necessary. Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83 Reviewed-on: https://gerrit.libreoffice.org/12133 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-16java: when overriding finalize(), keep it protectedNoel Grandin
Change-Id: I44456b3244f2190e33a7333c13492eea52bb06e4
2014-10-14fix use of RuntimeException constructorNoel Grandin
Change-Id: Ia8fe44ea365183a8dec8027812b4877be080070d