PreviewButtonDelegate: Debug assertion

[Uwe Klotz]

While entering a search query:

#4 0x000000000049d9ca in mixxx_debug_assert(char const*, char const*, int, char const*)
    (assertion=assertion@entry=0x10aacd8 "index != m_currentEditedCellIndex", file=file@entry=0x10aaca8 "../src/library/previewbuttondelegate.cpp", line=line@entry=177, function=function@entry=0x10aac68 "void PreviewButtonDelegate::cellEntered(const QModelIndex&)") at ../src/util/assert.h:11
#5 0x00000000004fbbc2 in PreviewButtonDelegate::cellEntered(QModelIndex const&) (this=0x2d63a220, index=...) at ../src/library/previewbuttondelegate.cpp:177
#6 0x00007ffff3eb04b0 in void doActivate<false>(QObject*, int, void**) () at /lib64/libQt5Core.so.5
#7 0x00007ffff71501c9 in QAbstractItemView::entered(QModelIndex const&) () at /lib64/libQt5Widgets.so.5
#8 0x00007ffff715dfd0 in QAbstractItemViewPrivate::checkMouseMove(QPersistentModelIndex const&) () at /lib64/libQt5Widgets.so.5
#9 0x00007ffff715e23d in QAbstractItemView::mouseMoveEvent(QMouseEvent*) () at /lib64/libQt5Widgets.so.5
#10 0x00007ffff6f3585e in QWidget::event(QEvent*) () at /lib64/libQt5Widgets.so.5

Continuing in the debugger worked without any issues.

Preview column was visible but not used.

[Jan Holthuis (holthuis-jan)]

I encountered this too by accident:

    DEBUG ASSERT: "index != m_currentEditedCellIndex" in function void PreviewButtonDelegate::cellEntered(const QModelIndex&) at /home/jan/Projects/mixxx/src/library/previewbuttondelegate.cpp:177

[Jan Holthuis (holthuis-jan)]

I think this only affects `main`, searching works on the `2.3` branch.

[Jan Holthuis (holthuis-jan)]

This also makes the browse view completely unusable. Clicking any cell in the track table triggers it.

[Uwe Klotz (uklotzde-deactivatedaccount)]

I remember to have experienced this once or twice in the past. No idea when or why it happens.

[Jan Holthuis (holthuis-jan)]

Just encountered this again:

> DEBUG ASSERT: "index != m_currentEditedCellIndex" in function void PreviewButtonDelegate::cellEntered(const QModelIndex&) at /home/jan/Projects/mixxx/src/library/previewbuttondelegate.cpp:177

I found how to reproduce this reliably:

1. Start Mixxx
2. Click somewhere in the tracks table
3. Press CTRL+A to select all tracks
4. Drag them on the analysis feature
5. Click a cell in the preview column -> DEBUG_ASSERT is triggered

[Uwe Klotz (uklotzde-deactivatedaccount)]

This bugs me frequently, will fix it.

[Uwe Klotz (uklotzde-deactivatedaccount)]

This might also affect 2.3, but only the development snapshots.

[Uwe Klotz (uklotzde-deactivatedaccount)]

Not critical, because it works as expected when ignoring the debug assertion.

[Uwe Klotz (uklotzde-deactivatedaccount)]

https://github.com/mixxxdj/mixxx/pull/4264

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/10418