summaryrefslogtreecommitdiff
path: root/extensions
AgeCommit message (Collapse)AuthorFilesLines
2017-02-06Add missing #includesStephan Bergmann20-5/+21
...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
2017-02-05.uno:ScriptOrganizer should be menu:menuitemMaxim Monastirsky1-1/+1
... so it won't show as an empty submenu in the customization dialog, and allow accidentally overwriting it. Change-Id: I9431e5240485a58f63a2333850a9052e475db63e
2017-02-02unnecessary use of OUStringBuffer in throwing exceptionsNoel Grandin1-11/+4
Change-Id: Iec1473264426f19c31e72260dfce9494389e474f Reviewed-on: https://gerrit.libreoffice.org/33788 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-31teach lolugin:stringconstant about calling constructorsNoel Grandin10-14/+14
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:stringconstant check for unnecessary OUString constructor..Noel Grandin1-2/+2
..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-27improve "unnecessary user-declared destructor" checkNoel Grandin1-1/+0
to look for inline&empty destructors, where we can just let the compiler do it's thing Change-Id: Ibde8800bdfed6b77649c30ebc19921167c33dec3 Reviewed-on: https://gerrit.libreoffice.org/32999 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-26Remove dynamic exception specificationsStephan Bergmann108-1480/+1233
...(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-24tdf#105467 new color widget doesn't support XStringListControlCaolán McNamara1-1/+1
so this threw and mangled the dialog. New color dialog doesn't need to be told the name of colors, it figures that out itself. Change-Id: I89382b6b26312fc90c967620c654ea611cbd9616
2017-01-23Change all Idle* LINKs to be Timer*Jan-Marek Glogowski5-10/+10
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-23use rtl::Reference in SubmissionPropertyHandlerNoel Grandin2-10/+8
rather than manual acquire/release Change-Id: I3276361153591a3af8c6b00b478365b2700dedf5
2017-01-23use rtl::Reference in bib::OComponentListenerNoel Grandin2-24/+9
rather than manual acquire/release Change-Id: Ib4b52fb4a85223f974e9905bc3d02eb839150030
2017-01-20use rtl::Reference in BibFrameCtrl_ImplNoel Grandin2-39/+37
instead of storing both a raw pointer and a uno::Reference Change-Id: I8caa56eade25561178aa6ffd144b00f452517974
2017-01-20use rtl::Reference in BibliographyLoaderNoel Grandin1-17/+14
instead of storing both a raw pointer and an uno::Reference Change-Id: Ic46c5cda34c1df818cbe1ffa4b2d44d1519b4d6f
2017-01-20use rtl::Reference in FormControlContainerNoel Grandin2-9/+7
instead of manual acquire/release Change-Id: Ie0fed7db217adea68aaa09cf9de699d488bf84dd
2017-01-19New loplugin:dynexcspec: Add @throws documentation, extensionsStephan Bergmann30-3/+67
Change-Id: Ia44f7b0a162d269bf38133575f986efd62eb6d52
2017-01-17replace #ifdef SOLARIS with #ifdef __sunMichael Stahl2-4/+4
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 Glogowski3-3/+3
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-16e*, f*: load BitmapEx resources instead of Image onesCaolán McNamara5-18/+13
Change-Id: I3383e222829042557a8fd9f575049c47aeddeb09
2017-01-16new loplugin: useuniqueptr: extensionsNoel Grandin2-12/+8
Change-Id: Iaafaf0117a339dbb435cec2bbf4dca55629ab8a2 Reviewed-on: https://gerrit.libreoffice.org/33154 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-14-Werror,-Wunused-lambda-captureStephan Bergmann1-1/+1
Change-Id: I0ceabed1f89b6b641d5282fa1c32719bf3ead19c
2017-01-11loplugin:externvar (clang-cl)Stephan Bergmann2-14/+14
Change-Id: I638199f1455bead71f0a03f15e4b6f418a0cd0e2
2017-01-09tdf#105160: word wrap at wizardheiko tietze2-5/+5
Vertical spacing fill now Change-Id: I1463df1a0b345a39c10ef83b2fa0c558bd135cdc Reviewed-on: https://gerrit.libreoffice.org/32829 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Heiko Tietze <tietze.heiko@googlemail.com>
2017-01-09New loplugin:externvar: extensionsStephan Bergmann3-7/+8
Change-Id: I596eecab083092cccbe67062ade2279e6e25baab
2017-01-06loplugin:salboolStephan Bergmann1-1/+1
Change-Id: I7dfd48a04e2dff1391d10acb75180711b403af9b
2017-01-01fold ResourceIndexAccessBase into ResourceStringIndexAccessCaolán McNamara1-16/+6
Change-Id: Icef518dc65d5edf4e7893df460aaae179eecfb45
2017-01-01so ResourceStringListIndexAccess isn't neededCaolán McNamara1-38/+0
Change-Id: Ic25fc6f2f8f3e471a90e10c3f2319f40888520e6
2017-01-01so we can drop StringList access from ResourceIndexAccessCaolán McNamara2-7/+2
Change-Id: Ie855a45f7994e9d58ddecc51b54c25424fc544d6
2016-12-26use std::vector instead of naked array in SfxItemPoolNoel Grandin4-11/+12
Change-Id: I2d4ac010ff5818e673567cee05700872588918e4 Reviewed-on: https://gerrit.libreoffice.org/32384 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
2016-12-24unwind RID_TOOLBAR_IMGLIST and RID_TOOLBAR_BIGIMGLIST ImageListsCaolán McNamara4-43/+44
Change-Id: Iae7ffce940f90e48dfb95b33abab35ca03a6b305
2016-12-22RID_SVXBMP->RID_EXTBMPCaolán McNamara4-83/+83
Change-Id: I19e5c60ea8b42223322da2a1e343b7c4c1c59928
2016-12-21loplugin:staticmethodsStephan Bergmann2-41/+40
Change-Id: I04d3095c657deb72a20fa803964912cf6baf268b
2016-12-21-Werror,-Wmacro-redefinedCaolán McNamara3-3/+3
Change-Id: Ifd246b642979df0d8b1aa20a59f8c73dc9ddc1b2
2016-12-21loplugin:staticmethodsTor Lillqvist2-3/+4
Change-Id: I30456ec3a0440364b8b4c460d198588aed241034
2016-12-21convert VclButtonsType to scoped enumNoel Grandin2-2/+2
Change-Id: I9b91108c18e190060dc71546977aa8a3c11f06e1 Reviewed-on: https://gerrit.libreoffice.org/32285 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
2016-12-21unwind RID_IL_FORMEXPLORER imagelistCaolán McNamara6-115/+194
Change-Id: I4c86aa99aecf22e842183966721976a921bafe7b
2016-12-20extensions: Drop empty fileTakeshi Abe2-31/+0
Change-Id: I4962d739af889595f026c250b8f10a917b1dfb17 Reviewed-on: https://gerrit.libreoffice.org/32130 Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp> Tested-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-12-13OSL_TRACE->SAL in chart2..ooxNoel Grandin7-30/+8
Change-Id: I133a6441824bfbefcfcda130119b5c5d706f86b2 Reviewed-on: https://gerrit.libreoffice.org/31907 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-12remove useless Mutex guardsNoel Grandin1-1/+0
Change-Id: Ib1edb681b47ce98e2e810d15111932002786aa51 Reviewed-on: https://gerrit.libreoffice.org/31875 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-07convert VCLEVENT constants to scoped enumNoel Grandin1-9/+10
Change-Id: Ic8ccb0a9715ec05182dacddab2c015b0de6a0fba Reviewed-on: https://gerrit.libreoffice.org/31675 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-05convert DecodeMechanism to scoped enumNoel Grandin4-4/+4
Change-Id: I5c56634b1bca8e37fa73d02d2428645301b6c547
2016-12-05convert INetURLObject::EncodeMechanism to scoped enumNoel Grandin1-1/+1
Change-Id: I50ea17c7779c7b5cacddf548f1773fd7d6c8bade
2016-12-05loplugin:unnecessaryoverride (dtors) in extensionsStephan Bergmann5-28/+0
Change-Id: Ia8250bc33c5d05861794619c5d240fdcb4002c13
2016-11-29extensions,sc,sd,vcl: de-obfuscate assignments in conditions to help GCCMichael Stahl1-2/+4
GCC 6.2.1 with -Og produces spurious -Werror=maybe-uninitialized on variables that are assigned in conditions; perhaps it's better to de-obfuscate the code if even GCC is confused about it. Change-Id: Ia2f8209df893a8e5659ca72f4cde3d7d847574e1 Reviewed-on: https://gerrit.libreoffice.org/31332 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-11-29loplugin:vclwidgetsStephan Bergmann1-3/+3
Change-Id: If88a200fa26bcc6b20ad3c1dbc160e862054318f
2016-11-29loplugin:vclwidgets check for assigning from VclPt<T> to T*Noel Grandin4-5/+5
Inspired by a recent bug report where we were assigning the result of VclPtr<T>::Create to a raw pointer. As a consequence, we also need to change various methods that were returning newly created Window subclasses via raw pointer, to instead return those via VclPtr Change-Id: I8118e0195a5b2b4780e646cfb0e151692e54ae2b Reviewed-on: https://gerrit.libreoffice.org/31318 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-11-24vcl: rename Window::Notify to EventNotifyMichael Stahl4-6/+6
There is annoying overloading between Window::Notify and SfxListener::Notify, and the Window one has apparently fewer implementations, so rename that and remove lots of disambiguating "using Notify" in multiply inheriting classes. Change-Id: I8b597fd9e70cf2e7103b9dfa7cc666e79e7aff49
2016-11-18loplugin:datamembershadowNoel Grandin1-1/+0
Change-Id: I8a737a2f22c7be9630a1f7562b4309e687bb85f9 Reviewed-on: https://gerrit.libreoffice.org/30948 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-11-11loplugin:countusersofdefaultparams in editeng..rscNoel Grandin4-5/+5
Change-Id: I8a8d13faf228cbc934ae21d6763d92d370eb42ec
2016-11-11update vclwidget loplugin to find ref-dropping assigmentNoel Grandin4-5/+5
Look for places where we are accidentally assigning a returned-by-value VclPtr<T> to a T*, which generally ends up in a use-after-free. Change-Id: I4f361eaca88820cdb7aa3b8340212db61580fdd9 Reviewed-on: https://gerrit.libreoffice.org/30749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-11-09make comphelper::containerToSequence a little smarterNoel Grandin2-6/+2
So we don't have to specify the source and destination type as often. Change-Id: Id9e286417a1cb246d163cbc3c536b231a4a92624 Reviewed-on: https://gerrit.libreoffice.org/30700 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-23 20:45:35 +0000