diff options
| author | Daniel Teske <daniel.teske@digia.com> | 2013-10-02 16:17:25 +0200 |
|---|---|---|
| committer | Daniel Teske <daniel.teske@digia.com> | 2013-10-07 12:13:18 +0200 |
| commit | 03e76337cae67bb718a4998c9e31badd295a9a61 (patch) | |
| tree | ad0a6fe05b254a8d51c4111f8e0f8f3e05061e56 /src/plugins/android/androidextralibrarylistmodel.h | |
| parent | 872bfb70948a8fc68eb6296a9a1a5ea6336b92d5 (diff) | |
AndroidExtraLibraryListModel: Fix crash on removing libs
Steps to reproduce the crash, start with the list a b a c, select the
second a and the c. Click on remove. The removeEntries will find that it
can remove two consecutive entries in one beginRemoveRows/endRemoveRows,
but will wrongly remove them starting at the first a.
The fix is too simply order the modelindexes in descendeding order such
that removing entries does not modify the rows.
Change-Id: I4be349f4bab8137075da0d8dfcef24f10dc25f92
Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@digia.com>
Diffstat (limited to 'src/plugins/android/androidextralibrarylistmodel.h')
| -rw-r--r-- | src/plugins/android/androidextralibrarylistmodel.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/android/androidextralibrarylistmodel.h b/src/plugins/android/androidextralibrarylistmodel.h index 54a486bd26..a4f5f203a3 100644 --- a/src/plugins/android/androidextralibrarylistmodel.h +++ b/src/plugins/android/androidextralibrarylistmodel.h @@ -51,7 +51,7 @@ public: int columnCount(const QModelIndex &parent) const; QVariant data(const QModelIndex &index, int role) const; - void removeEntries(const QModelIndexList &list); + void removeEntries(QModelIndexList list); void addEntries(const QStringList &list); private slots: |