Andrea Gelmini
ab6ec04dd8
Fix typos
...
Change-Id: I8b4a9933b3c3df1871a7eb94023f0d0fd451afd1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/209902
Reviewed-by: Julien Nabet <serval2412@yahoo.fr >
Tested-by: Jenkins
2026-08-25 11:32:16 +02:00
Gabor Kelemen
0a42ddbdf1
tdf#146619 Use more forward declarations in include/xmloff
...
Change-Id: I49e39d669ba77a99b77322275d2e54e211da5b11
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/199386
Tested-by: Jenkins
Reviewed-by: Gabor Kelemen <gabor.kelemen@collabora.com >
2026-02-22 10:42:36 +01:00
Gabor Kelemen
72df5a6fdf
Drop unneeded forward declarations from include/
...
test drive the new bin/find-unneeded-includes --fwdecl mode
Change-Id: I507fa2b172ec9e348d1d91066ea241f02187b5ed
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/179321
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com >
2025-01-03 09:32:27 +01:00
Noel Grandin
e10e5c15cc
improve the script for reducing symbol exports
...
And apply some of the results
Change-Id: If555476fdd951cbc1d01fb3ef3ab1cbca2b64960
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124896
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk >
2021-12-01 12:13:59 +01:00
Noel
8ef6067596
loplugin:stringviewparam check methods too
...
not just functions
Change-Id: Icca295dd159002b428b73f2c95d40725434f04d9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105789
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk >
2020-11-17 09:57:35 +01:00
Noel
d9f0e8db9a
loplugin:unusedmethods
...
Change-Id: I90d402a5c18ab16424a042bcea1470fa8a3a5839
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105629
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk >
2020-11-12 16:54:19 +01:00
Noel Grandin
3de38e9556
use fastparser in forms
...
Change-Id: I7d09d64857e24267b4b4baddb563e28ceea92f2e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105560
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk >
2020-11-11 06:33:53 +01:00
Andrea Gelmini
0edc370ac5
Fix typo in code
...
It passed "make check" on Linux
Change-Id: I8bcc1598c7b885a025dd990eb2f75551e15b5c13
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101794
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr >
2020-09-01 15:28:58 +02:00
Noel Grandin
47cd34a60f
use more compact namespace syntax in /include
...
excluding the UDK headers of course
Change-Id: Iac7ab83d60265f7d362c860776f1de9d5e444ec0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93268
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk >
2020-05-03 09:21:51 +02:00
Noel Grandin
8629be3093
loplugin:finalclasses in xmloff
...
Change-Id: I664aa4dce822bcf14368fd8d08f678e2cd41fd12
Reviewed-on: https://gerrit.libreoffice.org/81852
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk >
2019-11-01 07:58:01 +01:00
Gabor Kelemen
6e5cdd9d55
tdf#42949 Fix IWYU warnings in include/xmloff/*
...
Found with bin/find-unneeded-includes
Only removal proposals are dealt with here.
Change-Id: Id20e0bc42e39f868a7c5d49d756f2ad5e14a86c0
Reviewed-on: https://gerrit.libreoffice.org/66637
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com >
2019-01-22 09:27:34 +01:00
Stephan Bergmann
91dd2db17b
loplugin:override: No more need for the "MSVC dtor override" workaround
...
The issue of 362d4f0cd4 "Explicitly mark
overriding destructors as 'virtual'" appears to no longer be a problem with
MSVC 2013.
(The little change in the rewriting code of compilerplugins/clang/override.cxx
was necessary to prevent an endless loop when adding "override" to
OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager();
in chart2/source/inc/LifeTime.hxx, getting stuck in the leading
OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that
isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.)
Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
2016-09-13 13:19:22 +02:00
Chris Sherlock
a238b1f8d3
Remove excess newlines
...
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 >
2016-02-09 08:18:05 +00:00
Noel Grandin
e1d88a5745
use unique_ptr for pImpl's in xmloff/
...
Change-Id: Ib95118941938af83fed566a085837e17f092017a
2015-11-17 08:23:35 +02:00
Noel Grandin
1c909a13a0
com::sun::star->css in include/ucbhelper to include/xmlscript
...
Change-Id: Iaa7f0b8455a601d3992c08cde0943c709c417256
2015-10-23 12:05:04 +02:00
Noel Grandin
d1b27a1d9a
loplugin:staticmethods
...
Change-Id: Id3d6b50e07ea0850af18ab9bdadfffe0e4602aab
2015-04-07 10:24:54 +02:00
Julien Nabet
430484932e
Typo: beeing->being
...
Change-Id: I34937e400ac8576154d33e1e11b80690222f532c
2014-06-29 13:15:51 +02:00
Stephan Bergmann
362d4f0cd4
Explicitly mark overriding destructors as "virtual"
...
It appears that the C++ standard allows overriding destructors to be marked
"override," but at least some MSVC versions complain about it, so at least make
sure such destructors are explicitly marked "virtual."
Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
2014-04-01 19:22:54 +02:00
Alexander Wilms
8792ec7b21
Remove visual noise from include
...
Conflicts:
include/framework/preventduplicateinteraction.hxx
include/sfx2/sfxbasecontroller.hxx
include/sfx2/sfxbasemodel.hxx
include/toolkit/awt/vclxtabpagemodel.hxx
include/vcl/field.hxx
include/vcl/settings.hxx
Change-Id: Ibccf9f88c68267a3d7e656012b51eaf644c418c2
Reviewed-on: https://gerrit.libreoffice.org/8272
Reviewed-by: Caolán McNamara <caolanm@redhat.com >
Tested-by: Caolán McNamara <caolanm@redhat.com >
2014-03-01 09:51:32 -06:00
Norbert Thiebaud
d6fec4ba5d
fdo#65108 inter-module includes <> include/xmloff
...
Change-Id: I1d2da1dd3c82e95b8ad4883ce83b42f6dcb6200f
2013-11-09 18:48:45 -06:00
Thomas Arnhold
b95ee1be9b
fixincludeguards.sh: include/xml*
...
Change-Id: Iffab819621615c59709c087202cc578af00dd799
2013-10-23 23:22:28 +02:00
Michael Stahl
cdbbbdce99
include: convert <type> here too
...
sed -i "s,<type>\([^<]\+\)</type>,\1,"
Change-Id: Ibdaeb2c396ad29ced05716142b38c6a1c3e49847
2013-06-04 12:00:59 +02:00
Bjoern Michaelsen
b9337e22ce
execute move of global headers
...
see https://gerrit.libreoffice.org/#/c/3367/
and Change-Id: I00c96fa77d04b33a6f8c8cd3490dfcd9bdc9e84a for details
Change-Id: I199a75bc4042af20817265d5ef85b1134a96ff5a
2013-04-23 22:20:31 +02:00