52 Commits
Author SHA1 Message Date
Arnaud VERSINI 96b8567314 [cpputools] Remove an useless OUString and use string_view instead
Change-Id: I3276a5110d054df56a0fb4d364f3c26ceb395835
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/210640
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2026-09-11 12:04:29 +02:00
Arnaud Versini 8b2600bb4d sw : use more OUStringLiteral
Change-Id: Id0c3abbc2f44ca7ae818eeb357e435cc18793ea8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/209730
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2026-09-09 14:08:03 +02:00
Arnaud Versini 9de2f95044 oox : use more frozen unordered_map to avoid allocations
Change-Id: I05bc2deedf4e3d4a60ac59049f98596211ae40ca
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/205675
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2026-06-08 08:53:36 +02:00
Arnaud Versini a94bce360d chart2 : use frozen::unordered_map to avoid allocations
Change-Id: I920ed50880c11cf516448fc4197cb86722d5e2f3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/205674
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2026-06-08 08:53:05 +02:00
Arnaud VERSINI c9d878efcb [basic] Use more constexpr
Change-Id: I79fc5736f0468e4e4c05188a2504d836de0a9781
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/204721
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2026-05-04 12:37:10 +02:00
Arnaud VERSINI fc76052495 rust : use repr(transparent) when we should
Change-Id: Iaf8334aa8d83c170454dffc8438395a395246938
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/198557
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2026-04-28 08:13:34 +02:00
Arnaud VERSINI 64510d3c8e rust_uno : Cargo.lock must be in the git repository
Change-Id: I3d73e2ad55864d68a8405c1b03fbed8b5ea754f5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/198541
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2026-02-07 18:55:43 +01:00
Arnaud VERSINI 36eeec7eee rust code maker : use constexpr sets and maps
Change-Id: Iffedaf0075abda2c6b1de24dea4c28aa2dd26282
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/194067
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2026-02-07 18:53:58 +01:00
Arnaud VERSINI 772c251919 sd : use more OUString literal and simplify code
Change-Id: Ia481390554ba4b64764039b31a79893385afbb7f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/198537
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2026-02-03 14:29:45 +01:00
Arnaud Versini c1a3a0958f Use std::unordered_map instead of frozen one for non const usage.
This partially reverts f6f971ab7c.
See https://gerrit.libreoffice.org/c/core/+/188086 for context.

Change-Id: I15edd635028698ba0c269dfeb9f4c5e8773dcef3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/196268
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-12-29 12:05:25 +01:00
Arnaud VERSINI fd7aba091c package : some minor OUString cleanup
Change-Id: Id683af197cc1010de6d7e3e72e170348c3d801b3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/194068
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2025-11-16 14:02:02 +01:00
Arnaud VERSINI 4158f6ca75 rust_uno : compare rtl_uString pointers and use less unsafe code
Change-Id: I29f573f290d925c5c86cde890af6a7abdb278685
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190932
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2025-10-13 08:37:37 +02:00
Arnaud VERSINI afdc487513 sfx2 : use frozen::unordered_maps to avoid allocations
Change-Id: I1259bc5f33f1c0d91493f24b5993237530cd8adb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190617
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-09-10 21:17:10 +02:00
Arnaud VERSINI 4b8371bffd sax : use frozen::unordered_map instaed std::map
Change-Id: I845a71b22debf8b786048a8b4132e5ef87434600
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190136
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-08-25 14:57:20 +02:00
Arnaud VERSINI a23d5a4177 vcl : simplify appendDestinationName.
Usee rtl::isAsciiAlphanumeric and also foreach loop.

Change-Id: Ie4a494e32cacc36740ee2aa2ae58956c6a40776b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/189313
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2025-08-18 14:50:22 +02:00
Arnaud VERSINI fd975e8bb5 vcl : no need to use a OString for comparison
Change-Id: I396c5a553f9d645ee630c10f939121b1dab3b6a4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188437
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2025-07-30 08:38:50 +02:00
Arnaud VERSINI f6f971ab7c vcl : use more frozen::unordered_map
Change-Id: I1012be7df58cacde41e2a56e84e264edd4059a8f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188086
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-07-20 21:17:05 +02:00
Arnaud VERSINI 5ff4c96de5 comphelper : use more frozen map
Change-Id: Ie089de8952e951a098791b5d363c3b4deda3365d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/182669
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-07-06 19:29:28 +02:00
Arnaud VERSINI 234b42d449 comphelper : use more u16string_view instead of const OUString&
Change-Id: Ie571697e63a97e7a6d306ee6f551c4bcf9aa99fa
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/184941
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2025-06-09 16:26:55 +02:00
Arnaud VERSINI a232758528 writeperfect : use more frozen and OUString literal to avoid allocation
Change-Id: I3caebb786e8c09ff9c79cda7fe9aabed2422e79e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/184409
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-06-09 14:40:47 +02:00
Arnaud VERSINI 94c051433a bridges : use frozen unordered map
Change-Id: I8ba732ef5b0e7600c70a2ce880db343945b594bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/183246
Tested-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-03-23 15:57:08 +01:00
Arnaud VERSINI 83579ee9f1 package : reduce OUString allocation by using u16string_view
Change-Id: I6c52fe7dbf3072132853ad820b5eba58beb6d76c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174149
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Jenkins
2025-03-02 19:28:34 +01:00
Arnaud VERSINI 9fddd34834 basic : use more OUString literals in basic comp
Change-Id: Ib67bee7e4ba85602be266bb7acc23c52251548b4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177611
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2025-02-26 07:58:16 +01:00
Arnaud VERSINI c38e3562e3 vcl : use frozen unordered_map to avoid allocations
Change-Id: I6b0ecd18fe3aa79211c5f1f28a2b87eece98d0e9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/179581
Tested-by: Arnaud Versini <arnaud.versini@pm.me>
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2025-02-03 09:11:29 +01:00
Arnaud VERSINI e47aebcec5 tools : use constexpr array instead of std::vector
Change-Id: Ifade95457b4b7c614a4ec83857c822d349a9f88d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/179444
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-12-27 16:26:13 +01:00
Arnaud VERSINI 5d8336d70e vcl mtfxmldump : allocates less OUString
Change-Id: Ibd78e40817f6a0ec071c9f45bb41c38e476dd02d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176408
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
2024-12-18 14:46:18 +01:00
Arnaud VERSINI fb30ee5a63 tdf#147021 : use std::size in scaddins
Change-Id: I8a17b5a782aedd8784595c7febbacc24921bce1d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169357
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-11-30 19:27:58 +01:00
Arnaud VERSINI f39f318a6f vcl pdf : use frozen unordered_map instead of frozen map
Change-Id: I6e9c3562fa13bf5e1adf7de55c48d1e7326d288e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175969
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-11-11 09:06:06 +01:00
Arnaud VERSINI b48dab459a sdext replace OUStringLiteral with new OUString literal
Change-Id: I3f2730024fa3cd81c38927d495cdff9c608bddbe
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169358
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-10-26 20:33:38 +02:00
Arnaud VERSINI 5ea219eb86 package : no need to use OUString here.
Change-Id: I7e46e1c89e85c0baf941393237fee971f0a0863e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/173770
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-09-29 14:56:44 +02:00
Arnaud VERSINI af9b429a13 vcl builder : no need to use OUString here
Change-Id: I82219272ae06db45968f2aecf2583c0b7196e3b7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/173768
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2024-09-23 08:17:04 +02:00
Arnaud VERSINI be1c9ee5b3 i18npool : use new OUtring literals
Change-Id: I2000f4c5409d988c9030e2eec70247d8f79059e1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166970
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2024-07-14 15:00:25 +02:00
Arnaud VERSINI a123c5b39a basic : use more OUString literals
Change-Id: I38f57c64c0d110e892982eaed2619c0915fe93b1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166825
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-05-05 14:44:04 +02:00
Arnaud VERSINI 7ab8d33522 basic : use std::u16string_view and new OUString literal
Change-Id: Id3cd708b98e1b0551955562afe3ac594bdc6e50a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167153
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-05-05 13:42:05 +02:00
Arnaud VERSINI 7b8ef7cf92 i18npoool : use rtl::isAsciiDigit instead of local isnumber
Change-Id: I1e9df1b048169767e649fc825f7a49dac16c040f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166969
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-05-02 10:02:37 +02:00
Arnaud VERSINI 190156976d tdf#147021 : use std::size instead of SAL_N_ELEMENTS in generated code too
Change-Id: Ib11a703c0250b1f80b7e79fe24e3bda915e7198b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166385
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-04-29 18:45:26 +02:00
Arnaud VERSINI b6c2ff2f05 i18npool : remove useless stdio include generation
Change-Id: Ib37f2c7553de2045496a090cda7006257fcf5e20
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166386
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
Tested-by: Jenkins
2024-04-29 14:21:32 +02:00
Arnaud VERSINI 91d67c32f6 i18npool : limit string comparison using an enum class
Change-Id: Iec5327b6f0125b22a2a4ee8ccb789b5403a7fdb4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166070
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-04-21 17:42:11 +02:00
Arnaud VERSINI 2ced3022d2 svx : use string view to avoid OUString creation
Change-Id: I3e3488ed15cf314ff6a666c7ec4bd5b41f96100e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161331
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-03-13 17:23:22 +01:00
Arnaud VERSINI 68b73e7a29 basic : use OUstring literal for XServiceInfo implementation
Change-Id: Ic4501ec887bb3f1249ab4a8da1eae07f6b2a22ef
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164305
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-03-11 08:32:32 +01:00
Arnaud VERSINI be6243144e basegfx : use OUstring literal for XServiceInfo implementation
Change-Id: I18a17e38897c1feda7fbba330c14a79c9b6d6f93
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164306
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-03-10 17:17:00 +01:00
Arnaud VERSINI b7b029cd71 basctl : use OUstring literal for XServiceInfo implementation
Change-Id: Iee7585585b5e01fad4609f37a06d2aeeff492123
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164304
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-03-10 16:42:20 +01:00
Arnaud VERSINI 140a3719f8 avmedia : no need to use constexpr here
Change-Id: I1b9975841a702d7ad3e05adef3d849c63ddf264a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/163009
Tested-by: Jenkins
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2024-03-10 15:32:52 +01:00
Arnaud VERSINI 6a83118936 canvas : use more OUString literals
Change-Id: I539e1e6d04299154bfe1ad2cf0362bdf3d96537b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/163051
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2024-02-08 04:37:27 +01:00
Arnaud VERSINI fe012b3571 starmath : move static data to usage location
Change-Id: Id6e500f84ee6d121bdd67e78a8eccfbc10b6ef27
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/162884
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-02-07 09:00:43 +01:00
Arnaud VERSINI f504b2de4a writerfilter : use constexpr frozen unordered_set instead of std::unordered_set
Change-Id: I37b4b5ec56f846b5c46faeaa5aebb88340ac3c6b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161385
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-01-21 13:51:57 +01:00
Arnaud VERSINI c4fb690e6f animations : use string OUString literals in a11y for XServiceInfo impl
Change-Id: Ifd23fd38e5fa289e9606ba0a8d5a04449ddcb0ea
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161738
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-01-21 13:51:07 +01:00
Arnaud VERSINI 5b93f1f7cb starmath : remove useless delete indirection
Change-Id: Ib308ce7735bdd1dea8611b08a8b73dd2686ca0d7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161387
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Arnaud Versini <arnaud.versini@pm.me>
2023-12-30 15:26:05 +01:00
Arnaud VERSINI 0f6f5048d2 use string OUString literals in a11y for XServiceInfo impl.
Used in getImplementationName and getSupportedServiceNames

Change-Id: I003db960123c66d4d47b99aec678b156bf4c4d91
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161301
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2023-12-29 00:36:51 +01:00
Arnaud VERSINI 7a6fb93905 idl : remove useless define and copy the code directly !
Change-Id: I92d53beb110a44624b72f7ec76ab8d1089f9caf0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127438
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
2021-12-25 08:32:26 +01:00
Arnaud VERSINI 82807f810b bridges : remove redundant public
Change-Id: I836412674448acb2a047d3d8b4711fa8d0b67257
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127410
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2021-12-24 14:53:13 +01:00
Arnaud VERSINI 587c530d13 bridges : use std mutex instead of osl::Mutex.
The lock_guard moved from RTTI::GetRTTI to x86_64::getRtti to avoid recursive lock.

Change-Id: I0e71581dd57a4fb2655d4b9040fb9d943f73ab9e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127095
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Jenkins
2021-12-20 14:14:44 +01:00