summaryrefslogtreecommitdiffstats
path: root/hwpfilter
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2015-06-085-12/+12
| | | | Change-Id: I8a8ba86dd78e30146f7b9dd7071f5641d496a67a
* loplugin:cstylecast: deal with those that are (technically) const_castStephan Bergmann2015-06-021-1/+1
| | | | Change-Id: Ia48650597a4854cbb92ef203b3c1338b4f47becc
* loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann2015-05-111-2/+2
| | | | Change-Id: I10dfaab18f39df8766718370d0bee6c9e41d1a42
* cppcheck: memleakOnReallocCaolán McNamara2015-05-042-18/+9
| | | | Change-Id: Ibdf762b0d397f798372d9bf882aa82a6e5fd0229
* hstream.h->hstream.hxxCaolán McNamara2015-05-046-5/+5
| | | | Change-Id: Icc4a3d3661c3d40f9e71215bdc875c6a25f40ac1
* sequence operator twaddle + silly castCaolán McNamara2015-04-291-1/+2
| | | | Change-Id: I8804bfb39d8306762cbe39ac4b6df9128a8e0069
* redundant castCaolán McNamara2015-04-291-4/+4
| | | | Change-Id: Ib812a7cafabc4820946da67fb3f8896b6aad1461
* More loplugin:simplifyboolStephan Bergmann2015-04-273-6/+6
| | | | Change-Id: I64ec8a0525b935d12455f7ed236ffd367f47c070
* loplugin:simplifyboolStephan Bergmann2015-04-243-9/+9
| | | | Change-Id: Ic42cf05f9eddb0cd7724d16cc73a8d6495c193f2
* cppcheck: silence assertWithSideEffectCaolán McNamara2015-04-231-1/+1
| | | | Change-Id: Iaaf69ea5e8d5eeaa3a61660c3abe04ff3e83d061
* cppcheck: unsignedLessThanZeroCaolán McNamara2015-04-231-2/+2
| | | | Change-Id: I1043e24fdf0350e38b587f11a2916bd54e8f718f
* remove unnecessary use of void in function declarationsNoel Grandin2015-04-1512-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
* loplugin:staticmethodsNoel Grandin2015-04-133-10/+10
| | | | Change-Id: I33a8ca28b0c3bf1c31758d93238e74927bebde9c
* Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann2015-03-311-1/+1
| | | | Change-Id: I75514fe3a3c55691ca9adfa136bfcd5ff9a2b706
* Clean up C-style casts from pointers to voidStephan Bergmann2015-03-2811-32/+32
| | | | Change-Id: Ifbdbd2bb8a21afa76271728c46f88c11a887fc5a
* const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann2015-03-261-1/+1
| | | | Change-Id: I74898e791e17971a3105febe660a2140aafaece9
* loplugin:constantfunction: hwpfilterNoel Grandin2015-03-245-44/+0
| | | | Change-Id: I2504291e2167cbc4be91e01de654ca63f89dbd2a
* convert SFX_FILTER_ constants to enum classNoel Grandin2015-03-191-2/+2
| | | | Change-Id: I7e53cfc90cefd9da7d6ecd795b09214bd44b1613
* create new 'enum class' SotClipboardFormatId to unify typesNoel Grandin2015-03-181-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
* V668 no sense in testing the result of new against nullCaolán McNamara2015-03-113-7/+3
| | | | Change-Id: I4a33bd92fc8448638a4bfe1eab7e5041a4c5cc39
* V801: Decreased performanceCaolán McNamara2015-03-091-3/+3
| | | | Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
* TyposJulien Nabet2015-03-081-1/+1
| | | | Change-Id: Ibc378fa5f515de61bb768b4ef082638b40c94e00
* V805: Decreased performanceCaolán McNamara2015-03-021-5/+5
| | | | Change-Id: I822c5ebc321ebda87c238a1781a31793c1623e34
* more efficient check for empty stringsMarkus Mohrhard2015-03-011-2/+2
| | | | Change-Id: Ibc2f7412b6bf8ef75f93d649812836c01224e43f
* wrong skip set hereCaolán McNamara2015-02-261-1/+1
| | | | Change-Id: I86bdd1c2603c984380b52b080504b7583155b0fb
* fix bustageCaolán McNamara2015-02-251-1/+0
| | | | Change-Id: I249fe09319d6e0995d9ffdc7e4396b9b1787ad28
* check if reads were successfulCaolán McNamara2015-02-2512-269/+481
| | | | | | | 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>
* remove unnecessary parenthesis in return statementsNoel Grandin2015-02-231-1/+1
| | | | | | | found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
* Updated all precompiled headers.Ashod Nakashian2015-02-051-0/+1
| | | | | | | | 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>
* remove unused typedefsNoel Grandin2015-01-281-1/+0
| | | | | | | | | 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>
* followup code removal after changing virtual methods to non-virtualNoel Grandin2015-01-261-1/+1
| | | | | | | | | | 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
* new loplugin: change virtual methods to non-virtualNoel Grandin2015-01-263-8/+8
| | | | | | | | | | 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
* Just use plain memcpyStephan Bergmann2015-01-201-47/+3
| | | | Change-Id: I6f2014f3dfc650a61e8c0dc0bb7d7d4fdb88dbe9
* Some more loplugin:cstylecast: hwpfilterStephan Bergmann2015-01-204-16/+16
| | | | Change-Id: I0ba3fe95ab6ef317166b286531a718287308f62f
* brute-force find-and-remove of unused #define constants.Noel Grandin2015-01-083-25/+0
| | | | Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
* hwpfilter: Use appropriate OUString functions on string constantsStephan Bergmann2014-12-181-3/+3
| | | | Change-Id: Ib19462d2095ca46eace28e1d3170b559027a2acd
* fdo#39440 reduce scope of local variablesMichael Weghorn2014-12-176-31/+22
| | | | | | | | | 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>
* hwpfilter: Use appropriate OUString functions on string constantsStephan Bergmann2014-12-152-1248/+1247
| | | | Change-Id: I4e6b8077fd564be9508e5a08f2eb66032f5f3f40
* sal: clean up public headers with include-what-you-useMichael Stahl2014-11-171-0/+1
| | | | | | | | | 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
* Fix common typos. No automatic tools. Handmade…Andrea Gelmini2014-11-121-2/+2
| | | | | | | Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* coverity#1209713 Dereference null return valueCaolán McNamara2014-11-031-2/+6
| | | | Change-Id: I8aba67ae5bf5ffb7530be8f6d5b2511b554717db
* remove unnecessary 'using namespace rtl' declarationsNoel Grandin2014-10-292-2/+0
| | | | | | | | | It turns out that almost none of them were necessary. Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83 Reviewed-on: https://gerrit.libreoffice.org/12133 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* hwfilter: fix misuse of strncpyNorbert Thiebaud2014-10-041-25/+33
| | | | Change-Id: Ia52d6cd32aa3e3838d31ea2a994117e6820a503d
* fix indentationNoel Grandin2014-10-031-344/+346
| | | | Change-Id: Icd67d066eb72b951f3e4846d45d00e7863a6000e
* coverity#1242961 Unused valueCaolán McNamara2014-10-021-4/+4
| | | | | | | | I'm guessing that this is what was meant. Though hstr2ksstr and kstr2hstr still don't match, and I'm assuming they are supposed to be mirror images of each other. Change-Id: I0fccd086e16c043324af43a5c999c2fde7cf7505
* loplugin: cstylecastNoel Grandin2014-10-011-14/+14
| | | | Change-Id: I9fca98995131358b3b112b11e0b28feb1a3267d1
* Remove o3tl/heap_ptr.hxx, use std::unique_ptr insteadStephan Bergmann2014-09-242-3/+2
| | | | Change-Id: Iac70c9be13892a36bfb5975f62e5345b88d4f144
* cppcheck: Logical conjunction always evaluates to falseJulien Nabet2014-08-311-2/+2
| | | | | | | I don't know how to check it's ok, I just took example above It seems there was a wrong mix between header and footer test Change-Id: Ia5bb838e92306f3bb160069a7c62707e365e8ab2
* update_pchThomas Arnhold2014-08-131-0/+1
| | | | Change-Id: Ic1dae7aac2f4367b4196ba3128c0aea9be1fbbda
* unused returnCaolán McNamara2014-08-071-6/+4
| | | | Change-Id: I1b61151df81c75bf794fbeda967769501026bb05
generated by cgit (git 2.34.1) at 2024-08-11 05:24:07 +0000