The SAL_N_ELEMENTS() macro is defined in 'include/sal/macros.h'. It calculates the number of elements in an array at compile time. According the comments, "the argument must be an array and not a pointer". These are some examples from the above file: SAL_N_ELEMENTS("foo"); char aFoo[]="foo"; SAL_N_ELEMENTS(aFoo); This macro, uses sizeof() operator to calculate the "sizeof (arr) / sizeof ((arr)[0]))". Now we can use std::size() instead of SAL_N_ELEMENTS() macro. The result would be constexpr, compile-time constant expression. As an example, this code: for (unsigned i = 0; i != SAL_N_ELEMENTS(aFoo); ++i) { ... } can be converted into this one: for (unsigned i = 0; i != std::size(aFoo); ++i) { ... } One pitfall would be comparing the result with a signed value. Then, either you have to cast the result to a signed value, or use an unsigned value for the other side of the comparison.
(In reply to Hossein from comment #0) > One pitfall would be comparing the result with a signed value. Then, either > you have to cast the result to a signed value, or use an unsigned value for > the other side of the comparison. If there's need, we can make available C++20 > template<class T, ptrdiff_t N> constexpr ptrdiff_t ssize(const T (&array)[N]) noexcept; as o3tl::ssize for the time being. (Also, if it is known that the other side of the comparison, even though of signed type, must be non-negative, like e.g. a call to OUString::getLength, then o3tl::make_unsigned from o3tl/safeint.hxx is a good choice.)
Note also, that there are cases where std::size can't give a compile-time constant, unlike the SAL_N_ELEMENTS macro. That may be e.g. when a class contains an array member, and a static_assert checks the array dimension in its constructor. In that case, a "'this' can't be used in constant expression" error would result after the replacement. Usually, there's a way to restructure such cases, to avoid use of static asserts, and ensure the tested invariant some other way.
VaibhavMalik4187 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/8b327cd86d71d71d2f5f883321e5d53e3b42ed4e tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Hossein committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/a5c6fbe247ee9f9b2fba828d1360748c3fe4642b tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
(In reply to Commit Notification from comment #4) > Hossein committed a patch related to this issue. > It has been pushed to "master": This is an example for changing the type of the variable to a suitable unsigned type. https://git.libreoffice.org/core/commit/a5c6fbe247ee9f9b2fba828d1360748c3fe4642b The signed loop index variable of type 'int' is converted into 'size_t' to match the return value of std::size() which is unsigned
Gautham Krishnan committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/51fb84829afbc1c0957fd1a489085613ad199f1a tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
psidiumcode committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5b320aa058504becae6ac6746926b481b326a24d tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Sinduja Y committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/a40ed687a3e2f8e1d17705fcd7d397ba09398cb6 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Pragat Pandya committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/6abc09926c9b55a445b906303f56c6ec7fdeabf9 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Roman Kuznetsov committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/d694b572c7262a326ac7ceac2e558f3fc6c78df4 tdf#147021 Use std::size() instead SAL_N_ELEMENTS macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Roman Kuznetsov committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/81aad6d53fa792dd95732e401ecb298714f699f4 tdf#147021 Use std::size() instead SAL_N_ELEMENTS macro It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
jsala committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/2c68c419c1fce6de1a81e1f13a84b7069125a204 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5b04eed4d22c9e3caa76e195582b15862e9261b6 tdf#147021 Use std::size in {Read,Write}JobSetup It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Re-evaluating the EasyHack in 2022 This issue is still relevant. There are still several instances of SAL_N_ELEMENTS() macro in use: $ git grep SAL_N_ELEMENTS|wc -l 1897
ehsan committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/e010a57ab2489464719fb73ad0aea6e4e4c60383 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
jsala committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/9e7e95a57b7c16941d9fdc59f806c1f9e4263379 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
jsala committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/35a7e40d372d3211061465346825cf543a095f6d tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
jsala committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/8dc57426c3bd3385c30b9007991af8496b315f9a tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radhey Parekh committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/feb8d07333544ff2206489b786b7af30744d014a tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Tomoyuki Kubota committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/86498bcd73050e44ea5ea53c384edb0baf033d16 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
jsala committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/b26793b907ce823e5cce38f6fe93b420ebe38579 tdf#147021 REVERT Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Let's keep the default assignee, as this is a generic task that can be worked on by several different contributors.
(In reply to Aron Budea from comment #22) > Let's keep the default assignee, as this is a generic task that can be > worked on by several different contributors. Understood, thank you for your help
OmkarAcharekar committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/75e6ffe31d74ac36d8864b018008d4208a7efe73 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
insanetree committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/07b6e34397037722d40b2013eef21085eca4f235 tdf#147021: SAL_N_ELEMENTS changed to std::size It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
jsala committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/04a58d7eae2b26559efd48ff7cdd23cec1c2187c tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Leonid Ryzhov committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/49737dff4b7b46c44f8c8e1c4de0291309eef78e tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Vinit Agarwal committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/d898620cb47e0d794d4033fdfa13b049f972a9ff tdf#147021 - Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 7.6.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/4998370216bbea3bcaff7fac2d62cbb4ac978c5d tdf#145538,tdf#147021: sw/source: range based loops It will be available in 7.6.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/7bec1689fcbe9102aa220739b6113118d9a8dd1f tdf#145538,tdf#147021: qa: range based loops It will be available in 7.6.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/9bc3c4fa8d380bd7ae5689692cc80f815bed4707 tdf#147021: sw/source use std::size() instead of SAL_N_ELEMENTS It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/880bc4255099132557fe8d7d36a50c06f05cf5a1 tdf#147021: vcl/* KeyboardReplacements SAL_N_ELEMENTS removal It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/c437ccaab235320995af8c272705ac7c01784dd8 tdf#147021: sc/source/filter SAL_N_ELEMENTS usage It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
I'd appreciate review on: https://gerrit.libreoffice.org/c/core/+/153889 which takes out a chunk of the SAL_N_ELEMENTS using span types. (Large but mostly mechanical)
sahil committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/0c45d90cfb1cc60218fbce3743df442b1f1d3f70 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Adam Seskunas committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/b1b002c293ddf38be125dc9083f41eb27452c450 tdf#147021 use std::size instead of SAL_N_ELEMENTS It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Now our baseline allows the use of std::ssize()
Deepika Goyal committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/0cd703c699334ed96a5743aae977207ba9a2a74b tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Willian Pessoa committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/fec4d0fb9992811d1217624d8eda95eac6da4220 tdf#147021 Use std::size() in editeng It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Started working on this bug.
Forgot the multi-hackers part. Sorry
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/9924561a094f1b3b2b78700bd1f1638d9608f881 tdf#147021 Use std::span to avoid SAL_N_ELEMENTS in CustomShape It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/82f30ac55a90a0f0915d4016c760c5c38db087f1 tdf#147021 Use std::span to avoid SAL_N_ELEMENTS in CustomShape It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/4a49d7a3a98ccdca52ea0a4475d05235a111ec0a tdf#147021 Use std::span to avoid SAL_N_ELEMENTS in CustomShape It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/abdb93ce28febd7c4b4e5dad164ee8a394b499b9 tdf#147021 Use std::span to avoid SAL_N_ELEMENTS in CustomShape It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dr. David Alan Gilbert committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/80785bb26a1f96d5f5bced7940473d22af00cb60 tdf#147021 Use std::span to avoid SAL_N_ELEMENTS in CustomShape It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Keldin Maldonado (KNM) committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/c78679b349573d718f8641071928d5f2edb47efa tdf#147021 replace SAL_N_ELEMENTS() with std::size() It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Sujatro Bhadra committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/58d2abd2ed85beed48a7677f2b186914c6f2a6c9 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Mohit Marathe committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/913437f3cd79c4f64151b98366fcc8a90bb66dac tdf#147021 replace SAL_N_ELEMENTS() with std::size() It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
RMZeroFour committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/6817c668cd3a61e4129a414e35f4044c306d290e tdf#147021 Replace SAL_N_ELEMENTS with std::size It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Ashwani5009 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/ea462462db187d87d978cb659df8ca81309fd2e7 tdf#147021 Replace SAL_N_ELEMENTS with std::size It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Arnaud VERSINI committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/190156976dd4f1c023279eddd1eed61bfd037e7a tdf#147021 : use std::size instead of SAL_N_ELEMENTS in generated code too It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Zainab Abbasi committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/837c9af957a4935d44f7e4ad6d290be915bcced2 tdf#147021 Replace SAL_N_ELEMENTS with std::size It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Tuukka Orava committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/f0a79b4dc1e96eb5dda35097d0975aae9f3e0c98 tdf#147021 Use std::size instead of SAL_N_ELEMENTS in editeng It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Deepanshu Sharma committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/85699f54fec281aa0ddc94fce96cdc588c56d55e tdf#147021 Use std::size() or std::ssize() instead of SAL_N_ELEMENTS() It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Mohamed Ali committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/c069f50a9e05c796935a55e6b2db5f714cf9cff7 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Rosh committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/71ec40df421dea0540b4245c5d590f87c57e33e7 tdf#147021 - Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Arnaud VERSINI committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/fb30ee5a6390160cc482572adcab14cc242528da tdf#147021 : use std::size in scaddins It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Rosh committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/ece03018cb6e0bcc6bf56b1a703c29f04df88eed tdf#147021 - Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Rosh committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/e1775391a59927219dfb4e1308f9291a5a0cc4c0 tdf#147021 - Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
LeSci-0x1 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/388b8fa5b28df532cd87689a35a38905b0cd038a tdf#147021 - Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
LeSci-0x1 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/c3f04640979d4f9d9abe8dc3e645d058e1cfb182 tdf#147021 - Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/f90d00a43f9fd1e14ac42f43c11e5f7eb966a355 tdf#147021 avoid use of SAL_N_ELEMENTS macro It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
kkasliwal committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/25716e4b7e4d56530cbd6180d807c369fe0725ad tdf#147021 use std::size() instead of SAL_N_ELEMENTS() in ww8par.cxx It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/e4271c596d33b2e864e86517a15694967f05fa33 tdf#147021 avoid use of SAL_N_ELEMENTS macro in fmtcol.cxx It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/05c92d3bdc633186975d3191d9ae5629c57a9a60 tdf#147021 avoid use of SAL_N_ELEMENTS macro in keynames.cxx It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Rahul Anand committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/b5e1580544194c77ac0214c90c23fa47447ef1d9 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/4d6cf9434009fcbdd2445d7bcae5081595edae28 tdf#147021 avoid use of SAL_N_ELEMENTS macro in appuno.cxx It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
I submitted a commit for this bug but forgot to tag the bug ID on my initial commit message. https://git.libreoffice.org/core/commit/f5ed0f0b009a72b2586c233bc1599528eaa20d14
Nam Bui committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/0176752f6d9431d3b0b1e3c3d81acb2a8dca8fe0 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/7b9e27da2033192c628b23e4e1686209e951dadb tdf#147021 avoid use of SAL_N_ELEMENTS macro It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/1d78c4993c33bb505939243e3d5873f94e12219c tdf#147021 avoid use of SAL_N_ELEMENTS macro It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/50fcf2f5f2465bb2873d04b1270bc42aec76290b tdf#147021 avoid use of SAL_N_ELEMENTS macro in Condition.cxx It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Mohamed Zaghloul committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/a790b8f4d554f2e249cd63f524f88940b0166325 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Devashish Gupta committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/19ebfeb83d39d7ff246a89cfedcc4691c966dff1 tdf#147021 Use std::ssize() instead of SAL_N_ELEMENTS macro It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Ahmed Khaled committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/d87694cf9396ec557031a93cc8b92b01720b69e0 tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Vlad Ene committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/cf1c25e542ad58b9e242979cf1e6d9c426321c6f tdf#147021 Use std::size() instead of SAL_N_ELEMENTS() macro It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/17418dea6be97bfc5efbe8a47d1a8ee26fbd0a9c tdf#147021 avoid use of SAL_N_ELEMENTS macro in qproform.cxx It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Simon Chenery committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/25ece0a2c15f0dc637e0fb607aaa5114b2739998 tdf#147021 avoid use of SAL_N_ELEMENTS macro in langtab.cxx It will be available in 25.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.