| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Change-Id: I96b4d44e44389245f3ccc4a08198676461c6a38b
|
|
|
|
| |
Change-Id: Id78de487b36a1c939dc3a4a5b38ab0b77999478c
|
|
|
|
|
|
|
| |
...the generated .ilb, .lst, and .sid outputs requested by SdiTarget were
apparently unused.
Change-Id: I1abb2abc7945070451fce4a98a11d955515e3f63
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
...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: I31bbeb5f068754d6dc77c22d759058089b03c9b1
|
|
|
|
| |
Change-Id: Icca5a0dee296fae1abeb78ea8ffa2f9e934bb111
|
|
|
|
| |
Change-Id: Ie12862f8df701298db51ed45b5c24814a6f94def
|
|
|
|
|
|
|
| |
Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5
Reviewed-on: https://gerrit.libreoffice.org/9360
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|
|
|
|
| |
Change-Id: I8b15c97b1fcc6acaf567e8b1d2d37cbc99687a9d
|
|
|
|
| |
Change-Id: I665e901eb6f65308b8f130da1c6da95bc3255490
|
|
|
|
| |
Change-Id: I0ef2e67f6d61e0ce118c0f5e926b8194ef9d8058
|
|
|
|
|
|
|
|
| |
It appears that the C++ standard allows overriding destructors to be marked
"override," but at least some MSVC versions complain about it, so at least make
sure such destructors are explicitly marked "virtual."
Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
|
|
|
|
|
|
|
| |
...mostly done with a rewriting Clang plugin, with just some manual tweaking
necessary to fix poor macro usage.
Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
|
|
|
|
|
|
| |
This is working version of fc27a685d466dc380a8b68cd48f47db632ed3693
Change-Id: I48e861b5b7692ea051edf52d7ad9cc54ca989e22
|
|
|
|
|
|
| |
Forgot to test properly on Windows :-(.
This reverts commit fc27a685d466dc380a8b68cd48f47db632ed3693.
|
|
|
|
|
|
|
|
| |
Export these symbols from sfx library.
For cases when we link together more libraries, there are more symbols,
so I've kept __attribute__((__weak__)) hack there.
Change-Id: I8b7f6fc9bea479dca5062344c04b74c1f3eb36b2
|
|
|
|
| |
Change-Id: I5c34f9bc96ee23fca6182740e3279de1485c61ea
|
|
|
|
|
|
|
|
|
| |
in favour of ReadXXX methods.
Change-Id: Ic2c0a7b6b92ff4c236ae99b39d77f3d935b301e3
Reviewed-on: https://gerrit.libreoffice.org/7915
Tested-by: Michael Stahl <mstahl@redhat.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
|
|
| |
Change-Id: Ib652a8a1f81dde1f5ac3b6b7b73376c73777247c
|
|
|
|
|
|
|
|
|
| |
.. and convert the last few places still using those methods.
Change-Id: Id2cd8f9c0dd281df43af439d4fef65881f34a6fd
Reviewed-on: https://gerrit.libreoffice.org/7495
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
|
|
|
|
|
|
|
| |
This is in preparation for more conversion of SvStream::operator<< calls
to use more explicit method names.
This converts the subclasses that have their own convenience overloads
of operator<< to use normal methods.
Change-Id: I5efd5d9a24c264cb86d2471303dd5849bf91ba80
|
|
|
|
|
|
|
| |
According to the dmake documentation, ICC refers to Visual Age C++ for
OS/2, which is not a supported compiler (or platform).
Change-Id: Ic9e23bc7c44de110a3a312bd007beda3b660927d
|
|
|
|
| |
Change-Id: Id0d07ff9bcd4858cb74458eaf13fb9386387f455
|
|
|
|
| |
Change-Id: I4278999b9b7d184c26036bbe9e3b98420f461e8c
|
|
|
|
| |
Change-Id: Idf8e0e02ee2fde23959a281cc6ffa1a43ee7f239
|
|
|
|
|
|
|
|
| |
Modules sal, salhelper, cppu, cppuhelper, codemaker (selectively) and odk
have kept them, in order not to break external API (the automatic using declaration
is LO-internal).
Change-Id: I588fc9e0c45b914f824f91c0376980621d730f09
|
|
|
|
|
|
|
|
|
|
|
|
| |
Removed uses of fsys from database.cxx, command.cxx, module.cxx, svidl.cxx.
Also removed a lot of cruft from svidl.cxx and command.hxx; unused variables
and code relating to them, eg. variables aCxxFile, aHxxFile, aSrcFile, aODLFile.
Changed some Strings to OUStrings in svidl.cxx and command.hxx.
Change-Id: I708284f70435c8f65cdcfc954af43976e28b8c9d
Reviewed-on: https://gerrit.libreoffice.org/2935
Reviewed-by: Andras Timar <atimar@suse.com>
Tested-by: Andras Timar <atimar@suse.com>
|
|
|
|
|
|
|
| |
Change-Id: Ic547b4a59102afc8bdbc85ec95ac312e32ef2530
Reviewed-on: https://gerrit.libreoffice.org/1484
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
|
|
| |
Change-Id: I6fc91959f024b9a1f9e8633af8e16048752f6bdc
|
|
|
|
|
|
| |
Converts it to a template based on std::vector
Change-Id: Id7f89f494164c61a3a573cff443ac0e0488e93f1
|
|
|
|
| |
Change-Id: I184a7f9c65dd762407b2431d9c6557e718b954b2
|
|
|
|
| |
Change-Id: Ibf41655d40c9534629bbb0dd5ae600791b42922d
|
|
|
|
|
|
|
|
|
| |
cws mba34issues01: #i117709#: make sure that parent of error message box is visible
Patch contributed by Mathias Bauer
http://svn.apache.org/viewvc?view=revision&revision=1172346
i#119036 - improve user experience of certification dialog - only shown once
Patch contributed by Oliver-Rainer Wittmann
http://svn.apache.org/viewvc?view=revision&revision=1299727
|
|
|
|
| |
Change-Id: I0d6aad17c471b6edd584c668dc7ff054090c185c
|
|
|
|
| |
Change-Id: I6c145e984c885c7e06caa1c27bfb354ea49ad9ce
|
|
|
|
|
| |
Along the way, convert the table parameter passing to using references since we are never passing
a null pointer.
|
|
|
|
|
| |
Convert usage of tools/table.hxx to std::map in aPTable field of
SvPersistStream class.
|
|
|
|
|
| |
I can't see that WriteSrc, WriteCxx WriteHxx are anything other than unused
archaic remnants of the past.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|