diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2020-05-16 20:19:40 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2020-05-16 12:40:08 -0700 |
| commit | a7440ce2779734a59a7495ef44b9d62da4aa9eee (patch) | |
| tree | ad0ba5c2d254d4d0778acb89e3bbf15135180604 /CHANGELOG.md | |
| parent | 04b6bb8cc946486efa174722b75378028105250b (diff) | |
| download | subsurface-a7440ce2779734a59a7495ef44b9d62da4aa9eee.tar.gz | |
filter: properly search for tags
The old code used get_taglist_string() and split the resulting
string at commas to get the list of tags. This was wrong for two
reasons:
1) It was buggy. Every tag but the first would start with a leading
space and thus not be found.
2) It was inefficient. The tag list was concatenated, just to be split
again.
Turn the tag list directly into a QStringList and remove whitespace
for good measure.
Fixes #2842.
Reported-by: Hartley Horwitz <hhrwtz@gmail.com>
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'CHANGELOG.md')
| -rw-r--r-- | CHANGELOG.md | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md index 482e55a60..a5ff63f60 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,3 +1,4 @@ +Filter: fix searching for tags [#2842] Desktop: fix plotting of thumbnails on profile [#2833] Core: always include BT/BLE name, even for devices no recognized as dive computer Core: fix failure to recognize several Aqualung BLE dive computers |