reportdesign/source/ui/dlg/Condition.cxx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
New commits: commit 50fcf2f5f2465bb2873d04b1270bc42aec76290b Author: Simon Chenery <[email protected]> AuthorDate: Sat Mar 15 10:59:59 2025 +0100 Commit: Ilmari Lauhakangas <[email protected]> CommitDate: Mon Mar 17 15:20:20 2025 +0100 tdf#147021 avoid use of SAL_N_ELEMENTS macro in Condition.cxx Change-Id: I84925b1734f520fffb9f94255130f7156c6ff8a8 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/182944 Tested-by: Ilmari Lauhakangas <[email protected]> Tested-by: Jenkins Reviewed-by: Ilmari Lauhakangas <[email protected]> diff --git a/reportdesign/source/ui/dlg/Condition.cxx b/reportdesign/source/ui/dlg/Condition.cxx index 169d352862a5..7a5e1e2fd006 100644 --- a/reportdesign/source/ui/dlg/Condition.cxx +++ b/reportdesign/source/ui/dlg/Condition.cxx @@ -304,15 +304,15 @@ void Condition::setCondition( const uno::Reference< report::XFormatCondition >& void Condition::updateToolbar(const uno::Reference< report::XReportControlFormat >& _xReportControlFormat) { - OUString aItems[] = { u"bold"_ustr, u"italic"_ustr, u"underline"_ustr, u"fontdialog"_ustr }; + static constexpr OUString aItems[] = { u"bold"_ustr, u"italic"_ustr, u"underline"_ustr, u"fontdialog"_ustr }; OSL_ENSURE(_xReportControlFormat.is(),"XReportControlFormat is NULL!"); if ( !_xReportControlFormat.is() ) return; - for (size_t j = 0; j < SAL_N_ELEMENTS(aItems); ++j) + for (const auto& rItem : aItems) { - m_xActions->set_item_active(aItems[j], OReportController::isFormatCommandEnabled(mapToolbarItemToSlotId(aItems[j]), + m_xActions->set_item_active(rItem, OReportController::isFormatCommandEnabled(mapToolbarItemToSlotId(rItem), _xReportControlFormat)); }
