summaryrefslogtreecommitdiffstats
path: root/bridges
Commit message (Collapse)AuthorAgeFilesLines
* bridges: Use appropriate OUString functions on string constantsStephan Bergmann2014-12-161-3/+3
| | | | Change-Id: I95ec7503ab7cf0309427118cc5af95eba4f5785b
* java: remove some unused fields and variablesNoel Grandin2014-12-154-23/+3
| | | | | | | Change-Id: I5d6071096307adbe7df0178000346cf915afa3e7 Reviewed-on: https://gerrit.libreoffice.org/13477 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* add link to mkstemp coverity warning wrt threading fiascoCaolán McNamara2014-12-151-0/+1
| | | | Change-Id: Ic8fea32a163ca5e85ac3e2a34d04e4fa1a1943f9
* Fix JNI calls to AsynchronousFinalizerStephan Bergmann2014-12-113-56/+39
| | | | | | ...make sure the class is actually found, etc. Change-Id: I5459d531be39b07594a975ae708a7611d1667a2f
* Store JNI_info in JniUnoEnvironmentData instead of BridgeStephan Bergmann2014-12-116-199/+222
| | | | | | ...to have it available during JNI-UNO's uno_initEnvironment (see next) Change-Id: I7a2f27b512fc74f418b4648d92dafbf0304eaa96
* better documentationStephan Bergmann2014-12-111-0/+4
| | | | Change-Id: Ife9a98cfe2166ccc7aac3904c7be4ea71443d857
* Move JniUnoEnvironmentData into its own include fileStephan Bergmann2014-12-115-11/+53
| | | | Change-Id: I8e6fb1fc0acff781dd6e6b62018c7ccd5d0e2307
* Rename Context -> JniUnoEnvironmentDataStephan Bergmann2014-12-114-28/+33
| | | | Change-Id: I97879d250ed0ed20d5e129ff3af6dbc2f5759078
* bridges: fix gcc3_linux_arm buildMiklos Vajna2014-12-111-1/+1
| | | | Change-Id: I3bfaeab9dd9f2d8cd603c655ec3aa7c4f508c673
* reduce scope of local variablesMichael Weghorn2014-12-114-13/+7
| | | | | | | | | 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>
* java: reduce visibility of fields and methodsNoel Grandin2014-12-111-10/+10
| | | | | | | | | 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>
* java: improve use of WrappedTargetException to set cause properlyNoel Grandin2014-12-101-2/+2
| | | | | | | | | 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
* Typo: nuber=>numberJulien Nabet2014-12-102-3/+3
| | | | Change-Id: Idc87ab05f817a21e491537a1fa4c014c5a313bf6
* Uno bridge: Apply debug info setting to noopt codeDavid Ostrovsky2014-12-091-1/+1
| | | | | | | Change-Id: Id949718baf77bb7e2d276d3db08f68149c114796 Reviewed-on: https://gerrit.libreoffice.org/13364 Reviewed-by: David Ostrovsky <david@ostrovsky.org> Tested-by: David Ostrovsky <david@ostrovsky.org>
* Fix base+offset pointer calculations when synthesizing exception dataStephan Bergmann2014-12-091-12/+33
| | | | | | | | | | | | ...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
* Fix mscx_filterCppException base+offset pointer calculationsStephan Bergmann2014-12-091-12/+13
| | | | Change-Id: I9fbe8d7eba181cbfcab704761e6feebd78120644
* rhbz#1036877: Join Java AsynchronousFinalizer thread well before exitStephan Bergmann2014-12-097-49/+141
| | | | | | | | | | | | | | | | | | | | | | | | 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
* Mark classes as noncopyableStephan Bergmann2014-12-081-3/+6
| | | | Change-Id: I4262fb56d681c70a089638b876dc07b1f472f583
* Fold URE: WindowsStephan Bergmann2014-12-031-2/+1
| | | | | | | | ...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
* except.cxx: Fix pTypeDescr is unknown in this context errorDavid Ostrovsky2014-12-021-4/+4
| | | | | | | Change-Id: Ic47f69b01cf17a55901e9e3541419d9f477d9585 Reviewed-on: https://gerrit.libreoffice.org/13210 Tested-by: David Ostrovsky <david@ostrovsky.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* MSVC 2013: Compute uno bridge exception offset for X64David Ostrovsky2014-12-021-1/+3
| | | | | | | Change-Id: I34b308db422d861098fdf93cff8fea63128ba47a Reviewed-on: https://gerrit.libreoffice.org/13211 Tested-by: David Ostrovsky <david@ostrovsky.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* Fold URE: Linux ure/share/java/* -> program/classes/Stephan Bergmann2014-11-281-1/+1
| | | | Change-Id: I86864f832c0377d307cfa0b2c137f452e43797eb
* Fold URE: Linux ure/lib/* -> program/Stephan Bergmann2014-11-281-1/+1
| | | | | | | 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
* gcc3_linux_powerpc64 bridges: assert -> static_assert to fix buildRene Engelhard2014-11-231-1/+1
| | | | Change-Id: I9cd345de6895b38fc766b0fe16e218a146bfb7b9
* aarch63 bridges: add missing #include <osl/mutex.hxx>Rene Engelhard2014-11-231-0/+2
| | | | Change-Id: Ie422682f38752a5f92336106d40c79d2bf9006c7
* bridges: oops some semicolons got lostMichael Stahl2014-11-211-2/+2
| | | | Change-Id: I5c2f9286067237ec1a9df6a47009e25e82ea7f63
* bridges: oops, static_assert requires 2nd string parameterMichael Stahl2014-11-219-10/+10
| | | | Change-Id: I09b461473f514f2b7c4f256aedbe129372bae9e3
* bridges: forgot to remove some include diagnose.hMichael Stahl2014-11-213-6/+0
| | | | Change-Id: Ia4fb18a36769a3eecc20f92a6f5719b841964765
* ibridges: convert all legacy osl/diagnose.h assertions in source/cppu_unoMichael Stahl2014-11-2180-581/+431
| | | | Change-Id: I4e78fac76b6bb4923e3b680d910afe5bb9640c33
* Include <osl/diagnose.h>Tor Lillqvist2014-11-203-0/+6
| | | | Change-Id: I67cb459b1e9ab5638a5b405430d3e0a42fc6bfc3
* cppu: clean up public headers with include-what-you-useMichael Stahl2014-11-181-1/+0
| | | | | | | | 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
* java: fix some raw types warningsNoel Grandin2014-11-181-1/+1
| | | | Change-Id: I0e00739ac36ccc8704ad3c4e4adfa377a6b01f43
* java: make fields final where possibleNoel Grandin2014-11-181-4/+4
| | | | | | found by PMD Change-Id: I87780366119c141cd2dafe6ca1bf2d9798b10aec
* Blind fix attempt: include <osl/diagnose.h>Tor Lillqvist2014-11-171-0/+1
| | | | Change-Id: Ic734b56852212d4cee352fc3669dbb16838be8ae
* missing includeMatúš Kukan2014-11-171-0/+1
| | | | Change-Id: Iac01fc756a74cbaeebd1d9a01ddea74bbaf18f4b
* bridges: get at least the x86 bridge to buildMichael Stahl2014-11-172-0/+2
| | | | Change-Id: Idd2f223e0c83e0a6a19e70bd677058b6899bf766
* sal: clean up public headers with include-what-you-useMichael Stahl2014-11-179-0/+11
| | | | | | | | | 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
* bridges, vcl: turns out there is sal/alloca.h; replace #ifdeferyMichael Stahl2014-11-173-15/+5
| | | | Change-Id: I2eda8dab51e66af7856b55f2d5f297620e07ccdb
* fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava2014-11-141-1/+1
| | | | | | | Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
* Fix common typos. No automatic tools. Handmade…Andrea Gelmini2014-11-123-4/+4
| | | | | | | Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* java: convert fields to local variables where possibleNoel Grandin2014-11-121-2/+3
| | | | | | | | | 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>
* java: last statement in finalize() method should be call to super.finalize()Noel Grandin2014-11-122-2/+4
| | | | | | | Change-Id: I1785c6cef1fe7c1990207a76c263cff388cbb7e1 Reviewed-on: https://gerrit.libreoffice.org/12375 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Avoid -fsanitize=signed-integer-overflowStephan Bergmann2014-11-111-1/+1
| | | | Change-Id: Icec67e6eb57605102cabf494d3e2bb6249d785c7
* Revert "use the new OUString::fromUtf8 method"Stephan Bergmann2014-11-061-2/+2
| | | | | | | | 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).
* use the new OUString::fromUtf8 methodNoel Grandin2014-11-061-2/+2
| | | | Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
* markup with event type not checker typeCaolán McNamara2014-11-052-3/+5
| | | | Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
* some coverity false positive markup effortsCaolán McNamara2014-11-032-0/+3
| | | | Change-Id: Id17c3426cb4a5913881201a0bf2f569ca5002303
* fix sparc buildNoel Grandin2014-11-031-1/+1
| | | | | | | after my commit e7bc3cab019fbf040f9fb8b53ae2cf3f977d200b "remove boilerplate in UNO Exception constructor calls" Change-Id: Iccf79e3c41c860e15078b791fed551a9526aa453
* Fixed typos. No automatic tools (sed, and so on).Andrea Gelmini2014-10-301-1/+1
| | | | | | | | 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>
* coverity#704134 silence Resource leak in objectCaolán McNamara2014-10-301-0/+1
| | | | Change-Id: I7c2400e9f340562380423eb3300bbcf3eb4c4871