summaryrefslogtreecommitdiff
path: root/hwpfilter
AgeCommit message (Collapse)AuthorFilesLines
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
2015-02-25check if reads were successfulCaolán McNamara12-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>
2015-02-23remove unnecessary parenthesis in return statementsNoel Grandin1-1/+1
found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
2015-02-05Updated all precompiled headers.Ashod Nakashian1-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>
2015-01-28remove unused typedefsNoel Grandin1-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>
2015-01-26followup code removal after changing virtual methods to non-virtualNoel Grandin1-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
2015-01-26new loplugin: change virtual methods to non-virtualNoel Grandin3-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
2015-01-20Just use plain memcpyStephan Bergmann1-47/+3
Change-Id: I6f2014f3dfc650a61e8c0dc0bb7d7d4fdb88dbe9
2015-01-20Some more loplugin:cstylecast: hwpfilterStephan Bergmann4-16/+16
Change-Id: I0ba3fe95ab6ef317166b286531a718287308f62f
2015-01-08brute-force find-and-remove of unused #define constants.Noel Grandin3-25/+0
Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
2014-12-18hwpfilter: Use appropriate OUString functions on string constantsStephan Bergmann1-3/+3
Change-Id: Ib19462d2095ca46eace28e1d3170b559027a2acd
2014-12-17fdo#39440 reduce scope of local variablesMichael Weghorn6-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>
2014-12-15hwpfilter: Use appropriate OUString functions on string constantsStephan Bergmann2-1248/+1247
Change-Id: I4e6b8077fd564be9508e5a08f2eb66032f5f3f40
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl1-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
2014-11-12Fix common typos. No automatic tools. Handmade…Andrea Gelmini1-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>
2014-11-03coverity#1209713 Dereference null return valueCaolán McNamara1-2/+6
Change-Id: I8aba67ae5bf5ffb7530be8f6d5b2511b554717db
2014-10-29remove unnecessary 'using namespace rtl' declarationsNoel Grandin2-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>
2014-10-04hwfilter: fix misuse of strncpyNorbert Thiebaud1-25/+33
Change-Id: Ia52d6cd32aa3e3838d31ea2a994117e6820a503d
2014-10-03fix indentationNoel Grandin1-344/+346
Change-Id: Icd67d066eb72b951f3e4846d45d00e7863a6000e
2014-10-02coverity#1242961 Unused valueCaolán McNamara1-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
2014-10-01loplugin: cstylecastNoel Grandin1-14/+14
Change-Id: I9fca98995131358b3b112b11e0b28feb1a3267d1
2014-09-24Remove o3tl/heap_ptr.hxx, use std::unique_ptr insteadStephan Bergmann2-3/+2
Change-Id: Iac70c9be13892a36bfb5975f62e5345b88d4f144
2014-08-31cppcheck: Logical conjunction always evaluates to falseJulien Nabet1-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
2014-08-13update_pchThomas Arnhold1-0/+1
Change-Id: Ic1dae7aac2f4367b4196ba3128c0aea9be1fbbda
2014-08-07unused returnCaolán McNamara1-6/+4
Change-Id: I1b61151df81c75bf794fbeda967769501026bb05
2014-08-07coverity#1224982 Out-of-bounds accessCaolán McNamara1-4/+4
Change-Id: I028148a1112cdbe83ff7fb439cc4280a7723338b
2014-07-10coverity#1224982 out of boundNorbert Thiebaud1-1/+1
Change-Id: Ibf5e6475b3bbfbb25e0204e3999674e59555eddf