Age | Commit message (Collapse) | Author |
|
...with 09d5b31ebe187a4cd9f3efca5a7cb7f87e1e19f6 "loplugin:constantfunction:
store," which apparently over-eagerly reduced functionality that happened to be
constant in one specific implicit instantiation of a template, but not in
others.
(Plus, adapting the original code to silence false loplugin:staticmethods
warnings.)
Change-Id: I156df3aef1b3b0c9d36b2a54c57de29948749bde
|
|
rtl/string.hxx and rtl/ustring.hxx both unnecessarily #include <sal/log.hxx>
(and don't make use of it themselves), but many other files happen to depend on
it. Cleaned up some, but something like
grep -FwL sal/log.hxx $(git grep -Elw \
'SAL_INFO|SAL_INFO_IF|SAL_WARN|SAL_WARN_IF') -- \*.cxx)
shows lots more files that potentially need fixing before the include can be
removed from rtl/string.hxx and rtl/ustring.hxx.
Change-Id: Ibf033363e83d37851776f392dc0b077381cd8b90
|
|
ie.
void f(void);
becomes
void f();
I used the following command to make the changes:
git grep -lP '\(\s*void\s*\)' -- *.cxx \
| xargs perl -pi -w -e 's/(\w+)\s*\(\s*void\s*\)/$1\(\)/g;'
and ran it for both .cxx and .hxx files.
Change-Id: I314a1b56e9c14d10726e32841736b0ad5eef8ddd
|
|
Change-Id: I1f6c6d13697aa397067478d6b07429120106e6bd
|
|
Change-Id: Ie38450a082dc840714051ad65c297886038e68f3
|
|
Change-Id: Ibd23c1d96b093ae2670e7752691dad29c081639f
|
|
Change-Id: I13284409404ef77eb16164b7ce2074259d2cad12
|
|
...for a 32-bit build, similar to what ee11e221d2108212619e1bbe7f029e7d9afdba32
"tdf#43157: Fix format string violations in OSL_TRACE etc." did for a 64-bit
build
Change-Id: I05dd79ede3e66cb9ab7a33792319eb34b34c82dd
|
|
found with
$ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;'
Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
|
|
Change-Id: I87b252d3d6e5322dbc4b6cc77b846766a0d35893
|
|
and we can include a few less headers
Change-Id: Id742849ff4c1c37a2b861aa3d6ab823f00ea87f8
|
|
This cleanups up indentation and removes dead classes.
This is a followup patch to commit
272b1dd55797aacf511fb4342b0054e3697243f6
"new loplugin: change virtual methods to non-virtual"
Change-Id: I1c2139589cf8cb23bb9808defe22c51039d38de1
|
|
Where we can prove that the virtual method is never overriden.
In the case of pure-virtual methods, we remove the method entirely.
Sometimes this leads to entire methods and fields being
eliminated.
Change-Id: I138ef81c95f115dbd8c023a83cfc7e9d5d6d14ae
|
|
See http://nabble.documentfoundation.org/cppcheck-arrayIndexOutOfBounds-report-in-stortree-cxx-store-module-td4136432.html
Change-Id: I365befacf7d10bcbf0a159cfdcabd3eb1644d02b
|
|
...probably not worth dragging that unused code around any more
Change-Id: I4f22a704af862d8a1fe1f2202609f355ccee7c9c
|
|
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
|
|
and
coverity#1242867 Unused value
Change-Id: I8ad069a5a84b97e956cafcd925fd733fb5ed9b2e
|
|
Change-Id: I416144a4ccf2856ce85ab4b24a69958615276cb9
|
|
Change-Id: I16430567b3d57016458bd7877c36bc6e01197741
|
|
Change-Id: I650fab8227fd8b6aeb4fd4d420cb820aefd80271
|
|
Change-Id: I5f05fc312032c50639f147f27353aa560a69f65a
|
|
to replace hand-rolled version
Change-Id: I30635aec81313e4e0d1b67b30c8992fd63bb1f67
|
|
VS 2013 warns: Conversion rules for arithmetic operations in the
comparison mean that one branch cannot be executed.
Change-Id: I85f6df16b0ced94677568441d87a588bed1bfb9b
|
|
Change-Id: I2317522af91c8c60672c72a3b514767039930e22
|
|
Finish the work of ec8617568ff091026b2f17cce3b547b1633fa6c4
Only one is left: SC_PROGRESS_CXX
Change-Id: Ifdc07ec377d7a84e92bc4b0a66ab82b1fc93f76a
Reviewed-on: https://gerrit.libreoffice.org/9636
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|
|
Change-Id: I9464179a736b91f5bc32eb5e5e89b3b4e3da3494
|
|
Change-Id: Ia89a2dce799c7ba90bcd19919195bfd2c7a4a180
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Look for code like:
xxx ? yyy : false;
Which can be simplified to:
xxx && yyy
Change-Id: Ia33c0e452aa28af3f0658a5382895aaad0246b4d
|
|
Change-Id: Ic25bd678dc299627299b22145efd7bebcf2b39d0
|
|
Change-Id: Ie264bfec4a8c34adcaa45c0a5cc43c1109d31c95
|
|
Change-Id: Ib07a18d6592432e9e1716641938a6f336daa7b4b
|
|
Change-Id: I1dbb864d030b61417f6d137fc967f158e9243ed7
|
|
never defined, follow-up of 512de6e75d34d2144392d1e78c25446f2d0b3a35
|
|
Apparently this tool was available for early versions of NT; googling
finds a caplib.zip from 1992 but that is missing the CAP.dll so there's
no way to use this anyway.
http://cd.textfiles.com/cica/cica9308/UNZIPPED/NT/CAPLIB/CAP.TXT
Change-Id: I4718af0e175b78bcdad38bbca0fc9579b3b12111
|
|
...mostly done with a rewriting Clang plugin, with just some manual tweaking
necessary to fix poor macro usage.
Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
|
|
Change-Id: Ia58af7d32904de01fdb5291203938c748cae94c6
|
|
Change-Id: I6753eb480618af747ecf4a3ae3c666257761ce70
Reviewed-on: https://gerrit.libreoffice.org/8317
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I7d108e7ae387f9c07cce182a0bb09b69a6608226
|
|
Change-Id: Ibc5e3084e4bec40005779854ce7552a2eb39a7ed
|
|
Convert code like
aStr.compareToAscii("XXX") == 0
to
aStr.equalsAscii("XXX")
which is both easier to read and faster.
Change-Id: I448abf58f2fa0e7715dba53f8e8825ca0587c83f
|
|
Change-Id: Id476a0eb1f5965098c8cc2e2847547b1d23387eb
Reviewed-on: https://gerrit.libreoffice.org/4753
Reviewed-by: Petr Mladek <pmladek@suse.cz>
Tested-by: Petr Mladek <pmladek@suse.cz>
|
|
Change-Id: I1ec3d7845477cfb5bc6a3d7f70ddabef12583e1b
|
|
Change-Id: Id6b8f4ed6df14bd80a7937001ec1cfb98acd7f52
|
|
|
|
Change-Id: I8e9f70eb5d929c98b4379416c2259a74e31d587f
Reviewed-on: https://gerrit.libreoffice.org/3503
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
Change-Id: I400fad08c0ae7b6b34bad63693f54856867e4dac
Reviewed-on: https://gerrit.libreoffice.org/3502
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
see https://gerrit.libreoffice.org/#/c/3367/
and Change-Id: I00c96fa77d04b33a6f8c8cd3490dfcd9bdc9e84a for details
Change-Id: I199a75bc4042af20817265d5ef85b1134a96ff5a
|
|
Change-Id: I2b2099d8fc00062f67c42e73c4b8a17a689db89d
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
This reverts commit 5c60cebafdc6d6d773c794c0a4ee58d63782df22.
The "store" library is not actually a stable URE interface.
Conflicts:
odk/CustomTarget_doxygen.mk
Change-Id: I221696515ba07b0155406a503c33760c58d1338f
|
|
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
|