summaryrefslogtreecommitdiffstats
path: root/hwpfilter/source
Commit message (Collapse)AuthorAgeFilesLines
* update loplugin stylepolice to check local pointers varsNoel Grandin2016-04-261-52/+52
| | | | | | | | | | | | are actually pointer vars. Also convert from regex to normal code, so we can enable this plugin all the time. Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1 Reviewed-on: https://gerrit.libreoffice.org/24391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-201-3/+3
| | | | Change-Id: Icfa6686ee96e4c9ddf23abc3ece6beae6521473a
* Get rid of odd _HCHAR_, _UTYPE_, _ZZRECT_ guardsStephan Bergmann2016-04-121-9/+0
| | | | Change-Id: Ic44d1c751319897c55fdd03c912cd670624f928f
* Get rid of _DATECODE_WEEK_DEFINES_Stephan Bergmann2016-04-122-23/+22
| | | | Change-Id: I1b50fadd64df0a18339410068696a836850e9248
* use SAL_N_ELEMENTS more widelyNoel Grandin2016-03-311-1/+1
| | | | | | | found using git grep -n 'sizeof.*/.*sizeof.*[0]' Change-Id: Icd4a6cc1ca8ec8ebd68e1701a02789c74cf0eb2a
* -Werror,-WundefStephan Bergmann2016-03-041-1/+1
| | | | | | | see c665a1bc0f45aa4a4c679e6f004581168e03f93d for __STDC__ not defined under MSVC/clang-cl. Change-Id: I65d8a3b33ff40fc80c52b294b0c5bc75d9f5cf2f
* loplugin:unuseddefaultparams variousNoel Grandin2016-03-032-3/+3
| | | | | | | Change-Id: Ibf8489c957b307156689de4c7cb8440ddd4e4546 Reviewed-on: https://gerrit.libreoffice.org/22852 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:write only fieldsNoel Grandin2016-02-224-110/+1
| | | | Change-Id: I44f249a17d0a510ec63a488b656d57a1a392f821
* use consistent #define checks for the Windows platformNoel Grandin2016-02-168-34/+24
| | | | | | | | | | 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>
* Remove excess newlinesChris Sherlock2016-02-096-11/+0
| | | | | | | | | | | | | | | | | | | A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* Formatting changes across all modulesChris Sherlock2016-02-091-29/+31
| | | | | | | | | | | | | | + Removed comment cruft + Tab formatting in number of files + Some commented out code removed + Tab characters replaced with spaces + Newline cleanup in quite a few files + Tweak header guard #endifs Change-Id: I3208ff2f047da890edcc49b73389aca22442f5fc Reviewed-on: https://gerrit.libreoffice.org/22221 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* coverity#1349757 Unchecked return valueCaolán McNamara2016-01-211-1/+1
| | | | Change-Id: Ia2286fa9e1c351c4e4c9579b31b94ce1eac3b841
* loplugin:unusedmethodsNoel Grandin2016-01-203-65/+0
| | | | | | | Change-Id: Ib2dc804f55d1a96ae01f2cf31b7d55956040a72e Reviewed-on: https://gerrit.libreoffice.org/21603 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin: unused return valuesNoel Grandin2016-01-186-30/+26
| | | | | | | Change-Id: I9c61a46c57894bc63a57740206c0bcb4a16553af Reviewed-on: https://gerrit.libreoffice.org/21571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedmethods unused return value in hwpfilterNoel Grandin2016-01-1112-73/+65
| | | | Change-Id: Ib225db89e60708f6b55b3cb86f8e881386fab86f
* Fix typosAndrea Gelmini2016-01-101-12/+12
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* cppcheck: duplInheritedMemberCaolán McNamara2016-01-042-2/+2
| | | | Change-Id: I451a7a04fd1578d3095e5931f0792a32870a647e
* cppcheck: allocaCalledCaolán McNamara2016-01-021-32/+3
| | | | Change-Id: I253f71f851621d8a3d1244468b9da63ba2d10b09
* cppcheck: noExplicitConstructorCaolán McNamara2015-12-291-1/+1
| | | | Change-Id: Ib43e53d5b6c9c130adb765ac9b769f58060ac640
* loplugin:nullptr: More NULL -> nullptr automatic rewriteStephan Bergmann2015-12-101-6/+6
| | | | Change-Id: Ie83819e2bcdc5fa160b39296b005ca9a5ff74b1d
* unused Picture::Type()Juergen Funk2015-12-072-12/+0
| | | | | | | | | and removed unused TxtBox::Type(), too Change-Id: I4a1d7f1cbbc5bb4d929fe7420db8f827dd8b142d Reviewed-on: https://gerrit.libreoffice.org/20424 Reviewed-by: Tor Lillqvist <tml@collabora.com> Tested-by: Tor Lillqvist <tml@collabora.com>
* Adapt to true wchar_t in clang-clStephan Bergmann2015-11-281-4/+4
| | | | Change-Id: Id127668e6786665b819b7c0b1c0cd424ebff96bf
* '#pragma setlocale("C")' is probably no longer necessaryStephan Bergmann2015-11-242-10/+0
| | | | | | | | | | ...as in both files the direct use of non-ASCII characters in ordinary string literals has since been changed to use \xXX escapes instead Change-Id: Ic3e17a9849288a02dc69d7702782fefccb7026ee Reviewed-on: https://gerrit.libreoffice.org/20148 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* Translate Korean commentsStephan Bergmann2015-11-241-3/+3
| | | | | | ...via translate.google.com, to make file plain ASCII Change-Id: Icbf25074bd92eb1f0fb453f9887b1f1f1ca82b57
* Remove non-ASCII U+0086 PARAGRAPH SIGN from commentStephan Bergmann2015-11-241-1/+1
| | | | Change-Id: Ifcbb2a61c662f135badc01b7beaef10fa3d75515
* Replace non-ASCII content in ordinary string literals with UTF-8 bytesStephan Bergmann2015-11-241-4/+25
| | | | Change-Id: Ib92221527550beb35e2263b150a509b3ef68545f
* cppcheck: noExplicitConstructorCaolán McNamara2015-11-246-10/+10
| | | | Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
* use unique_ptr for pImpl in extensions,hwpfilter/Noel Grandin2015-11-182-12/+6
| | | | Change-Id: I476ba7ec2ebee8c61bf9bb6ff0309cc40affb275
* Don't assume sal_Unicode is unsigned shortStephan Bergmann2015-11-172-59/+59
| | | | Change-Id: I72b0f9956e866afd22f7335a57d7dca0297e3b8a
* use initialiser syntax for Sequence<OUString>Noel Grandin2015-11-151-2/+1
| | | | | | | | | | | | | | replaced using the script: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence< OUString > (\w+)\(1\); .*\[0\] = (\S+);/Sequence< OUString > \1 { \2 };/g" Change-Id: I23688a91562051a8eed11fc2a85599545c285c34 Reviewed-on: https://gerrit.libreoffice.org/19967 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* new loplugin: memoryvarNoel Grandin2015-11-111-15/+8
| | | | | | | | | | detect when we can convert a new/delete sequence on a local variable to use std::unique_ptr Change-Id: Iecae4e4197eccdfacfce2eed39aa4a69e4a660bc Reviewed-on: https://gerrit.libreoffice.org/19884 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-1021-418/+418
| | | | Change-Id: Idec97093ca48c14b825c7b87ec050cc99aadc526
* yyyyyNoel Grandin2015-11-042-19/+19
| | | | Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
* no need to be so verbose in constructing uno::ReferenceNoel Grandin2015-11-011-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>
* com::sun::star->css in hwpfilter/Noel Grandin2015-10-192-9/+9
| | | | Change-Id: I2ec12da777d04855d34233d5781f2304e181bbbd
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-124-67/+67
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* Reduce variable scopeStephan Bergmann2015-10-071-2/+1
| | | | Change-Id: I5d2d7e7991c3727ed6367fe22ebddd5c2e2e9265
* clang-analyzer-deadcode.DeadStoresStephan Bergmann2015-10-071-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
* loplugin:unusedmethodsNoel Grandin2015-10-014-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>
* Fix typosAndrea Gelmini2015-09-304-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>
* loplugin:removeunusedmethods, remove unused stuffNoel Grandin2015-09-291-35/+0
| | | | Change-Id: I35456b2a3ad2a84a1d045f09cdfb29e4c19b8350
* boost->stdCaolán McNamara2015-09-181-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>
* boost->stdCaolán McNamara2015-09-141-5/+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>
* clang-tidy clang-analyzer-deadcode.DeadStoresStephan Bergmann2015-09-031-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
* tdf#39440 reduce scope of local variablesMichael Weghorn2015-08-222-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>
* add hwp to fftesterCaolán McNamara2015-08-211-0/+21
| | | | Change-Id: Ibf819307905295580cb6a813f6d003a7dabc86b5
* hwpfilter: tdf#88206 replace cppu::WeakImplHelper*Takeshi Abe2015-08-183-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>
* loplugin:unusedmethodsNoel Grandin2015-08-051-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>
* loplugin:unusedmethodsNoel Grandin2015-07-301-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>
* formatting of public/private/protected section qualifiersNoel Grandin2015-07-171-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>