summaryrefslogtreecommitdiffstats
path: root/basctl
Commit message (Collapse)AuthorAgeFilesLines
* update loplugin stylepolice to check local pointers varsNoel Grandin2016-04-261-2/+2
| | | | | | | | | | | | are actually pointer vars. Also convert from regex to normal code, so we can enable this plugin all the time. Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1 Reviewed-on: https://gerrit.libreoffice.org/24391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Resolves: tdf#99425 crash when closing dialog editorCaolán McNamara2016-04-211-1/+4
| | | | | | with undocked "Object Catalog" toolbar Change-Id: I70d51cec699333c5cfc1461d61734c1bbb5323da
* clang-tidy modernize-loop-convert in b*Noel Grandin2016-04-215-27/+22
| | | | Change-Id: I8ac6eb59e213eafa78e3dc4578738b53e8adef5b
* clang-tidy performance-unnecessary-copy-initializationNoel Grandin2016-04-181-8/+8
| | | | | | | | | | | probably not much performance benefit, but it sure is good at identifying leftover intermediate variables from previous refactorings. Change-Id: I3ce16fe496ac2733c1cb0a35f74c0fc9193cc657 Reviewed-on: https://gerrit.libreoffice.org/24026 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* clean-up: unused using declarations and includesJochen Nitschke2016-04-184-18/+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>
* Remove the hardly used SyntaxLineSet typedefRyan McCoskrie2016-04-182-3/+2
| | | | | | | Change-Id: Ib38bbbdcc65620329d1471d0d61ede65c4f834e6 Reviewed-on: https://gerrit.libreoffice.org/24176 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#97380: don't destroy completely window when replace dialogJulien Nabet2016-04-161-1/+1
| | | | | | | | | | | | | | | | | 0x00002aaaf64ff222 in basctl::ScriptDocument::Impl::isApplication (this=0x9999999999999999) at /home/julien/lo/libreoffice/basctl/source/basicide/scriptdocument.cxx:210 210 inline bool isApplication() const { return m_bValid && m_bIsApplication; } (gdb) bt 0 0x00002aaaf64ff222 in basctl::ScriptDocument::Impl::isApplication (this=0x9999999999999999) at /home/julien/lo/libreoffice/basctl/source/basicide/scriptdocument.cxx:210 1 0x00002aaaf64fa12e in basctl::ScriptDocument::isApplication (this=0x576b458) at /home/julien/lo/libreoffice/basctl/source/basicide/scriptdocument.cxx:1311 2 0x00002aaaf648f6d7 in basctl::MarkDocumentModified (rDocument=...) at /home/julien/lo/libreoffice/basctl/source/basicide/basobj3.cxx:256 3 0x00002aaaf645e2a9 in basctl::implImportDialog (pWin=0x576b210, rCurPath=<error reading variable: Cannot access memory at address 0x999999999999999d>, rDocument=..., aLibName="Standard") at /home/julien/lo/libreoffice/basctl/source/basicide/baside3.cxx:1154 4 0x00002aaaf645eda9 in basctl::DialogWindow::ImportDialog (this=0x576b210) at /home/julien/lo/libreoffice/basctl/source/basicide/baside3.cxx:1213 Change-Id: I7365b2cacc8a3783fdc8e573bbc10ff574d9c4ed Reviewed-on: https://gerrit.libreoffice.org/24139 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
* basctl: prefix some membersJulien Nabet2016-04-164-142/+142
| | | | | | | Change-Id: I59660153a33e83dc9457fbee082e8da444b04b58 Reviewed-on: https://gerrit.libreoffice.org/24126 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
* loplugin:passstuffbyref in variousNoel Grandin2016-04-143-3/+3
| | | | Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
* Avoid reserved identifierStephan Bergmann2016-04-121-2/+2
| | | | Change-Id: I5c166398df5a16c5c9123ce3c046cbf6d64b9fa4
* Get rid of _SVX_NOIDERESIDSStephan Bergmann2016-04-122-10/+4
| | | | Change-Id: Iab728d51c24ccbedad5fa181850a673129d489ae
* clang-tidy performance-unnecessary-value-param in basctlNoel Grandin2016-04-127-69/+59
| | | | Change-Id: Ie48eb08edd1335661d1ef57ae652713143951317
* cleanup: remove unused com/sun/star/uno includesJochen Nitschke2016-04-121-1/+0
| | | | | | | | | | | | Sequence.h(xx), Any.h(xx) and Type.h(xx) and remove unused using-declarations from these files. Add a few missing includes provided by them. Change-Id: I6b91b6d1fdf9d0496dd546c0aab9bdcc6831a5d4 Reviewed-on: https://gerrit.libreoffice.org/23805 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* fix copy assignment signaturesJochen Nitschke2016-04-111-1/+1
| | | | | | | | | | | | | get value by reference introduced in commit 8c2f2e1dd77cdce9bdf63beff5a79f91adc44630 and commit f9b200ce54cd67ddc04747f9676568a86e14d864 Change-Id: Iba84dd752f4cbdde9f3676c434d107efabe63a38 Reviewed-on: https://gerrit.libreoffice.org/23974 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* sequence->vector in xmlscriptNoel Grandin2016-04-051-1/+2
| | | | | | | Change-Id: I4f99cd9dc659f54bd4818559dd3e0dbce1e8f5d4 Reviewed-on: https://gerrit.libreoffice.org/23795 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* cppcheck: noExplicitConstructorCaolán McNamara2016-04-041-2/+4
| | | | Change-Id: I9510a16bf4ac5dd2c3da16c618841918daea8cfb
* tdf#94306 replace boost::noncopyable in ..Jochen Nitschke2016-04-043-7/+10
| | | | | | | | | | .. accessibility, avmedia, basctl, basic and bridges remove boost from module if not needed anymore Change-Id: I6177f8276766a0a7df1703e81bf1b448912df6e2 Reviewed-on: https://gerrit.libreoffice.org/23744 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* cleanup: remove unused Reference.h(xx) includesJochen Nitschke2016-04-021-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>
* tdf#97966 Drop 'static' keywordsWastack2016-04-011-1/+1
| | | | | | | | | Including no keywords from extern "C" blocks Change-Id: Id0304994a692f1004993dda2ffd7fb819ab8e8d0 Reviewed-on: https://gerrit.libreoffice.org/23670 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* tdf#97525 vcl: make DeletionListener internalJochen Nitschke2016-03-311-1/+0
| | | | | | | | | | | | move include/vcl/impdel.hxx to vcl/inc/ include impdel.hxx in salframe.hxx remove vcl/impdel.hxx includes in pch add missing <list> includes Change-Id: Id146363b2e20ce0238542929c26a83efb1e8c4bd Reviewed-on: https://gerrit.libreoffice.org/23664 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* sfx2: make SfxDispatcher::Execute() less variadic ...Michael Stahl2016-03-316-36/+36
| | | | | | | | | | | ... and less overloaded; C++11 initializer_list should make undefined behavior as fixed in ba0a866b83f016fd460320e7d057dd107e019f19 less likely. Change-Id: I15cc0804451b6a4fcbfaa623f9a30db222207865 Reviewed-on: https://gerrit.libreoffice.org/23666 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* use SAL_N_ELEMENTS in for loopsNoel Grandin2016-03-311-1/+1
| | | | | | | | | | for with git grep -n 'for.*sizeof' Change-Id: I6211024385e03ac5eeeb38690d2c1c699e015c2f Reviewed-on: https://gerrit.libreoffice.org/23569 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#96505: fix wrong SfxDispatcher::Execute calls ...Michael Stahl2016-03-306-15/+29
| | | | | | | | | | | | | | ... that are now busted on all 64-bit platforms since commit 57d0caacee2f395be2e89b78f8ece2d47b2c8683 but were already busted only on 64-bit WNT before. SfxDispatcher::Execute(sal_uInt16 nSlot, SfxCallMode eCall, const SfxPoolItem* pArg1, ...) assigns the varargs to a "const SfxPoolItem *", so passing in "0L" is only correct if long is 64-bit, but on WNT 64-bit long is 32-bit, so use "nullptr" already. Change-Id: I50448d8ad121c4881be549623d44c44b00e56d98
* loplugin:constantparam in basctlNoel Grandin2016-03-233-4/+4
| | | | Change-Id: I08ab924a9bb69bede4247cfe8c50921520a4f4a7
* convert SvButtonState to scoped enumNoel Grandin2016-03-181-3/+3
| | | | Change-Id: I601f9e2fe33e85cf3f7dc52ee20d68202bd2b09d
* drop unused params from SvLBoxItem and all it's subclassesNoel Grandin2016-03-181-3/+3
| | | | | | | | looks like this is fallout from commit ac7acb0a "Merged SvTreeListBox and SvLBox." (in year 2012) Change-Id: Iaebeae64fc7cd3ba11f5f45b53bcb65bf906e906
* loplugin:constantparam in sfx2Noel Grandin2016-03-172-2/+2
| | | | | | | Change-Id: I53e690ab8d50fb3ce43ba633fc018e7b66e591cd Reviewed-on: https://gerrit.libreoffice.org/23321 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:constantparam in toolsNoel Grandin2016-03-171-3/+3
| | | | | | | Change-Id: I3774661799c074561c694515baba42a375d0a077 Reviewed-on: https://gerrit.libreoffice.org/23301 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:constantparamNoel Grandin2016-03-159-24/+23
| | | | Change-Id: I270e068b3c83e966e741b0a072fecce9d92d53f5
* tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar2016-03-121-7/+0
| | | | | | | | | | I removed the OSL_DEBUG_LEVEL > 1 conditional and replaced it to OSL_DEBUG_LEVEL > 0 as suggested in easy hack. Change-Id: If5f086f280d01e7a80be59f3c992dc2d0a7663cf Reviewed-on: https://gerrit.libreoffice.org/23062 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* Resolves: tdf#95428 backspace in readonly macro editor hangsCaolán McNamara2016-03-091-4/+3
| | | | Change-Id: I16568b00e92d18f32818188ba3ab8df92bbd13aa
* loplugin:constantparam in basctlNoel Grandin2016-03-092-15/+9
| | | | Change-Id: I07d22585e768335f61d70bf18807479a2675804a
* drop now-needless depsDavid Tardon2016-03-071-4/+0
| | | | Change-Id: I2ef0b873a58710de399889cee3abf6b6ecf4fb1e
* remove remaining uses of gb_SrsTemplateTarget_get_targetDavid Tardon2016-03-071-1/+0
| | | | Change-Id: I5ddf0ba5fc289a124b39b9c14b7f5fdd3d7b54a4
* these ResIds are used solely for their number by nowCaolán McNamara2016-03-071-1/+1
| | | | | | so cut out the middle-man ResId Change-Id: Ic83db355cc1a0c7dc70958bca79b466baadaedd0
* tdf#96505 Remove long integer literal specifier when not neededAleksas Pantechovskis2016-03-046-17/+17
| | | | | | | | | | Remove L from integer literals if it does not produce warnings and does not change semantics Change-Id: I9573a47d3e3401993a1f0fbcf7df4be20f9e6eba Reviewed-on: https://gerrit.libreoffice.org/22867 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unuseddefaultparams variousNoel Grandin2016-03-031-1/+1
| | | | | | | Change-Id: Ibf8489c957b307156689de4c7cb8440ddd4e4546 Reviewed-on: https://gerrit.libreoffice.org/22852 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* convert HighlighterLanguage to scoped enumNoel Grandin2016-02-241-1/+1
| | | | Change-Id: Ibf0871601e52ea4fa49e9622176431943c55abdd
* convert TokenTypes to scoped enumNoel Grandin2016-02-243-28/+29
| | | | Change-Id: I17c0a616dd6cf48a22896b6cd6b0df157d1f9a9f
* tdf#93837 Convert RID_POPUP_DLGED to xmlMaxim Monastirsky2016-02-216-19/+24
| | | | Change-Id: I0588063aef397ee95923ad5acd332bc6052fafc7
* use consistent #define checks for the Windows platformNoel Grandin2016-02-171-1/+1
| | | | | | | | | | | | 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>
* tdf#95857 Sort out German plurals ...danielt9982016-02-124-21/+21
| | | | | | | | | Made a start in removing the incorrect 'Infos' German plural Change-Id: Ie989351a7473fc35b563e63ce6a4fb229093af60 Reviewed-on: https://gerrit.libreoffice.org/22301 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* Remove excess newlinesChris Sherlock2016-02-0919-122/+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>
* vcl: cmdevt.[hc]xx -> commandevent.[hc]xxChris Sherlock2016-02-081-1/+1
| | | | | | | Change-Id: I77b30f28ae5a6fad360d7cada9acfaa9c324408b Reviewed-on: https://gerrit.libreoffice.org/22216 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
* remove unused HelpText attribute from *.sdi filesNoel Grandin2016-02-081-1/+0
| | | | Change-Id: I21bcf66c552cd38eaae1bdc85626aa9bd1782ebd
* remove unnecessary module GUIDs in *.sdi filesNoel Grandin2016-02-081-2/+0
| | | | Change-Id: I78784abc1031027d69bbe31d150bc78c8bfbfcf4
* vcl: split exception types from input typesChris Sherlock2016-02-061-5/+15
| | | | | | | Having them in the file apptypes.hxx isn't necessary helpful, IMO so I've split the types into inputtypes.hxx and exceptiontypes.hxx Change-Id: I89a1ff168c3ae276b2f5486669d4ec2dda062d57
* Fix typosAndrea Gelmini2016-02-052-3/+3
| | | | | | | Change-Id: Ice72f8d9971e15dd6ef365e64cd567b8581a92d3 Reviewed-on: https://gerrit.libreoffice.org/21797 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* remove unused UUID and VERSION from .SDI filesNoel Grandin2016-02-051-6/+0
| | | | Change-Id: Ieb3e47c10c497a6f642f28a6741ac0fd2ecfd419
* remove unused Automation from .SDI filesNoel Grandin2016-02-051-2/+2
| | | | | | | Change-Id: I65e1ca6e022dc76ac96ed75da2c5e78e9356a3e9 Reviewed-on: https://gerrit.libreoffice.org/22107 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>