summaryrefslogtreecommitdiff
path: root/vcl/source/fontsubset
AgeCommit message (Collapse)Author
2016-10-02cppcheck invalidPrintfArgTypeJochen Nitschke
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>
2016-09-15loplugin:countusersofdefaultparams in vclNoel Grandin
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>
2016-06-27loplugin:singlevalfields in vcl(part2)Noel Grandin
Change-Id: I4782c6f6d3d090ba0f9e29af8afdd7d88aa2d382 Reviewed-on: https://gerrit.libreoffice.org/26598 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-17convert DBG_ASSERT in vclNoel Grandin
Change-Id: I732fb1a789f90ca7a7f393cc41a6afe84fecf3d3 Reviewed-on: https://gerrit.libreoffice.org/26200 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-10tdf#96099 Remove various smart pointer typedefs in vclMark Page
Change-Id: I4ac1bb25d3d63f150b42d9e1708efc344bbdb6a7 Reviewed-on: https://gerrit.libreoffice.org/26113 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-25vcl: replace boost::dynamic_bitset with boost::optional<std::bitset>Michael Stahl
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>
2016-05-25remove (un-)definitions of NDEBUG from source filesMichael Stahl
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
2016-05-14cppcheck: uninitialised members in vclJochen Nitschke
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>
2016-05-10Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann
...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
2016-04-11Don't get excited about "inline"Stephan Bergmann
Change-Id: I4f067bd60706814b4d2cc6057a0342696021ad75
2016-03-31use SAL_N_ELEMENTS in for loopsNoel Grandin
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>
2016-03-30tdf#96099 Remove trivial typedefs from vcl moduleslideon
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>
2016-03-10Avoid reserved identifier, remove unnecessary typedefStephan Bergmann
Change-Id: I2dfce92ef98ca18ac0fe2c415240216228b4ee0a
2016-03-10Avoid reserved identifierStephan Bergmann
Change-Id: I554fde45000114dd19f117d93ef5c7a780231594
2016-03-10Avoid reserved identifierStephan Bergmann
Change-Id: Ie62bbf64a9cdb74725fd48a8f8dcc1ab76d97219
2016-03-10Remove unnecessary typedef (that introduced a reserved identifier)Stephan Bergmann
Change-Id: I6c0aaac10b2244271f3cdf45f4eceb6d685b213c
2016-02-26loplugin:unuseddefaultparam in vcl/Noel Grandin
Change-Id: Ic09d160ed6e3cdcd95bc04844ee8f20cfcb286ec Reviewed-on: https://gerrit.libreoffice.org/22698 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-17use consistent #define checks for the Windows platformNoel Grandin
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>
2016-02-16use consistent #define checks for the Windows platformNoel Grandin
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>
2016-01-31cppcheck: fix redundantCopyJulien Nabet
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
2016-01-25vcl: move vcl/inc/list.h to vcl/source/fontsubset/Chris Sherlock
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
2016-01-18loplugin: unused return values in vcl/Noel Grandin
Change-Id: I879bd3bec4dd2cf002681f1e50b6c33cce0934a9
2016-01-10Fix typosAndrea Gelmini
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2015-12-27sal_uLong to sal_uInt32Matteo Casalin
Change-Id: Ia07f4057f7a48766974f80f7a23f62b6cef6f573
2015-11-17Don't assume sal_Unicode is unsigned shortStephan Bergmann
Change-Id: I7a741130689721c69fd2879be6bda27fc6ec3646
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann
Change-Id: I05e89f9896170d4df3d1377549ea074f06b884a0
2015-10-24cppcheck: invalidPrintfArgType_sintCaolán McNamara
Change-Id: I4103c8e1549db44055fbfcb8ced6ec36d27fc126
2015-10-22crashtesting+ubsan: rhbz1003919-1.svg left shift of negative numberCaolán McNamara
Change-Id: I6175b4b28af3257f5734685b8fa323f0b9078512
2015-10-22crashtesting+ubsan: abi2079-1.html memcpy null source with 0 lenCaolán McNamara
Change-Id: I50b9df6a428bd2ddb57819a9da6f07df346e7e61
2015-10-10valgrind: zero padding bytesCaolán McNamara
uninitialized memory warnings on --convert-to pdf of ooo115771-3.doc Change-Id: I2dac6fe29d23d6e25c2f1d1511886d8b585e498c
2015-10-09malloc + memset(0,..) -> callocCaolán McNamara
Change-Id: Ic7c496259258bf36cd885ed68912b43117f348f0
2015-10-07Make this compile for iOS againTor Lillqvist
Change-Id: Idd4a1e6d50652a879493d8411c59605ca1a53dfb
2015-10-02loplugin:unusedmethods in vclNoel Grandin
Change-Id: Ib404310761fd555b033ab6594ba8636d37f23463 Reviewed-on: https://gerrit.libreoffice.org/19074 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-09-14Refactor graphite integration and update graphiteMartin Hosken
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>
2015-09-04Reduce nVal scopeStephan Bergmann
Change-Id: Id5756033ccfbe392ae8264afbd3e2e0afd487f6b
2015-09-04Uniformly use if --- else if brachnes hereStephan Bergmann
Change-Id: I80d9d0d4c4d56aa8d52dc2e108a9b24025fb0f76
2015-09-04clang-tidy clang-analyzer-deadcode.DeadStoresStephan Bergmann
Change-Id: I1588f55b4be2d32dcafae27ac45eb5def1407fa6
2015-08-21No NO_MAPPERS, NO_TYPE3, NO_TYPE42 or NO_TTCR anywhereTor Lillqvist
Change-Id: Icab5608900108c89e649e78a2e35d02e941d0ac0
2015-08-11loplugin: defaultparamsNoel Grandin
Change-Id: I79a889c68e91712d2abdacc559c78813f730e623
2015-07-04coverity#1213369 Untrusted value as argumentCaolán McNamara
Change-Id: I3560bc689b2ffd44662678b62e18ce047efb22cc
2015-07-04Fix typosAndrea Gelmini
Change-Id: Id9296115f30858e7fd470a199e59343a96d7deec Reviewed-on: https://gerrit.libreoffice.org/16712 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2015-07-02coverity#1213369 Untrusted value as argumentCaolán McNamara
Change-Id: Ie929aee9c78a89d9ebed15cc59d33d7f2fdb3fad
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin
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
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann
Change-Id: I07bf1403e6b992807541a499b786d47f835b2f81
2015-05-25cppcheck: noExplicitConstructorCaolán McNamara
Change-Id: I62076450ab77472bfd09b3fb9824f54b6ea1e0f7
2015-05-11loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann
Change-Id: I0f844c7265b52d668bc6a2368321950a1a00cce5
2015-04-15remove unnecessary use of void in function declarationsNoel Grandin
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
2015-03-31-fsanitize=functionStephan Bergmann
Change-Id: I47bb1acebf409f0d391a06ca7a7399d8654ff143
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann
Change-Id: Iad602cece6e328c7f5d5f36adb294c97b152ade3
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann
Change-Id: I17c57a85bcda98ef36ddefd6562d4681d0c7d5fc