Age | Commit message (Collapse) | Author |
|
Change-Id: I40d03ab9acb67ab72b9047017452f069ce88fd4b
|
|
Change-Id: Ieae67cbf917cdf4bca2b0d6c1697eddc6137dbe2
|
|
...detected with a modified trunk Clang with
> Index: lib/Sema/SemaDeclCXX.cpp
> ===================================================================
> --- lib/Sema/SemaDeclCXX.cpp (revision 219190)
> +++ lib/Sema/SemaDeclCXX.cpp (working copy)
> @@ -1917,9 +1917,10 @@
> const Type *T = FD.getType()->getBaseElementTypeUnsafe();
> // FIXME: Destruction of ObjC lifetime types has side-effects.
> if (const CXXRecordDecl *RD = T->getAsCXXRecordDecl())
> - return !RD->isCompleteDefinition() ||
> - !RD->hasTrivialDefaultConstructor() ||
> - !RD->hasTrivialDestructor();
> + return !RD->hasAttr<WarnUnusedAttr>() &&
> + (!RD->isCompleteDefinition() ||
> + !RD->hasTrivialDefaultConstructor() ||
> + !RD->hasTrivialDestructor());
> return false;
> }
>
> @@ -3517,9 +3518,11 @@
> bool addFieldInitializer(CXXCtorInitializer *Init) {
> AllToInit.push_back(Init);
>
> +#if 0
> // Check whether this initializer makes the field "used".
> if (Init->getInit()->HasSideEffects(S.Context))
> S.UnusedPrivateFields.remove(Init->getAnyMember());
> +#endif
>
> return false;
> }
to warn about members of SAL_WARN_UNUSED-annotated class types, and warn about
initializations with side effects (cf.
<http://lists.cs.uiuc.edu/pipermail/cfe-dev/2014-October/039602.html>
"-Wunused-private-field distracted by side effects").
Change-Id: I3f3181c4eb8180ca28e1fa3dffc9dbe1002c6628
|
|
Temp files created before first test method don't see LO_TESTNAME
Change-Id: Iff74abf574f5151980f463646f11c1b32ad0c241
|
|
Change-Id: Ic90e806d279f8180d78f78597dbc549264338295
|
|
Change-Id: If067f755b99480b7b7bd0bd3bb7a71a447794273
|
|
try and silence
coverity#1240258 Operands don't affect result
and
coverity#1240255 Operands don't affect result
Change-Id: If53ff385585567e9b46372f45420c4321811bca0
|
|
see can we silence these false positives
coverity#1240259 Operands don't affect result
coverity#1240254 Operands don't affect result
coverity#1240264 Operands don't affect result
coverity#1240267 Operands don't affect result
Change-Id: Ieca40474c231e33a516b70f7693346ac73babd61
|
|
Change-Id: I7e6490c5b50a236a5f070e98d4501e7dd5a90307
|
|
Change-Id: I9e0e06331f134fe89c9942f93b7546b0e67bb821
|
|
Change-Id: Ibf4f2d46bfbb977f0ba27110d86d7b7f1173631c
|
|
Change-Id: I997dfa3b1fe651db56ad2a6a24ec0cb6d5044550
|
|
Change-Id: I86fe850b4b7bc8fc4c44717ca813772114ca70ff
|
|
Change-Id: I214884e9ee1d49dcc4db71f8744513468b912691
|
|
Change-Id: I82113fcd4182f9557ba0e7258498b5d4e798bf03
|
|
(Spotted while valgrinding an unrelated issue)
Change-Id: I2eab4c08e251d79f427fd01442c4dce20d7d89f0
Reviewed-on: https://gerrit.libreoffice.org/11785
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
Change-Id: I4412a0c1e4dee94cd717f24b8df5e257d895f721
|
|
Change-Id: I68eee76c8710c89af8e3c1e1006345f908923ece
|
|
Change-Id: If997720635f99726e14c00132308529f96e639c8
|
|
Change-Id: I4e4281ac8767c9a6b6b85bc759f28bcf074d8e39
|
|
Change-Id: I54fd66b8788171884c6d3d6e7645871615d48080
|
|
Change-Id: I6cb46a51dda3fda51a3b6413656da15fc5bdb04d
|
|
Put the TOOLS Time class in the tools namespace. Avoids clash with the X11
Time typedef.
Change-Id: Iac57d5aef35e81ace1ee0d5e6d76cb278f8ad866
Reviewed-on: https://gerrit.libreoffice.org/11684
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Icd477e433f0c294c823c64147d49fb4ac6bfb447
|
|
Change-Id: If8fda4b27adb2f83cbdfdbb20d90ced77e467a54
|
|
As discussed with Stephan Bergmann, these files, which contained unused
cppunittests, could be removed.
Change-Id: I91844b96967e3c3e0e8367452f2dda420f479ffe
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
So no need to expand it at configure time in ure/source/unorc, and no need to
handle the related builddir!=srcdir complications.
Change-Id: Ifa34d25fab9ad2da13ed039bf6c5921b0fb58703
|
|
The default on Fedora 20 is 8MB, which causes stack overflows from
CPython code in some of the JunitTests when built with clang 3.4
-fsanitize=address.
Change-Id: I3de9975564aad668e746cea74ae10931ef36a608
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I96d081f394b0b62d99ec1034bf5e99da9aedd9d9
|
|
In other words, only executable files go in the MacOS folder. Dynamic
libraries and bundled frameworks (i.e., LibreOfficePython), and
nothing else, go in the Frameworks folder, and all other files go in
the Resources folder.
Especially, note that Java class files and rc (.ini) files also go in
Resources.
Such an app bundle structure is what Apple strongly suggests one
should use, and it has been hinted that future versions of code
signing and/or Gatekeeper will require such a structure.
There is still some ugliness thanks to traces of the historical
separation of URE from "the office". Like there are two separate
"unorc" files, one for URE, one for the LibreOffice application. IMHO,
this should be cleaned up, but is probably controversial.
(Eek! I now see there are actually *three* unorc files in the app
bundle. Not intentional. Need to fix that later.)
Change-Id: Idcf235038deb5b8e1d061734993e9f31869b7606
|
|
Change-Id: I42bff46998b3d516161068af3d53bf92ee2f2b15
|
|
Change-Id: I5a01e4e71e69824f23998d23fc0bea60ddb27cc2
|
|
The build does not get far before it runs into trouble in the GNU libstdc++
headers, though:
android-ndk-r10/sources/cxx-stl/gnu-libstdc++/4.9/libs/x86/include/bits/opt_random.h:33:23:
fatal error: x86intrin.h: No such file or directory
Change-Id: I9d459c64980091ba8bf5b3d631d47342625f6be9
|
|
Change-Id: I08086caacb7abe0c85aeb184935b3f73666e9faf
|
|
Change-Id: I4c247916256618312e7d98673d9db4d26e26fa39
|
|
Nice to see what weird threads are running.
Change-Id: Ic9888aa3260d0aeb5858cde5415571ab23c75484
|
|
No point in doing it in build-time tools like cppumaker which don't
run as sandboxed processes. Just slows them down a lot, while cfprefsd
consumes lots of CPU doing user preference lookups in vain for every
file accessed through the uunxapi functions.
Change-Id: I83e55a8e8d0c4f2c60c60ecad2c831e42c9e5bfd
|
|
Change-Id: I1b5d3f881e82cea90fcc4e97c837251e340abffe
|
|
They were never working, at least since
65f82ca62922e2c487ba72b6d67ea7e3a03f8ebc
Which changed some mappings, for example:
-check( RTL_TEXTENCODING_IBM_437, RTL_TEXTENCODING_IBM_437 );
+check( RTL_TEXTENCODING_IBM_437, RTL_TEXTENCODING_MS_1252 );
But I don't know if that's the root of the problem.
Change-Id: I09f640a6c92a782f99d79405aab5f5dd2b992c6d
|
|
quite useful in this case
Change-Id: I2f9a5ef8ea9faf7a1ce62bd9b4925b3f36b19d1a
|
|
Change-Id: Ic1dae7aac2f4367b4196ba3128c0aea9be1fbbda
|
|
Argh, this is getting even uglier.
We cheerfully ignore for now the theoretical possibility that the URE
unorc used by build-time tools (i.e. the configure-expanded
ure/source/unorc) could be different for HOST and BUILD (in case they
use different --enable-canonical-installation-tree-structure), and use
the HOST one for the BUILD tools.
The right thing would probably be to construct the URE unorc in the
relevant Makefile, like we do for fundamentalrc? Or then to just
re-design the whole mess of rc files into some simpler (good luck).
Change-Id: I654309503d0e696778910acadcbf2f6b90ffa02a
|
|
Change-Id: Ib1b6abf7a2aca80e7206d792d31f2cdd0fbe3ff8
|
|
Change-Id: I082611edd88daae36181d62f9fc56b588630629b
Reviewed-on: https://gerrit.libreoffice.org/10828
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
so the strncpy branch is guaranteed null terminated too
Change-Id: I79df96d69633a4db76f865301690c7ce7533fce9
|
|
Change-Id: Ib795dc1b64d908942a836d09bfcde9f19d318b1b
|
|
Change-Id: I6b1f10e74aafc47e5618290d92087563c60e3c49
|
|
It fails with "assertion failed":
- Expression: compare_environments() [osl_process.cxx:370]
- Expression: compare_merged_environments(different_child_env_vars) [osl_process.cxx:428]
Just disable it (yet) for OSX <10.6 as it's already done for Windows.
Change-Id: I88e838481a25ee7e977512e33e23afa2b9d394ff
Reviewed-on: https://gerrit.libreoffice.org/10641
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I012027c38a6d2b06a4bb0cb53743d350b124ccf4
|
|
"Assertion failed: sequence not ordered" printed when running smoketest
Change-Id: Id9b7541f43342adf62137718dc332c0187c32f57
|