[nautilus/wip/oholy/deep-count-loops] properties-window: Fix endless content size calculations
- From: Ondrej Holy <oholy src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [nautilus/wip/oholy/deep-count-loops] properties-window: Fix endless content size calculations
- Date: Mon, 2 Mar 2020 14:27:13 +0000 (UTC)
commit c96a4fbca9974046d5008e67fa626c4cfe5fe4ea
Author: Ondrej Holy <oholy redhat com>
Date: Mon Mar 2 14:53:36 2020 +0100
properties-window: Fix endless content size calculations
Deep count calculations are currently restarted with each "changed"
event of `NautilusFile` object(s). This is not usually a problem if
only one file is selected, but it is a pretty big issue when more
files areselected. It is common that the calculation never ends. This
is because the "changed" events are emitted in many irrelevant cases
(e.g. free space change) and it totally doesn't make sense to restart
the calculation in most of the cases. The initial idea was to react
on ongoing file operations, however, the calculation currently doesn't
react on file changes deeper in the tree anyway, or on changes, which
happened after the calculation is done. Thus the current result can be
outdated anyway. Let's ignore `NautilusFile` changes at all when
calculating the content size as it is pretty impossible to implement
this properly to dynamically react on all size changes in the tree.
Fixes: https://gitlab.gnome.org/GNOME/nautilus/issues/363
src/nautilus-properties-window.c | 4 ----
1 file changed, 4 deletions(-)
---
diff --git a/src/nautilus-properties-window.c b/src/nautilus-properties-window.c
index 9250adbe3..52a2b3fce 100644
--- a/src/nautilus-properties-window.c
+++ b/src/nautilus-properties-window.c
@@ -1137,10 +1137,6 @@ properties_window_update (NautilusPropertiesWindow *window,
{
dirty_target = TRUE;
}
- if (changed_file != NULL)
- {
- start_deep_count_for_file (window, changed_file);
- }
}
if (dirty_original)
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]