summaryrefslogtreecommitdiff
path: root/jvmfwk
AgeCommit message (Collapse)AuthorFilesLines
2016-07-20Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann1-1/+1
...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in preparation of enabling -Wimplicit-fallthrough. (This is only relevant for C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.) Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but that would require adding back in dependencies on boost_headers to many libraries where we carefully removed any remaining Boost dependencies only recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its future evolution will not have any impact on the stable URE interface.) C++17 will have a proper [[fallthroug]], eventually removing the need for a macro altogether. (cherry picked from commit 14cd5182c5f64c43581c82db8c958369152226ac) Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
2016-07-20Fix typosAndrea Gelmini1-1/+1
Reviewed-on: https://gerrit.libreoffice.org/24317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> (cherry picked from commit fc2590cfa112222500a6c847917d7545f60024c6) Change-Id: Id81b16ff26283611f0b84929d831c827f847ab73
2016-07-20Fix typosAndrea Gelmini1-1/+1
Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com> (cherry picked from commit 64d624b65124ac02d8ee59b135593fd9d8eb9067) Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86
2016-07-13Turn JavaInfo sal_Sequence* member into rtl::ByteSequenceStephan Bergmann3-16/+6
Change-Id: Iecd476970b0b7a46afe223f71e95b0010048d7b1 (cherry picked from commit 8546831b3b36e29c1ee42b790cbecd3fd8d8bbaf)
2016-07-13Turn JavaInfo rtl_uString* members into OUStringStephan Bergmann3-31/+12
Change-Id: Ieb23b0c36ef56a4793a56cdb450df34e4d9bce1d (cherry picked from commit 5dcdb35ab1e99dbeb283d3694ab7ebba354e9197)
2016-07-13Manage JavaInfo instances via new/deleteStephan Bergmann3-17/+9
Change-Id: I10a113718e525b646c51aa8a19f9f2b75a36714a (cherry picked from commit 81dd7115e8da9a365a2d26e225f6ec4d2d8ccb5f)
2016-07-13No need for extern "C" hereStephan Bergmann1-6/+6
...and fix resulting loplugin:salbool fallout Change-Id: I5ae1497608d31c20b0d10676450a7673dee1c651 (cherry picked from commit 8471ba7b97a2a01039133674d3960c5c16803833)
2016-07-13include/jvmfwk/*.h -> .hxx (only ever included from C++ code)Stephan Bergmann4-6/+6
Change-Id: Ia912e937d5a48afb6f8f5345b20bb7bd517fc4f1 (cherry picked from commit 220d9c24d0e81668862010ed3e68e4f46fe9924b)
2016-07-12Remove excess newlinesChris Sherlock8-19/+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 Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> (cherry picked from commit a238b1f8d304bf1e2ffb357937f3ec888ee8ac89) Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c
2016-07-12update loplugin stylepolice to check local pointers varsNoel Grandin1-12/+10
are actually pointer vars. Also convert from regex to normal code, so we can enable this plugin all the time. Reviewed-on: https://gerrit.libreoffice.org/24391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> (cherry picked from commit e8fd5a07eca70912ddee45aaa34d434809b59fb7) Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1
2015-11-24cppcheck: noExplicitConstructorCaolán McNamara4-11/+11
Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
2015-11-17Don't assume sal_Unicode is unsigned shortStephan Bergmann1-1/+1
Change-Id: I0439e40b0843ba9a0939eb47e339f396d26f3c64
2015-11-10loplugin:nullptr (automatic rewrite; Mac-specific code)Stephan Bergmann2-2/+2
Change-Id: Iee5d66a1c14e1583edb0945d23cec9986ce7a830
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann10-133/+133
Change-Id: I9dfcec9488b2a25e32d3934766d6d4b208a8c2dc
2015-11-09new loplugin: oncevarNoel Grandin2-15/+7
Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-05java: combine nested if statementsNoel Grandin1-4/+2
Change-Id: I0457b81668e9427a3c8d6a4af93438b7fb2bb7ba
2015-11-04Avoid using reserved identifiers, add missing includesStephan Bergmann1-2/+6
Change-Id: I23b548de1f07a4ec5acfeef7c553030a1f098311
2015-11-04Add mode linesStephan Bergmann2-0/+8
Change-Id: I4026040cfa2cd34c3a33a0a3dd4651caeb11c76f
2015-11-04loplugin:passstuffbyrefStephan Bergmann2-2/+2
Change-Id: If6e877b326bb630f1b4631a60241a93a6e6cbaca
2015-11-04yyyyyNoel Grandin1-13/+5
Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
2015-11-04tdf#94716 allow Oracle's JDK to be used on OS X 10.10 and 10.11Patrick Luby6-11/+195
Change-Id: Ide9b4beebb407e4ceee30f1d99f29d028c848d8c Reviewed-on: https://gerrit.libreoffice.org/19131 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
2015-10-27loplugin:unusedmethodsNoel Grandin3-52/+0
Change-Id: I161cd52606c11b6008f5d8b1d8ee391692f91861 Reviewed-on: https://gerrit.libreoffice.org/19231 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-25tdf#39440: fix several warnings reported by cppcheckSerge Krot1-2/+2
Change-Id: I560d28b7cc67740c6479494d0e5aa62d2ac6ffae Reviewed-on: https://gerrit.libreoffice.org/19587 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann4-11/+11
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2-9/+9
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
2015-10-01loplugin:unusedmethodsNoel Grandin1-4/+0
- improvements to the plugin to find more method calls - improvements to python script to remove more false+ - fix the FORCE_COMPILE_ALL build flag to include code in the $WORKDIR Change-Id: I4d6015dcb9b9d60c26f0bcee8abad807177a7836 Reviewed-on: https://gerrit.libreoffice.org/19064 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-09-30Fix typosAndrea Gelmini1-2/+2
Change-Id: I44bdb63bef76e9686e995661a24a3bef7cd50ab7 Reviewed-on: https://gerrit.libreoffice.org/18959 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-08-31loplugin:stringconstant: OUStringBuffer: appendAscii -> appendStephan Bergmann3-6/+6
Change-Id: I09552b4ddd8900952f499c8b9c44933c7240c866
2015-08-27Clean up sEmptyStephan Bergmann1-4/+3
Change-Id: If1b2bfe308caa2bce92e73d2c5c86ee273faed93
2015-08-23loplugin:defaultparamsStephan Bergmann1-1/+1
Change-Id: I797b986b227381bc066ea3aa305c0abf78cd156a
2015-08-22tdf#39440 reduce scope of local variablesMichael Weghorn2-2/+2
This addresses some cppcheck warnings. Change-Id: I6ab170c4426494ab9fc53017f88e47fbfdbd9aad Reviewed-on: https://gerrit.libreoffice.org/17920 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-08-11loplugin: defaultparamsNoel Grandin1-1/+1
Change-Id: I5afe7f9f5caa1f5e55c6218bdb3771571ff0dafa
2015-07-16loplugin:simplifyboolStephan Bergmann4-6/+6
Change-Id: Iaa3c04ba936512f511de238f5a3ca5b0f9afc06d
2015-07-11tdf#92483 fix initializing JRE on Win 64bit (load java's msvcr100.dll)Christian Lohmaier1-1/+1
problem is that the offsets/sizes are of different type than ptrdiff_t, and when using without casting, the resulting address is way off... Change-Id: I7b1cdd611c8c4b317cd33ca8fbbda2e7e8f5f4fc Reviewed-on: https://gerrit.libreoffice.org/16938 Reviewed-by: David Ostrovsky <david@ostrovsky.org> Tested-by: Jenkins <ci@libreoffice.org>
2015-07-10jvmfwk: pluginlib: Add support for mips64.Heiher2-2/+10
Change-Id: I525266468ae2ef04976d7835cc22c930293a0dc3 Reviewed-on: https://gerrit.libreoffice.org/16911 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-07-03Fix typosAndrea Gelmini2-2/+2
Change-Id: I8e429d1f03aac7c7cdb7ff4b43b3f46d40292510 Reviewed-on: https://gerrit.libreoffice.org/16709 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com> Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2015-06-30Fix typosAndrea Gelmini1-2/+2
Change-Id: I1c5e4970d4e90bdea68a901294594104de5e189e Reviewed-on: https://gerrit.libreoffice.org/16582 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Andras Timar <andras.timar@collabora.com>
2015-06-24jvmfwk: pass along -Xss to JVM on AMD64 tooMichael Stahl1-2/+2
The PythonTest_dbacces_python in a clang ASAN build on F21 started to fail with StackOverflowError in JNI_CreateJavaVM on the main thread, and this fixes the problem. Seen with java-1.8.0-openjdk-1.8.0.45-39.b14.fc21.x86_64 Change-Id: I87dfb4b62d547b334c19261aad88b4856489776f
2015-06-24jvmfwk: slighly misleading error messageMichael Stahl1-1/+1
Change-Id: I64165a03c16d852781173fe2de952e6b803efaf7
2015-06-17Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe4-13/+13
This may reduce some degree of dependency on boost. Done by running a script like: git grep -l '#include *.boost/scoped_array.hpp.' \ | xargs sed -i -e 's@#include *.boost/scoped_array.hpp.@#include <memory>@' git grep -l '\(boost::\)\?scoped_array<\([^<>]*\)>' \ | xargs sed -i -e 's/\(boost::\)\?scoped_array<\([^<>]*\)>/std::unique_ptr<\2[]>/' ... and then killing duplicate or unnecessary includes, while changing manually m_xOutlineStylesCandidates in xmloff/source/text/txtimp.cxx, extensions/source/ole/unoconversionutilities.hxx, and extensions/source/ole/oleobjw.cxx. Change-Id: I3955ed3ad99b94499a7bd0e6e3a09078771f9bfd Reviewed-on: https://gerrit.libreoffice.org/16289 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-06-15cppcheck:redundantAssignmentNoel Grandin2-10/+7
Change-Id: I1167d0ce6b6f6e48309d0551c1d2a283d79546a7
2015-06-11java:regulatize the order of 'final' and public/privateNoel Grandin1-1/+1
Make the order be 'public static' or 'private static' Just makes the code nicer to read. Change-Id: I182424bda45a2d68642e5d04c6091d268ace1fe2 Reviewed-on: https://gerrit.libreoffice.org/16202 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2-5/+5
Change-Id: I62a6714749eba8226ba3a5416ee81f2d76cf0afc
2015-06-02loplugin:cstylecast: deal with those that are (technically) const_castStephan Bergmann1-1/+1
Change-Id: Ie8e3bde0eeeec6cd35a7f73f9a29c28cdd03ad02
2015-06-01loplugin:loopvartoosmallNoel Grandin1-1/+1
Change-Id: I86ff38a90018a2ddfb2db3babf67168b0e6257a5
2015-05-29Support Azul Zulu JRE on Linux and Mac OS X, tooStephan Bergmann3-2/+9
Change-Id: I317506319b91daf006a0ba79874ff265ba6ed4da
2015-05-27Support Azul Zulu JRE (at least on Windows)Stephan Bergmann3-1/+6
(<http://www.azulsystems.com/products/zulu/downloads> also offers it for Linux and Mac OS X, so it could be added to more javavendors_*.xml if need be; I only checked it actually works with a Windows 64-bit LO build) Change-Id: If549a0d5bf1459eba2f6ba2bd545cfd55a20cea6
2015-05-27fprintf -> SAL_INFOStephan Bergmann4-51/+29
Change-Id: I4808e89cf5b90c99df3d4c4474c578a8c8f611a6
2015-05-12java dir for powepc64 and powepc64le can differDavid Tardon1-1/+5
E.g., RHEL >= 7.1 uses ppc64le for reasons listed in https://bugs.openjdk.java.net/browse/JDK-8073139 . This is a follow-up to commit 7a66dbe982623be620ad6ac426fe689a434e5f51. Change-Id: I685f76d51e9775788a7fb225c6a5e2309a45ceb7
2015-05-11loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann1-1/+1
Change-Id: I21cac03d5100822ef99d4b7fb7c192c0d2adf1b2