summaryrefslogtreecommitdiffstats
path: root/sot
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:unnecessaryparen include c++ castsNoel Grandin2017-09-041-1/+1
| | | | | | | Change-Id: I132d3c66f0562e2c37a02eaf4c168d06c2b473eb Reviewed-on: https://gerrit.libreoffice.org/41874 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* remove UL/L suffixes from integer constants on the RHS of expressionsNoel Grandin2017-08-172-4/+4
| | | | | | | Change-Id: I899a8126c9d971601fea6c77eca165718aea0ac5 Reviewed-on: https://gerrit.libreoffice.org/41237 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* remove UL/L suffixes from integer constants in initialiser/call expressionsNoel Grandin2017-08-177-23/+23
| | | | | | | Change-Id: Iae081567c4fa5b88edbd12cf2fbafd2b8f31b300 Reviewed-on: https://gerrit.libreoffice.org/41214 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unusedenumconstantsNoel Grandin2017-08-101-5/+0
| | | | | | | Change-Id: I54c0c1fca8ea5c89e71c6cd1110fad081ed3d334 Reviewed-on: https://gerrit.libreoffice.org/40953 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* EditEngine: Removing BIN format from sotVarun Dhall2017-08-071-3/+2
| | | | | | | Change-Id: I495e2749b961dfca63bb1e27239ac6a09b80fd19 Reviewed-on: https://gerrit.libreoffice.org/40333 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:constparams in i18nutil,sotNoel Grandin2017-07-176-12/+12
| | | | | | | Change-Id: I8e91b11baa855b1049130746b34f53158010948b Reviewed-on: https://gerrit.libreoffice.org/40050 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Translate German comments/debug strings (leftovers in the sot dir)Johnny_M2017-07-1311-77/+71
| | | | | | | | | | Translates leftovers found using a custom regex and manually checking the rest of the affected file. Change-Id: I0e44957d58e089b82738c65a8774d1a45ea7068d Reviewed-on: https://gerrit.libreoffice.org/39879 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* sot: fix loplugin:unnecessaryparen warningMiklos Vajna2017-07-071-1/+1
| | | | Change-Id: Ia022f6cb1e9c91c9aa49025e54699a5c7718fc75
* new loplugin unnecessaryparenNoel Grandin2017-07-051-1/+1
| | | | | | | 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:unusedfields in slideshow..svtoolsNoel Grandin2017-07-032-6/+0
| | | | | | | Change-Id: I74d5a4b8cfc4b18267f99648a3112b163c91fd8c Reviewed-on: https://gerrit.libreoffice.org/39474 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#39468 Translate German comments/termsJens Carl2017-07-032-23/+18
| | | | | | | | Change-Id: I9edef3138964d3f43745be39956bc6bb66d0cc4d Reviewed-on: https://gerrit.libreoffice.org/39403 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Johnny_M <klasse@partyheld.de> Reviewed-by: Eike Rathke <erack@redhat.com>
* aKey is demonstratedly always emptyStephan Bergmann2017-06-301-2/+1
| | | | | | | ...since cb958bb5e0e81d343c91c08a8513006a7bf1d913 "loplugin:constantparam in sot..svl" Change-Id: Id74078a26efb9b7a5a8b58ea28f3142cef4ac71f
* aContentType is unusedStephan Bergmann2017-06-301-5/+0
| | | | | | | ...ever since f14feb754eba67b2420c67c310440d16845f62f5 "INTEGRATION: CWS warnings01" Change-Id: I96bbbf55fe3136550dac229e4f247457bcb66364
* loplugin:useuniqueptr in variousNoel Grandin2017-06-271-5/+4
| | | | | | | | | extending it to find places we can use std::unique_ptr on arrays Change-Id: I9feb1d12d738d6931e752ecb6dd51cbc1540c81b Reviewed-on: https://gerrit.libreoffice.org/39255 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:oncevar in scaddins..sotNoel Grandin2017-06-261-2/+1
| | | | | | | Change-Id: I657a131374b9f1153135e00ed429889518b91427 Reviewed-on: https://gerrit.libreoffice.org/39258 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unusedfields store..svlNoel Grandin2017-06-215-21/+2
| | | | | | | Change-Id: I6070a683e5128271b84a10caccb548d07c950927 Reviewed-on: https://gerrit.libreoffice.org/39021 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Fix typosAndrea Gelmini2017-06-151-1/+1
| | | | | | | Change-Id: Iea1f82e95c37bdf778941a568d61f17f179f838d Reviewed-on: https://gerrit.libreoffice.org/38795 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
* use ERRCODE_NONE instead of 0Noel Grandin2017-06-141-5/+5
| | | | | | | | | peeling off a small chunk of my ErrCode strong_int conversion Change-Id: Idc89e8496083beed7608cba705cd981139eb7111 Reviewed-on: https://gerrit.libreoffice.org/38777 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* EditEngine Exporting flat XML from actual clipboardVarun Dhall2017-06-071-1/+2
| | | | | | | | | | Allowing export of EditEngine document into flat XML from actual clipboard by modifying SotClipboardFormatId Change-Id: I9c7dfd3ca428d0fa355bd3dfb7bfa0f0e30dada6 Reviewed-on: https://gerrit.libreoffice.org/38226 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
* replace SVSTREAM_OK with ERRCODE_NONENoel Grandin2017-06-068-31/+31
| | | | | | | | | since the first is #define'd to the second, and offers no extra value Change-Id: I2c67e09ea3aa5361b8e7dfe7a20858c6ae054450 Reviewed-on: https://gerrit.libreoffice.org/38406 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Improved loplugin:cstylecast to reference types: sotStephan Bergmann2017-06-053-5/+5
| | | | Change-Id: I14135642998ef08dfb4aa4c7e10b327cf8d33de7
* Improved loplugin:redundantcast, const/static_cast combinations: sotStephan Bergmann2017-06-051-2/+2
| | | | Change-Id: I32a9d3f9b7cb978c649461e01a802f4356217f8f
* convert FatError to scoped enumNoel Grandin2017-06-053-57/+57
| | | | | | | Change-Id: Ic9da7c88077d0f8a6998971122c64ce20a1990dd Reviewed-on: https://gerrit.libreoffice.org/38402 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Remove some unnecessary scaffolding around SAL_WARN callsStephan Bergmann2017-05-311-3/+0
| | | | Change-Id: Iaec105d85a283ad15fdca6ffc5cf3d2ec5acac18
* teach redundantcast plugin about functional castsNoel Grandin2017-05-302-3/+3
| | | | | | | 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>
* remove unnecessary use of OString::getStrNoel Grandin2017-05-281-3/+3
| | | | | | | 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>
* remove unused uno::Reference varsNoel Grandin2017-05-121-1/+0
| | | | | | | | | found by temporarily marking Reference as SAL_WARN_UNUSED. Change-Id: I18809b62654467f890016adcc92576980ced393b Reviewed-on: https://gerrit.libreoffice.org/37511 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* cleanup osl/diagnose.h includesJochen Nitschke2017-05-091-1/+0
| | | | | | | | | | | | | | with command > git grep -l osl/diagnose.h *.cxx | xargs grep -L -w 'OSL_\w*' | xargs sed -i '/#include *\(<\|\"\)osl\/diagnose.h\(>\|\"\).*/d' headers need more work Change-Id: I906519ebbd47a04703b4fa5943b2f7abea7a97ab Reviewed-on: https://gerrit.libreoffice.org/37350 Tested-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* tdf#39486: Translate some randomly found German commentsJens Carl2017-04-281-16/+16
| | | | | | | Change-Id: I4c2950180f504c06666f055d166c51e32a6d5294 Reviewed-on: https://gerrit.libreoffice.org/36969 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* clang-tidy readability-simplify-boolean-exprNoel Grandin2017-04-282-8/+4
| | | | | | | Change-Id: Iea7ab64683f0b29794d50d774cc482b54a00e70a Reviewed-on: https://gerrit.libreoffice.org/36450 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* errinf.hxx moved out of tools and into vcl moduleChris Sherlock2017-04-251-1/+1
| | | | | | | | | | ErrorInfo has a hard depency on VCL, yet is in the tools package. It is more appropriate to have it reside in the VCL module. Change-Id: Ica54a46c3a7f86cf0331ed7245234bea69c05650 Reviewed-on: https://gerrit.libreoffice.org/36839 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
* tools: svstream.hxx needs only errcode.hxx & not errinf.hxxChris Sherlock2017-04-256-1/+10
| | | | | | | Change-Id: Ia28e35ae5af4f601e9a586a3deffbcd61702b0ca Reviewed-on: https://gerrit.libreoffice.org/36896 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* gbuild: Remove MSVC 2013 legacy codeDavid Ostrovsky2017-04-212-2/+0
| | | | | | | | | Uwinapi is discontinued. Change-Id: I063b4d0d8fab2d60de168e960a63b8181158ac01 Reviewed-on: https://gerrit.libreoffice.org/23198 Reviewed-by: David Ostrovsky <david@ostrovsky.org> Tested-by: David Ostrovsky <david@ostrovsky.org>
* remove unnecessary explicit linefeeds from end of SAL and OSL log callsNoel Grandin2017-04-211-1/+1
| | | | | | | Change-Id: I3fa363c8e76e6cfb297f4ec346e3f031c09d6fbf Reviewed-on: https://gerrit.libreoffice.org/36727 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Fix typosAndrea Gelmini2017-04-071-1/+1
| | | | | | | Change-Id: Ibad3f8e9f55af5e652b6be198bebace2b1bfb35b Reviewed-on: https://gerrit.libreoffice.org/36235 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
* these are file urls, so don't need ucbCaolán McNamara2017-04-031-7/+8
| | | | | | | | Change-Id: I25beaeab002db1de34be3b5f4a774bd15b71f5d3 Reviewed-on: https://gerrit.libreoffice.org/36045 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* tdf#96505 Get rid of cargo cult long integer literalsdilekuzulmez2017-03-211-3/+3
| | | | | | | Change-Id: I6bda2915547980b33d059257c8dbd65f181f9f16 Reviewed-on: https://gerrit.libreoffice.org/35291 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* fix indentCaolán McNamara2017-03-211-1/+1
| | | | Change-Id: I6facc99a16c29275771f46d397162bcd4f4a27b0
* check full chain upwardsCaolán McNamara2017-03-142-12/+6
| | | | | | | | Change-Id: I3a620824b987bf78eaabffa913a3a62a842ba0d9 Reviewed-on: https://gerrit.libreoffice.org/35193 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Remove uses of SAL_CONSTEXPR in LIBO_INTERNAL_ONLYStephan Bergmann2017-03-031-2/+2
| | | | | | | Change-Id: I9a7dc7c83302b3361f056fcf6636bbba7672f15f Reviewed-on: https://gerrit.libreoffice.org/34840 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* new loplugin unoanyNoel Grandin2017-02-281-1/+1
| | | | | | | Change-Id: I5d6c4a67cb2a09e7cd5bd620c6b262d188701b89 Reviewed-on: https://gerrit.libreoffice.org/34714 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:loopvartoosmallStephan Bergmann2017-02-271-1/+1
| | | | Change-Id: I3ada4fc8c8ebcf04962be03c048fb87bc0d6dbf8
* loplugin:subtlezeroinit: sotStephan Bergmann2017-02-211-1/+1
| | | | Change-Id: Ibb99c0e1dce1dd3ae66161af010d071f1d46c75a
* convert RepresentModes to scoped enumNoel Grandin2017-02-151-15/+7
| | | | | | and drop unused xinputstream enumerator Change-Id: I7c608a4a80b0aec29140dd481cdafd646ad7a05f
* convert InsertOperation to scoped enumNoel Grandin2017-02-101-1/+1
| | | | | | and drop unused LINK enumerator Change-Id: I6c42424ae5a7503b3e14e5f04fcf7a49b595b416
* Add missing #includesStephan Bergmann2017-02-062-0/+3
| | | | | | | | | ...and remove some unncessary using directives/declarations, in preparation of removing now-unnecessary #includes from cppumaker-generated files, post e57ca02849c3d87142ff5ff9099a212e72b8139c "Remove dynamic exception specifications". Change-Id: Iaf1f268871e2ee1d1c76cf90f03557527ebc9067
* new loplugin:unusedenumconstantsNoel Grandin2017-02-031-1/+1
| | | | | | | | | | These are the simple removals, where it is obviously safe, the more complex ones will come in separate commits Change-Id: I7211945a6a5576354b60d9c709940ce9b674f308 Reviewed-on: https://gerrit.libreoffice.org/33828 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* convert method names in tools::SvRef to be more like our other..Noel Grandin2017-02-023-34/+34
| | | | | | | | | | | reference classes, uno::Reference and rtl::Reference. Specifically rename Is()->is() and Clear()->clear(). Change-Id: Icb7e05e2d09cb9977121508b837ba0961dabb4ae Reviewed-on: https://gerrit.libreoffice.org/33576 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* inline macrosJochen Nitschke2017-02-011-1265/+1231
| | | | | | | | | | move static arrays to anonymous namespace try creating the SotAction arrays at compile time with constexpr Change-Id: If0d0563b35936574beea617da08880a6c32569d9 Reviewed-on: https://gerrit.libreoffice.org/33750 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* teach lolugin:stringconstant about calling constructorsNoel Grandin2017-01-311-2/+2
| | | | | | | | | | | so we can remove unnecessary calls to the OUString(literal) constructor when calling constructors like this: Foo(OUString("xxx"), 1) Change-Id: I1de60ef561437c86b27dc9cb095a5deb2e103b36 Reviewed-on: https://gerrit.libreoffice.org/33698 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>