Commit graph

1448 commits

Author SHA1 Message Date
Noel Grandin
2cb43c6b88 clang-tidy: performance-unnecessary-copy-initialization in various
Change-Id: Ib7d75d4d8207261898568402e7983e0f11da14bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176417
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-11-11 21:02:55 +01:00
Noel Grandin
33d4760998 loplugin:passstuffbyref in lotuswordpro..pyuno
Change-Id: I9cd7a98d74b1896951124740b66d873ca41cceae
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176197
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
2024-11-07 13:17:27 +01:00
Bogdan Buzea
c818bbfb2c tdf#163486: PVS: Identical branches
V1037 Two or more case-branches perform the same actions. Check lines: 352, 369, 549
V1037 Two or more case-branches perform the same actions. Check lines: 672, 689, 784
V1037 Two or more case-branches perform the same actions. Check lines: 740, 771, 801

Change-Id: Ifc34cc2c283138b226b5487a838a93d97e45056e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175312
Tested-by: Jenkins
Reviewed-by: David Gilbert <freedesktop@treblig.org>
2024-11-01 01:43:43 +01:00
Bogdan Buzea
929075ef07 tdf#163486: PVS: Identical branches
V1037 Two or more case-branches perform the same actions. Check lines: 876, 881

Change-Id: I03f7b39a9f1d2226bf85904ffa2c4165b84b0aba
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175264
Reviewed-by: David Gilbert <freedesktop@treblig.org>
Tested-by: Jenkins
2024-10-30 01:43:09 +01:00
Caolán McNamara
70d3278ad4 cid#1608255 silence Overflowed constant
Change-Id: I3b518782a82bd1f066a36c87d1a52e5e44e4abda
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175259
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-10-21 12:33:48 +02:00
Caolán McNamara
23d7910c1e cid#1554844 COPY_INSTEAD_OF_MOVE
and

cid#1554878 COPY_INSTEAD_OF_MOVE
cid#1554907 COPY_INSTEAD_OF_MOVE
cid#1555082 COPY_INSTEAD_OF_MOVE
cid#1555224 COPY_INSTEAD_OF_MOVE
cid#1555301 COPY_INSTEAD_OF_MOVE
cid#1555378 COPY_INSTEAD_OF_MOVE
cid#1555395 COPY_INSTEAD_OF_MOVE
cid#1555452 COPY_INSTEAD_OF_MOVE
cid#1555466 COPY_INSTEAD_OF_MOVE
cid#1555514 COPY_INSTEAD_OF_MOVE
cid#1555566 COPY_INSTEAD_OF_MOVE
cid#1555753 COPY_INSTEAD_OF_MOVE
cid#1555782 COPY_INSTEAD_OF_MOVE
cid#1555825 COPY_INSTEAD_OF_MOVE
cid#1555842 COPY_INSTEAD_OF_MOVE
cid#1555891 COPY_INSTEAD_OF_MOVE
cid#1555917 COPY_INSTEAD_OF_MOVE
cid#1555961 COPY_INSTEAD_OF_MOVE
cid#1556021 COPY_INSTEAD_OF_MOVE
cid#1556055 COPY_INSTEAD_OF_MOVE
cid#1556307 COPY_INSTEAD_OF_MOVE
cid#1556349 COPY_INSTEAD_OF_MOVE
cid#1556436 COPY_INSTEAD_OF_MOVE
cid#1556665 COPY_INSTEAD_OF_MOVE
cid#1556696 COPY_INSTEAD_OF_MOVE
cid#1556718 COPY_INSTEAD_OF_MOVE
cid#1556775 COPY_INSTEAD_OF_MOVE
cid#1556835 COPY_INSTEAD_OF_MOVE
cid#1556877 COPY_INSTEAD_OF_MOVE
cid#1556994 COPY_INSTEAD_OF_MOVE
cid#1557010 COPY_INSTEAD_OF_MOVE
cid#1557074 COPY_INSTEAD_OF_MOVE
cid#1557111 COPY_INSTEAD_OF_MOVE
cid#1557142 COPY_INSTEAD_OF_MOVE
cid#1557163 COPY_INSTEAD_OF_MOVE
cid#1557164 COPY_INSTEAD_OF_MOVE
cid#1557214 COPY_INSTEAD_OF_MOVE
cid#1557254 COPY_INSTEAD_OF_MOVE
cid#1557322 COPY_INSTEAD_OF_MOVE
cid#1557327 COPY_INSTEAD_OF_MOVE
cid#1557492 COPY_INSTEAD_OF_MOVE
cid#1557565 COPY_INSTEAD_OF_MOVE
cid#1557663 COPY_INSTEAD_OF_MOVE
cid#1557798 COPY_INSTEAD_OF_MOVE

Change-Id: I2ae9c184dcc18d44a65437e18c49e16b27f4d81d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174893
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-10-14 15:28:24 +02:00
Caolán McNamara
26b8818481 cid#1608296 silence Overflowed integer argument
and

cid#1606815 Overflowed integer argument
cid#1606617 Overflowed integer argument

Change-Id: I4569190edd9b8d65e9b080a7ad0fac391f4a657e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/172348
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-08-25 17:36:28 +02:00
Caolán McNamara
ceec55249e cid#1607844 Overflowed integer argument
Change-Id: If6fc59a827983840c27701382c201ee16e690472
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/172254
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-08-22 18:21:13 +02:00
Gabor Kelemen
e1a351ca19 Remove now unneeded osl/diagnose.h
followup to commit 57c228803e

Change-Id: Iebfb23bb65e2bf898bf27f367cc9641f47a14cf3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167998
Reviewed-by: Gabor Kelemen <gabor.kelemen.extern@allotropia.de>
Tested-by: Jenkins
2024-07-20 10:20:31 +02:00
Noel Grandin
5c997461bf make XFCellListener single-arg constructor explicit
Change-Id: If519af401dca2ec77ff448409888bbc2871a0c9a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170682
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
2024-07-18 15:33:48 +02:00
Caolán McNamara
4504d921a6 cid#1555694 COPY_INSTEAD_OF_MOVE
and

cid#1555695 COPY_INSTEAD_OF_MOVE
cid#1555696 COPY_INSTEAD_OF_MOVE
cid#1555705 COPY_INSTEAD_OF_MOVE
cid#1555711 COPY_INSTEAD_OF_MOVE
cid#1555713 COPY_INSTEAD_OF_MOVE
cid#1555727 COPY_INSTEAD_OF_MOVE
cid#1555742 COPY_INSTEAD_OF_MOVE
cid#1555745 COPY_INSTEAD_OF_MOVE
cid#1555749 COPY_INSTEAD_OF_MOVE
cid#1555760 COPY_INSTEAD_OF_MOVE
cid#1555773 COPY_INSTEAD_OF_MOVE
cid#1555774 COPY_INSTEAD_OF_MOVE
cid#1555778 COPY_INSTEAD_OF_MOVE
cid#1555790 COPY_INSTEAD_OF_MOVE
cid#1555831 COPY_INSTEAD_OF_MOVE
cid#1555847 COPY_INSTEAD_OF_MOVE
cid#1555851 COPY_INSTEAD_OF_MOVE
cid#1555853 COPY_INSTEAD_OF_MOVE
cid#1607763 COPY_INSTEAD_OF_MOVE
cid#1607504 COPY_INSTEAD_OF_MOVE

Change-Id: I991f15f12e773d325ed9725498df5725b049a858
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170625
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-07-17 21:49:14 +02:00
Caolán McNamara
0d72c7f9de cid#1554775 COPY_INSTEAD_OF_MOVE
Change-Id: I75b3205b23b43ba639b75808d0a832418f7d7c24
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169976
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-07-04 12:57:54 +02:00
Mike Kaganski
2cb5e2256c Simplify solar.h a bit
Use std::copy / std::reverse_copy for double serialization. Also
remove __cplusplus conditions, because the header is not included
in any C files. And simplify the functions uses, where casts were
unneeded.

Change-Id: I1d8375e601851231fdffcf52e3f3e3bdcbcbde7f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169706
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2024-06-28 22:53:27 +02:00
Caolán McNamara
cc3a8a025b cid#1597280 Dereference null return value
Change-Id: I0275c178a478b7143b0cf4ed19f030009b7bd2fe
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167504
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-05-11 14:30:33 +02:00
Noel Grandin
e00f6e0b9d loplugin:ostr in lotuswordpro
Change-Id: Ifea660ab9e0663312f9013670c27d3feda314d81
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167427
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-05-10 09:51:45 +02:00
Nima Mahanloo
d04b6c775d tdf#145614 Convert #define to enum
Change-Id: I0a360a93775d2c46b7e02009879d7a18315497b1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166168
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Tested-by: Jenkins
2024-04-30 08:39:15 +02:00
Mike Kaganski
fce1fb034d Use more o3tl::convert in lotuswordpro
Change-Id: I7ec74dc105f846307f5bf3d3c8cedcc4aebb64a4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166058
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2024-04-14 06:57:06 +02:00
Rafał Dobrakowski
c8b8a2c2cb tdf#141908 - CppUnittests: replace usage of sal_Int32 with colors
Conversion of hex/dec colour notation
(example entry Color( 255, 255, 255), Color(0xFFFFFF) - COL_WHITE)
For the other available colour definitions.

Change-Id: I9eed0cd64adcbc8d25e1c22143a000906a457586
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/163729
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2024-03-14 02:33:03 +01:00
Keldin Maldonado (KNM)
5dadcd1ea9 tdf#145538 use range based for loops
using range based for loops instead of index based
to increase readability in codebase

Change-Id: Ib8c3ec3796fce9228cee1d90beb924128b0dea3f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/162950
Tested-by: Jenkins
Reviewed-by: Hossein <hossein@libreoffice.org>
2024-02-12 13:30:40 +01:00
Mike Kaganski
0fa827dbb2 Drop std::as_const from css::uno::Sequence iterations
Obsoleted by commit 2484de6728 (Remove
non-const Sequence::begin()/end() in internal code, 2021-10-15) and
commit fb3c04bd19 (Drop non-const
Sequence::operator[] in internal code, 2021-11-05).

Change-Id: Idbafef5d34c0d4771cbbf75b9db9712e504164cd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/162640
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2024-01-27 12:43:48 +01:00
Mike Kaganski
a01a642ebb UnoApiTest::loadFromURL -> UnoApiTest::loadFromFile
The old name was misleading (it doesn't take an URL, but a filename);
also, now it's easier to grep for it - doesn't get mixed with
vcl::graphic::loadFromURL.

Change-Id: Ib88d2194200a6a54d2326971e0306ba39f0c7025
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161578
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2024-01-03 07:51:28 +01:00
Caolán McNamara
dd82bf96ac ofz#64860 Timeout
Change-Id: Ib475cbba118f0705b9475dbe61ec1b96a68af47c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161544
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2024-01-02 18:11:41 +01:00
Caolán McNamara
f0a8b5b81d move IsFuzzing to comphelper
and try something a bit more generic

Change-Id: I1d8256576cd02f0a589df350ba7b53059dd586a5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161250
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2023-12-30 12:45:45 +01:00
Julien Nabet
250f6d0caf Static map initialization in lotuswordpro/lwpnumericfmt
Change-Id: I7894e8b7bf9b4d4c25e1c75551cae2b58b26d6a7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160881
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins
2023-12-17 13:47:00 +01:00
Caolán McNamara
5d3d7cd656 cid#1546281 COPY_INSTEAD_OF_MOVE
and

cid#1546278 COPY_INSTEAD_OF_MOVE
cid#1546276 COPY_INSTEAD_OF_MOVE
cid#1546247 COPY_INSTEAD_OF_MOVE
cid#1546244 COPY_INSTEAD_OF_MOVE
cid#1546243 COPY_INSTEAD_OF_MOVE
cid#1546220 COPY_INSTEAD_OF_MOVE
cid#1546209 COPY_INSTEAD_OF_MOVE
cid#1546207 COPY_INSTEAD_OF_MOVE
cid#1546206 COPY_INSTEAD_OF_MOVE
cid#1546205 COPY_INSTEAD_OF_MOVE
cid#1546197 COPY_INSTEAD_OF_MOVE
cid#1546180 COPY_INSTEAD_OF_MOVE
cid#1546172 COPY_INSTEAD_OF_MOVE
cid#1546165 COPY_INSTEAD_OF_MOVE
cid#1546164 COPY_INSTEAD_OF_MOVE
cid#1546158 COPY_INSTEAD_OF_MOVE
cid#1546151 COPY_INSTEAD_OF_MOVE
cid#1546135 COPY_INSTEAD_OF_MOVE
cid#1546132 COPY_INSTEAD_OF_MOVE
cid#1546129 COPY_INSTEAD_OF_MOVE
cid#1546128 COPY_INSTEAD_OF_MOVE
cid#1546122 COPY_INSTEAD_OF_MOVE
cid#1546117 COPY_INSTEAD_OF_MOVE
cid#1546113 COPY_INSTEAD_OF_MOVE
cid#1546106 COPY_INSTEAD_OF_MOVE
cid#1546099 COPY_INSTEAD_OF_MOVE
cid#1546091 COPY_INSTEAD_OF_MOVE
cid#1546085 COPY_INSTEAD_OF_MOVE
cid#1546069 COPY_INSTEAD_OF_MOVE
cid#1546063 COPY_INSTEAD_OF_MOVE
cid#1546062 COPY_INSTEAD_OF_MOVE
cid#1546058 COPY_INSTEAD_OF_MOVE
cid#1546056 COPY_INSTEAD_OF_MOVE
cid#1546051 COPY_INSTEAD_OF_MOVE
cid#1546040 COPY_INSTEAD_OF_MOVE
cid#1546030 COPY_INSTEAD_OF_MOVE
cid#1546028 COPY_INSTEAD_OF_MOVE
cid#1546015 COPY_INSTEAD_OF_MOVE
cid#1546001 COPY_INSTEAD_OF_MOVE

Change-Id: Ib954c92a300fc323b29f27880fdf8bc46ed98862
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160520
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2023-12-10 11:57:12 +01:00
Caolán McNamara
a58c0d4cef cid#1556778 COPY_INSTEAD_OF_MOVE
and

cid#1546474 COPY_INSTEAD_OF_MOVE
cid#1546473 COPY_INSTEAD_OF_MOVE
cid#1546469 COPY_INSTEAD_OF_MOVE
cid#1546467 COPY_INSTEAD_OF_MOVE
cid#1546458 COPY_INSTEAD_OF_MOVE
cid#1546448 COPY_INSTEAD_OF_MOVE
cid#1546446 COPY_INSTEAD_OF_MOVE
cid#1546441 COPY_INSTEAD_OF_MOVE
cid#1546424 COPY_INSTEAD_OF_MOVE
cid#1546377 COPY_INSTEAD_OF_MOVE
cid#1546374 COPY_INSTEAD_OF_MOVE
cid#1546352 COPY_INSTEAD_OF_MOVE
cid#1546334 COPY_INSTEAD_OF_MOVE
cid#1546328 COPY_INSTEAD_OF_MOVE
cid#1546310 COPY_INSTEAD_OF_MOVE
cid#1546303 COPY_INSTEAD_OF_MOVE

Change-Id: I8a43fdd574857ba1d3ec416f23c8269a9a08e12f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160515
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
2023-12-09 22:38:50 +01:00
sahil
e2c9b90561 changed sal_uInt8 to BenError (enum)
Change-Id: Ia7cb2f879b7be833d1a5ccf845a8dc8bfb94befd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156900
Tested-by: Jenkins
Reviewed-by: Hossein <hossein@libreoffice.org>
2023-10-24 09:41:12 +02:00
Stephan Bergmann
bd6fb0cd3e Extended loplugin:ostr: Rewrite some O[U]StringLiteral -> O[U]String
...in include files.  This is a mix of automatic rewriting in include files and
manual fixups (mostly addressing loplugin:redundantfcast) in source files that
include those.

Change-Id: I1f3cc1e67b9cabd2e9d61a4d9e9a01e587ea35cc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/158337
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2023-10-23 11:04:42 +02:00
Stephan Bergmann
e0c33ec15f loplugin:ostr: automatic rewrite
Change-Id: I2d09b2b83e1b50493ec88d0b2c323a83c0c86395
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/157647
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Jenkins
2023-10-07 00:47:20 +02:00
sahil
f5bcc34580 tdf#114441 - Convert sal_uLong to better integer types
Change-Id: Idc6de0c0275ce9913e248548466b77aaef180d7d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155873
Tested-by: Jenkins
Reviewed-by: Hossein <hossein@libreoffice.org>
2023-09-12 02:36:35 +02:00
Caolán McNamara
81e8cc7aa3 ofz#58529 Null-dereference READ
Change-Id: I9d8ea7d40429faee94889e4d2d9df7440b18e65f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151255
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-05-02 12:54:04 +02:00
Noel Grandin
dba55c304a loplugin:unnecessarygetstr extend to createFromAscii
idea from mike kaganski

Change-Id: I0ecb9cad091d7a048d2ddae73165bf22748f3872
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/150907
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-04-24 12:45:22 +02:00
Caolán McNamara
ffdee63185 ofz#57800 Timeout
Change-Id: I14b4906ce61b3d8a86cb9a766986c49e0b0e6abc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/150171
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-04-10 10:40:01 +02:00
jpuronah
6201b6468f tdf#143148: Use pragma once instead of include guards
Change-Id: I8369e0bdb6a01838d87e7cddc1cbaed911fe461a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149952
Tested-by: Jenkins
Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
2023-04-03 12:32:47 +02:00
Noel Grandin
7f101a5148 no need to allocate these separately
they are all one or two words in size

Change-Id: I09a0392f8d0b067a3a8e4105568b578ef4795c58
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148341
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-03-06 19:57:50 +00:00
Mike Kaganski
f2f008c52a Merge SvXMLAttributeList to comphelper::AttributeList
And simplify the latter, to always use "CDATA" type (as the former did).
"CDATA" was used in all cases but one, where an empty string was used.

Change-Id: I1b3bfae40e29628e4094d9a6e58a69a66865874c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145526
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2023-01-15 12:32:29 +00:00
Caolán McNamara
13cd316bc7 ofz#55034 Stack-buffer-overflow in vcl::bitmap::CreateFromData
since:

commit 21734247d5
Date:   Tue Jan 10 12:56:58 2023 +0200

    drop internal support for 1-bit images

the input data is still 1 bit here, even if the output will be 8 bit

Change-Id: Ib7eb301b2e62ebc61ee61ebb99e6de92c1f4931b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145410
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-12 20:03:30 +00:00
Noel Grandin
21734247d5 drop internal support for 1-bit images
on modern hardware, these are less efficient than regular alpha
channels.

Also, this greatly simplies the range of stuff that vcl needs to deal
with, which will make the upcoming transparency->alpha patch easier to
deal with.

Enhance vcl::CreateFromData to convert incoming 1-bit data to 8-bit
image.

Change-Id: I35829da750029fe373d0d2911a669d10bab6ad23
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145321
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-12 06:52:42 +00:00
Noel Grandin
4504d6ddf1 loplugin:unusedfields make some fields private
this is one of the secondary analyses this plugin performs

Change-Id: I1c496e234513362338600180e51f7e71e605a7cc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143180
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-11-24 07:00:02 +01:00
Xisco Fauli
248be1fe7d UnoApiTest: simplify code by using tempfile member
SwModelTestBase does the same. this will help to make
SwModelTestBase inherit from UnoApiTest

Change-Id: If1c824cf92f0e8b70253e4d5fdeddcaa521d4632
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142287
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2022-11-08 11:54:22 +01:00
Stephan Bergmann
34cecc1f9c -Werror,-Wdeprecated-declarations (sprintf, macOS 13 SDK): lotuswordpro
Change-Id: I6b9707c1f416eb1ab890ff0d783151066cf695d4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142329
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-11-06 01:14:44 +01:00
Xisco Fauli
8c018651a0 CppunitTest_lotuswordpro_import_test: inherit from UnoApiTest
Change-Id: I6287ea3b0ed24a8dca002b95d59120ce302fc2c4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142100
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2022-11-01 12:33:55 +01:00
Noel Grandin
425255567e use more string_view
Change-Id: Ie826234aa9064b08b8f0647738b57c47ac0ed91a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/141369
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-10-17 10:39:11 +02:00
Caolán McNamara
3642db420a ofz#51149 avoid Timeout
Change-Id: I7709e69154d231e597d364210bc876afca3df4a3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139719
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-09-09 13:40:46 +02:00
Caolán McNamara
7a7cd04488 rearrange to have a single ConvertTable call
Change-Id: Ic48ebac04d6e95df8748f38b1f23a8295fba77b8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139718
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-09-09 13:40:23 +02:00
Caolán McNamara
b715118b8e ofz#50756 Direct-leak
Change-Id: I76404a154bcf5b3724970ce546f64c0a3cf2f282
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139015
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-30 12:20:55 +02:00
Andrea Gelmini
df31e72e58 Fix typos
Change-Id: I62b11b80e4d3d2a8bd4c461804f39d8da6a58066
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137298
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2022-07-21 16:16:44 +02:00
Bartosz Kosiorek
a0bcd1d187 tdf33787 lotuswordpro: fix increments of ordered bullets
With previous implementation there was an bug which causing missing
incrementing ordered bullets. For example:

        A. Lettered bullet uppercase 1 (A. )
        A. Lettered bullet uppercase 2 (B. )
        A. Lettered bullet uppercase 3 (C. )

This patch is fixing that, and increments is working correctly:

        A. Lettered bullet uppercase 1 (A. )
        B. Lettered bullet uppercase 2 (B. )
        C. Lettered bullet uppercase 3 (C. )

It is also make sure that nested and skipped bullets are working correctly:

        1) Numbered bullet w/ bracket one (1) )
           Skipped bullet
        2) Numbered bullet w/ bracket two (2) )
            a) Lettered bullet w/ bracket lowercase 1 (a) )
            b) Lettered bullet w/ bracket lowercase 2 (b) )

Change-Id: I1d1ad9ee549039077b418756efefb10bacfa1ddd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137112
Tested-by: Jenkins
Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
2022-07-21 09:11:31 +02:00
Bartosz Kosiorek
7bb911d118 tdf#112626 lotuswordpro: switch to constexpr and update documentation
Change-Id: I566c22285978f7b31961d105d02d99d0696908b9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137041
Tested-by: Jenkins
Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
2022-07-14 10:32:23 +02:00
Noel Grandin
41e6e40d72 clang-tidy modernize-pass-by-value in lotuswordpro
Change-Id: Ia20d653bdfedf49430f8ba46c8200c2b96734d81
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136336
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-06-23 18:24:26 +02:00