Age | Commit message (Collapse) | Author |
|
Change-Id: I822c5ebc321ebda87c238a1781a31793c1623e34
|
|
Change-Id: I8c73d95d92dc4a30dd6a71260817ab4fc3096f27
|
|
Change-Id: I9219619b538b6530a89f5932ac51eb3b62eb396a
|
|
This was unused since the earlier cleanup.
Change-Id: Ia56641c4242037a0ce501e43939b8dc862499f0e
|
|
Change-Id: I55f0a5bd01a3e250764e1eb34d674b535b85b56f
|
|
Change-Id: I132d22e2af3cf673c17d8964f690d48990834884
|
|
Change-Id: Ie54d340478412e62b87d66e287fd8a3963e97898
|
|
Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
|
|
Change-Id: I0ad9681a8b31d78cefce5b66040415154a1c7a99
|
|
Change-Id: I3475be796cf2655d9b619b86c9686aeef4b97b82
|
|
Change-Id: I5514898f588b21bafceefca95e3276826cb9a882
|
|
Change-Id: Iaa65658aed6bb4abb20a4d95dc9c6caf7c1c764b
|
|
...superseded by CppunitTest_odk_checkapi
Change-Id: I5df89a1a12f496984deab153f0ba2734323c1293
|
|
Change-Id: I09049db51668176b90dabe2c82d525b2fab60299
|
|
...for cases where ? "a" : "bb" does not work, as well as to work around the
MSVC bug for cases like ? "a" : "b".
Change-Id: Id404716047aca5cc81440f291616d92365379b8f
|
|
...to use single ASCII character literals "more directly" in the OUString API
(instead of having to go via an intermediary OUString ctor call). Especially
useful for character literals that are defined as const variables or via macros
("direct" uses of character literals in the OUString API can often simply be
replaced with single-character string literals, for improved readability).
(The functions overloaded for OUStringLiteral1 are those that are actually used
by the existing LO code; more could potentially be added. The asymmetry in the
operator ==/!= parameter types is by design, though---writing code like
'x' == s
is an abomination that shall not be abetted.)
Change-Id: Ic5264714be7439eed56b5dfca6ccaee277306f1f
|
|
Change-Id: I4382504549eb1e3a8c34495a21d6df10a86ac23b
|
|
Change-Id: I568c2e0c9b5b234d941e17d57bcb53374626909e
|
|
Change-Id: I4a42c9f795cfbaa0f6c7b68d8189204a63da2321
|
|
Change-Id: If89443c1668ca01c826c8fc353b20097c7da7bf4
|
|
Signed-off-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>, undid one remove that was
detrimental to loplugin:unreffun
Change-Id: I18d8252084d828f94ef7a954e1dbfb45743d7970
|
|
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
|
|
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959
Reviewed-on: https://gerrit.libreoffice.org/12164
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
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).
|
|
Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
|
|
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
|
|
It turns out that almost none of them were necessary.
Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83
Reviewed-on: https://gerrit.libreoffice.org/12133
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
...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
|
|
Change-Id: Ic90e806d279f8180d78f78597dbc549264338295
|
|
Change-Id: If067f755b99480b7b7bd0bd3bb7a71a447794273
|
|
Change-Id: I7e6490c5b50a236a5f070e98d4501e7dd5a90307
|
|
Change-Id: I9e0e06331f134fe89c9942f93b7546b0e67bb821
|
|
Change-Id: Ibf4f2d46bfbb977f0ba27110d86d7b7f1173631c
|
|
Change-Id: I4412a0c1e4dee94cd717f24b8df5e257d895f721
|
|
Change-Id: I68eee76c8710c89af8e3c1e1006345f908923ece
|
|
Change-Id: Icd477e433f0c294c823c64147d49fb4ac6bfb447
|
|
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>
|
|
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
|
|
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: Ic3ab061ab8d3578130dfa8a637e5351d4f5d766e
|
|
Change-Id: Id7c517fb37bc28797c45fc0dde83e866f2aa4aac
|
|
Change-Id: I4a6280f47ca3c4a77b4e42fe05d79ded7fc30ef1
|
|
Change-Id: Ia4111a0fb76c848d3bd2d01f3835995e4c007b6b
|
|
Added header guards to files in directories sal/, sal/, and scripting/
Change-Id: Ieb7f224f2d27bd671618c516f47f5b7f08c1d294
Reviewed-on: https://gerrit.libreoffice.org/9582
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|
|
Change-Id: I7acba38a00dfc1101e6217bff098ef7d6f032497
|
|
Change-Id: Ib849ff0839bba80d5695c0f78dd18448b668ec78
|
|
Change-Id: If20b1108af89dec75d8c6a232290ab28e51d09d4
|
|
Change-Id: I17dc424d485053d3eb09e8eb0dd01dd4b4672bfd
|