mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

filter: avoid Windows crash

Browse source 
The scope confusion between s (the for loop variable) and s (the function
argument) caused a crash in the s.split() on Windows.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2020-11-11 13:50:50 -08:00
parent 6133796cda
commit 10a026b2ff
2 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -1,3 +1,4 @@
- Windows: avoid crash when setting filter values for tag-style fields
- mobile: add location service warning as required by Google Play - mobile: add location service warning as required by Google Play
- mobile: fix manually adding dives in the past [#2971] - mobile: fix manually adding dives in the past [#2971]

4
core/filterconstraint.cpp
View file

@ -686,8 +686,8 @@ void filter_constraint_set_stringlist(filter_constraint &c, const QString &s)
return; return;
} }
c.data.string_list->clear(); c.data.string_list->clear();
for (const QString &s: s.split(",", SKIP_EMPTY)) for (const QString &part: s.split(",", SKIP_EMPTY))
c.data.string_list->push_back(s.trimmed()); c.data.string_list->push_back(part.trimmed());
} }
void filter_constraint_set_timestamp_from(filter_constraint &c, timestamp_t from) void filter_constraint_set_timestamp_from(filter_constraint &c, timestamp_t from)