jeirmeister/Cockatrice
1
0
Fork 0
mirror of https://github.com/Cockatrice/Cockatrice.git synced 2026-07-19 08:52:15 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

reduce redundant recursion in updateVisibility

Browse source 
findChildren is recursive by default, so only the top-level updateVisibility needs to loop through the found children
This commit is contained in:
RickyRister 2025-02-26 22:55:33 -08:00
parent 2da0deff62
commit 49c884f8da
2 changed files with 9 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
cockatrice/src/client/ui/widgets/visual_deck_storage/visual_deck_storage_folder_display_widget.cpp
View file

@ -98,10 +98,11 @@ void VisualDeckStorageFolderDisplayWidget::createWidgetsForFiles()
} }
/** /**
* Updates the visibility of this folder and all its DeckPreviewWidgets, as well as all its subfolders and their * Updates the visibility of this folder and all its DeckPreviewWidgets
* DeckPreviewWidgets. *
* @param recursive Also update the visibility of all subfolders and their DeckPreviewWidgets.
*/ */
void VisualDeckStorageFolderDisplayWidget::updateVisibility() void VisualDeckStorageFolderDisplayWidget::updateVisibility(bool recursive)
{ {
bool atLeastOneWidgetVisible = checkVisibility(); bool atLeastOneWidgetVisible = checkVisibility();
if (atLeastOneWidgetVisible) { if (atLeastOneWidgetVisible) {
@ -109,8 +110,10 @@ void VisualDeckStorageFolderDisplayWidget::updateVisibility()
for (DeckPreviewWidget *display : flowWidget->findChildren<DeckPreviewWidget *>()) { for (DeckPreviewWidget *display : flowWidget->findChildren<DeckPreviewWidget *>()) {
display->updateVisibility(); display->updateVisibility();
} }
for (VisualDeckStorageFolderDisplayWidget *subFolder : findChildren<VisualDeckStorageFolderDisplayWidget *>()) { if (recursive) {
subFolder->updateVisibility(); for (auto *subFolder : findChildren<VisualDeckStorageFolderDisplayWidget *>()) {
subFolder->updateVisibility(false);
}
} }
} else { } else {
setVisible(false); setVisible(false);

2
cockatrice/src/client/ui/widgets/visual_deck_storage/visual_deck_storage_folder_display_widget.h
View file

@ -28,7 +28,7 @@ public:
}; };
public slots: public slots:
void updateVisibility(); void updateVisibility(bool recursive = true);
bool checkVisibility(); bool checkVisibility();
void updateShowFolders(bool enabled); void updateShowFolders(bool enabled);