summaryrefslogtreecommitdiff
path: root/package
AgeCommit message (Collapse)AuthorFilesLines
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin1-10/+7
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-13TyposJulien Nabet1-1/+1
Change-Id: I21c352a63d668c174eef212dbfbe6346c678ce4d
2015-06-11cppcheck: noExplicitConstructorCaolán McNamara4-5/+5
Change-Id: Id6291335945a8f10cef3afd1a2b3bae65fdf4562
2015-06-11convert expressions like 'size() == 0' to 'empty()'Noel Grandin2-7/+7
Change-Id: Ia5c8c0f38a347f398d587970a22e03f29ffd37af
2015-06-11convert 'it.begin() == it.end()' to 'it.empty()'Noel Grandin1-1/+1
Change-Id: I244a9eb6bce6b1c649653ec38ebb9a39f8b4c145 Reviewed-on: https://gerrit.libreoffice.org/16212 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2-4/+4
Change-Id: I1a3766bba2c157f20422d3ffb0c08f631b537724
2015-06-05WaE: unreferenced local variableCaolán McNamara1-1/+1
Change-Id: Ie04966936a406fefbbcd1d96d7d3d299f98a31ae
2015-06-05uncaught exceptions in parallel deflate threads on re-export of HTB23106 odgsCaolán McNamara3-2/+20
Change-Id: Ib6aef478b8dcf1c7a7262ab60eef4d44da0de632
2015-06-01loplugin:loopvartoosmallNoel Grandin2-2/+2
Change-Id: I86ff38a90018a2ddfb2db3babf67168b0e6257a5
2015-05-08Rephrase comparisons between bool and sal_BoolStephan Bergmann1-1/+1
...to cater for forthcoming loplugin:implicitboolconversion improvements Change-Id: I88c0c4681137022005c3a4c418e91cb17bc17148
2015-04-24loplugin:simplifyboolStephan Bergmann2-3/+3
Change-Id: Ia4177cf1a4b0c06ef614b54298fd6f9224956afc
2015-04-23Improved loplugin:literaltoboolconversion looking into cond. exprs.Stephan Bergmann2-3/+3
...automatic rewriter fixes Change-Id: I6b04ca80f08f8a71ff94e309fd52f44d736751ee
2015-04-15remove unnecessary use of void in function declarationsNoel Grandin16-19/+19
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-12unnecessary castCaolán McNamara1-4/+3
Change-Id: I7c040b2539b9f7f923620a02d0a27e8e000fee4a
2015-04-09loplugin:staticmethodsNoel Grandin4-5/+5
Change-Id: Ie348778ea666c24e95e048386547f301083a0017
2015-04-08A UNO Any can't contain an AnyStephan Bergmann1-1/+1
...and css::uno::makeAny<css::uno::Any>() was never meant to be used. Introduce css::uno::toAny for the (template-code) cases that shall return an Any for both Any and non-Any inputs. Change-Id: Ifa977d73f1da71b2fedde7e8140b19497c4a0257
2015-03-31Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann2-2/+2
Change-Id: I6979d9af3a97918593f0596df17c096bd3f4c4ad
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann1-2/+2
Change-Id: I313d61fc6e7be3a5513da83d3de9202237306739
2015-03-25tdf#90010 Partially revert "package: Do not bother with deflating jpeg images"Matúš Kukan1-1/+2
This reverts commit 0843ade74e5848c0babad509b10c334b01847be2 for the case, when the document is protected by password. Would be nice to fix properly later. Change-Id: If159639d6f74de4e106e9dad382faac9d5de4a09
2015-03-24loplugin:constantfunction: variousNoel Grandin1-4/+0
Change-Id: I6eddda9f4b31c7ce413c328b6a857a81bd222eed
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-13V803 decreased performance postfix incrementMichael Stahl1-1/+1
These are pretty silly anyway, but apparently it complains even about integer variables which make this rather a waste of time. Change-Id: I15e847d33d5decd2adcab04e4f1567d3997d28a2
2015-03-12package: PVS-Studio V595 'm_pData' pointer could be nullMichael Stahl1-27/+15
... so convert the last member of StorInternalData_Impl that needs manual cleanup to rtl::Reference Change-Id: Ie1549c1f52d50dc3c63715176c02821b63c7cab1
2015-03-12package: convert StorInternalData_Impl member to std::unique_ptrMichael Stahl1-20/+12
Change-Id: If523dbd4596f4023bda6509c6dc09b8ab3d74874
2015-03-12package: convert OStorage::m_pData to std::unique_ptrMichael Stahl2-6/+5
Change-Id: I875432aeb98072882df1f228d9fcf43c6027e8d5
2015-03-12package: move WSInternalData_Impl out of headerMichael Stahl2-16/+17
Change-Id: Ie7c01b37e9a4eb2221d4b40fc6b13672d9b3a684
2015-03-12package: convert OWriteStream::m_pData to std::unique_ptrMichael Stahl2-16/+12
Change-Id: I9e72325679b2758cd4297d2693cf49fd533f8baa
2015-03-12package: PVS-Studio V595 'm_pData' pointer could be nullMichael Stahl2-23/+18
... just convert its explicitly deleted member to unique_ptr. Change-Id: I826257ff512632a2aedd53a7ce5e4bedf49c3cfe
2015-03-11redundant checkCaolán McNamara1-2/+1
Change-Id: Id6b72a1a3133e47a07ee274f4082dde0e7032894
2015-03-09V801: Decreased performanceCaolán McNamara2-7/+7
Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
2015-02-06fdo#75757: Remove inheritance from std::vectorMatthew Pottage2-3/+3
Deprecated comphelper/sequenceasvector.hxx. Rewritten code using it. Using instead the functions containerToSequence and sequenceToContainer, found in include/comphelper/sequence.hxx. One class that inherits from it (in framework/inc/stdtypes.h), and the code using that has been left. Signed-off-by: Michael Stahl <mstahl@redhat.com> Conflicts: writerfilter/source/dmapper/DomainMapper.cxx writerfilter/source/dmapper/TblStylePrHandler.hxx writerfilter/source/dmapper/WrapPolygonHandler.hxx Change-Id: Ice61c94ffb052e389281aebb7cdf185134221061
2015-02-05Updated all precompiled headers.Ashod Nakashian2-0/+4
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-26Visible function type RTTI for Clang -fsanitize=functionStephan Bergmann1-0/+3
...by adding some further SAL_DLLPUBLIC_RTTI type annotations (cf. b4f6b26b5a1a78fecfa95ec2eb7ac8b80495d8aa "SAL_DLLPUBLIC_RTTI for proper RTTI visibility for LLVM") and by making sure relevant function types do not use incomplete types in their parameter and return types (which would make the RTTI hidden). Change-Id: Id7aadcbc0704b9759968ae36266fc9ce11a2e340
2015-01-20Some more loplugin:cstylecast: packageStephan Bergmann6-18/+19
Change-Id: Icf7351082cf63aeb70ec47a8562a89d452ae9d8e
2015-01-15remove unnecessary castsNoel Grandin1-20/+20
leftovers from conversion of SvStream operator>> to Write methods Change-Id: Id1d1e60ae42afb689910845a3c6ec75e3388fb56
2015-01-08brute-force find-and-remove of unused #define constants.Noel Grandin1-1/+0
Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
2015-01-04cppcheck: Possible inefficient checking for <var> emptinessJulien Nabet1-1/+1
Change-Id: I6ca0e477a4429e762c48c721951b9876db3a5c6c
2015-01-02Fix simple typos.Andrea Gelmini1-11/+11
Change-Id: I1fdbe47ed7a6e05d2cf20f4c296bffad4bbc6599 Reviewed-on: https://gerrit.libreoffice.org/13719 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2015-01-02include algorithmCaolán McNamara1-0/+2
Change-Id: I55abf17a1da2616ca2b1303747ca8cc7e402634e
2015-01-02include algorithmCaolán McNamara1-0/+1
Change-Id: Icd44b7c128bf72236206d7c753a9ef986f50dad2
2015-01-02boost::unordered_map->std::unordered_mapCaolán McNamara4-8/+8
you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
2014-12-19fdo#39440 reduce scope of local variablesMichael Weghorn1-2/+4
This addresses some cppcheck warnings. Change-Id: Idcedd908c653d5a5700884f233ad134dde8be018 Reviewed-on: https://gerrit.libreoffice.org/13540 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-18comphelper: Use appropriate OUString functions on string constantsStephan Bergmann3-9/+9
Change-Id: Id1d5c3cf2f76dbb33606cec1c0f17d4a1f282247
2014-12-15fdo#87210: Re-enable parallel deflate, which was not the root cause.Kohei Yoshida1-2/+0
With f92183833fa569006602ac7e93c906d2094e0d4d, export no longer crashes, and there is no reason to leave this piece disabled any more. Let's re-enable this. Change-Id: Ibeca8869f152cbcd80f1dcb55f8199110125741d
2014-12-15package: Use appropriate OUString functions on string constantsStephan Bergmann1-1/+1
Change-Id: I8d01dbad13cf26473c03d0c3dd01ae7baca991fb
2014-12-14package: Better to use temporary files for huge memory zip streamsMatúš Kukan3-8/+26
ZipPackageBuffer was holding the whole compressed data stream in one uno::Sequence which seems to be a lot for big documents in some cases. Change-Id: Ib10d00ac54df9674231f4bbf047fab7e9b0a7d45
2014-12-14package: Create memory buffer only when we need it - if we use parallelismMatúš Kukan6-15/+33
Otherwise write directly to the resulting zip file. Change-Id: I75097969f0cccf0b45da591c71221e5ae18668cb
2014-12-12callcatcher: remove newly unused codeCaolán McNamara6-227/+129
and rework reads to just return the read value Change-Id: I5d2f01064465c65859ec4ba031ec9dfa16403487
2014-12-11remove operator>> methodsNoel Grandin7-109/+109
in favour of ReadXXX methods. Change-Id: I242b92da7dbcafe22aef69dc7348ac58dadc9241