Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959
Reviewed-on: https://gerrit.libreoffice.org/12164
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Ic5edb7c0977b6c589329cd80616aa320d6cc929b
Signed-off-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: Idc40752b4beee932b5912e6df9fe6acb15571201
Signed-off-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I5ac499df1f4063a80a5d41f60778106a7b96940d
|
|
Change-Id: I0d4f9b14f2a41a3acb08ba1b6c13fdf1174021e7
|
|
...because rtl_ImplGetFastUTF8UnicodeLen has already checked that.
Change-Id: I17f2b80f374073934a8f0b1a97099d4dec89ce4e
|
|
Keep the fast path fast, fall back to the text encoder in case there's a
fly in the ointment.
Change-Id: I94507856a7f3170f770adb741aa1e282d0d2400c
|
|
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
|
|
...so (highbit(ave) - 1) will never be negative and never cause undefined shift by negative
value.
Change-Id: I86703f03d30d48c74d2e814c182b9edb85d18186
|
|
...so arena->m_hash_shfit will never be negative and never cause undefined shift by negative
value.
Change-Id: Ifc3d28d53bae38bc8deea72473c81f1d043dc18e
|
|
...there should not be any legitimate reason these fcntl calls can fail, anyway.
Change-Id: Ib5cbe5b40ff42656fe3bad2ac50dcf6faeca0e9d
|
|
Change-Id: I02e81d526c07297a5bc3664cc475302a552187eb
|
|
Change-Id: Ia46417525e655a5a80f9e70f25f65e4916e4c8d8
|
|
Change-Id: I92ce690a6f8183ce77c8ed53bd2fcb45d39d756e
|
|
Change-Id: Ia43976d84eede6f699381bc4f3daf89b95e4cb4f
Reviewed-on: https://gerrit.libreoffice.org/12150
Reviewed-by: Bryan Quigley <gquigs@gmail.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I808da98a3b825eb2213dbcb6d435baa58bf54a9d
Reviewed-on: https://gerrit.libreoffice.org/12151
Reviewed-by: Bryan Quigley <gquigs@gmail.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
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>
|
|
Change-Id: I08a5fb662a4a5d756b72340cb9328fc1dc1fe9fe
|
|
etc.
Change-Id: I1548d60280ab4f74fe023d026435a05f7865b516
|
|
Change-Id: I626bd8d29a7efe967608efbb21fda87e5692c0c8
|
|
...in sal/osl/unx/signal.c ReportCrash, so do not implicitly burden that on every
--enable-dbgutil --disable-crashdump build.
Change-Id: I2d03ea2715697bf7960ca7e9e55c72489ce24bae
|
|
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
|