summaryrefslogtreecommitdiffstats
path: root/desktop/source
Commit message (Collapse)AuthorAgeFilesLines
* Clean up android specific template functions defined in std namespacebrainbreaker2017-07-181-19/+1
| | | | | | | Change-Id: I2094ee8eec00587f166d96723a02eb5957dbf79b Reviewed-on: https://gerrit.libreoffice.org/39890 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* use more OUString::operator== in dbaccess..filterNoel Grandin2017-07-137-17/+17
| | | | | | | Change-Id: Ib7b4f2b2403ce766a7db2f6ffc118468e7677776 Reviewed-on: https://gerrit.libreoffice.org/39889 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:oncevar: empty strings: desktopStephan Bergmann2017-07-132-8/+4
| | | | Change-Id: I8c88b295d797a98f0374e3ce57b9c875b72c7bd3
* tdf#106359: use SvFileStream to read iqy; fixes *nix compatibilityMike Kaganski2017-07-121-37/+27
| | | | | | | Change-Id: I42dc6559a57eaedcc64d2a4e59e16677b9dfeb1c Reviewed-on: https://gerrit.libreoffice.org/39862 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* strip some linefeeds from the end of debug outputNoel Grandin2017-07-121-2/+1
| | | | | | | Change-Id: I2821969d86b7f8cee53404e6a0acfbebbe53b3ac Reviewed-on: https://gerrit.libreoffice.org/39824 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* teach unnecessaryparen loplugin about identifiersNoel Grandin2017-07-101-2/+2
| | | | | | | Change-Id: I5710b51e53779c222cec0bf08cd34bda330fec4b Reviewed-on: https://gerrit.libreoffice.org/39737 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use OString::operator== in preference to ::equalsNoel Grandin2017-07-061-1/+1
| | | | | | | Change-Id: Ib291521963a791a9c6175964571e9d9895072acf Reviewed-on: https://gerrit.libreoffice.org/39646 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* new loplugin unnecessaryparenNoel Grandin2017-07-053-4/+4
| | | | | | | Change-Id: Ic883a07b30069ca6342d7521c8ad890f4326f0ec Reviewed-on: https://gerrit.libreoffice.org/39549 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:casttovoid (clang-cl): desktopStephan Bergmann2017-07-051-6/+0
| | | | Change-Id: Id14f6d0f234188b9e4576380f45dd3eec8d857b7
* loplugin:unusedfields variousNoel Grandin2017-07-052-8/+0
| | | | Change-Id: Ia7bcf2471381aa6a2768ea795fe6572e38209100
* loplugin:unusedfields in cui..idlNoel Grandin2017-07-042-12/+3
| | | | | | | Change-Id: Icb393cc0b2f79ded154e186ab6975b95e5126903 Reviewed-on: https://gerrit.libreoffice.org/39496 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:casttovoid in VCL_BUILDER_DECL_FACTORYStephan Bergmann2017-07-042-4/+2
| | | | Change-Id: I4b0dd08963cf50daa41901975c6f92fe21db2048
* C++11 remove std::unary_function bases from functorsJochen Nitschke2017-07-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | std::unary_function is deprecated since C++11 and removed in C++17 90% done with regexp magic. removed obsolete <functional> includes. The std::unary_function base class was used in 3 places: * chart2/source/tools/DataSeriesHelper.cxx: lcl_MatchesRole is used in a std::not1 function helper who uses the members return_type and argument_type. - replace deprecated std::not1 with a lambda * chart2/source/tools/ModifyListenerHelper.cxx: lcl_weakReferenceToSame used the argument_type member in the operator() parameter. - inline the parameter type. * xmloff/source/chart/SchXMLExport.cxx: lcl_SequenceToMapElement used result_type and argument_type in operator(). - inline the types Also fix compile error with gcc about finding std::for_each. Change-Id: I073673beb01410c3108e7d0346d9e7d6b9ad2e2f Reviewed-on: https://gerrit.libreoffice.org/39358 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:casttovoid: desktopStephan Bergmann2017-07-027-11/+3
| | | | Change-Id: Iea973ac2047c030019b70e6db37328d05058af08
* Handle osl::File::setPos failureStephan Bergmann2017-07-021-3/+11
| | | | Change-Id: Icf21c14f44250e1680dd7521dbb0e1260614a573
* loplugin:oncevarStephan Bergmann2017-06-301-4/+4
| | | | Change-Id: I070f06ed3195925e525066c6ba25a0bb01b824d0
* remove default-able copy ctors and copy operatorsJochen Nitschke2017-06-281-10/+0
| | | | | | | | | | | | | | | The implicit defined ctors and operators will copy all members (and bases). Since C++11 implicit copy is depreciated if there is a non-default dtor, keep such copies. This commit includes only types that had either copy ctor or copy operator and were found by cppcheck. Change-Id: I93ee687fb3b3c5884f475a2c6054955cdde57ed7 Reviewed-on: https://gerrit.libreoffice.org/39351 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* updater: add a way to manually test updater command lineMarkus Mohrhard2017-06-242-7/+17
| | | | | | | | | | | Without this hack it is not possible to debug the updater invocation easily. Each time this code will be called the updater would be executed and an update directory already present. Change-Id: I1159ee504b8f4ebf211e2a719369a48ae2e32345 Reviewed-on: https://gerrit.libreoffice.org/39080 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: forward the command line arguments to the updaterMarkus Mohrhard2017-06-241-1/+12
| | | | | | | | | The arguments will then be used again to restart LibreOffice with the same arguments. Change-Id: I17697c997a1dfeebecd9ec64e8f4ea66cadc4690 Reviewed-on: https://gerrit.libreoffice.org/39079 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* loplugin:unusedfields in desktop..editengNoel Grandin2017-06-232-5/+1
| | | | | | | Change-Id: Ieea12d0790a1034eba8975fe74fa205f02003b7d Reviewed-on: https://gerrit.libreoffice.org/39134 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:oncevar in cppcanvas..drawinglayerNoel Grandin2017-06-237-30/+18
| | | | | | | Change-Id: I5456aad61fb0dfe6830eae62b91d1a6399d6343f Reviewed-on: https://gerrit.libreoffice.org/39128 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* updater: handle the SeeAlso property after the update succeededMarkus Mohrhard2017-06-222-0/+49
| | | | | | | Change-Id: Ib90f1f7def6de05144bf8215a97a6d6e15207b88 Reviewed-on: https://gerrit.libreoffice.org/39073 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* convert ErrCode to strong typedefNoel Grandin2017-06-211-1/+1
| | | | | | | | | | | would have preferred to re-use o3tl::strong_int, of which this is a modified copy, but there are lots of convenience accessors which are nice to define on the class. Change-Id: I301b807aaf02fbced3bf75de1e1692cde6c0340a Reviewed-on: https://gerrit.libreoffice.org/38497 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* updater: check every 12 hours for updates for nowMarkus Mohrhard2017-06-181-1/+1
| | | | Change-Id: I566a0177f9bdbc523e88a17be746b3528f67403d
* remove unused osl/mutex.hxx includesJochen Nitschke2017-06-186-8/+0
| | | | | | | Change-Id: I3b50e45fdb99e9cd8bfda07356ee3ddb4dd0f8bb Reviewed-on: https://gerrit.libreoffice.org/38905 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
* updater: detect when user can not write to installation directoryMarkus Mohrhard2017-06-171-0/+38
| | | | | | In this case we can not update right now. Change-Id: I19cda5ddef448ff81e1ca457774b2db036038c88
* updater: only download language packs for installed languagesMarkus Mohrhard2017-06-161-2/+15
| | | | Change-Id: I8c605e2c64fee4d369732f783cc9dd76f7dc858c
* Make SfxItemSet ranges correct by constructionStephan Bergmann2017-06-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a follow-up to 45a7f5b62d0b1b21763c1c94255ef2309ea4280b "Keep WID ranges sorted, and join adjacent ones". While SfxItemSet::MergeRange relies on the m_pWhichRanges being sorted (and, under DBG_UTIL, asserts if they are not), the various SfxItemSet constructors curiously only check (via assert or DBG_ASSERT) that each individual range has an upper bound not smaller than its lower bound. Arguably, all SfxItemSet instances should fulfill the stronger guarantees required and checked by MergeRange. And in many cases the ranges are statically known, so that the checking can happen at compile time. Therefore, replace the two SfxItemSet ctors taking explicit ranges with two other ctors that actually do proper checking. The (templated) overload taking an svl::Items struct should be used in all cases where the range values are statically known at compile time, while the overload taking a std::initializer_list<Pair> is for the remaining cases (that can only do runtime checking via assert). Most of those latter cases are simple cases with a single range covering a single item, but a few are more complex. (At least some of the uses of the existing SfxItemSet overload taking a const sal_uInt16* pWhichPairTable can probably also be strengthened, but that is left for another day.) This commit is the first in a series of two. Apart from the manual changes to compilerplugins/clang/store/sfxitemsetrewrite.cxx, include/svl/itemset.hxx, and svl/source/items/itemset.cxx, it only consists of automatic rewriting of the relevant SfxItemSet ctor calls (plus a few required manual fixes, see next). But it does not yet check that the individual ranges are properly sorted (see the TODO in svl::detail::validGap). That check will be enabled, and the ensuing manual fixes will be made in a follow-up commit, to reduce the likelyhood of accidents. There were three cases of necessary manual intervention: * sw/source/core/unocore/unostyle.cxx uses eAtr of enum type RES_FRMATR in braced-init-list syntax now, so needs explicit narrowing conversion to sal_uInt16. * In sw/source/uibase/uiview/formatclipboard.cxx, the trailiing comma in the definition of macro FORMAT_PAINTBRUSH_FRAME_IDS needed to be removed manually. * In svx/source/svdraw/svdoashp.cxx, svx/source/svdraw/svdotext.cxx, sw/source/uibase/app/docstyle.cxx, sw/source/uibase/shells/frmsh.cxx, sw/source/uibase/shells/grfsh.cxx, and sw/source/uibase/shells/textsh1.cxx, some comments had to be put back (see "TODO: the replaced range can contain relevant comments" in compilerplugins/clang/store/sfxitemsetrewrite.cxx). A few uses of the variadic form erroneously used nullptr instead of 0 for termination. But this should have been harmless even if promoted std::nullptr_t is larger than promoted sal_uInt16, assuming that the part of the nullptr value that was interpreted as sal_uInt16/promoted int was all-zero bits. Similarly, some uses made the harmless error of using 0L instead of 0. Change-Id: I2afea97282803cb311b9321a99bb627520ef5e35 Reviewed-on: https://gerrit.libreoffice.org/38861 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* use more SAL_N_ELEMENTS part 3Noel Grandin2017-06-152-5/+3
| | | | | | | Change-Id: I82e366fefd2e31928b99840fe76649cc3521e623 Reviewed-on: https://gerrit.libreoffice.org/38789 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* cleanup unused css/frame/* includesJochen Nitschke2017-06-124-11/+1
| | | | | | | Change-Id: I173a29fd1ee889127369d2bc2fce8e010b89ca65 Reviewed-on: https://gerrit.libreoffice.org/38633 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use the shared path codeMarkus Mohrhard2017-06-121-4/+1
| | | | | | | Change-Id: I046b8f497b78c4067abbf22180edb1b210020f4e Reviewed-on: https://gerrit.libreoffice.org/38684 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* better variable namesMarkus Mohrhard2017-06-121-3/+3
| | | | | | | Change-Id: I14e667f10ae472f24fbb6107eee449ff9cf7f4a4 Reviewed-on: https://gerrit.libreoffice.org/38683 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* make it easier to extend the responseMarkus Mohrhard2017-06-121-1/+1
| | | | | | | Change-Id: I95d60fe38bbb16c502eccaae3bc25a35dd269d1a Reviewed-on: https://gerrit.libreoffice.org/38682 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* use the shared path codeMarkus Mohrhard2017-06-121-8/+4
| | | | | | | Change-Id: I02cc24d7a9e3b679a7bd883cd7085682063520ee Reviewed-on: https://gerrit.libreoffice.org/38681 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* better updater loggingMarkus Mohrhard2017-06-122-0/+15
| | | | | | | Change-Id: I6012f2159d058cbb2afb4b685badb8c870adf07b Reviewed-on: https://gerrit.libreoffice.org/38679 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Add --script-cat parameter to dump scripts content.Michael Meeks2017-06-098-90/+190
| | | | | | | | | Also factor out over-long print method into its own helper. Change-Id: Ica98dc9c999d5655e78662774e0140d70369d0b7 Reviewed-on: https://gerrit.libreoffice.org/38579 Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* tdf#36019: Disable context menu entry for extension removal properlyMuhammet Kara2017-06-061-1/+4
| | | | | | | | | | | When ExtensionSecurity::DisableExtensionRemoval is set in expert configurations, context menu entry of 'remove' for installed extensions should also be disabled along with the "Remove" button on the dialog. Change-Id: I66e4a8aafbaa633ab5626844c8df9e036045cd09 Reviewed-on: https://gerrit.libreoffice.org/38283 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
* Remove some unnecessary scaffolding around SAL_WARN callsStephan Bergmann2017-05-316-10/+0
| | | | Change-Id: Iaec105d85a283ad15fdca6ffc5cf3d2ec5acac18
* tdf#36019: Fix tooltipMuhammet Kara2017-05-311-1/+1
| | | | | | | | Change-Id: I8c1a01fa54addfdf78eaff45952e306de812e954 Reviewed-on: https://gerrit.libreoffice.org/38128 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Muhammet Kara <muhammet.kara@pardus.org.tr> Tested-by: Muhammet Kara <muhammet.kara@pardus.org.tr>
* clang-tidy readability-redundant-control-flowNoel Grandin2017-05-311-2/+0
| | | | | | | Change-Id: I832f7ef0f1bd55e365db7e49823fe8bc30390c04 Reviewed-on: https://gerrit.libreoffice.org/38215 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* teach redundantcast plugin about functional castsNoel Grandin2017-05-301-1/+1
| | | | | | | Change-Id: Iac8ccd17d9e46ebb2cb55db7adb06c469bbd4ea0 Reviewed-on: https://gerrit.libreoffice.org/37910 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* lok: Notify about the Formatting Marks changes.Jan Holesovsky2017-05-291-0/+1
| | | | | | | Change-Id: Ibcbd370c99cf63789637ad6642c91775066ecfad Reviewed-on: https://gerrit.libreoffice.org/38157 Reviewed-by: Jan Holesovsky <kendy@collabora.com> Tested-by: Jan Holesovsky <kendy@collabora.com>
* remove unnecessary use of OString::getStrNoel Grandin2017-05-2812-68/+34
| | | | | | | Change-Id: I0490efedf459190521f4339854b3394d57765fdb Reviewed-on: https://gerrit.libreoffice.org/38058 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* allow to override settings based updater URL and channel through envMarkus Mohrhard2017-05-271-0/+10
| | | | Change-Id: Ie2d96c1f6c15f91db6410997185161e5fb443dfc
* tdf#36019: Dynamically enable/disable extension removal via GUIMuhammet Kara2017-05-262-1/+21
| | | | | | | | | | | | Now extension removal via the Extension Manager dialog can be dynamically enabled/disabled from the expert configurations. This doesn't affect the unopkg commands. Change-Id: I2b718d48977ca18cc3d808e98cf7804d4a47dcff Reviewed-on: https://gerrit.libreoffice.org/37990 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Muhammet Kara <muhammet.kara@pardus.org.tr>
* tdf#36019: Dynamically enable/disable extension installation via GUIMuhammet Kara2017-05-242-1/+35
| | | | | | | | | | | | | Now extension installation via the Extension Manager dialog can be dynamically enabled/disabled from the expert configurations. This also affects the installation by double-clicking the extension file in the file manager, but doesn't affect the unopkg commands. Change-Id: I70c2c83c2458115b366c7fdcc06f72e11df4726b Reviewed-on: https://gerrit.libreoffice.org/36688 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Muhammet Kara <muhammet.kara@pardus.org.tr>
* cleanup unused css/uri/ includesJochen Nitschke2017-05-202-3/+0
| | | | | | | Change-Id: I08c7981ecce45e343ff9e98277dd3aea4ed68ab9 Reviewed-on: https://gerrit.libreoffice.org/37860 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* cleanup unused css/script/ includesJochen Nitschke2017-05-201-1/+0
| | | | | | | | | but keep exception includes in headers for now Change-Id: I826828675a2d14b906e57068cbced2e790e12bce Reviewed-on: https://gerrit.libreoffice.org/37846 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:stringcopy: desktopStephan Bergmann2017-05-192-2/+2
| | | | Change-Id: Ic27d5296dc6854a27f4848dc9047feef90cb6ff7
* make string translation loading more uniformCaolán McNamara2017-05-199-49/+49
| | | | | | | | | | | | | | | change various ResId classes that use conversion operator to OUString to functions that return a OUString drop various defines drop unnecessary toString calls Change-Id: Ibeccdf2b91a46a2ed5b4b74e6024e301a023bc92 Reviewed-on: https://gerrit.libreoffice.org/37817 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>