[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #36 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #35 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #34 from Dave Gilbert --- 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) -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #33 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #32 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #31 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Commit Notification changed: What|Removed |Added Whiteboard|target:7.4.0 target:7.5.0 |target:7.4.0 target:7.5.0 |reviewed:2022 target:7.6.0 |reviewed:2022 target:7.6.0 ||target:24.2.0 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #30 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #29 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Commit Notification changed: What|Removed |Added Whiteboard|target:7.4.0 target:7.5.0 |target:7.4.0 target:7.5.0 |reviewed:2022 |reviewed:2022 target:7.6.0 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #27 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #26 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #25 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #24 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #23 from treeb...@tutanota.com --- (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 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Aron Budea changed: What|Removed |Added Assignee|treeb...@tutanota.com |libreoffice-b...@lists.free ||desktop.org Whiteboard|target:7.4.0 reviewed:2022 |target:7.4.0 target:7.5.0 |target:7.5.0 reviewed:2022 |reviewed:2022 --- Comment #22 from Aron Budea --- Let's keep the default assignee, as this is a generic task that can be worked on by several different contributors. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 treeb...@tutanota.com changed: What|Removed |Added Assignee|libreoffice-b...@lists.free |treeb...@tutanota.com |desktop.org | -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #21 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #20 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #19 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #18 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #17 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #16 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #15 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Hossein changed: What|Removed |Added Whiteboard|target:7.4.0 reviewed:2022 |target:7.4.0 reviewed:2022 |target:7.5.0|target:7.5.0 reviewed:2022 --- Comment #14 from Hossein --- 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 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #13 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #12 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Commit Notification changed: What|Removed |Added Whiteboard|target:7.4.0 reviewed:2022 |target:7.4.0 reviewed:2022 ||target:7.5.0 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Hossein changed: What|Removed |Added Whiteboard|target:7.4.0|target:7.4.0 reviewed:2022 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #11 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #10 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #9 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #8 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #7 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #6 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Aron Budea changed: What|Removed |Added CC||aron.bu...@gmail.com Blocks||143781 Referenced Bugs: https://bugs.documentfoundation.org/show_bug.cgi?id=143781 [Bug 143781] [META] Development- and code-related bug reports -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #5 from Hossein --- (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 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #4 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #3 from Commit Notification --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Commit Notification changed: What|Removed |Added Whiteboard||target:7.4.0 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Hossein changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever confirmed|0 |1 -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 --- Comment #2 from Mike Kaganski --- 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. -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Stephan Bergmann changed: What|Removed |Added CC||sberg...@redhat.com --- Comment #1 from Stephan Bergmann --- (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 constexpr ptrdiff_t ssize(const T ()[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.) -- You are receiving this mail because: You are the assignee for the bug.
[Libreoffice-bugs] [Bug 147021] Use std::size() instead of SAL_N_ELEMENTS() macro
https://bugs.documentfoundation.org/show_bug.cgi?id=147021 Hossein changed: What|Removed |Added Keywords||difficultyBeginner, ||easyHack, skillCpp -- You are receiving this mail because: You are the assignee for the bug.