summaryrefslogtreecommitdiffstats
path: root/jvmfwk
Commit message (Collapse)AuthorAgeFilesLines
* Related: tdf#101057 inherit MalformedVersionException from std::exceptionCaolán McNamara2016-07-221-1/+1
| | | | | | | so we don't get std::terminate on MacOSX if its thrown up to JavaVirtualMachine::getJavaVM Change-Id: I1c6b79bf16cbaa66b8f79d495239e301d3ca58de
* Related: tdf#101057 allow java update version number > 99Caolán McNamara2016-07-221-3/+3
| | | | Change-Id: Icc37cf248fbf35f108eb5bbb25c36fbd54a9fe85
* loplugin:redundantcast: redundant static_casts in jvmfwkStephan Bergmann2016-07-081-1/+1
| | | | Change-Id: I6aef36bd21043460e74b5e3d478665a35213bb1a
* Fix some spelling errors in comments and stringsOtto Kekäläinen2016-07-041-1/+1
| | | | | | | Change-Id: Iecd6b5e13d6be14651f77d8e37f01117ba15a11e Reviewed-on: https://gerrit.libreoffice.org/26883 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* passing a NULL pointer to fileno is not allowedMarkus Mohrhard2016-06-151-0/+3
| | | | | | | | | | | | See crash reports at http://crashreport.libreoffice.org/stats/signature/do_msvcr_magic+0x7 and documentation at https://msdn.microsoft.com/en-us/library/zs6wbdhx.aspx Change-Id: Ia9166d3b9fa10b87585821504e39cdfecbd22eda Reviewed-on: https://gerrit.libreoffice.org/26317 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Convert javaPluginError to scoped enumNoel Grandin2016-05-223-97/+97
| | | | | | | Change-Id: Ib3a32227d50fa9e0f9cc4106ce40a4c3e840f8c7 Reviewed-on: https://gerrit.libreoffice.org/25206 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Add SAL_FALLTHROUGH, where apparently appropriateStephan Bergmann2016-05-101-0/+2
| | | | Change-Id: I0dccdf984f4b171feab20e8400376cf8846ffe2e
* Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann2016-05-101-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. Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
* Fix typosAndrea Gelmini2016-04-301-1/+1
| | | | | | | Change-Id: Id81b16ff26283611f0b84929d831c827f847ab73 Reviewed-on: https://gerrit.libreoffice.org/24317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* update loplugin stylepolice to check local pointers varsNoel Grandin2016-04-261-7/+7
| | | | | | | | | | | | are actually pointer vars. Also convert from regex to normal code, so we can enable this plugin all the time. Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1 Reviewed-on: https://gerrit.libreoffice.org/24391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-204-11/+11
| | | | Change-Id: I65fdd933e2a3973053359024d8d9c733f79a6477
* clang-tidy performance-unnecessary-copy-initializationNoel Grandin2016-04-181-2/+1
| | | | | | | | | | | probably not much performance benefit, but it sure is good at identifying leftover intermediate variables from previous refactorings. Change-Id: I3ce16fe496ac2733c1cb0a35f74c0fc9193cc657 Reviewed-on: https://gerrit.libreoffice.org/24026 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* clang-tidy performance-unnecessary-value-param in variousNoel Grandin2016-04-142-2/+2
| | | | | | | Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907 Reviewed-on: https://gerrit.libreoffice.org/24019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* cleanup: remove unused com/sun/star/uno includesJochen Nitschke2016-04-123-4/+0
| | | | | | | | | | | | Sequence.h(xx), Any.h(xx) and Type.h(xx) and remove unused using-declarations from these files. Add a few missing includes provided by them. Change-Id: I6b91b6d1fdf9d0496dd546c0aab9bdcc6831a5d4 Reviewed-on: https://gerrit.libreoffice.org/23805 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#94306 replace boost::noncopyable ...Jochen Nitschke2016-04-111-2/+4
| | | | | | | | | | | | | ... in modules editeng to oox. Replace with C++11 delete copy-constructur and copy-assignment. Remove boost/noncopyable.hpp includes and one unused boost/checked_delete.hpp include in linguistic. Change-Id: I5a38d8e5ac1b4286bdeb3858d56490a53d13fe80 Reviewed-on: https://gerrit.libreoffice.org/23928 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Values read from the Windows registry shouldn't be assert'edStephan Bergmann2016-04-051-2/+3
| | | | Change-Id: I1a6cc78025790fa22e91ae91dda5644bdc384ca3
* tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar2016-03-151-6/+2
| | | | | | | | | I replaced OSL_ASSERT() with standard C++ assert Change-Id: I92e07d62f3dfe2ad914c49e2b596aef28c35e225 Reviewed-on: https://gerrit.libreoffice.org/23231 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* Fix memory leaksStephan Bergmann2016-03-101-8/+7
| | | | Change-Id: If9f7dc4a28d5e005959f0d4a0a2ed317b699f292
* No more need for jfw_freeJavaInfoStephan Bergmann2016-03-103-19/+12
| | | | Change-Id: I2426a76936b4099a243ce8c102da867e7868aac3
* Turn JavaInfo sal_Sequence* member into rtl::ByteSequenceStephan Bergmann2016-03-103-16/+6
| | | | Change-Id: Iecd476970b0b7a46afe223f71e95b0010048d7b1
* Turn JavaInfo rtl_uString* members into OUStringStephan Bergmann2016-03-103-31/+12
| | | | Change-Id: Ieb23b0c36ef56a4793a56cdb450df34e4d9bce1d
* Manage JavaInfo instances via new/deleteStephan Bergmann2016-03-103-17/+9
| | | | Change-Id: I10a113718e525b646c51aa8a19f9f2b75a36714a
* No need to repeat SAL_CALL in definitionsStephan Bergmann2016-03-101-20/+18
| | | | Change-Id: Ief75af9d08d9b7e31d821773db2363fa201f3038
* No need for extern "C" hereStephan Bergmann2016-03-101-6/+6
| | | | | | ...and fix resulting loplugin:salbool fallout Change-Id: I5ae1497608d31c20b0d10676450a7673dee1c651
* include/jvmfwk/*.h -> .hxx (only ever included from C++ code)Stephan Bergmann2016-03-095-7/+7
| | | | Change-Id: Ia912e937d5a48afb6f8f5345b20bb7bd517fc4f1
* use consistent #define checks for the Windows platformNoel Grandin2016-02-178-17/+17
| | | | | | | | | | | | stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Remove excess newlinesChris Sherlock2016-02-099-20/+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 Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* loplugin:unusedmethodsNoel Grandin2016-02-082-3/+0
| | | | | | | | | | | | | | | using an idea from dtardon: <dtardon> noelgrandin, hi. could you try to run the unusedmethods clang plugin with "make build-nocheck"? that would catch functions that are only used in tests. e.g., i just removed the whole o3tl::range class, which has not been used in many years, but htere was a test for it... <noelgrandin> dtardon, interesting idea! Sure, I can do that. Change-Id: I5653953a426a2186a1e43017212d87ffce520387 Reviewed-on: https://gerrit.libreoffice.org/22041 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2016-01-101-1/+1
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* jvmfwk/plugins: clean-up in util.cxxDouglas Mencken2016-01-061-15/+4
| | | | | | | | | | These are some remnants from the-good-old-days of Mac OS X 10.4 support and 2005's commit b8871d23ea14d72ba1a22376e7812ea5840d1084 Change-Id: I9a14aaac94e4a6e4961a2fa9a5020d9cf3a07c9d Reviewed-on: https://gerrit.libreoffice.org/21124 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
* cppcheck: noExplicitConstructorCaolán McNamara2015-11-244-11/+11
| | | | Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
* Don't assume sal_Unicode is unsigned shortStephan Bergmann2015-11-171-1/+1
| | | | Change-Id: I0439e40b0843ba9a0939eb47e339f396d26f3c64
* loplugin:nullptr (automatic rewrite; Mac-specific code)Stephan Bergmann2015-11-102-2/+2
| | | | Change-Id: Iee5d66a1c14e1583edb0945d23cec9986ce7a830
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-1010-133/+133
| | | | Change-Id: I9dfcec9488b2a25e32d3934766d6d4b208a8c2dc
* new loplugin: oncevarNoel Grandin2015-11-092-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>
* java: combine nested if statementsNoel Grandin2015-11-051-4/+2
| | | | Change-Id: I0457b81668e9427a3c8d6a4af93438b7fb2bb7ba
* Avoid using reserved identifiers, add missing includesStephan Bergmann2015-11-041-2/+6
| | | | Change-Id: I23b548de1f07a4ec5acfeef7c553030a1f098311
* Add mode linesStephan Bergmann2015-11-042-0/+8
| | | | Change-Id: I4026040cfa2cd34c3a33a0a3dd4651caeb11c76f
* loplugin:passstuffbyrefStephan Bergmann2015-11-042-2/+2
| | | | Change-Id: If6e877b326bb630f1b4631a60241a93a6e6cbaca
* yyyyyNoel Grandin2015-11-041-13/+5
| | | | Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
* tdf#94716 allow Oracle's JDK to be used on OS X 10.10 and 10.11Patrick Luby2015-11-046-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>
* loplugin:unusedmethodsNoel Grandin2015-10-273-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>
* tdf#39440: fix several warnings reported by cppcheckSerge Krot2015-10-251-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>
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-124-11/+11
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-122-9/+9
| | | | Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
* loplugin:unusedmethodsNoel Grandin2015-10-011-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>
* Fix typosAndrea Gelmini2015-09-301-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>
* loplugin:stringconstant: OUStringBuffer: appendAscii -> appendStephan Bergmann2015-08-313-6/+6
| | | | Change-Id: I09552b4ddd8900952f499c8b9c44933c7240c866
* Clean up sEmptyStephan Bergmann2015-08-271-4/+3
| | | | Change-Id: If1b2bfe308caa2bce92e73d2c5c86ee273faed93
* loplugin:defaultparamsStephan Bergmann2015-08-231-1/+1
| | | | Change-Id: I797b986b227381bc066ea3aa305c0abf78cd156a