diff options
author | frosch <frosch@openttd.org> | 2011-03-13 21:34:21 +0000 |
---|---|---|
committer | frosch <frosch@openttd.org> | 2011-03-13 21:34:21 +0000 |
commit | 652c9cb6635cc5f72a32077fdcd4e0f4bcc859e0 (patch) | |
tree | 3b78a1acc0ee0efe1e64ada470e815b44dc63442 /src/signs_gui.cpp | |
parent | 5689b3dbda6b3e4adee2c20e32d06a9e5a158c07 (diff) | |
download | openttd-652c9cb6635cc5f72a32077fdcd4e0f4bcc859e0.tar.xz |
(svn r22247) -Codechange: Update comments wrt. ForceRebuild() in command-scope.
Diffstat (limited to 'src/signs_gui.cpp')
-rw-r--r-- | src/signs_gui.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/signs_gui.cpp b/src/signs_gui.cpp index d639e1dcc..40039c3cf 100644 --- a/src/signs_gui.cpp +++ b/src/signs_gui.cpp @@ -374,15 +374,11 @@ struct SignListWindow : QueryStringBaseWindow, SignList { */ virtual void OnInvalidateData(int data = 0, bool gui_scope = true) { - if (!gui_scope) return; /* When there is a filter string, we always need to rebuild the list even if * the amount of signs in total is unchanged, as the subset of signs that is - * accepted by the filter might has changed. - * - * We can only set the trigger for resorting/rebuilding. - * We cannot safely resort at this point, as there might be multiple scheduled invalidations, - * and a rebuild needs to be done first though it is scheduled later. */ + * accepted by the filter might has changed. */ if (data == 0 || !StrEmpty(this->filter_string)) { // New or deleted sign, or there is a filter string + /* This needs to be done in command-scope to enforce rebuilding before resorting invalid data */ this->signs.ForceRebuild(); } else { // Change of sign contents while there is no filter string this->signs.ForceResort(); |