summaryrefslogtreecommitdiffstats
path: root/compilerplugins
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:unusedfields variousNoel Grandin2015-11-261-3/+18
| | | | Change-Id: I18f94269a1172cf195ee402384f7144610e1e82d
* compilerplugins: adapt commentMiklos Vajna2015-11-261-1/+1
| | | | | | | ...after 49c2b9808df8a6b197dec666dfc0cda6321a4306 "bin/rename-sw-abbreviations.sh" Change-Id: Ibd74246403bead8100d181b335f9145013f31e5e
* Adapt whitelisted namesStephan Bergmann2015-11-261-4/+4
| | | | | | | ...after 49c2b9808df8a6b197dec666dfc0cda6321a4306 "bin/rename-sw-abbreviations.sh" Change-Id: I90cec551bdc52ea4d76dd9e32b45692e8a3be70f
* loplugin:unusedfields variousNoel Grandin2015-11-251-1/+2
| | | | Change-Id: I59d9f7f73677358b4ae57efda965d43718bdf0d5
* loplugin:unusedfields in include/sot,include/svl,include/svtoolsNoel Grandin2015-11-251-1/+2
| | | | Change-Id: Ia2dd13c2c48b127ab85311a668fa1f37608d360a
* update unusedfields plugin to use new clang warn_unused attribute supportNoel Grandin2015-11-242-24/+11
| | | | Change-Id: I7b84de29b672e40cbf3c3d340d235f334d2be8cb
* loplugin:unusedfields in vcl/Noel Grandin2015-11-241-1/+5
| | | | | | | | | and remove the unused SALEVENT_MOUSEACTIVATE stuff Change-Id: Ieb85872eca68621c6a7be47ff5dbea12f7690507 Reviewed-on: https://gerrit.libreoffice.org/20140 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedfields in tools/Noel Grandin2015-11-241-1/+2
| | | | Change-Id: Ic460bf42cbcf356cdcebbd5060a0dad3d3caa4ef
* loplugin:unusedfields in sw/Noel Grandin2015-11-241-1/+2
| | | | Change-Id: I323a038e5581b00cd3a4ea2f362c66540377759e
* loplugin:unusedfields in scNoel Grandin2015-11-231-3/+7
| | | | Change-Id: If5123e676a27302f3e11475309bc748977c6f430
* loplugin:unusedfields in dbaccessNoel Grandin2015-11-201-0/+2
| | | | Change-Id: I563c3727c1719fe21acced269e5469c2de7112e8
* loplugin:unusedfields in basctlNoel Grandin2015-11-201-2/+43
| | | | | | | and improve the plugin to search for only WARN_UNUSED and fundamental types Change-Id: Ic06207758e28d44d64d76d8119fd76b5b098bb05
* new loplugin:unusedfieldsNoel Grandin2015-11-203-0/+417
| | | | | | run it over the framework module Change-Id: I1220a4be0936ba30136ce22ffd78633c8a7b9d35
* loplugin:sallogareasStephan Bergmann2015-11-192-2/+35
| | | | Change-Id: I2220ab194384fb397716bf3227d38716ba54f537
* isLookupContext already available in Clang 3.7.0Stephan Bergmann2015-11-191-1/+1
| | | | Change-Id: Ia17a116b1f7605b5a8d572c6a6f4ef64adea8f94
* Adapt compilerplugins to old Clang versionsStephan Bergmann2015-11-192-3/+12
| | | | Change-Id: I91335f1e81e251f0578792517dded9fae239fb61
* vcl, sd: rename some overly generic static variablesMichael Stahl2015-11-171-2/+2
| | | | Change-Id: I89159df36361f9ceff3401ef379c8230465617b7
* Fix check for std::weak_ptr (that may be in an inline namespace)Stephan Bergmann2015-11-172-1/+132
| | | | Change-Id: I048aef08df43d07544aafc69b711d258dc40bc21
* Work around loplguins:badstaticsStephan Bergmann2015-11-171-0/+2
| | | | Change-Id: I597dbc3e4288acb5b71f7a344ae78ba568451614
* Work around loplguins:badstaticsStephan Bergmann2015-11-171-0/+2
| | | | Change-Id: I9d8d0f6cc001de09d8d3f96d8ad7e243e07f426d
* Work around loplguins:badstaticsStephan Bergmann2015-11-171-0/+6
| | | | Change-Id: I243dcb86cc1a99550b26962daf47c19eb62aa8df
* Improve commentStephan Bergmann2015-11-171-1/+2
| | | | Change-Id: I25931d59fc1b4feb1fd1eaa036a64a28a269f409
* Work around loplugin:badstaticsStephan Bergmann2015-11-171-0/+1
| | | | | | | | | | | | | | | | | | | | ...about static ImplImageTree instance; in ImplImageTree::get (vcl/source/gdi/impimagetree.cxx) due to StyleIconSet ImplImageTree::maIconSet; (of type std::unordered_map<OUString, IconSet, OUStringHash>) due to IconCache ImplImageTree::IconSet::maIconCache; (of type std::unordered_map<OUString, std::pair<bool, BitmapEx>, OUStringHash>), reported at least when building against Mac libc++. This is a false positive, as ImplImageTree::shutDown cleans this up. Change-Id: Idb1a99aa33f71286e57b31b7adb833e3ee8b5f5a
* Improve loplugin:badstatics diagnostic outputStephan Bergmann2015-11-171-17/+20
| | | | Change-Id: Ia2f088a79b03ff866429e1cae73edccd36a01b4f
* Use a DiagnosticsEngine::NoteStephan Bergmann2015-11-171-1/+1
| | | | | | ...a Remark prints the whole stack of includes leading up to the source Change-Id: I41e5bd2855b4a55b54e63432703f2e65b4c44249
* loplugin:unnecessaryvirtualNoel Grandin2015-11-172-200/+118
| | | | | | | | | | update the plugin with lessons learned from the mergeclasses plugin and re-run it Change-Id: I9d622eb3d05fceaf8fa764c533c8fa5dfb4c7711 Reviewed-on: https://gerrit.libreoffice.org/20015 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* compilerplugins: enhance badstatics plugin to follow pointersMichael Stahl2015-11-171-8/+70
| | | | | | | | | | .... and references. This gives numerous false positives as pointers may be re-set prior to shutdown, so needs a white-list. Change-Id: I19a011c6f19501cc31b3d9ae76b599296f132478 Reviewed-on: https://gerrit.libreoffice.org/19949 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Move CommandInfoProvider to vcl and use it from thereSamuel Mehrbrodt2015-11-161-1/+1
| | | | | | | Change-Id: If7e3ef81cad8974bc54f359096e3ecc700fa62d6 Reviewed-on: https://gerrit.libreoffice.org/19984 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* Cleanup: Consolidate the different ways to get command labels/tooltipsSamuel Mehrbrodt2015-11-161-1/+1
| | | | Change-Id: Ieab809a3122c9d592894b84ec2e68195a4e02dde
* cppcheck:stlIfStrFindNoel Grandin2015-11-151-1/+1
| | | | | | | | | "Inefficient usage of string::find() in condition; string::compare() would be faster." Change-Id: I90403b1d05eff6499c10be33068e5fd4fed30b62 Reviewed-on: https://gerrit.libreoffice.org/19966 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* new loplugin ptrvector - find bad comparison of container<unique_ptr>Noel Grandin2015-11-131-0/+81
| | | | Change-Id: Idb18ae1ca2f7c644680703dc3b7dd3cd6000e040
* sc: loplugin:badstatics: remove pActiveEdWnd global variableMichael Stahl2015-11-131-1/+3
| | | | | | | Just use a Link, or rather std::function to set a member in the tab page. Unfortunately loplugin:vclwidgets complains about the new member. Change-Id: Ie2f9cb73c38292d02057d43b12694c6609fa0db8
* Generalize COM_GCC_IS_CLANG -> COM_IS_CLANGStephan Bergmann2015-11-121-1/+1
| | | | | | | | | ...in anticipation of building with clang-cl.exe on Windows Change-Id: I1d723c9d3b5ca8a2bc6b27ef0189a7b053581398 Reviewed-on: https://gerrit.libreoffice.org/19928 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:mergeclass, fix and update resultsNoel Grandin2015-11-112-4/+172
| | | | Change-Id: I0660833db69604c5c4dd3c5826f1baac3086e052
* 5th step to remove tools/rtti.hxxOliver Specht2015-11-111-2/+0
| | | | | | | | | | | tools/rtti.hxx removed completed the interface of some Sdr.* Items and removed pseudo items Change-Id: I0cdcd01494be35b97a27d5985aa908affa96048a Reviewed-on: https://gerrit.libreoffice.org/19837 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
* disable loplugin memoryvar for nowNoel Grandin2015-11-111-1/+1
| | | | | | seems to be crashing with latest(trunk) clang sometimes Change-Id: Ice92b35deb6cb4610dbd53f570d0b43e9d2a2834
* new loplugin: memoryvarNoel Grandin2015-11-111-0/+239
| | | | | | | | | | detect when we can convert a new/delete sequence on a local variable to use std::unique_ptr Change-Id: Iecae4e4197eccdfacfce2eed39aa4a69e4a660bc Reviewed-on: https://gerrit.libreoffice.org/19884 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Missing isInLokIncludeFile checkStephan Bergmann2015-11-101-0/+2
| | | | Change-Id: I6577987f3f3717337e6607ffbe38397f229978f0
* Keep LibreOfficeKit.hxx compatible with C++03Stephan Bergmann2015-11-101-0/+9
| | | | Change-Id: Ic2f123c9b341dbb421b766c3bba1fc56c1bfb41d
* new loplugin: oncevarNoel Grandin2015-11-091-0/+146
| | | | | | | Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* compilerplugins: enhance "badstatics" plugin to find membersMichael Stahl2015-11-061-7/+52
| | | | | | Finds static variables of types that have bad non-static members. Change-Id: I81ee940580c0f043feb543840ea4d3dd27734202
* compilerplugins: getAsTagDecl doesn't exist in clang 3.5Michael Stahl2015-11-062-1/+11
| | | | | | | Change-Id: If7277820227486e2eb578a167e0fd9f05c8f74ae Reviewed-on: https://gerrit.libreoffice.org/19823 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:stringconstant: elide explicit ctor usageStephan Bergmann2015-11-061-66/+161
| | | | Change-Id: I962db9583ef9cada42a61b6a95eeea818fceeead
* fix tdf#95488, crash in report wizardNoel Grandin2015-11-051-0/+1
| | | | | | | This reverts commit bb76b8f10697f3d5ca1f9a2f19902b043bd61cd7 "loplugin:mergeclasses" Change-Id: Ib65459ab45cd4cefa859ed75f9ddc4f070879b1d
* Improve warning messagesStephan Bergmann2015-11-051-12/+12
| | | | Change-Id: I75eb9f17a3dcc688314355fa957e3f34086b161a
* Do not implicitly run this for now; it breaks the buildStephan Bergmann2015-11-051-1/+1
| | | | Change-Id: I84679c4ca395f934bca8f92f0587f6cf16cfa5e6
* complete initial compiler plugin for the ServiceDocumenterBjoern Michaelsen2015-11-051-10/+88
| | | | | | | | | | | | - this creates a set of static html-meta-redirect-pages linking the UNO implementation name to its C++ class, while doing a clang compile from scratch - the output is written to $(WORKDIR)/ServiceImplementations - still quite some corner cases missing, e.g: - anonymous namespaces - non-trivial getImplementationName() functions Change-Id: I19a0e8a3901277dc3811eb428cac9d00737f14a9
* skeleton implementation for ServiceDocumenter compiler pluginStephan Bergmann2015-11-051-0/+228
| | | | Change-Id: I6a9c957c0c4dac16365d269e57c30210619d23c9
* yyyyyNoel Grandin2015-11-041-0/+43
| | | | Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
* tdf#94269: Replace "n" prefix for bool variables with "b"Benjamin Ni2015-11-027-102/+102
| | | | | Change-Id: I178545792c7354a362658ac7ef8b1d4cf0865797 Signed-off-by: Michael Stahl <mstahl@redhat.com>