Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: I8563922204d3ec93bf138cbd1aaba620c0215bf7
|
|
Change-Id: I04e53d7de9f2f26e9338a82f7d5ae5dab1682712
|
|
Change-Id: I977b0a51a609d3cbc30abd308715f91cad2cfdfa
|
|
Change-Id: I7285c90c82b04e3ebb2db371b675689de9c183d7
|
|
Change-Id: I3458ae46c256206e31ad268f47e8b92a30f3738e
|
|
Change-Id: I6af399ef26d3f3ee97a49592cf97c590c0781eb0
|
|
Change-Id: Ieda14d984f1614056c701a50ce637ea6fe9620f3
|
|
Change-Id: I5f114f4ca92d991d17ad871d11a78a5306191f9c
|
|
Change-Id: If7652a7c0251b741660365848a717c06954ca419
|
|
Change-Id: I8d5363bb2ad813ef29b4078848914f4b0989b9f1
|
|
Change-Id: I481c153815fff793e9334153d863f26982b879cb
|
|
Turn the Link class into a template abstracting over the link's argument and
return types, but provide default template arguments that keep the generic,
unsafe "void* in, sal_IntPtr out" behvior. That way, individual uses of the
Link class can be updated over time.
All the related macros are duplicated with ..._TYPED counterparts, that
additionally take the RetType (except for LINK_TYPED, which manages to infer the
relevant types from the supplied Member).
(It would have been attractive to change the "untyped" LinkStubs from taking a
void* to a properly typed ArgType parameter, too, but that would cause
-fsanitize=function to flag uses of "untyped" Link::Call.)
Change-Id: I3b0140378bad99abbf240140ebb4a46a05d2d2f8
|
|
Change-Id: If2d2970160803bcaa1feabb8274f83bba0ba722c
|
|
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
|
|
Change-Id: I10d50f76dff5d5e456b28393c48b4de21201779b
|
|
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
|
|
...which removes the need to abstract over the standard URI '%' escape prefix
vs. the silly vim '=' special case invention.
Change-Id: I54a52dd912c3aafc38275a0ac2466a6daeec328f
|
|
...to avoid clashes with macros and poor warnings about hiding global FILE
Change-Id: I444e8b958f029201b228937e088efabc4b6d3050
|
|
...and fix o3tl::enumarray::operator [] const overload
Change-Id: I749b1b9d68686b03a97074253478d9d2d9d32b0b
|
|
Change-Id: I3ae555fd9f0dc3c11ea301ee1cb2c799fbec820d
|
|
Change-Id: I07d6cd7191c4ac1eeca4bba5edf8f86dd9d9467e
|
|
Change-Id: I413b3716b3c5bda204619e8b1ff2b724f95a5c38
|
|
Change-Id: Ia0441caec222227f173a9508530d7c4cefd7892e
|
|
Change-Id: I4b7b04031fa74956379c5a1b21abe10b0717f3e9
|
|
Change-Id: I1b49c020d597b569e330482f4dbf20c15ccdae3f
|
|
Change-Id: I1d157bfc82f09f2438e59eca4fcd92931ca38723
|
|
Change-Id: Ib6842b6215ff70a31299d1410ed66c3ea34e7c78
|
|
this appears to be completely unused
Change-Id: I74dad8b0c478a1aed035ce9cc663a74de157816e
|
|
Change-Id: I7e53cfc90cefd9da7d6ecd795b09214bd44b1613
|
|
Change-Id: Ice4963da8a856b1d2ada13e5b699157a0217e06c
|
|
of which there are several.
There are some issues here I am unsure of
- the SW and SC and CHART2 modules essentially ignore the enum values and assign their own ids
Perhaps I should change them to use the common values and create new enum values where necessary?
- the sc/qa/ and sq/qa/ and starmath/qa/ code was doing some dodgy stuff. I translated the code to pass down the stuff
numeric values to the underlying code, but perhaps further fixing is necessary?
Change-Id: Ic06d723e404481e3f1bca67c43b70321b764d923
|
|
Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
|
|
Change-Id: I8484b985e46ad34fa45b02aa07130d3259336082
|
|
found with
$ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;'
Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
|
|
Change-Id: I6144d7d6527dc401b7f1b577d1a227361e39e7bb
|
|
Change-Id: I955c8ac4dbe002d23531df7eb10fb4444d6b5157
Reviewed-on: https://gerrit.libreoffice.org/14292
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
i.e lots now able to be detected after...
commit b44cbb26efe1d0b0950b1e1613e131b506dc3876
Author: Noel Grandin <noel@peralex.com>
Date: Tue Jan 20 12:38:10 2015 +0200
new loplugin: change virtual methods to non-virtual
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: I605e2fa56f7186c3d3a764f3cd30f5cf7f881f9d
|
|
Change-Id: I7192a4d29937fc6222457a4a0f53db11e620fcdc
|
|
left over from "SVStream operator>> to Write method" conversion
Change-Id: I619eb743d7890d5c70d0a94e51ce263567fa6f3b
|
|
Change-Id: I4d0e1de89d0bbdbea23bc5a46bf75ae0ce4e2796
|
|
and a handful of other currently uncompiled spots,
after my commit 7f8f277b94704a289fbbd1b836e4e5d66311580d
"fdo#84938: convert STREAM_ #defines to 'enum class'"
Change-Id: I550f6fb850e1d71a6f08767eeb222a18071b89d5
|
|
Change-Id: Ibbf14c7e9a5c1883c1311d4c86f948f74f8e473e
|
|
Change-Id: I82f668ef72e916d2ff11df5cda2a02653999f66f
|
|
XclImpStream and elsewhere does an initial seek to the end, then a
tell to work out the true length of the stream. In order to let us
attempt to rescue data from the beginning of otherwise corrupt /
truncated streams, we should avoid setting an error here.
Interestingly, we also (probably) don't want to return the true length
of the valid data - as this is how SotStorage has worked historically.
Change-Id: Ie0ff0e183220b81871ae3bf83980a24b57ac8694
|
|
Change-Id: Ic11c397984602bf8a2e292bc901cd7bf71ad555d
|
|
found by UCDetector
Change-Id: I0a0a00d1fae1fed2e6aef198b7a1482d7e7e29f0
|
|
Change-Id: Id787ebbfaa95c29b3f6337286fa78b089038258e
|
|
found by PMD
Change-Id: Id6737916b68ccbdbdeec5d314747a38410923ac6
Reviewed-on: https://gerrit.libreoffice.org/13409
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Ib459799f4a3224f8c9683ac4b6cf37982d2077a3
Reviewed-on: https://gerrit.libreoffice.org/13406
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I4d15a0e606f468346bc5d2f43af2e14bcc758e13
Reviewed-on: https://gerrit.libreoffice.org/13405
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|