Age | Commit message (Collapse) | Author |
|
signedness and long vs int corrections
Change-Id: I67c6b9e05b16f5d8d4693879f1656db50dc7ec48
Reviewed-on: https://gerrit.libreoffice.org/29355
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
|
|
Change-Id: I046e6f16c5b171a06e2be2da2f72340634f3e979
Reviewed-on: https://gerrit.libreoffice.org/28891
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I4782c6f6d3d090ba0f9e29af8afdd7d88aa2d382
Reviewed-on: https://gerrit.libreoffice.org/26598
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I732fb1a789f90ca7a7f393cc41a6afe84fecf3d3
Reviewed-on: https://gerrit.libreoffice.org/26200
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I4ac1bb25d3d63f150b42d9e1708efc344bbdb6a7
Reviewed-on: https://gerrit.libreoffice.org/26113
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
The getTTCoverage either leaves the bitset empty or inits it with 128 bits,
so it's not particularly dynamic.
Change-Id: Iac0aa6a023acc54da86d681e75ca550faf91ef26
Reviewed-on: https://gerrit.libreoffice.org/25456
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
The build system already defines NDEBUG only if OSL_DEBUG_LEVEL=0 AND
ASSERT_ALWAYS_ABORT=FALSE so the C++ code shouldn't override that.
Change-Id: Ie67a576646cc7e18c3ffac67a3c81b80747ea438
|
|
and remove clear() on std::vector types in ctors
Change-Id: I8e239676d46e30049d4d5886a5b8de99cadd3f45
Reviewed-on: https://gerrit.libreoffice.org/24983
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in
preparation of enabling -Wimplicit-fallthrough. (This is only relevant for
C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.)
Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but
that would require adding back in dependencies on boost_headers to many
libraries where we carefully removed any remaining Boost dependencies only
recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its
future evolution will not have any impact on the stable URE interface.) C++17
will have a proper [[fallthroug]], eventually removing the need for a macro
altogether.
Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
|
|
Change-Id: I4f067bd60706814b4d2cc6057a0342696021ad75
|
|
for with
git grep -n 'for.*sizeof'
Change-Id: I6211024385e03ac5eeeb38690d2c1c699e015c2f
Reviewed-on: https://gerrit.libreoffice.org/23569
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
This typedef was not needed.
Change-Id: I442068a3e394a23551655294dc8f1765bc8f2887
Reviewed-on: https://gerrit.libreoffice.org/23618
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I2dfce92ef98ca18ac0fe2c415240216228b4ee0a
|
|
Change-Id: I554fde45000114dd19f117d93ef5c7a780231594
|
|
Change-Id: Ie62bbf64a9cdb74725fd48a8f8dcc1ab76d97219
|
|
Change-Id: I6c0aaac10b2244271f3cdf45f4eceb6d685b213c
|
|
Change-Id: Ic09d160ed6e3cdcd95bc04844ee8f20cfcb286ec
Reviewed-on: https://gerrit.libreoffice.org/22698
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
stage 2 of replacing usage of various checks for the windows platform
with the compiler-defined '_WIN32' macro
In this stage we focus on replacing usage of the WIN macro
Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84
Reviewed-on: https://gerrit.libreoffice.org/22393
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
stage 1 of replacing usage of various checks for the windows platform
with the compiler-defined '_WIN32' macro
Change-Id: Iece73abdee530937e0737190b1aa97a46cd3075f
Reviewed-on: https://gerrit.libreoffice.org/22390
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
scalloc calls calloc which allocates and zero-initializes array
(see http://www.cplusplus.com/reference/cstdlib/calloc/)
Note: there was another useless memset to 0 not detected by cppcheck
Change-Id: Icc6458aef2ffb05fd33a858a65e0ec2dd36a8307
|
|
This appears to be an implementation of a linked list for processing
truetype fonts. I genuinely do NOT want anyone to use this anywhere
in the codebase except where it's already being used! If we must use
a list, then we can use the standard template library...
Thus, I'm moving this closer to the code that uses it - which means
moving it directly into the fontsubset directory
Change-Id: I67fb324a004cd30e32043bf9d5a3b193351d58c0
|
|
Change-Id: I879bd3bec4dd2cf002681f1e50b6c33cce0934a9
|
|
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86
Reviewed-on: https://gerrit.libreoffice.org/21209
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
|
|
Change-Id: Ia07f4057f7a48766974f80f7a23f62b6cef6f573
|
|
Change-Id: I7a741130689721c69fd2879be6bda27fc6ec3646
|
|
Change-Id: I05e89f9896170d4df3d1377549ea074f06b884a0
|
|
Change-Id: I4103c8e1549db44055fbfcb8ced6ec36d27fc126
|
|
Change-Id: I6175b4b28af3257f5734685b8fa323f0b9078512
|
|
Change-Id: I50b9df6a428bd2ddb57819a9da6f07df346e7e61
|
|
uninitialized memory warnings on --convert-to pdf
of ooo115771-3.doc
Change-Id: I2dac6fe29d23d6e25c2f1d1511886d8b585e498c
|
|
Change-Id: Ic7c496259258bf36cd885ed68912b43117f348f0
|
|
Change-Id: Idd4a1e6d50652a879493d8411c59605ca1a53dfb
|
|
Change-Id: Ib404310761fd555b033ab6594ba8636d37f23463
Reviewed-on: https://gerrit.libreoffice.org/19074
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I09db7262c2a6180a078620ba086dd7a4c132bd82
Reviewed-on: https://gerrit.libreoffice.org/18459
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
Reviewed-by: Martin Hosken <martin_hosken@sil.org>
|
|
Change-Id: Id5756033ccfbe392ae8264afbd3e2e0afd487f6b
|
|
Change-Id: I80d9d0d4c4d56aa8d52dc2e108a9b24025fb0f76
|
|
Change-Id: I1588f55b4be2d32dcafae27ac45eb5def1407fa6
|
|
Change-Id: Icab5608900108c89e649e78a2e35d02e941d0ac0
|
|
Change-Id: I79a889c68e91712d2abdacc559c78813f730e623
|
|
Change-Id: I3560bc689b2ffd44662678b62e18ce047efb22cc
|
|
Change-Id: Id9296115f30858e7fd470a199e59343a96d7deec
Reviewed-on: https://gerrit.libreoffice.org/16712
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
|
|
Change-Id: Ie929aee9c78a89d9ebed15cc59d33d7f2fdb3fad
|
|
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: I07bf1403e6b992807541a499b786d47f835b2f81
|
|
Change-Id: I62076450ab77472bfd09b3fb9824f54b6ea1e0f7
|
|
Change-Id: I0f844c7265b52d668bc6a2368321950a1a00cce5
|
|
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: I47bb1acebf409f0d391a06ca7a7399d8654ff143
|
|
Change-Id: Iad602cece6e328c7f5d5f36adb294c97b152ade3
|
|
Change-Id: I17c57a85bcda98ef36ddefd6562d4681d0c7d5fc
|