summaryrefslogtreecommitdiff
path: root/filter
AgeCommit message (Collapse)AuthorFilesLines
2016-04-21new plugin stylepoliceNoel Grandin6-29/+29
check for local variables which follow our member field naming convention, which is highly confusing Change-Id: Idacedf7145d09843e96a584237b385f7662eea10
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann9-39/+39
Change-Id: I887a9161107e58c9343d240c2b54e02f4e59567f
2016-04-20clang-tidy clang-analyzer-deadcode.DeadStoresNoel Grandin1-4/+3
suspect that some of these may reveal latent bugs.... Change-Id: I9de9aabbda9c8952715631577c29c1830df1eb71 Reviewed-on: https://gerrit.libreoffice.org/24200 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-18Avoid reserved identifiersStephan Bergmann8-33/+33
Change-Id: I8ee75a8ba2ca579cf945a7ef2a9746a2eb13611a
2016-04-18clang-tidy performance-unnecessary-copy-initializationNoel Grandin8-28/+20
probably not much performance benefit, but it sure is good at identifying leftover intermediate variables from previous refactorings. Change-Id: I3ce16fe496ac2733c1cb0a35f74c0fc9193cc657 Reviewed-on: https://gerrit.libreoffice.org/24026 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-18clean-up: unused using declarations and includesJochen Nitschke2-10/+0
Searched source for using declarations. Checked if those symbols reappear in the source file, even in comments or dead code but not in #include statements. If they don't reappear, remove the declaration. Remove includes whose symbol got removed. Change-Id: Ibb77163f63c1120070e9518e3dc0a78c6c59fab0 Reviewed-on: https://gerrit.libreoffice.org/24148 Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-13loplugin:passstuffbyref in filterNoel Grandin4-7/+7
Change-Id: I68e69620d4725aac66ded11e7d316caf03098249
2016-04-12Avoid reserved identifiersStephan Bergmann1-177/+177
Change-Id: Icaf083db79b2ceb88e9b0e9e4e2a131cf6ba498b
2016-04-12Avoid reserved identifiersStephan Bergmann2-4/+4
Change-Id: Ia3a9d2edb9ce7bcf9450dd10589d823383210269
2016-04-12Remove unconditionally enabled _SVG_WRITE_EXTENTS flagStephan Bergmann1-5/+2
Change-Id: I0bcf1e4929c4bdc0781637dfe95a8a7b47554430
2016-04-12Avoid reserved identifiersStephan Bergmann7-44/+44
Change-Id: I175de0aa53559fb092644d18e05e2d890fe05fca
2016-04-12Avoid reserved identifiersStephan Bergmann1-1/+1
Change-Id: I6c8565653ece37251a98103f71408ace2a0946a8
2016-04-12Elide macros used only onceStephan Bergmann1-5/+2
Change-Id: I1f1adf0790490a600ec164793ab7ee8ed0adc29e
2016-04-12clang-tidy performance-unnecessary-value-param in filterNoel Grandin21-38/+38
Change-Id: I6818f6c9593bfe7105cc194fcf6f69131046e4f3
2016-04-12cleanup: remove unused com/sun/star/uno includesJochen Nitschke8-10/+0
Sequence.h(xx), Any.h(xx) and Type.h(xx) and remove unused using-declarations from these files. Add a few missing includes provided by them. Change-Id: I6b91b6d1fdf9d0496dd546c0aab9bdcc6831a5d4 Reviewed-on: https://gerrit.libreoffice.org/23805 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-11tdf#94306 replace boost::noncopyable ...Jochen Nitschke1-3/+3
... in modules editeng to oox. Replace with C++11 delete copy-constructur and copy-assignment. Remove boost/noncopyable.hpp includes and one unused boost/checked_delete.hpp include in linguistic. Change-Id: I5a38d8e5ac1b4286bdeb3858d56490a53d13fe80 Reviewed-on: https://gerrit.libreoffice.org/23928 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-05tdf#91794 OSL_DEBUG_LEVEL > 1 removed in libo/filter moduleRohan Kumar4-22/+21
I replaced the occurences of OSL_DEBUG_LEVEL > 1 with the debug macro DEBUG_{module}_{area}. Change-Id: Ibbf7345d3dc43354ff80a8ceafb4a812749513c7 Reviewed-on: https://gerrit.libreoffice.org/22355 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-04-04tdf#97499 Fixed containers parameters clearing #3tymyjan2-6/+3
Change-Id: I1c105de7e1df251533179fba77e3797ef1c4e2d4 Reviewed-on: https://gerrit.libreoffice.org/23752 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-01sequence->vector in filterNoel Grandin3-21/+20
Change-Id: I2b18a4affed75dfc36d44ba485b2ac44d0e082cb Reviewed-on: https://gerrit.libreoffice.org/23690 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-01tdf#97966 Drop 'static' keywordsWastack1-2/+2
Including no keywords from extern "C" blocks Change-Id: I8bcd7e8a492ee4dbfa5141416c7c038a1391cf20 Reviewed-on: https://gerrit.libreoffice.org/23673 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-29Avoid reserved identifiersStephan Bergmann2-7/+7
Change-Id: I759939aa74570be63f5d41814c22aad1851d65c4 Reviewed-on: https://gerrit.libreoffice.org/23587 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-24filter: [loplugin:simplifybool]Michael Stahl1-1/+1
Change-Id: I491708d12c0c220705fe79a16ca85cf874692b77
2016-03-24disable export notes pages when printing selectionDavid Tardon3-2/+26
Change-Id: I71d348ed3b6ed5d6bcd24f7932591bafc1ef569f
2016-03-24tdf#39271 allow to export only notes pagesDavid Tardon5-15/+67
Change-Id: Ia03062cf31b6bab8196dc317a7e8fbcfc86fadf0
2016-03-23loplugin:constantparam in filterNoel Grandin7-105/+77
Change-Id: I383770bc8bd75e5b5325e133587eda2fe28f9fdd
2016-03-20Revert "tdf#93124: Fix incorrect text fit in imported PPT"Thorsten Behrens1-36/+90
Change broke autofit feature, see bugreport. This reverts commit 5ca1f04976930c6fd656ebf89d667c80e2466897.
2016-03-19tdf#93124: Fix incorrect text fit in imported PPTMike Kaganski1-90/+36
To make text adjustment to full width, the alignment optimizations had to be removed, and bAutoFit was removed, too. Allso, to fix tdf#41245 again, the SDRTEXTFIT_AUTOFIT that is set in SdStyleSheetPool::CreateLayoutStyleSheets had to be overridden. I touch the following commits: http://cgit.freedesktop.org/libreoffice/core/commit/?id=d2000efb31f864e912c6cf52760eea0e602b6893 http://cgit.freedesktop.org/libreoffice/core/commit/?id=3550256daa5451c1d51d220d5489c1b20150c374 http://cgit.freedesktop.org/libreoffice/core/commit/?id=812aff65ad0661fed9687a429c763dc8f2144a0c Change-Id: I7f7934e2982baf0240c740e09fcb9bd348079064 Reviewed-on: https://gerrit.libreoffice.org/18895 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-03-17loplugin:constantparam in svxNoel Grandin2-4/+4
Change-Id: I50fa7e4c7525d2f8107a11d8203957a47680eb80 Reviewed-on: https://gerrit.libreoffice.org/23303 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-17loplugin:constantparam in toolsNoel Grandin1-1/+1
Change-Id: I3774661799c074561c694515baba42a375d0a077 Reviewed-on: https://gerrit.libreoffice.org/23301 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-16coverity#1242658 Untrusted loop boundCaolán McNamara1-2/+24
Change-Id: Iae002c0f925772a56f56162b0e4ac63de2b50d10
2016-03-16loplugin:constantparams in vcl/Noel Grandin4-5/+4
also some improvements to the plugin Change-Id: I0e3a519d70756e577fcb1bd47dd66864b5b4c871 Reviewed-on: https://gerrit.libreoffice.org/23289 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-15remove unnecessary parentheses in case labelsNoel Grandin1-56/+56
Change-Id: I987bf68c767709080c67c8b183c036a4109c6caa
2016-03-14Fix build after oor:nillable TSAURLs changeJan-Marek Glogowski1-4/+7
Fixes the fallout from bd1a3e75738d65a3bdd615c106ea57a999d51376 Change-Id: Idfc019a034f3dd855ae85676962382879051e900
2016-03-11these don't need to be fields on the classNoel Grandin2-30/+11
Change-Id: I6f8bab0b9c286270f1e896629379762e38d448c5 Reviewed-on: https://gerrit.libreoffice.org/23120 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-11tdf#97966: Remove 'static' keywordsWastack1-3/+3
Change-Id: I0d9837a7ccecd606e0d43b53f8b920547b0d0bcd Reviewed-on: https://gerrit.libreoffice.org/23132 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-11work on sane lifecylce for SfxFilterMarkus Mohrhard2-2/+2
all SfxFilter instances should now be hold inside of a std::shared_ptr. This fixes a number of huge memory leaks in the test framework and removes one huge source of memory issue in sfx2. SfxMedium contains a pointer to the SfxFilter but does not own. Therefore it is required that any SfxFilter belonging to a SfxMedium lives longer. However this seems to work mostly by hoping that all SfxFilter instances are stored in a global array. As we have seen with the tests this is not true (there are also some cases inside of sd that seem to not follow that pattern as well). Change-Id: I12fd04a504cc4efc0a94967abd91c6fe2c6a8ce8 Reviewed-on: https://gerrit.libreoffice.org/23140 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-03-10Extract Directories from BootstrapFixtureBaseStephan Bergmann12-18/+18
(as some tests derive from the latter only for the Directories part, not for the setUp/tearDown overrides: those tests will be cleaned up next) Change-Id: Ib6b78eea868b8bc21d4cc6e8fd9e1d025deca05f Reviewed-on: https://gerrit.libreoffice.org/23078 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-09tdf#82335 Register Libreoffice for Kingsoft file typesJaskaran6-6/+6
Registered LO for Kingsoft document , spreadsheet, presentation and their template types. Change-Id: If2b9f428b22f6d104c5ffe699f5264a22b814a0e Reviewed-on: https://gerrit.libreoffice.org/23037 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-03-08tdf#97703 Removed empty setUp/tearDown methodsChirag Manwani1-6/+0
Change-Id: Ia62de30ae94bdae87cd5109c44eab40af4d020e5 Reviewed-on: https://gerrit.libreoffice.org/22985 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-07loplugin:write only fieldsNoel Grandin3-173/+0
Change-Id: Ia0fb487c5590e096659b81f76c4dc543e184c59c
2016-03-07improve defaultparams lopluginNoel Grandin11-24/+12
to catch calling params with defaults like "= OUSString()" Change-Id: Iad060e318ed492c22f8be44e326174fe6d28fff9 Reviewed-on: https://gerrit.libreoffice.org/22932 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-04crashtesting: moz397551-1.svg detect self-referencing 'use' loopCaolán McNamara1-5/+34
Change-Id: Ibc9001f02c89bcfdff396cd694d0de378c6d1863
2016-03-04add config namespace cause we use config: elementsCaolán McNamara1-0/+1
Change-Id: I34d47e854999eb2e8fd118351ff3b3577a1083d7
2016-03-04loplugin:unuseddefaultparam in filterNoel Grandin5-24/+16
Change-Id: Ieaad3145c01dc4ea7b3e7eb7859db89ebd1d7fe1
2016-03-03loplugin:unuseddefaultparams in filterNoel Grandin10-122/+83
Change-Id: Ife21ac3fc2ede30694a1c7e2f9296105aa548218
2016-02-22loplugin:commaoperator in filter/Noel Grandin12-36/+91
Change-Id: I88697862bfb160d4d7d257bf425507567deae837
2016-02-22loplugin:write only fieldsNoel Grandin3-6/+0
Change-Id: I44f249a17d0a510ec63a488b656d57a1a392f821
2016-02-22loplugin:write only fieldsNoel Grandin1-3/+0
Change-Id: I45895e9845a9037da207f001fece427452ed499f
2016-02-17use consistent #define checks for the Windows platformNoel Grandin1-3/+3
stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-16tdf#39440: reduce scope of local variablesArnold Dumas1-2/+1
Change-Id: I6ba411d2e07240821518281996d543f71acf3259 Reviewed-on: https://gerrit.libreoffice.org/22378 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>