summaryrefslogtreecommitdiff
path: root/shell
AgeCommit message (Collapse)AuthorFilesLines
2016-05-11clang-tidy modernize-loop-convert in scripting to svtoolsNoel Grandin1-3/+3
Change-Id: I98229d14109cf243839d632feabde1391ea9bad5 Reviewed-on: https://gerrit.libreoffice.org/24847 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann1-8/+8
Change-Id: I97f0ed6c3f4fa36523b6ecc05ab8b4e79a3a3c91
2016-04-19WIP tdf#89609 Ignore subsecond precision in iso8601Muhammet Kara2-1/+11
Explorer shell extension fails to convert date into locale specific string if it has sub-second precision. Change-Id: I11bd38fc2876aa1f8235dbfb8c7850ff22ac2a8b Reviewed-on: https://gerrit.libreoffice.org/24220 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-04-18clean-up: unused using declarations and includesJochen Nitschke6-11/+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-13tdf#94306 replace boost::noncopyable in sfx2 to sotJochen Nitschke4-11/+13
Replace with C++11 delete copy-constructur and copy-assignment. Remove boost/noncopyable.hpp includes. Add missing default ctors. Make some overloaded ctors explicit, seems cppcheck doesn't flag: T(const template<X>& rx). Change-Id: I63c5a3ac84a33ea6d49868e2d9aa281ada79958e Reviewed-on: https://gerrit.libreoffice.org/24050 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-04-07Drop pointless "internal" directory level for already internal include filesTor Lillqvist49-108/+108
Change-Id: I1ece44616704483cd4d9d2b6204329414f82a98c
2016-04-02cleanup: remove unused Reference.h(xx) includesJochen Nitschke2-2/+0
and unused using-declarations from Reference.h Change-Id: I297a7ae6044fa329d245ecf08fd5c4cb930f5b19 Reviewed-on: https://gerrit.libreoffice.org/23735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-04-01tdf#97966 Drop 'static' keywordsWastack1-1/+1
Including no keywords from extern "C" blocks Change-Id: I87f2ed75888b51ec9e0cb75566bf7c2351b479b4 Reviewed-on: https://gerrit.libreoffice.org/23675 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-31use SAL_N_ELEMENTS more widelyNoel Grandin5-10/+10
found using git grep -n 'sizeof.*/.*sizeof.*[0]' Change-Id: Icd4a6cc1ca8ec8ebd68e1701a02789c74cf0eb2a
2016-03-11tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar1-3/+3
I replaced OSL_DEBUG_LEVEL > 2 conditionals with OSL_DEBUG_LEVEL > 0 conditionals Change-Id: Ibfb3a4e00da7ef6a213dcd4659c7963935cfad1f Reviewed-on: https://gerrit.libreoffice.org/23081 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-03-08tdf#97703 Removed empty setUp/tearDown methodsChirag Manwani1-2/+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-07tdf#96099 Reduce number of trivial typedefsChirag Manwani1-5/+3
Change-Id: I39e9fcfdf2203239ac56d1c8195ca7ac07054817 Reviewed-on: https://gerrit.libreoffice.org/22898 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-02shell, scp2: clean up empty script packagesMichael Stahl4-39/+0
Change-Id: I93abddac491578d470f85bf3e814a0d7a91f0351
2016-03-02Just use xdg-openBryan Quigley7-197/+2
Remove kde4/tde/open-url.sh and have LO just use xdg-open. It's available on TDE builds as well as any recent KDE. We recently did this just for gnome-open-url and have had no reported issues. Change-Id: I4e1ee6caef368051af92023fa382b6fed9d9397e Reviewed-on: https://gerrit.libreoffice.org/21173 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2016-03-01loplugin:unuseddefaultparam in shellNoel Grandin2-3/+3
Change-Id: Ib14fd4f7f0a23f83801f448aa2c22e550057ec71
2016-02-17Typo: namesapce(s)->namespace(s)Julien Nabet1-1/+1
Change-Id: Ic7c14c2e39a5ade1f5622a8350f9197d84cf9cc8
2016-02-17use consistent #define checks for the Windows platformNoel Grandin2-8/+8
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-16use consistent #define checks for the Windows platformNoel Grandin1-4/+0
stage 1 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro Change-Id: Iece73abdee530937e0737190b1aa97a46cd3075f Reviewed-on: https://gerrit.libreoffice.org/22390 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-02-09Remove excess newlinesChris Sherlock34-129/+0
A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-05Fix typosAndrea Gelmini2-2/+2
Change-Id: Ice72f8d9971e15dd6ef365e64cd567b8581a92d3 Reviewed-on: https://gerrit.libreoffice.org/21797 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2016-01-19-Werror=terminate (GCC 6)Stephan Bergmann1-3/+5
...when throwing an exception from implicitly noexcept ~GErrorWrapper. So make ~GErrorWrapper noexcept(false) and see that no code executed while a GErrorWrapper instance is active on the stack would throw an exception. But the design of GErrorWrapper, to throw an exception from a dtor, is very brittle and should be fixed. Change-Id: Ib04ff2b0497c160dc7ee44c3354fa9619285ec88
2016-01-08cppcheck: noExplicitConstructorCaolán McNamara8-17/+17
Change-Id: If1ddb112c85f127295eb55566360b066e7173ba2 Reviewed-on: https://gerrit.libreoffice.org/21245 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-01-02tdf#84263 KDE4: unify includesJan-Marek Glogowski2-6/+6
Use Qt submodule notation for Qt includes and also "correct" some KDE headers AKA use <> notation for external headers. Change-Id: I9457f75baae56514989800b016a11348a72a242b
2015-12-17Remove gnome-open supportBryan Quigley4-21/+0
Gnome-open has been deprecate for some time. xdg-open replaces it. Change-Id: I7fc0f32a1f2fbadbe3d1cee58d0089213665c3a5 Reviewed-on: https://gerrit.libreoffice.org/20347 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2015-12-11Do not assume time_t is long.Pedro Giffuni1-2/+2
time_t is long on some platforms but not on all. This plagues a warning on FreeBSD. Taken from: Jung-uk Kim (cherry picked from commit 1c030e286ac7a82eaa5b87c0f950894ab6be5027)
2015-12-08Resolve Linker Warning LNK4104Mike Kaganski9-20/+19
See https://msdn.microsoft.com/en-us/library/e821azx1.aspx Some export symbols should be PRIVATE Also, don't export DLLMain. It is internal function used by CRT, and not intended to be exported. Change-Id: I48139b985aa2ee499c2005d8b0d966a2cc6dcd0c Reviewed-on: https://gerrit.libreoffice.org/20419 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Matúš Kukan <matus@libreoffice.org>
2015-11-28-Werror,-Wdelete-non-virtual-dtorStephan Bergmann1-1/+1
"thumbviewer.cxx(168,9) : error: delete called on 'StreamOnZipBuffer' that has virtual functions but non-virtual destructor" Change-Id: I34510d54b9a99d8f1f9e8da0236625023c3fd94c
2015-11-28-Werror,-Wduplicate-decl-specifier (extern "C" already in STDAPI)Stephan Bergmann1-2/+2
Change-Id: I2f170c8ad2ef30f49062fdeee4bcdcf5baf552db
2015-11-28-Werror,-Wunused-functionStephan Bergmann1-274/+0
...since ce59f8851472776f2ea364f7597ac887e77081c5 "INTEGRATION: CWS desktintgr03: #i34294#" commented out the uses (whicht got later removed completely with 45f0cc2ec176f02592a4e67942b1220f7837ab9f "Remove commented code in libs-core/shell") Change-Id: Icb7f97b72634d07045f5880af33b372be36e66b2
2015-11-23loplugin:unusedfields in shell/Noel Grandin2-19/+3
Change-Id: I3a3f7144483cebaad6d64588ff0570f29ab24e27
2015-11-20shell: rename CppunitTest to the same name as the makefileMichael Stahl1-6/+6
Change-Id: I3d1ef0573a460a3101cb8c9afd30162397d8012c
2015-11-20Seems more natural to pass a homogenous list by initializer_listStephan Bergmann1-1/+1
...than by template parameter pack (even if that requires using ServiceDecl*, as initializer_list cannot take reference types) Change-Id: Ia986201b52d8daedfe925f132ebc79bc2c0ba378
2015-11-18-Werror,-Wunused-variableStephan Bergmann1-3/+0
Change-Id: I7ac641d052643b205f7e470de77d962dd0499dd8
2015-11-18-Werror,-Wduplicate-decl-specifier (extern "C" already in STDAPI)Stephan Bergmann1-4/+4
Change-Id: Ibd633b37e134483b778dcf7bdee1a1a3f377b699
2015-11-18-Werror,-Wunused-const-variableStephan Bergmann1-1/+0
Change-Id: Id3885ec6323393d07def1915b95ee02e0c3dc402
2015-11-18-Werror,-Wunused-const-variableStephan Bergmann1-7/+0
Change-Id: I6e4aa7c0a3b420820d8581ce1a4a826fa777525c
2015-11-18-Werror,-Wsizeof-pointer-memaccessStephan Bergmann1-1/+2
Change-Id: I08d4331e5b2c9d17d16b1415d321c4f38322c251
2015-11-18-Wmissing-braces is globally disabled in com_GCC_defs.mk anywayStephan Bergmann1-6/+0
Change-Id: Ie2a1e6785ac925f268e63775f2282df3f3c6bcd4
2015-11-17Don't assume sal_Unicode is unsigned shortStephan Bergmann1-1/+1
Change-Id: I721f53d79cd3e2c24542382d519a909a86a31111
2015-11-17Don't assume sal_Unicode is unsigned shortStephan Bergmann1-1/+2
Change-Id: Id3f43ba115d519e789b7eaf014734d2fd31d39cc
2015-11-16use initialiser list for Sequence<OUString>Noel Grandin2-4/+2
Change-Id: Ia5e47261d1fc6fac2d046656c05a1c5eedb07e02 Reviewed-on: https://gerrit.libreoffice.org/19978 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser for Sequence<OUString>Noel Grandin1-2/+1
using variations of: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<OUString> (\w+)\(1\)\; \s*OUString\* pArray.*; .*\[0\]\s*=\s*(\S+)\;/Sequence<OUString> \1 { \2 };/g" Change-Id: I03c64334ff30ee14dce0d17b67f5122a3893bbe3 Reviewed-on: https://gerrit.libreoffice.org/19971 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser for Sequence<OUString>Noel Grandin1-4/+2
replaced using: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<\s*OUString\s*> (\w+)\(\s*1\s*\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I20ad0489da887a9712982531c3b127339bb8b3b9 Reviewed-on: https://gerrit.libreoffice.org/19969 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser for Sequence<OUString>Noel Grandin2-4/+2
performed using: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence<OUString> (\w+)\(1\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I4da56c80fa09bfc1e8f868794001e9921431e09f Reviewed-on: https://gerrit.libreoffice.org/19968 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser syntax for Sequence<OUString>Noel Grandin4-8/+4
replaced using the script: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence< OUString > (\w+)\(1\); .*\[0\] = (\S+);/Sequence< OUString > \1 { \2 };/g" Change-Id: I23688a91562051a8eed11fc2a85599545c285c34 Reviewed-on: https://gerrit.libreoffice.org/19967 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-10loplugin:nullptr (automatic rewrite; Mac-specific code)Stephan Bergmann3-14/+14
Change-Id: Iaa14c583c55c3582e33a1d073ea92b388cd6f3a4
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann12-34/+34
Change-Id: Ied61699170bd5d2c3878135b1a43b4dc1deff4d4
2015-11-06loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann1-6/+3
Change-Id: I7cf9fe48cf1c38a4327290e9393a5f88f3d4a97d
2015-11-05use uno::Reference::set method instead of assignmentNoel Grandin1-1/+1
Change-Id: Ib93d762cf523e0029bbed16e08beebd9f418ae24
2015-11-04loplugin:stringconstantStephan Bergmann1-3/+1
Change-Id: Ib579c499904bf10844ed58dfd3e531002b308dcf