summaryrefslogtreecommitdiff
path: root/hwpfilter
AgeCommit message (Collapse)AuthorFilesLines
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann21-418/+418
Change-Id: Idec97093ca48c14b825c7b87ec050cc99aadc526
2015-11-04yyyyyNoel Grandin2-19/+19
Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
2015-11-01no need to be so verbose in constructing uno::ReferenceNoel Grandin1-4/+3
Change-Id: I187a26e200e9ecaff2adaf53a2ba3f6e87346030 Reviewed-on: https://gerrit.libreoffice.org/19724 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-01no need to use OUString constructor in call to createInstanceNoel Grandin1-4/+2
Change-Id: Iaf3d83ba1490cb1d97a5bd4d1f7cd6943d4a7296 Reviewed-on: https://gerrit.libreoffice.org/19704 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-19com::sun::star->css in hwpfilter/Noel Grandin2-9/+9
Change-Id: I2ec12da777d04855d34233d5781f2304e181bbbd
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann5-69/+69
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-07Reduce variable scopeStephan Bergmann1-2/+1
Change-Id: I5d2d7e7991c3727ed6367fe22ebddd5c2e2e9265
2015-10-07clang-analyzer-deadcode.DeadStoresStephan Bergmann1-2/+1
The code had been like this ever since 16cba77220efab8204eeecb49ccd3ec033efca38 "#112673# initial checkin of HWP filter." Assuming that the second line should rather read angle = 1800 - angle * 10; instead of angle = 1800 - prop->angle * 10; does not look too plausible: It would keep mapping -1 -> 181 0 -> 180 1 -> 179 : 179 -> 1 but then would discontinuously map 180 -> 180 181 -> 179 : instead of continuously mapping 180 -> 0 181 -> -1 : Change-Id: I8cf97eeb53409b18bda6777b09a20331f3c8132a
2015-10-01loplugin:unusedmethodsNoel Grandin4-165/+0
- 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>
2015-09-30Fix typosAndrea Gelmini4-4/+4
Change-Id: Iab78219aff60a7a45a319a96f326e27a6e8e25b8 Reviewed-on: https://gerrit.libreoffice.org/18953 Reviewed-by: Oliver Specht <oliver.specht@cib.de> Tested-by: Oliver Specht <oliver.specht@cib.de>
2015-09-29loplugin:removeunusedmethods, remove unused stuffNoel Grandin1-35/+0
Change-Id: I35456b2a3ad2a84a1d045f09cdfb29e4c19b8350
2015-09-18boost->stdCaolán McNamara1-2/+2
Change-Id: Ie490bf2c6921f393bdeed96b1a8815996b701bf0 Reviewed-on: https://gerrit.libreoffice.org/18670 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-09-14boost->stdCaolán McNamara2-6/+3
Change-Id: I3fd9e1599c5ad812879a58cf1dabbcd393105e1c Reviewed-on: https://gerrit.libreoffice.org/18564 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-09-03clang-tidy clang-analyzer-deadcode.DeadStoresStephan Bergmann1-1/+1
...ever since its inception in 16cba77220efab8204eeecb49ccd3ec033efca38 "#112673# initial checkin of HWP filter" (but curr is needed as a local scratch pad in the STARTT macro) Change-Id: I7233bf99b7d6f9ac0d3d3384ba4508f84ba8fa38
2015-08-22tdf#39440 reduce scope of local variablesMichael Weghorn2-4/+3
This addresses some cppcheck warnings. Change-Id: I6ab170c4426494ab9fc53017f88e47fbfdbd9aad Reviewed-on: https://gerrit.libreoffice.org/17920 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-08-21add hwp to fftesterCaolán McNamara2-0/+22
Change-Id: Ibf819307905295580cb6a813f6d003a7dabc86b5
2015-08-18hwpfilter: tdf#88206 replace cppu::WeakImplHelper*Takeshi Abe3-8/+6
with the variadic variants. Change-Id: Ibfe59dc7631cf499f42ff998066ed73d1eb257b3 Reviewed-on: https://gerrit.libreoffice.org/17767 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-08-05loplugin:unusedmethodsNoel Grandin1-11/+0
Change-Id: I6801618efb5a66d24156fa429e026acb6ca03aba Reviewed-on: https://gerrit.libreoffice.org/17506 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-30loplugin:unusedmethodsNoel Grandin1-8/+0
Change-Id: Ib4d77ee01e7362f5951f81fceeca3c489872d971 Reviewed-on: https://gerrit.libreoffice.org/17378 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-17formatting of public/private/protected section qualifiersNoel Grandin1-1/+1
make it consistent across the codebase, no space between keyword and the colon Change-Id: Idca61ddfc74ad2460fb05fe417499324b05e5de5 Reviewed-on: https://gerrit.libreoffice.org/17148 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-16tdf#91067: Translate Korean commentsJihui Choi13-309/+308
Change-Id: Idbee9cb5a1745bb2cc3c4cb1238773da7ff2a0a3 Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
2015-07-06loplugin:unusedmethods hwpfilter,i18npoolNoel Grandin7-44/+0
Change-Id: Ied85d93019d0f6c01c14045758b405f2ac316676 Reviewed-on: https://gerrit.libreoffice.org/16783 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-03Fix typosAndrea Gelmini2-2/+2
Change-Id: Ie2bbe020fc6e3a4a4f913208c245f395849bb9ee Reviewed-on: https://gerrit.libreoffice.org/16708 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin1-2/+1
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 Bergmann5-12/+12
Change-Id: I8a8ba86dd78e30146f7b9dd7071f5641d496a67a
2015-06-02loplugin:cstylecast: deal with those that are (technically) const_castStephan Bergmann1-1/+1
Change-Id: Ia48650597a4854cbb92ef203b3c1338b4f47becc
2015-05-11loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann1-2/+2
Change-Id: I10dfaab18f39df8766718370d0bee6c9e41d1a42
2015-05-04cppcheck: memleakOnReallocCaolán McNamara2-18/+9
Change-Id: Ibdf762b0d397f798372d9bf882aa82a6e5fd0229
2015-05-04hstream.h->hstream.hxxCaolán McNamara6-5/+5
Change-Id: Icc4a3d3661c3d40f9e71215bdc875c6a25f40ac1
2015-04-29sequence operator twaddle + silly castCaolán McNamara1-1/+2
Change-Id: I8804bfb39d8306762cbe39ac4b6df9128a8e0069
2015-04-29redundant castCaolán McNamara1-4/+4
Change-Id: Ib812a7cafabc4820946da67fb3f8896b6aad1461
2015-04-27More loplugin:simplifyboolStephan Bergmann3-6/+6
Change-Id: I64ec8a0525b935d12455f7ed236ffd367f47c070
2015-04-24loplugin:simplifyboolStephan Bergmann3-9/+9
Change-Id: Ic42cf05f9eddb0cd7724d16cc73a8d6495c193f2
2015-04-23cppcheck: silence assertWithSideEffectCaolán McNamara1-1/+1
Change-Id: Iaaf69ea5e8d5eeaa3a61660c3abe04ff3e83d061
2015-04-23cppcheck: unsignedLessThanZeroCaolán McNamara1-2/+2
Change-Id: I1043e24fdf0350e38b587f11a2916bd54e8f718f
2015-04-15remove unnecessary use of void in function declarationsNoel Grandin12-32/+32
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-04-13loplugin:staticmethodsNoel Grandin3-10/+10
Change-Id: I33a8ca28b0c3bf1c31758d93238e74927bebde9c
2015-03-31Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann1-1/+1
Change-Id: I75514fe3a3c55691ca9adfa136bfcd5ff9a2b706
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann11-32/+32
Change-Id: Ifbdbd2bb8a21afa76271728c46f88c11a887fc5a
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann1-1/+1
Change-Id: I74898e791e17971a3105febe660a2140aafaece9
2015-03-24loplugin:constantfunction: hwpfilterNoel Grandin5-44/+0
Change-Id: I2504291e2167cbc4be91e01de654ca63f89dbd2a
2015-03-19convert SFX_FILTER_ constants to enum classNoel Grandin1-2/+2
Change-Id: I7e53cfc90cefd9da7d6ecd795b09214bd44b1613
2015-03-18create new 'enum class' SotClipboardFormatId to unify typesNoel Grandin1-2/+2
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
2015-03-11V668 no sense in testing the result of new against nullCaolán McNamara3-7/+3
Change-Id: I4a33bd92fc8448638a4bfe1eab7e5041a4c5cc39
2015-03-09V801: Decreased performanceCaolán McNamara1-3/+3
Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
2015-03-08TyposJulien Nabet1-1/+1
Change-Id: Ibc378fa5f515de61bb768b4ef082638b40c94e00
2015-03-02V805: Decreased performanceCaolán McNamara1-5/+5
Change-Id: I822c5ebc321ebda87c238a1781a31793c1623e34
2015-03-01more efficient check for empty stringsMarkus Mohrhard1-2/+2
Change-Id: Ibc2f7412b6bf8ef75f93d649812836c01224e43f
2015-02-26wrong skip set hereCaolán McNamara1-1/+1
Change-Id: I86bdd1c2603c984380b52b080504b7583155b0fb
2015-02-25fix bustageCaolán McNamara1-1/+0
Change-Id: I249fe09319d6e0995d9ffdc7e4396b9b1787ad28