Age | Commit message (Collapse) | Author |
|
Change-Id: Iecd03d0343b9b7c3a8b23b5a1e9654148b94d44c
|
|
Some versions of gcc clobber one of the registries that are used to pass
arguments in the function's prologue, like:
Dump of assembler code for function (anonymous namespace)::privateSnippetExecutor():
510 {
0x00003fffaffe8454 <+0>: mflr r0
0x00003fffaffe8458 <+4>: std r0,16(r1)
0x00003fffaffe845c <+8>: std r29,-24(r1)
0x00003fffaffe8460 <+12>: std r30,-16(r1)
0x00003fffaffe8464 <+16>: std r31,-8(r1)
0x00003fffaffe8468 <+20>: stdu r1,-352(r1)
0x00003fffaffe846c <+24>: mr r31,r1
=> 0x00003fffaffe8470 <+28>: ld r8,-28688(r13)
0x00003fffaffe8474 <+32>: std r8,312(r31)
0x00003fffaffe8478 <+36>: li r8,0
Reading the registries through variables makes gcc aware that they are
used, so it does not touch them. It has got no negative effect on
performance, as it produces the same object code as the current asm
block.
Change-Id: I3b99b0aa9944f9f33de9a42508e9d4dd23cec5e0
|
|
rtl/string.hxx and rtl/ustring.hxx both unnecessarily #include <sal/log.hxx>
(and don't make use of it themselves), but many other files happen to depend on
it. Cleaned up some, but something like
grep -FwL sal/log.hxx $(git grep -Elw \
'SAL_INFO|SAL_INFO_IF|SAL_WARN|SAL_WARN_IF') -- \*.cxx)
shows lots more files that potentially need fixing before the include can be
removed from rtl/string.hxx and rtl/ustring.hxx.
Change-Id: Ibf033363e83d37851776f392dc0b077381cd8b90
|
|
ie.
void f(void);
becomes
void f();
I used the following command to make the changes:
git grep -lP '\(\s*void\s*\)' -- *.cxx \
| xargs perl -pi -w -e 's/(\w+)\s*\(\s*void\s*\)/$1\(\)/g;'
and ran it for both .cxx and .hxx files.
Change-Id: I314a1b56e9c14d10726e32841736b0ad5eef8ddd
|
|
Change-Id: If5d839b05c1babecc84b76d2bcdf47009d0f713c
|
|
Change-Id: I43264a975febf0d62271ab4e7d8cc96246956124
|
|
Change-Id: I88259ffaffc73979c240721d2db166c79d3085f1
|
|
Change-Id: I37d1a939c1a98f77152ed90dfb201446101f3157
|
|
Change-Id: Ie95f74966ff951b2b3161efeb073f0648c09ff3b
|
|
found with
git grep -E '\* *\<reinterpret_cast\>[^>]+\<OUString\>'
Change-Id: I9306d4ad8e3b1664f54cb7df86f2d79bfd3c6cb9
|
|
Change-Id: I19b86fba87df57c4c65c658d051b664c527214eb
|
|
Change-Id: I390fc815c32fa0eeeabf4d80a17bc4deedad2d2c
|
|
Change-Id: I7fd4ec85eac0693ae6a61ba0b854dd6091bdc6ef
|
|
Change-Id: I5c0372bce2564e58c78daf544cfcb106387c6894
|
|
Change-Id: Ic97da0ff2b0f62e0351271bf03329b1903b8d45c
|
|
Change-Id: I89c3c2c63c1c1b7478d94c713341db1cd24f0ab8
|
|
Change-Id: Idbde54c641531b34f397d07c5f929870c3a82371
|
|
...similarly to 0fdbb5b0eabbaa571f3747fda12a56c938cba474 "Make
cpp_uno/gcc3_linux_x86-64 bridge work with GCC 4.7"
Change-Id: Idcafcb07678d02446172d7fde30631a342f6437e
|
|
Change-Id: Ifac86f4d7e637e9fdbce7e60f6d383e7dca69508
|
|
and we can include a few less headers
Change-Id: Id742849ff4c1c37a2b861aa3d6ab823f00ea87f8
|
|
Change-Id: Ia367fcd3b204b9dd96f5fa6d3a52d0895bd9c0c9
|
|
Change-Id: I08f17dd9cc092206083ff41bbbc178e0322e86d0
|
|
Change-Id: Ia8f2c9f1c1c284708a2cbde379197ec6ba58742f
|
|
Change-Id: I5f2393b13729ab43ad2cfd4a3f960a507e3e608b
|
|
Change-Id: Icbab4d030877f978babfc51f984fb4793b60f681
|
|
Change-Id: I132d22e2af3cf673c17d8964f690d48990834884
|
|
found with some minor modifications to find/find-unused-defines.sh
Change-Id: I18cc479adedc7a0dada68a4aeef08300e62631dd
Reviewed-on: https://gerrit.libreoffice.org/14194
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Iaefde6a2fbe1b37f31435217c9f57d611d255b11
|
|
Change-Id: Ib8575109cfe0339f2d8b56741d3ad2a538ecf164
|
|
Change-Id: Ic224abf67acb212ee20ccf9eb81b5ed5edf851b9
|
|
...when write+exec mmap fails (due to SELinux deny_execmem). This avoids the
tmp file creation in environments that don't need it and which in turn have
problems of their own with that tmp file business.
An alternative would be to first check whether SELinux deny_execmem is enforced
and only then try double mmap first. An advantage could be that it might avoid
false SELinux alerts in that case. The disadvantage would be the overhead of
introducing a conditional dependency on libselinux here. And given that for one
deny_execmem typically appears to be off by default (as at least both
contemporary GNOME desktop and OpenJDK malfunction when it is enabled), and for
another I guess deny_execmem could still change its value between the time of
checking for it and the time of requesting a write+exec mmap, that just does not
seem worth it.
Change-Id: I3560803139b630557b6219d3db52945c7e0cdcd2
|
|
Change-Id: Iffc8cbf108310099318e37378c4b3033ea087cee
|
|
Change-Id: I7c41b90c9af045fd452ee62ed0c5d9b261236855
|
|
This reverts commit 3976739f2378391fa09379c48844daf0e2790f5b,
the problem mentioned there was caused by a different commit, and
has meanwhile been fixed.
|
|
cf. 9a745cbf549aa391be2b67f41c83056bd44db97a "Introdude SAL_JNI_EXPORT and use
that instead of JNIEXPORT"
Change-Id: I81dcc8dfcb878d3e935d807f491b99927637c23c
|
|
as an experiment to see if that's somehow the cause of
NEXT An uncaught exception of type com.sun.star.sdbc.SQLException
NEXT - General error: java.lang.UnsatisfiedLinkError: com.sun.star.sdbcx.comp.hsqldb.StorageFileAccess.isStreamElement(Ljava/lang/String;Ljava/lang/String;)Z
NEXT ##Failure Location unknown## : Error
Test name: HSQLDBTest::testEmptyDBConnection
NEXT An uncaught exception of type com.sun.star.sdbc.SQLException
NEXT - General error: java.lang.UnsatisfiedLinkError: com.sun.star.sdbcx.comp.hsqldb.StorageFileAccess.isStreamElement(Ljava/lang/String;Ljava/lang/String;)Z
under clang
This reverts commit ce7f442bd0b600c0acc74d4757e894a2ba382c53.
Change-Id: Ieed0be5721953b9644e4be411173e0ea73f33ed8
|
|
Using std::unordered_map causes a complex multi-line error message, call to
implicitly-deleted copy constructor of 'jni_uno::JNI_type_info_holder' etc.
Revert ce7f442bd0b600c0acc74d4757e894a2ba382c53 for one source file.
Change-Id: I24453498d3fcaadf900f2bb56a2812f8bce55dd4
|
|
Change-Id: I3a16703727f1a421e0ed18079e14219a4feeb8c8
|
|
found by CodePro
Change-Id: If1b75e43f81d70984422e437147048a491395b66
|
|
found by looking for unused parameters (in Eclipse)
Change-Id: I03cf9bc8312e59747b2d0ac153ee2fc8d76be893
|
|
Change-Id: I5d458f43616edc395faa8c27edaddc7d515166db
|
|
I had introduced it with d83de4b1a93ba7ed7bc3243073be3de96a44bfa9 in
2012, when the C++/UNO bridge for 64-bit OS X was created mostly as a
copy of the Linux one.
Stephan says that the only need for USE_DOUBLE_MMAP should be on
SELinux anyway, so most likely also its use for the various BSDs and
Android are copypasta or cargo cult.
Change-Id: I1c16e830e5e8269b78b14837a9127a98612a6e54
|
|
Rewrite of 32-Bit-Code to work under 64 Bit:
To use the 32 Bit offset values in the ExceptionType we have to
allocate a single allocation block and use it for all code and data.
All offsets inside this area are guaranteed to be in 32 bit address
range. So we have to calc total memory allocation size for D-tor,
C-Tors, ExceptionType and type_info. ExceptionType is allocated via
placement new operator to locate everything inside our mem block.
There is one caveat: Struct type_info is kept in a map and was
referenced from class ExceptionType. Therefore type_info now is also
member of ExceptionType and can be referenced via 32 bit offset.
Change-Id: I6b524e36975a66fb64ca0cfb014c207053acec90
Reviewed-on: https://gerrit.libreoffice.org/13653
Reviewed-by: David Ostrovsky <david@ostrovsky.org>
Tested-by: David Ostrovsky <david@ostrovsky.org>
|
|
Change-Id: I7866959b9ea36c81003259cd387a001e9f34f26a
|
|
Change-Id: I72e0df381bd9525ea4fff1f4bbd57ffe84ce241f
|
|
Change-Id: I95ec7503ab7cf0309427118cc5af95eba4f5785b
|
|
Change-Id: I5d6071096307adbe7df0178000346cf915afa3e7
Reviewed-on: https://gerrit.libreoffice.org/13477
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Ic8fea32a163ca5e85ac3e2a34d04e4fa1a1943f9
|
|
...make sure the class is actually found, etc.
Change-Id: I5459d531be39b07594a975ae708a7611d1667a2f
|
|
...to have it available during JNI-UNO's uno_initEnvironment (see next)
Change-Id: I7a2f27b512fc74f418b4648d92dafbf0304eaa96
|