summaryrefslogtreecommitdiff
path: root/sd/source
AgeCommit message (Collapse)Author
2017-02-06Typo: s/dependend/dependent/iTor Lillqvist
Change-Id: If02798894ad6f0e0442ed60aaec6eca40e6dcb61
2017-02-04flatten STR_IMPRESS_PRINT_UI_OPTIONS resourceCaolán McNamara
Change-Id: Icef2a5ae31973c09620112dba062525d42bdd227
2017-02-04simplify SfxWhichIterJochen Nitschke
only one of 164 calls used the ctor with a restricting range. handle this special case. remove nFrom and nTo members. use operator[] on range pointer to highlight we access lower [0] and upper [1] range bound. remove dtor. Change-Id: I87b28502515e45bb190425eb88aa187ac0b5b2d2 Reviewed-on: https://gerrit.libreoffice.org/33911 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
2017-02-03TyposJulien Nabet
Change-Id: Ic54695e86b4b462419fa7d5ded7b1ddb19ee8ed5 Reviewed-on: https://gerrit.libreoffice.org/33904 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2017-02-03TyposJulien Nabet
Change-Id: I36f443c266670bd26aa6b813075e18653f372f52 Reviewed-on: https://gerrit.libreoffice.org/33900 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2017-02-02convert method names in tools::SvRef to be more like our other..Noel Grandin
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>
2017-02-01tdf#105677: sd: fix crash in FuPage::ApplyItemSetMichael Stahl
(regression from f3a90d13056ca23708cc4fd9562c99d0b41a4b6f) Change-Id: I54e3f06a74b2ac715061b5da54ac4db2b0a1010a
2017-02-01tdf#98034 - SIDEBAR: Add delay spinbox to Custom Animation tabAkshay Deep
Change-Id: Ie13a5e04207475794fef75c568786f37eb2c406f Reviewed-on: https://gerrit.libreoffice.org/33230 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Heiko Tietze <tietze.heiko@googlemail.com> Reviewed-by: Akshay Deep <akshaydeepiitr@gmail.com>
2017-02-01loplugin:unnecessaryoverrideStephan Bergmann
Change-Id: I53699ef386dad99b38559defe985057492a5df76
2017-02-01loplugin:useuniqueptr extend to check local varsNoel Grandin
just the simple and obvious case for now, of a local var being allocated and deleted inside a single local block, and the delete happening at the end of the block Change-Id: I3a7a094da543debdcd2374737c2ecff91d644625 Reviewed-on: https://gerrit.libreoffice.org/33749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-02-01loplugin:stringconstant handle calls to constructors with one argNoel Grandin
Change-Id: Ide9148a908bef46ba14640dfa6f556beaf6e3f60 Reviewed-on: https://gerrit.libreoffice.org/33772 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-02-01defaults are good enough ..Jochen Nitschke
for copy ctor/assignment and dtor. inline MEMBER_CONSTRUCTOR_LIST macro Change-Id: I50296cebca3bc1aa43376ccd538c84df68a232a2 Reviewed-on: https://gerrit.libreoffice.org/33771 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-31more SAL_WARN_UNUSEDNoel Grandin
Change-Id: Idf44fe0302b4504c443f083e527b63b2461ada85
2017-01-31teach lolugin:stringconstant about calling constructorsNoel Grandin
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>
2017-01-28loplugin:unnecessaryoverrideJochen Nitschke
seen on tinderbox Linux-F19-x86_64_14-with-check Change-Id: Ic494f75df7b2ca879e7bb312166b4c91263ee214 Reviewed-on: https://gerrit.libreoffice.org/33644 Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Tested-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
2017-01-28loplugin:stringconstant check for unnecessary OUString constructor..Noel Grandin
..calls when creating exceptions Change-Id: I3bc58a5aa4dc6f0508ecb88b3a843b96b8c7ebfe Reviewed-on: https://gerrit.libreoffice.org/33617 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-26Remove dynamic exception specificationsStephan Bergmann
...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2017-01-25move Image ctor variant with mask color to callersCaolán McNamara
Change-Id: I81a52aeb855edca823881360831b26e252780053
2017-01-25Fall back on o3tl::make_unique for nowStephan Bergmann
Change-Id: I58f688004804aed8b749b72caaa53afc06387ec7
2017-01-25Missing includeStephan Bergmann
Change-Id: I60177c2b10b973e9c8f09ea7c71d44df08e50cc4
2017-01-25use a std::unique_ptrCaolán McNamara
Change-Id: I1ab99995e35714d6ef3358400b0805723c44678c
2017-01-25valgrind: fix leakCaolán McNamara
since... commit e94d5233dd7939c54eb52fff456e817cecdf0a4c Date: Fri Mar 11 06:43:06 2016 +0100 work on sane lifecylce for SfxFilter which removed "delete pFilter" though it was pMediumFilter which was in question in this method Change-Id: I598dd44a8498ebd6b3e63d1c89147bee5ab3bb55
2017-01-25tdf#105502 sd increase font size: handle table selectionMiklos Vajna
In part of a table shape is selected, then only operate on the selected cells, not on all of them. Change-Id: I3a9ba2b99bcaa2e355b6fcdafdd142d4a809bce6 Reviewed-on: https://gerrit.libreoffice.org/33524 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2017-01-25use rtl::Reference in SdLayerNoel Grandin
instead of storing both a raw pointer and an uno::Reference Change-Id: Id608c50cb4cb3a7cf7ad972303a3c5f9fc8e79b0
2017-01-25loplugin: unnecessary destructor scaddins..sdextNoel Grandin
Change-Id: I6ae02ad25863ec7e0f780020dbc69c9a99488c24 Reviewed-on: https://gerrit.libreoffice.org/33511 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-24teach unusedvariablecheck plugin about SfxPoolItem subclassesNoel Grandin
which can all be treated as SAL_WARN_UNUSED The eehtml.cxx change probably fixes some CJK/CTL bug somewhere Change-Id: I6852129540f316075aee907971ac19418d71dd9a
2017-01-24coverity#1399550 Dereference after null checkCaolán McNamara
Change-Id: I4a1ba212b64c7b561f0c1b1f7f75fe255c8d1415
2017-01-23Change all Idle* LINKs to be Timer*Jan-Marek Glogowski
Seem UBSAN doesn't like my forced reinterpret_cast to set the Idles Link in the Timer class. Now there are two possible solution: 1. convert all (DECL|IMPL).*_LINK call sites to use a Timer* or 2. split the inheritance of Idle from Timer again to maintain different Link<>s and move all common code into a TimerBase. While the 1st is more correct, the 2nd has a better indicator for Idles. This implements the first solution. And while at it, this also converts all call sites of SetTimeoutHdl and SetIdleHdl to SetInvokeHandler and gets rid of some local Link objects, which are just passed to the SetInvokeHandler call. It also introduces ClearInvokeHandler() and replaces the respective call sites of SetInvokeHandler( Link<Timer *, void>() ). Change-Id: I40c4167b1493997b7f136add4dad2f4ff5504b69
2017-01-23pass ownership by pointerJochen Nitschke
to show the intend of the code. assert on invalid objects. remove useless OSL_ENSURE. make sure list of owning pointers can't be copied. Change-Id: I3cccf8ea87585ec9fe62921203a4d12e617ce15c Reviewed-on: https://gerrit.libreoffice.org/33393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
2017-01-22align commentCaolán McNamara
Change-Id: I50a728e785589e3ec2d9128679f3e69451c5102a
2017-01-21coverity#1399024 Explicit null dereferencedCaolán McNamara
Change-Id: I9e8f4bf693734625ec3544e5d4f35a7d048ea609
2017-01-20Missing 'const'Stephan Bergmann
Change-Id: I203ada1b4c527311ded56e6af0de7a7eb711f3f8
2017-01-20Missing 'const'Stephan Bergmann
Change-Id: Ia1cced46dc8c76a252952129fb93be9d5a281057
2017-01-20don't need an intermediate ImageCaolán McNamara
Change-Id: I23d712846a9b198cc2fcb87f3b097766405f8bce
2017-01-19New loplugin:dynexcspec: Add @throws documentation, sdStephan Bergmann
Change-Id: I01e47c52eb4313cf43561d7a57b71656fb66b2ef
2017-01-19sd: load BitmapEx resources instead of Image onesCaolán McNamara
Change-Id: Ic3f416300194d8fa9dfebc2b7a700396ba5641a5
2017-01-19flatten resource, drop LocalResource classCaolán McNamara
drop cargo-cult second LocalResource use Change-Id: I0c655f8ca15ad146bb11290fdefad67846ec8006
2017-01-19can use Bitmaps directly hereCaolán McNamara
Change-Id: I75f4148ff864657e747b02d80a120177b469f28a
2017-01-19make SfxItemSet with SAL_WARN_UNUSEDNoel Grandin
Change-Id: Ia74ca329d8438f614169031acfbe0406faf7c281 Reviewed-on: https://gerrit.libreoffice.org/33268 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-18new loplugin: useuniqueptr: sdNoel Grandin
Change-Id: I2fc8a2fa57cc00edf2edab9e3722c824e75cb7e5 Reviewed-on: https://gerrit.libreoffice.org/33204 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-17replace #ifdef SOLARIS with #ifdef __sunMichael Stahl
Check for a macro that is defined by the compiler, we don't really need one defined by the build system. Change-Id: Iccb8e3198396881395c97a6b81690ebe64b7e9d2
2017-01-17Refactor Scheduler to add Task classJan-Marek Glogowski
Moves all the "task-specific" stuff into a Task class and just keeps the "real" static Scheduler functions in the original Scheduler class. Change-Id: I9eb02d46e2bcf1abb06af5bab1fa0ee734d1984c
2017-01-17tdf#97087 GDB pretty print the Scheduler task listJan-Marek Glogowski
In addition to the GDB pretty printer, this annotates a lot more Timers and Idles. Change-Id: I5b93fab02161b23bb753e65ef92643a04fb0789c
2017-01-14rename from bmp to pngCaolán McNamara
Change-Id: Ibcb08acfd8ac518388c33b97121fc04a7b7dbf03
2017-01-13tdf#105188 sd: fix shutdown crash after accessing master pagesMichael Stahl
The problem here is that the destructor of SdModule does a lot of things, including destroying an SdXImpressDocument that is referenced from some SdGlobalResourceContainer. This calls SD_MOD() to get the SdModule to get some resource, but at that point SfxApplication::GetModule() returns null, because the sequence was changed from first deleting the SfxModules, then clearing the pointer in ~SfxModule to null, to unique_ptr::reset(), which, at least in libstdc++, is implemented via std::swap, so it clears the pointer before deleting the SfxModule. It appears rather brittle to rely on such a subtle detail, so refactor things so that SdGlobalResourceContainer is no longer owned by SdModule but has its own pet XTerminationListener, which means it will be destroyed earlier, while the SdModule is still fully alive. (regression from f7b1cd66167050afecf487e3d89ea12de74200b5) Change-Id: I7f03f3adf431be8728ef3d65a078b536cb96f959
2017-01-12tdf#64588 - prevent loss of non-uniquely-named slideJustin Luth
Although the UI prevents someone from renaming slides so that two have the same name, it is still possible to have such non-uniquely named pages apparently. Since drag/drop uses the name to add a slide to the clipboard, it resulted in duplicates of the first found instance of the slidename, and loss of the similarly-named slides when they were dragged to a new location. Change-Id: I2986a7c6c3360162adf9430f645d0dfc73f1d4e2 Reviewed-on: https://gerrit.libreoffice.org/32481 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Justin Luth <justin_luth@sil.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2017-01-12silence coverity#1371305 Missing move assignment operatorCaolán McNamara
Change-Id: I761e07d1583262d83074addbff4ca67b01640387
2017-01-11tdf#105182 sd: avoid iterating too far in SdOutliner::Initialize()Michael Stahl
Considering the valid indexes are extended by "-1" for backwards iterators and "size()" for forward iterators, it's obvious that a not-yet-at-the-end-in-the-other-direction iterator can be incremented once, but not necessarily twice. Why this code even wants to increment it twice isn't obvious to me. Change-Id: I578c8c6202049ebe6dbed41b8276a6bfa0566bbc
2017-01-11Some loplugin:conststringvar/stringconstant improvements: sdStephan Bergmann
Change-Id: If44cce6e4c241f6a2203d73f6d191f270fb5ca5a
2017-01-10replace namespace url strings with function callsRosemary Sebastian
Change-Id: I1b496a93449caa4a9699f71874f293eeaa1fa320 Reviewed-on: https://gerrit.libreoffice.org/32916 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
generated by cgit (git 2.34.1) at 2024-08-10 18:35:24 +0000