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