fixup! Playlists: keep correct track selection (position) when sorting

Author: ronso0

src/library/basesqltablemodel.cpp

@@ -265,7 +265,7 @@ void BaseSqlTableModel::select() {
         // the the first column always contains the id?
         DEBUG_ASSERT(idColumn == kIdColumn);
 
-        VERIFY_OR_DEBUG_ASSERT(idColumn >= 0) {
+        VERIFY_OR_DEBUG_ASSERT(idColumn != -1) {
             qCritical()
                     << "ID column not available in database query results:"
                     << m_idColumn;
@@ -280,10 +280,10 @@ void BaseSqlTableModel::select() {
         // current position defines the ordering
         rowInfo.order = rowInfos.size();
 
-        if (posColumn < 0 && !m_positionColumn.isEmpty()) {
+        if (posColumn == -1 && !m_positionColumn.isEmpty()) {
             posColumn = sqlRecord.indexOf(m_positionColumn);
         }
-        if (posColumn >= 0) {
+        if (posColumn != -1) {
             bool ok = false;
             int pos = sqlRecord.value(posColumn).toInt(&ok);
             if (ok) {
@@ -351,7 +351,7 @@ void BaseSqlTableModel::select() {
     DEBUG_ASSERT(trackIdToRows.size() <= rowInfos.size());
 
     TrackPos2Row trackPosToRows;
-    if (posColumn >= 0) {
+    if (posColumn != -1) {
         // We expect as many positions as we have rows
         trackPosToRows.reserve(rowInfos.size());
         for (int i = 0; i < rowInfos.size(); ++i) {