Age | Commit message (Collapse) | Author |
|
and which got horribly destroyed by merge-conflict resolution of
78f6407978aa48c3cbb116bc281940dbf1f2dcbc
Change-Id: Ie560416f4aa435e80c57a4449ee499fc5af20022
|
|
...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: Ib3763f20d74c22e28d519a9ac47f6f3ab4e31f51
Reviewed-on: https://gerrit.libreoffice.org/11983
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I33ca88f38210140931b12a05e426d1373243156e
|
|
Change-Id: I84250c6a40baeabd0d12958dfe0aea346d2b4fc8
|
|
Change-Id: Icb1732d867d86e2a1b180ce58cec4c91fc630881
|
|
Change-Id: I7146ee946c7f882decae2c8d93c63002d00dd511
|
|
Change-Id: I8b6743c4f2c5c3d52c7df33953ed3dd7b32a1a46
|
|
Change-Id: I59c96846aa5bf4780e4c5bfb0b71242e3b928976
|
|
Change-Id: Ibf43bf9ca241c6286659a16906a45bbafa798748
Reviewed-on: https://gerrit.libreoffice.org/11961
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Iee42ab31a82ed4459f956706cef513afcf70ef94
|
|
Currently the oox import creates a temp file and leaks it, and there is
no way to clean it up afterwards. Unfortunately it turns out that
SdrModel has no way to access the imported OOXML storage, so add a
really ugly hack to get the embedded media into the SdrMediaObj by
setting both MediaURL and PrivateStream properties (currently oox really
wants to set the properties in alphabetical order too...)
Change-Id: I5a235fbeb08e7bc17faf066de52b94867e9a79a2
|
|
Change-Id: I312fd6b964555b3b4aa4e22cec6054c9a83b2c52
|
|
Change-Id: I9572d4c7fb5f50b869e69436302ac8c20e500392
|
|
Change-Id: I5e16f68f8565a26857f4ef233e06a002209e2566
Reviewed-on: https://gerrit.libreoffice.org/11915
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I76cd8502f9cb2af379dd732d2081feb094599aca
|
|
and fix a couple of bugs in SC and SW where the call mode was
being passed to the hints parameter by accident
Change-Id: Ief805410b3f7035e012e229e77f92d5832430f58
Reviewed-on: https://gerrit.libreoffice.org/11916
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Ifa46523619ae7fb8d112525b8df594836e51bb8f
Reviewed-on: https://gerrit.libreoffice.org/11843
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Idcd31ea07c460aaecff3ea56072cf65e26040bda
|
|
Change-Id: I18d4f6af97fcfb9341711751a8d550e421f7a381
|
|
Change-Id: Ifbad1fc30665ff6b9238b5fb4728fc302b0458fa
|
|
Change-Id: I830210b44459cf94eabb983b99bfcc104f865492
|
|
there was two of these, see
commit 164bd64255d71d96e35e908f6425acf8372795fb
Date: Thu Oct 2 04:44:33 2014 -0500
coverity#1242757 Result is not floating-point
Change-Id: I1f66191f329b2d87bdeb52aa1a7528ca662064a6
|
|
Change-Id: I3c4aeb3318af7cdb964128dd78f7d2cfba48000e
|
|
Change-Id: I02b9c31146396a39a624fbb869a5880b176e0f9e
|
|
Change-Id: Iedf06cc78cda022112699be3db650d11c7249adc
|
|
The problem is that utl::TempFile::GetURL() will return an empty string if
the file UCP is not available. There will be an entry in the log file:
warn:legacy.osl:25344:1:ucb/source/core/provprox.cxx:366: UcbContentProviderProxy::getContentProvider - No provider for 'com.sun.star.ucb.FileContentProvider.
Change-Id: I2741a04cf4b8b420294ae521199706921b37e5c6
|
|
Change-Id: I479da454235a7cfc084ae1517ab9ffa07580ab0c
|
|
Change-Id: I98526bfd6f69d95d5c309e67c397bc85fb8817ab
|
|
* Added rational util functions used by Fraction class not
available in the boost::rational class.
* Replaced usage of Fraction by boost::rational<long>
* Removed code that relies on:
1. fraction.IsValid() -- rational only allow valid values, ie
denominator() != 0
2. rational.denominator() == 0 -- always false
3. rational.denominator() < 0 -- always false but implementation
detail: http://www.boost.org/doc/libs/release/libs/rational/rational.html#Internal%20representation
* Simplified code that relies on:
1. rational.denominator() != 0 -- always true
* BUGS EXIST because Fraction allows the creation of invalid values but
boost::rational throws the exception boost::bad_rational
Change-Id: I84970a4956afb3f91ac0c8f726547466319420f9
Reviewed-on: https://gerrit.libreoffice.org/11551
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
Apparently fc04f76336fdf8c96e35382cdeb497e2f939705c used some sed script
to change all ...
Change-Id: Ie609bd02a2c5d70109fc6185cf4440480f29d8f5
|
|
after my hasty commit
bc00203 "convert PathKind enum in SD to enum class"
Change-Id: I97263e907bf7363121dc1b437eaf564efdb66ad2
|
|
to get the NONE constant out of the global namespace
Change-Id: Iecbe39126c1ec08aa852fcb903cee1a6e6f4594b
|
|
If you right click on a field in an inactive text box, then a mouse down,
context command, mouse up sequence is sent. The mouse up is supposed to be
captured by the context menu here. But it doesn't because the mouse down in
draw sends an artificial mouse down to the editengine which eventually sets up
a MouseCapture window so the context menu doesn't get considered (because it
occurs before the capture window is released) so the mouse up gets send to sd
while the context menu is up and that causes all sorts of havoc.
Change-Id: If3b7c501586f8561be484d0828c630d84644f770
|
|
Change-Id: Ifa0c1b1983d1cc7f22d55b15dee88b7f709dbcf8
|
|
It was intended to disable sdremote by default when it exited
experimental mode.
This reverts commit 576943b9bf7506829de97d2194c4bee35a485436.
Change-Id: I2c1b5443e334021bd9574316167f48b1af6200e5
|
|
Change-Id: I76582bb6a1a8dee56611f1463d3e63246eefb310
|
|
Change-Id: I47ddb95018109e5e771324b699dfdc10f529dfc4
|
|
from PptSlidePersistList.
This also puts in assertions that every entry in a PptSlidePersistList
is non-null.
Cf. 9536a28ede89e8c29940e8ba225215a45667062c
Change-Id: Iea35532ed668e58a3f02d3f3433a66c2d4e1fe2a
Reviewed-on: https://gerrit.libreoffice.org/11807
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
and automatically seed from time on first use
coverity#1242393 Don't call rand
coverity#1242404 Don't call rand
coverity#1242410 Don't call rand and additionally allow 0xFF as a value
coverity#1242409 Don't call rand
coverity#1242399 Don't call rand
coverity#1242372 Don't call rand
coverity#1242377 Don't call rand
coverity#1242378 Don't call rand
coverity#1242379 Don't call rand
coverity#1242382 Don't call rand
coverity#1242383 Don't call rand
coverity#1242402 Don't call rand
coverity#1242397 Don't call rand
coverity#1242390 Don't call rand
coverity#1242389 Don't call rand
coverity#1242388 Don't call rand
coverity#1242386 Don't call rand
coverity#1242384 Don't call rand
coverity#1242394 Don't call rand
Change-Id: I241feab9cb370e091fd6ccaba2af941eb95bc7cf
|
|
Change-Id: I05b63dc00023945d92fd837bd64042135d3f9aed
Reviewed-on: https://gerrit.libreoffice.org/11648
Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
|
|
Change-Id: Iae8ac2ad9d1713512619c5c33eeee9a21b5835f1
|
|
Change-Id: I2be3e6d88472ba8b69f012aaf975a91f0ab4f693
|
|
Change-Id: I7031e7408f5bcd9ae85fc50952253e14d2fb2edf
|
|
The same pattern as introduced elsewhere recently.
Change-Id: Ie7b4fd1ec26d417f63bee26dd336dfc54f4847dc
|
|
Change-Id: I18a9355bb902b19f0eeecb26d33eba4fb25db8cb
|
|
We need to set the "paragraph summation" flag also for .pptx import. (As is
already done for .ppt import.)
This is what in the Options (Preferences) UI is called "Add spacing between
paragraphs and tables (in current document)", on the LibreOffice
Impress:General page. This is a setting that when set becomes local to the
document. I actually don't really understand why this needs to be a
user-visible option, wouldn't it be enough for it to be an internal flag that
is set for documents imported from .ppt and .pptx, and is clear otherwise?
Change-Id: I525178957276e7d587b6ac4be8ae7919fc245a27
|
|
Change-Id: I88b96d42f5ee96641dc21e3e55415f6dc32b3951
|
|
Change-Id: I419b217df306d41ed99e603bae9381f54c633598
|
|
Change-Id: I2c4295c7d308257290aaf04181805d7dc71d6553
|