summaryrefslogtreecommitdiff
path: root/hwpfilter
AgeCommit message (Collapse)Author
2015-09-03clang-tidy clang-analyzer-deadcode.DeadStoresStephan Bergmann
...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 Weghorn
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 McNamara
Change-Id: Ibf819307905295580cb6a813f6d003a7dabc86b5
2015-08-18hwpfilter: tdf#88206 replace cppu::WeakImplHelper*Takeshi Abe
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 Grandin
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 Grandin
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 Grandin
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 Choi
Change-Id: Idbee9cb5a1745bb2cc3c4cb1238773da7ff2a0a3 Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
2015-07-06loplugin:unusedmethods hwpfilter,i18npoolNoel Grandin
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 Gelmini
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 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: I8a8ba86dd78e30146f7b9dd7071f5641d496a67a
2015-06-02loplugin:cstylecast: deal with those that are (technically) const_castStephan Bergmann
Change-Id: Ia48650597a4854cbb92ef203b3c1338b4f47becc
2015-05-11loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann
Change-Id: I10dfaab18f39df8766718370d0bee6c9e41d1a42
2015-05-04cppcheck: memleakOnReallocCaolán McNamara
Change-Id: Ibdf762b0d397f798372d9bf882aa82a6e5fd0229
2015-05-04hstream.h->hstream.hxxCaolán McNamara
Change-Id: Icc4a3d3661c3d40f9e71215bdc875c6a25f40ac1
2015-04-29sequence operator twaddle + silly castCaolán McNamara
Change-Id: I8804bfb39d8306762cbe39ac4b6df9128a8e0069
2015-04-29redundant castCaolán McNamara
Change-Id: Ib812a7cafabc4820946da67fb3f8896b6aad1461
2015-04-27More loplugin:simplifyboolStephan Bergmann
Change-Id: I64ec8a0525b935d12455f7ed236ffd367f47c070
2015-04-24loplugin:simplifyboolStephan Bergmann
Change-Id: Ic42cf05f9eddb0cd7724d16cc73a8d6495c193f2
2015-04-23cppcheck: silence assertWithSideEffectCaolán McNamara
Change-Id: Iaaf69ea5e8d5eeaa3a61660c3abe04ff3e83d061
2015-04-23cppcheck: unsignedLessThanZeroCaolán McNamara
Change-Id: I1043e24fdf0350e38b587f11a2916bd54e8f718f
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-04-13loplugin:staticmethodsNoel Grandin
Change-Id: I33a8ca28b0c3bf1c31758d93238e74927bebde9c
2015-03-31Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann
Change-Id: I75514fe3a3c55691ca9adfa136bfcd5ff9a2b706
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann
Change-Id: Ifbdbd2bb8a21afa76271728c46f88c11a887fc5a
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann
Change-Id: I74898e791e17971a3105febe660a2140aafaece9
2015-03-24loplugin:constantfunction: hwpfilterNoel Grandin
Change-Id: I2504291e2167cbc4be91e01de654ca63f89dbd2a
2015-03-19convert SFX_FILTER_ constants to enum classNoel Grandin
Change-Id: I7e53cfc90cefd9da7d6ecd795b09214bd44b1613
2015-03-18create new 'enum class' SotClipboardFormatId to unify typesNoel Grandin
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 McNamara
Change-Id: I4a33bd92fc8448638a4bfe1eab7e5041a4c5cc39
2015-03-09V801: Decreased performanceCaolán McNamara
Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
2015-03-08TyposJulien Nabet
Change-Id: Ibc378fa5f515de61bb768b4ef082638b40c94e00
2015-03-02V805: Decreased performanceCaolán McNamara
Change-Id: I822c5ebc321ebda87c238a1781a31793c1623e34
2015-03-01more efficient check for empty stringsMarkus Mohrhard
Change-Id: Ibc2f7412b6bf8ef75f93d649812836c01224e43f
2015-02-26wrong skip set hereCaolán McNamara
Change-Id: I86bdd1c2603c984380b52b080504b7583155b0fb
2015-02-25fix bustageCaolán McNamara
Change-Id: I249fe09319d6e0995d9ffdc7e4396b9b1787ad28
2015-02-25check if reads were successfulCaolán McNamara
Change-Id: I69ab0ca9c017c9a1c10d18fd850f32a92c641d12 Reviewed-on: https://gerrit.libreoffice.org/14631 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-02-23remove unnecessary parenthesis in return statementsNoel Grandin
found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
2015-02-05Updated all precompiled headers.Ashod Nakashian
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>
2015-01-28remove unused typedefsNoel Grandin
found with some minor modifications to find/find-unused-defines.sh Change-Id: I18cc479adedc7a0dada68a4aeef08300e62631dd Reviewed-on: https://gerrit.libreoffice.org/14194 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-01-26followup code removal after changing virtual methods to non-virtualNoel Grandin
This cleanups up indentation and removes dead classes. This is a followup patch to commit 272b1dd55797aacf511fb4342b0054e3697243f6 "new loplugin: change virtual methods to non-virtual" Change-Id: I1c2139589cf8cb23bb9808defe22c51039d38de1
2015-01-26new loplugin: change virtual methods to non-virtualNoel Grandin
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: I138ef81c95f115dbd8c023a83cfc7e9d5d6d14ae
2015-01-20Just use plain memcpyStephan Bergmann
Change-Id: I6f2014f3dfc650a61e8c0dc0bb7d7d4fdb88dbe9
2015-01-20Some more loplugin:cstylecast: hwpfilterStephan Bergmann
Change-Id: I0ba3fe95ab6ef317166b286531a718287308f62f
2015-01-08brute-force find-and-remove of unused #define constants.Noel Grandin
Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
2014-12-18hwpfilter: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: Ib19462d2095ca46eace28e1d3170b559027a2acd
2014-12-17fdo#39440 reduce scope of local variablesMichael Weghorn
This addresses some cppcheck warnings. Change-Id: I9812658e8a96dd35d686c7ae7a8b829267c5c8bc Reviewed-on: https://gerrit.libreoffice.org/13499 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-15hwpfilter: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: I4e6b8077fd564be9508e5a08f2eb66032f5f3f40
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl
Sadly cannot forward declare "struct {...} TimeValue;". rtl/(u)?string.hxx still include sal/log.hxx but removing osl/diagnose.h was painful enough for now... Change-Id: Id41e17f3870c4f24c53ce7b11f2c40a3d14d1f05