Age | Commit message (Collapse) | Author |
|
Change-Id: Ifafdaf6da0225f244853a0042a6458643b570623
|
|
Change-Id: Iadba30885ba3bab7aaee58e298a9dc6b22c8b977
|
|
Change-Id: Ie5a37a49f1b1c08a20f89e167ca415fc645ed40d
|
|
Change-Id: I5b2b7044c77de899ce8dc1e44f3673835bd3ce4c
|
|
Change-Id: I5c6fae963c4aabe15d82a51f5801e62656188051
|
|
Change-Id: Ia058e42700c7724efd74fd6c5560ccca919fb2e8
|
|
just makes it harder to grep for methods
Change-Id: I0925db2f9284972fcf7e1e3adf90cbe21f69eb15
|
|
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
|
|
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
|
|
...to silence a clang-analyzer-deadcode.DeadStores on the last update of pBuffer
Change-Id: I144b64c330c178f78109fefadacf62fca3765694
|
|
- improvements to the plugin to find more method calls
- improvements to python script to remove more false+
- fix the FORCE_COMPILE_ALL build flag to include code in
the $WORKDIR
Change-Id: I4d6015dcb9b9d60c26f0bcee8abad807177a7836
Reviewed-on: https://gerrit.libreoffice.org/19064
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I42c0ab2012553e11f8c1372103488ba1d291d5a9
Reviewed-on: https://gerrit.libreoffice.org/18965
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
|
|
Change-Id: I35456b2a3ad2a84a1d045f09cdfb29e4c19b8350
|
|
This addresses some of cppcheck's "redundantAssignment" warnings
("Variable '<varName>' is reassigned a value before the old one
has been used").
Change-Id: I8fd2950d5aa1a3f4bd9e1c4c336abe465d1f657e
Reviewed-on: https://gerrit.libreoffice.org/17926
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I3968bcd55ab4d0ff6215f76be242acc17cf5ccab
|
|
find places where we should be using std::unique_ptr
Change-Id: I5b9defe778fdc4738ecea381215396874db59e66
|
|
Change-Id: Id2654555c4042f8c0bdbd6bab6507e705f08326b
|
|
Change-Id: Ic0115e76076e84cc132a9fc9b0ce6a90cd9f639c
|
|
Change-Id: I020c7ce4d9aa93cb0292378a410cdb9f7788e920
Signed-off-by: Michael Stahl <mstahl@redhat.com>
|
|
(and note the operator == vs. = typo)
Change-Id: I9c25149967ed9b80f5e8f014de0dfee2e8f485d8
|
|
Change-Id: I313adfb72272a631299efb3d2464873853a126be
|
|
Change-Id: Ic4058c133a676646bc0c5d1e52207471bb78a1cb
|
|
(and note the operator == vs. = typo)
Change-Id: I4567927f3f702f8e74b168becce4002e84d5c7a0
|
|
Change-Id: I43710d057043a258ec9ab94bca0703434098e6c9
|
|
Change-Id: Ib615ff7eb12232b0b83664af134744d96754ffd8
|
|
Change-Id: Ie453a7c97a762932884c5c41b13607a170368107
|
|
Change-Id: I72bdd6627a191a65d5c3b479c100e30442a8bb9e
|
|
Change-Id: Ie78eb881a7fc47d0cd7b9862bd0cd200153ce77d
Reviewed-on: https://gerrit.libreoffice.org/16679
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Idea originally from caolan.
Found using the following command:
find . -name *.cxx | xargs /opt/local/bin/grep -zlP '(?m)if\s*\(\s*\w+\s*\)\s*delete\s+\w+\;'
Change-Id: I3338f4e22193a6dfd6219c8c75835224a3392763
|
|
Change-Id: Id6291335945a8f10cef3afd1a2b3bae65fdf4562
|
|
Idea from bubli - look for loops where the index variable is of such
size that it cannot cover the range revealed by examining the length
part of the condition.
So far, I have only run the plugin up till the VCL module.
Also the plugin deliberately excludes anything more complicated than a
straightforward incrementing for loop.
Change-Id: Ifced18b01c03ea537c64168465ce0b8287a42015
|
|
Change-Id: I07d9567da23094836134a54afce660500bf1b3f5
|
|
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: Id4194f4d5bb6fcf064985fddc6f7344a4d34ca04
|
|
more 'extern C' fallout from my conversion of enum to scoped enum
Change-Id: I4c9aabbfbd255775a8f3edc2b7c8c62647f539eb
|
|
since after my converting stuff in include/registry/types.h to use 'enum
class', it can obviously no longer be "extern C", so drop the "extern C"
and rename the file to reflect that.
Change-Id: Ia30f5731316525e48531c4785ab7471a428bcf6f
|
|
Change-Id: I8320f6f42d5579fbd09450ddca61c4c066de98e4
|
|
Change-Id: I81599570698eb92abf14fa6386d8545c2031e863
|
|
Change-Id: Ifef0aaae6208f18c1352fef7c471deb60a97820d
|
|
Change-Id: I55977c38578cb59deb195d91f8948da2e0905b6a
|
|
Change-Id: Ic672e75db4d7323760577b19490ffa28d38965b6
|
|
since we no longer support the RG_LINKTYPE stuff
Change-Id: If388ecfa0c475471b99b26155ad554ec702ca734
|
|
Change-Id: I11d92218c5e0678d497f8964723033e2bd8b3300
|
|
Change-Id: I192bc9653d8492a850a5e259e15782bd3bb24922
|
|
Change-Id: I10c30ef28e7321882a720d1e7869a75a89febfc5
|
|
Change-Id: I32d7a1d4fed6ef5c56f485cd92464f2d0960e5fd
|
|
Change-Id: I0556481aee672451221aa72a8a90d465f3670313
|
|
found with
$ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;'
Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
|
|
Change-Id: I52253f6b925a943edc2984ce4d9462bd1edb5f51
|