sc/source/ui/Accessibility/AccessibleDocument.cxx | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
New commits:
commit d81894231794c7a570bef1a5991056a325a7d61e
Author: Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Thu Nov 28 13:52:30 2024 +0100
Commit: Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Thu Nov 28 23:11:22 2024 +0100
sc a11y: Make 2 members non-mutable
... and use a non-const pointer when iterating instead.
Change-Id: I165b0ce907cadacc76562bff974ff913a3a711da
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177480
Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>
Tested-by: Jenkins
diff --git a/sc/source/ui/Accessibility/AccessibleDocument.cxx
b/sc/source/ui/Accessibility/AccessibleDocument.cxx
index e6f474a606df..d4eca4f99487 100644
--- a/sc/source/ui/Accessibility/AccessibleDocument.cxx
+++ b/sc/source/ui/Accessibility/AccessibleDocument.cxx
@@ -90,9 +90,9 @@ struct ScAccessibleShapeData
ScAccessibleShapeData(css::uno::Reference< css::drawing::XShape > xShape_);
~ScAccessibleShapeData();
mutable rtl::Reference< ::accessibility::AccessibleShape > pAccShape;
- mutable std::optional<ScAddress> xRelationCell; // if it is NULL this
shape is anchored on the table
+ std::optional<ScAddress> xRelationCell; // if it is NULL this shape is
anchored on the table
css::uno::Reference< css::drawing::XShape > xShape;
- mutable bool bSelected;
+ bool bSelected;
bool bSelectable;
// cache these to make the sorting cheaper
std::optional<sal_Int16> mxLayerID;
@@ -720,7 +720,7 @@ void ScChildrenShapes::DeselectAll()
}
if (bSomethingSelected)
- for (const ScAccessibleShapeData* pAccShapeData : maZOrderedShapes)
+ for (ScAccessibleShapeData* pAccShapeData : maZOrderedShapes)
if (pAccShapeData)
{
pAccShapeData->bSelected = false;
@@ -746,7 +746,7 @@ void ScChildrenShapes::SelectAll()
try
{
- for (const ScAccessibleShapeData* pAccShapeData : maZOrderedShapes)
+ for (ScAccessibleShapeData* pAccShapeData : maZOrderedShapes)
{
if (pAccShapeData && pAccShapeData->bSelectable)
{
